[{"id":3674015,"web_url":"http://patchwork.ozlabs.org/comment/3674015/","msgid":"","list_archive_url":null,"date":"2026-04-07T09:07:00","subject":"Re: [PATCH v5 8/9] driver core: Replace dev->of_node_reused with\n dev_of_node_reused()","submitter":{"id":64666,"url":"http://patchwork.ozlabs.org/api/people/64666/","name":"Johan Hovold","email":"johan@kernel.org"},"content":"On Mon, Apr 06, 2026 at 04:23:01PM -0700, Doug Anderson wrote:\n> In C, bitfields are not necessarily safe to modify from multiple\n> threads without locking. Switch \"of_node_reused\" over to the \"flags\"\n> field so modifications are safe.\n\nThis flag is only set before registering a device with driver core so\nthere is no issue using the existing bitfield here (with the caveat that\nPCI pwrctrl may have gotten that wrong). I haven't checked the other\nflags, but I assume most of them work the same way.\n\nBut apart from the commit message being misleading, switching to using\natomic ops and accessors for consistency is fine.\n\n> Cc: Johan Hovold \n> Acked-by: Mark Brown \n> Reviewed-by: Rafael J. Wysocki (Intel) \n> Reviewed-by: Danilo Krummrich \n> Signed-off-by: Douglas Anderson \n> ---\n> Not fixing any known bugs; problem is theoretical and found by code\n> inspection. Change is done somewhat manually and only lightly tested\n> (mostly compile-time tested).\n \n> diff --git a/drivers/regulator/bq257xx-regulator.c b/drivers/regulator/bq257xx-regulator.c\n> index dab8f1ab4450..40e0f1a7ae81 100644\n> --- a/drivers/regulator/bq257xx-regulator.c\n> +++ b/drivers/regulator/bq257xx-regulator.c\n> @@ -143,7 +143,7 @@ static int bq257xx_regulator_probe(struct platform_device *pdev)\n> \tstruct regulator_config cfg = {};\n> \n> \tpdev->dev.of_node = pdev->dev.parent->of_node;\n> -\tpdev->dev.of_node_reused = true;\n> +\tdev_set_of_node_reused(&pdev->dev);\n> \n> \tpdata = devm_kzalloc(&pdev->dev, sizeof(struct bq257xx_reg_data), GFP_KERNEL);\n> \tif (!pdata)\n> diff --git a/drivers/regulator/rk808-regulator.c b/drivers/regulator/rk808-regulator.c\n> index e66408f23bb6..8297d31cde9f 100644\n> --- a/drivers/regulator/rk808-regulator.c\n> +++ b/drivers/regulator/rk808-regulator.c\n> @@ -2115,7 +2115,7 @@ static int rk808_regulator_probe(struct platform_device *pdev)\n> \tint ret, i, nregulators;\n> \n> \tpdev->dev.of_node = pdev->dev.parent->of_node;\n> -\tpdev->dev.of_node_reused = true;\n> +\tdev_set_of_node_reused(&pdev->dev);\n> \n> \tregmap = dev_get_regmap(pdev->dev.parent, NULL);\n> \tif (!regmap)\n\nThese two uses are broken currently though and should be using\ndevice_set_of_node_from_dev() so that an extra reference is taken to\nbalance the one dropped by the platform bus code.\n\nI'll send two fixes to Mark, any merge conflict should be trivial to\nfixup.\n\nReviewed-by: Johan Hovold \n\nJohan","headers":{"Return-Path":"\n ","X-Original-To":["incoming@patchwork.ozlabs.org","linux-aspeed@lists.ozlabs.org"],"Delivered-To":"patchwork-incoming@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256\n header.s=k20201202 header.b=stgpB9oZ;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=lists.ozlabs.org\n (client-ip=2404:9400:21b9:f100::1; helo=lists.ozlabs.org;\n envelope-from=linux-aspeed+bounces-3851-incoming=patchwork.ozlabs.org@lists.ozlabs.org;\n receiver=patchwork.ozlabs.org)","lists.ozlabs.org;\n arc=none smtp.remote-ip=\"2600:3c04:e001:324:0:1991:8:25\"","lists.ozlabs.org;\n dmarc=pass (p=quarantine dis=none) header.from=kernel.org","lists.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256\n header.s=k20201202 header.b=stgpB9oZ;\n\tdkim-atps=neutral","lists.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=kernel.org\n (client-ip=2600:3c04:e001:324:0:1991:8:25; helo=tor.source.kernel.org;\n envelope-from=johan@kernel.org; receiver=lists.ozlabs.org)"],"Received":["from lists.ozlabs.org (lists.ozlabs.org\n [IPv6:2404:9400:21b9:f100::1])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519 server-signature ECDSA (secp384r1 raw public key)\n server-digest SHA384)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fqgLh42Ksz1xy1\n\tfor ; Tue, 07 Apr 2026 19:07:08 +1000 (AEST)","from boromir.ozlabs.org (localhost [127.0.0.1])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 4fqgLh0Xj8z2yhD;\n\tTue, 07 Apr 2026 19:07:08 +1000 (AEST)","from tor.source.kernel.org (tor.source.kernel.org\n [IPv6:2600:3c04:e001:324:0:1991:8:25])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519 server-signature RSA-PSS (2048 bits) server-digest\n SHA256)\n\t(No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 4fqgLg0QP9z2ySk\n\tfor ; Tue, 07 Apr 2026 19:07:07 +1000 (AEST)","from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58])\n\tby tor.source.kernel.org (Postfix) with ESMTP id 090A1600CB;\n\tTue, 7 Apr 2026 09:07:04 +0000 (UTC)","by smtp.kernel.org (Postfix) with ESMTPSA id A41DDC116C6;\n\tTue, 7 Apr 2026 09:07:03 +0000 (UTC)","from johan by xi.lan with local (Exim 4.98.2)\n\t(envelope-from )\n\tid 1wA2Oi-0000000AjGf-2EeN;\n\tTue, 07 Apr 2026 11:07:00 +0200"],"ARC-Seal":"i=1; a=rsa-sha256; d=lists.ozlabs.org; s=201707; t=1775552827;\n\tcv=none;\n b=Nvkgl/T6KdsqqcOUC2766INsdeghTKLD+RIJyMX1TvhUuaQVYiT1fa5LT/IRKKJJdxkUeCY13kZ9Bmze7CsJikT1TLccLonHZCumR25rxbcIhTwV6KBfCvzAvDgAN1KYcK05aHDdunjG3k1AnFhYsd1xzFf1sAwngfwBFdVFV6SGzfTHVEFTom0uUYTlMy8UbKQfkmOfOmH0cWRrT5NlLX48ocqr+htEK+qzDu2IfzgQ3TF7IlikrdCqYhz7p5ckeM12EIlcenPbVKR9WyWEr2V/X3Jx3LsfeiNvz9GFb0oskMzsWlSE34FQ9j2jO8AjleULQtXikVoNzLuTiNfGfA==","ARC-Message-Signature":"i=1; a=rsa-sha256; d=lists.ozlabs.org; s=201707;\n\tt=1775552827; c=relaxed/relaxed;\n\tbh=hO0i5DF8EvnkES+hyeEAy75jRio1nx20ZDks5J2KtzU=;\n\th=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version:\n\t Content-Type:Content-Disposition:In-Reply-To;\n b=mPFIZ8AXslF2l2LiUZwa5nBlVilhQBXuMf+1Y85wcS5FEelyoLBzFpdqk5fswPElE3SQkhckeHaZAahBBZB+/LrTSGQaWO6wN9vTzdq6AM3FzggfqD2sDbgs+UvljoVg62PaQqf5ItcS9w6mV/EV+bRarjWycnZ0/y8JyYOn3SdCgaJT1lf31OEMTWrY1cAD0hmsLJFmy65FFa5/fwKVIuNCNAta3gnarusJwYK3r36gfmnfBJjtldfk0erqxiZ6ecPmRslJD+Wchi26KmlERwIm4B656qt9dxqWu5oY5joaLXmqp5aFcpMIcHSdnvQ85VEw+6E/3TaY+0PXk7zD5A==","ARC-Authentication-Results":"i=1; lists.ozlabs.org;\n dmarc=pass (p=quarantine dis=none) header.from=kernel.org;\n dkim=pass (2048-bit key;\n unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256\n header.s=k20201202 header.b=stgpB9oZ; dkim-atps=neutral;\n spf=pass (client-ip=2600:3c04:e001:324:0:1991:8:25;\n helo=tor.source.kernel.org; envelope-from=johan@kernel.org;\n receiver=lists.ozlabs.org) smtp.mailfrom=kernel.org","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org;\n\ts=k20201202; t=1775552823;\n\tbh=4NERHreOtOHtxPxrg9hJfCL0sb0Gtcuiwi8WJRXUxIs=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=stgpB9oZ+np9GYARlzUA9ZMs/QvN4KIBqwa1qRYDAwyfYPqyHDzuHCzXPSnlyzgtg\n\t EAHXWcSv4E0rrXHIwU/F9B+i1mivT8/hiuEewvUz63bUM7ytEH5SQzK2thj+lYKQTs\n\t o2tisu8aVFvQIZG+/d8iAyn9lfuaOQqIZdUeq7k3DZiLr0hgqrYjSRZ2PwzXfNPDL4\n\t eowKFIJwq3yekc5lX5NoSnG1UiqaPzINQrHDk6adQBuAm9YxWQDDa+YyftbK/zR8JS\n\t kxyiSzhZcnKMojNpglf1PuMqJl8hSj8Ux7HAXRlz/T13loQq/CWQfn1V+eHwkI8KMb\n\t 5kNMKJjzp6Bvg==","Date":"Tue, 7 Apr 2026 11:07:00 +0200","From":"Johan Hovold ","To":"Douglas Anderson ","Cc":"Greg Kroah-Hartman ,\n\t\"Rafael J . Wysocki\" ,\n\tDanilo Krummrich ,\n\tAlan Stern ,\n\tAlexey Kardashevskiy ,\n\tEric Dumazet ,\n\tLeon Romanovsky , Christoph Hellwig ,\n\tRobin Murphy , maz@kernel.org,\n\tAlexander Lobakin ,\n\tSaravana Kannan ,\n\tMark Brown , alexander.stein@ew.tq-group.com,\n\tandrew@codeconstruct.com.au, andrew@lunn.ch,\n\tandriy.shevchenko@linux.intel.com, astewart@tektelic.com,\n\tbhelgaas@google.com, brgl@kernel.org, davem@davemloft.net,\n\tdevicetree@vger.kernel.org, driver-core@lists.linux.dev,\n\thkallweit1@gmail.com, jirislaby@kernel.org, joel@jms.id.au,\n\tkees@kernel.org, kuba@kernel.org, lgirdwood@gmail.com,\n\tlinux-arm-kernel@lists.infradead.org, linux-aspeed@lists.ozlabs.org,\n\tlinux-kernel@vger.kernel.org, linux-pci@vger.kernel.org,\n\tlinux-serial@vger.kernel.org, linux-usb@vger.kernel.org,\n\tlinux@armlinux.org.uk, mani@kernel.org, netdev@vger.kernel.org,\n\tpabeni@redhat.com, robh@kernel.org","Subject":"Re: [PATCH v5 8/9] driver core: Replace dev->of_node_reused with\n dev_of_node_reused()","Message-ID":"","References":"<20260406232444.3117516-1-dianders@chromium.org>\n <20260406162231.v5.8.I806b8636cd3724f6cd1f5e199318ab8694472d90@changeid>","X-Mailing-List":"linux-aspeed@lists.ozlabs.org","List-Id":"","List-Help":"","List-Owner":"","List-Post":"","List-Archive":",\n ","List-Subscribe":",\n ,\n ","List-Unsubscribe":"","Precedence":"list","MIME-Version":"1.0","Content-Type":"text/plain; charset=us-ascii","Content-Disposition":"inline","In-Reply-To":"\n <20260406162231.v5.8.I806b8636cd3724f6cd1f5e199318ab8694472d90@changeid>","X-Spam-Status":"No, score=-0.2 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,\n\tDKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,SPF_PASS\n\tautolearn=disabled version=4.0.1","X-Spam-Checker-Version":"SpamAssassin 4.0.1 (2024-03-25) on lists.ozlabs.org"}},{"id":3674509,"web_url":"http://patchwork.ozlabs.org/comment/3674509/","msgid":"","list_archive_url":null,"date":"2026-04-07T07:27:19","subject":"Re: [PATCH v5 8/9] driver core: Replace dev->of_node_reused with\n dev_of_node_reused()","submitter":{"id":78905,"url":"http://patchwork.ozlabs.org/api/people/78905/","name":"Manivannan Sadhasivam","email":"mani@kernel.org"},"content":"On Mon, Apr 06, 2026 at 04:23:01PM -0700, Douglas Anderson wrote:\n> In C, bitfields are not necessarily safe to modify from multiple\n> threads without locking. Switch \"of_node_reused\" over to the \"flags\"\n> field so modifications are safe.\n> \n> Cc: Johan Hovold \n> Acked-by: Mark Brown \n> Reviewed-by: Rafael J. Wysocki (Intel) \n> Reviewed-by: Danilo Krummrich \n> Signed-off-by: Douglas Anderson \n\nAcked-by: Manivannan Sadhasivam # PCI_PWRCTRL\n\n- Mani\n\n> ---\n> Not fixing any known bugs; problem is theoretical and found by code\n> inspection. Change is done somewhat manually and only lightly tested\n> (mostly compile-time tested).\n> \n> (no changes since v4)\n> \n> Changes in v4:\n> - Use accessor functions for flags\n> \n> Changes in v3:\n> - New\n> \n> drivers/base/core.c | 2 +-\n> drivers/base/pinctrl.c | 2 +-\n> drivers/base/platform.c | 2 +-\n> drivers/net/pcs/pcs-xpcs-plat.c | 2 +-\n> drivers/of/device.c | 6 +++---\n> drivers/pci/of.c | 2 +-\n> drivers/pci/pwrctrl/core.c | 2 +-\n> drivers/regulator/bq257xx-regulator.c | 2 +-\n> drivers/regulator/rk808-regulator.c | 2 +-\n> drivers/tty/serial/serial_base_bus.c | 2 +-\n> drivers/usb/gadget/udc/aspeed-vhub/dev.c | 2 +-\n> include/linux/device.h | 7 ++++---\n> 12 files changed, 17 insertions(+), 16 deletions(-)\n> \n> diff --git a/drivers/base/core.c b/drivers/base/core.c\n> index 8a83d7c93361..30825bf83234 100644\n> --- a/drivers/base/core.c\n> +++ b/drivers/base/core.c\n> @@ -5283,7 +5283,7 @@ void device_set_of_node_from_dev(struct device *dev, const struct device *dev2)\n> {\n> \tof_node_put(dev->of_node);\n> \tdev->of_node = of_node_get(dev2->of_node);\n> -\tdev->of_node_reused = true;\n> +\tdev_set_of_node_reused(dev);\n> }\n> EXPORT_SYMBOL_GPL(device_set_of_node_from_dev);\n> \n> diff --git a/drivers/base/pinctrl.c b/drivers/base/pinctrl.c\n> index 6e250272c843..0bbc83231234 100644\n> --- a/drivers/base/pinctrl.c\n> +++ b/drivers/base/pinctrl.c\n> @@ -24,7 +24,7 @@ int pinctrl_bind_pins(struct device *dev)\n> {\n> \tint ret;\n> \n> -\tif (dev->of_node_reused)\n> +\tif (dev_of_node_reused(dev))\n> \t\treturn 0;\n> \n> \tdev->pins = devm_kzalloc(dev, sizeof(*(dev->pins)), GFP_KERNEL);\n> diff --git a/drivers/base/platform.c b/drivers/base/platform.c\n> index d44591d52e36..199e6fb25770 100644\n> --- a/drivers/base/platform.c\n> +++ b/drivers/base/platform.c\n> @@ -856,7 +856,7 @@ struct platform_device *platform_device_register_full(\n> \tpdev->dev.parent = pdevinfo->parent;\n> \tpdev->dev.fwnode = pdevinfo->fwnode;\n> \tpdev->dev.of_node = of_node_get(to_of_node(pdev->dev.fwnode));\n> -\tpdev->dev.of_node_reused = pdevinfo->of_node_reused;\n> +\tdev_assign_of_node_reused(&pdev->dev, pdevinfo->of_node_reused);\n> \n> \tif (pdevinfo->dma_mask) {\n> \t\tpdev->platform_dma_mask = pdevinfo->dma_mask;\n> diff --git a/drivers/net/pcs/pcs-xpcs-plat.c b/drivers/net/pcs/pcs-xpcs-plat.c\n> index b8c48f9effbf..f4b1b8246ce9 100644\n> --- a/drivers/net/pcs/pcs-xpcs-plat.c\n> +++ b/drivers/net/pcs/pcs-xpcs-plat.c\n> @@ -349,7 +349,7 @@ static int xpcs_plat_init_dev(struct dw_xpcs_plat *pxpcs)\n> \t * up later. Make sure DD-core is aware of the OF-node being re-used.\n> \t */\n> \tdevice_set_node(&mdiodev->dev, fwnode_handle_get(dev_fwnode(dev)));\n> -\tmdiodev->dev.of_node_reused = true;\n> +\tdev_set_of_node_reused(&mdiodev->dev);\n> \n> \t/* Pass the data further so the DW XPCS driver core could use it */\n> \tmdiodev->dev.platform_data = (void *)device_get_match_data(dev);\n> diff --git a/drivers/of/device.c b/drivers/of/device.c\n> index f7e75e527667..be4e1584e0af 100644\n> --- a/drivers/of/device.c\n> +++ b/drivers/of/device.c\n> @@ -26,7 +26,7 @@\n> const struct of_device_id *of_match_device(const struct of_device_id *matches,\n> \t\t\t\t\t const struct device *dev)\n> {\n> -\tif (!matches || !dev->of_node || dev->of_node_reused)\n> +\tif (!matches || !dev->of_node || dev_of_node_reused(dev))\n> \t\treturn NULL;\n> \treturn of_match_node(matches, dev->of_node);\n> }\n> @@ -192,7 +192,7 @@ ssize_t of_device_modalias(struct device *dev, char *str, ssize_t len)\n> {\n> \tssize_t sl;\n> \n> -\tif (!dev || !dev->of_node || dev->of_node_reused)\n> +\tif (!dev || !dev->of_node || dev_of_node_reused(dev))\n> \t\treturn -ENODEV;\n> \n> \tsl = of_modalias(dev->of_node, str, len - 2);\n> @@ -254,7 +254,7 @@ int of_device_uevent_modalias(const struct device *dev, struct kobj_uevent_env *\n> {\n> \tint sl;\n> \n> -\tif ((!dev) || (!dev->of_node) || dev->of_node_reused)\n> +\tif ((!dev) || (!dev->of_node) || dev_of_node_reused(dev))\n> \t\treturn -ENODEV;\n> \n> \t/* Devicetree modalias is tricky, we add it in 2 steps */\n> diff --git a/drivers/pci/of.c b/drivers/pci/of.c\n> index 9f8eb5df279e..1f9b669abdb0 100644\n> --- a/drivers/pci/of.c\n> +++ b/drivers/pci/of.c\n> @@ -38,7 +38,7 @@ int pci_set_of_node(struct pci_dev *dev)\n> \tstruct device *pdev __free(put_device) =\n> \t\tbus_find_device_by_of_node(&platform_bus_type, node);\n> \tif (pdev)\n> -\t\tdev->bus->dev.of_node_reused = true;\n> +\t\tdev_set_of_node_reused(&dev->bus->dev);\n> \n> \tdevice_set_node(&dev->dev, of_fwnode_handle(no_free_ptr(node)));\n> \treturn 0;\n> diff --git a/drivers/pci/pwrctrl/core.c b/drivers/pci/pwrctrl/core.c\n> index 7754baed67f2..72963a92362a 100644\n> --- a/drivers/pci/pwrctrl/core.c\n> +++ b/drivers/pci/pwrctrl/core.c\n> @@ -39,7 +39,7 @@ static int pci_pwrctrl_notify(struct notifier_block *nb, unsigned long action,\n> \t\t * If we got here then the PCI device is the second after the\n> \t\t * power control platform device. Mark its OF node as reused.\n> \t\t */\n> -\t\tdev->of_node_reused = true;\n> +\t\tdev_set_of_node_reused(dev);\n> \t\tbreak;\n> \t}\n> \n> diff --git a/drivers/regulator/bq257xx-regulator.c b/drivers/regulator/bq257xx-regulator.c\n> index dab8f1ab4450..40e0f1a7ae81 100644\n> --- a/drivers/regulator/bq257xx-regulator.c\n> +++ b/drivers/regulator/bq257xx-regulator.c\n> @@ -143,7 +143,7 @@ static int bq257xx_regulator_probe(struct platform_device *pdev)\n> \tstruct regulator_config cfg = {};\n> \n> \tpdev->dev.of_node = pdev->dev.parent->of_node;\n> -\tpdev->dev.of_node_reused = true;\n> +\tdev_set_of_node_reused(&pdev->dev);\n> \n> \tpdata = devm_kzalloc(&pdev->dev, sizeof(struct bq257xx_reg_data), GFP_KERNEL);\n> \tif (!pdata)\n> diff --git a/drivers/regulator/rk808-regulator.c b/drivers/regulator/rk808-regulator.c\n> index e66408f23bb6..8297d31cde9f 100644\n> --- a/drivers/regulator/rk808-regulator.c\n> +++ b/drivers/regulator/rk808-regulator.c\n> @@ -2115,7 +2115,7 @@ static int rk808_regulator_probe(struct platform_device *pdev)\n> \tint ret, i, nregulators;\n> \n> \tpdev->dev.of_node = pdev->dev.parent->of_node;\n> -\tpdev->dev.of_node_reused = true;\n> +\tdev_set_of_node_reused(&pdev->dev);\n> \n> \tregmap = dev_get_regmap(pdev->dev.parent, NULL);\n> \tif (!regmap)\n> diff --git a/drivers/tty/serial/serial_base_bus.c b/drivers/tty/serial/serial_base_bus.c\n> index a12935f6b992..5f23284a8778 100644\n> --- a/drivers/tty/serial/serial_base_bus.c\n> +++ b/drivers/tty/serial/serial_base_bus.c\n> @@ -74,7 +74,7 @@ static int serial_base_device_init(struct uart_port *port,\n> \tdev->parent = parent_dev;\n> \tdev->bus = &serial_base_bus_type;\n> \tdev->release = release;\n> -\tdev->of_node_reused = true;\n> +\tdev_set_of_node_reused(dev);\n> \n> \tdevice_set_node(dev, fwnode_handle_get(dev_fwnode(parent_dev)));\n> \n> diff --git a/drivers/usb/gadget/udc/aspeed-vhub/dev.c b/drivers/usb/gadget/udc/aspeed-vhub/dev.c\n> index 2ecd049dacc2..8b9449d16324 100644\n> --- a/drivers/usb/gadget/udc/aspeed-vhub/dev.c\n> +++ b/drivers/usb/gadget/udc/aspeed-vhub/dev.c\n> @@ -593,7 +593,7 @@ int ast_vhub_init_dev(struct ast_vhub *vhub, unsigned int idx)\n> \t\td->gadget.max_speed = USB_SPEED_HIGH;\n> \td->gadget.speed = USB_SPEED_UNKNOWN;\n> \td->gadget.dev.of_node = vhub->pdev->dev.of_node;\n> -\td->gadget.dev.of_node_reused = true;\n> +\tdev_set_of_node_reused(&d->gadget.dev);\n> \n> \trc = usb_add_gadget_udc(d->port_dev, &d->gadget);\n> \tif (rc != 0)\n> diff --git a/include/linux/device.h b/include/linux/device.h\n> index 5b0fb6ad4c72..a79865a212e9 100644\n> --- a/include/linux/device.h\n> +++ b/include/linux/device.h\n> @@ -483,6 +483,8 @@ struct device_physical_location {\n> *\t\tdriver/bus sync_state() callback.\n> * @DEV_FLAG_DMA_COHERENT: This particular device is dma coherent, even if the\n> *\t\tarchitecture supports non-coherent devices.\n> + * @DEV_FLAG_OF_NODE_REUSED: Set if the device-tree node is shared with an\n> + *\t\tancestor device.\n> */\n> enum struct_device_flags {\n> \tDEV_FLAG_READY_TO_PROBE = 0,\n> @@ -492,6 +494,7 @@ enum struct_device_flags {\n> \tDEV_FLAG_DMA_OPS_BYPASS = 4,\n> \tDEV_FLAG_STATE_SYNCED = 5,\n> \tDEV_FLAG_DMA_COHERENT = 6,\n> +\tDEV_FLAG_OF_NODE_REUSED = 7,\n> \n> \tDEV_FLAG_COUNT\n> };\n> @@ -573,8 +576,6 @@ enum struct_device_flags {\n> *\n> * @offline_disabled: If set, the device is permanently online.\n> * @offline:\tSet after successful invocation of bus type's .offline().\n> - * @of_node_reused: Set if the device-tree node is shared with an ancestor\n> - * device.\n> * @flags:\tDEV_FLAG_XXX flags. Use atomic bitfield operations to modify.\n> *\n> * At the lowest level, every device in a Linux system is represented by an\n> @@ -681,7 +682,6 @@ struct device {\n> \n> \tbool\t\t\toffline_disabled:1;\n> \tbool\t\t\toffline:1;\n> -\tbool\t\t\tof_node_reused:1;\n> \n> \tDECLARE_BITMAP(flags, DEV_FLAG_COUNT);\n> };\n> @@ -715,6 +715,7 @@ __create_dev_flag_accessors(dma_skip_sync, DEV_FLAG_DMA_SKIP_SYNC);\n> __create_dev_flag_accessors(dma_ops_bypass, DEV_FLAG_DMA_OPS_BYPASS);\n> __create_dev_flag_accessors(state_synced, DEV_FLAG_STATE_SYNCED);\n> __create_dev_flag_accessors(dma_coherent, DEV_FLAG_DMA_COHERENT);\n> +__create_dev_flag_accessors(of_node_reused, DEV_FLAG_OF_NODE_REUSED);\n> \n> #undef __create_dev_flag_accessors\n> \n> -- \n> 2.53.0.1213.gd9a14994de-goog\n>","headers":{"Return-Path":"\n ","X-Original-To":["incoming@patchwork.ozlabs.org","linux-aspeed@lists.ozlabs.org"],"Delivered-To":"patchwork-incoming@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256\n header.s=k20201202 header.b=m30OgZ4+;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=lists.ozlabs.org\n (client-ip=2404:9400:21b9:f100::1; helo=lists.ozlabs.org;\n envelope-from=linux-aspeed+bounces-3856-incoming=patchwork.ozlabs.org@lists.ozlabs.org;\n receiver=patchwork.ozlabs.org)","lists.ozlabs.org;\n arc=none smtp.remote-ip=172.105.4.254","lists.ozlabs.org;\n dmarc=pass (p=quarantine dis=none) header.from=kernel.org","lists.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256\n header.s=k20201202 header.b=m30OgZ4+;\n\tdkim-atps=neutral","lists.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=kernel.org\n (client-ip=172.105.4.254; helo=tor.source.kernel.org;\n envelope-from=mani@kernel.org; receiver=lists.ozlabs.org)"],"Received":["from lists.ozlabs.org (lists.ozlabs.org\n [IPv6:2404:9400:21b9:f100::1])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fr2J73x7jz1xtJ\n\tfor ; Wed, 08 Apr 2026 09:21:11 +1000 (AEST)","from boromir.ozlabs.org (localhost [127.0.0.1])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 4fr2J60sl8z2ySW;\n\tWed, 08 Apr 2026 09:21:10 +1000 (AEST)","from tor.source.kernel.org (tor.source.kernel.org [172.105.4.254])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519 server-signature RSA-PSS (2048 bits) server-digest\n SHA256)\n\t(No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 4fqd7v1WWnz2ySS\n\tfor ; Tue, 07 Apr 2026 17:27:39 +1000 (AEST)","from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58])\n\tby tor.source.kernel.org (Postfix) with ESMTP id 7B29360126;\n\tTue, 7 Apr 2026 07:27:36 +0000 (UTC)","by smtp.kernel.org (Postfix) with ESMTPSA id 9F639C116C6;\n\tTue, 7 Apr 2026 07:27:23 +0000 (UTC)"],"ARC-Seal":"i=1; a=rsa-sha256; d=lists.ozlabs.org; s=201707; t=1775546860;\n\tcv=none;\n b=Q0g4lEcz7JNSjCRwN84OpSPY9/j2SY9nGDKFsNavdJQDdGX5VAJ7wzA0hHr4NQBMYSfMKtopdIytV9PWADajmPYrQGpEtACmMsSbfnsStJh1tXit9tFVC1oAcSjBo84TY0oCyZFIBsT5s2bDtpPubqq1sXt9xisJZqGYA5MUnZIBSBa4sPg6p5ZAp5chtlDhqxZiIDgi+l7kutmMSMiFYxGBBbNNaun4wmxEVms9MMAIysdavEpaVLKNxucGNnSNyxpARPLT4GJFwlrDdQEjnP2gE1zjdcJz5WsbZ1qXnz9lFBpcO+tU/jjysDNx1g2o6cEvFqIMbnZfRMtxo2mERQ==","ARC-Message-Signature":"i=1; a=rsa-sha256; d=lists.ozlabs.org; s=201707;\n\tt=1775546860; c=relaxed/relaxed;\n\tbh=kT01Kxko0Mdfvco+cwdX7xfZEHOufw2OA9hulGs7J8w=;\n\th=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version:\n\t Content-Type:Content-Disposition:In-Reply-To;\n b=TYfMyUKw+MvUd2hlpho7WyyFnABMlrooyBJ3wXJd/9KhAKHORdh0yw3pivSIlpkiqN9gq+RCudxMv5H0Nys2s4B+3mpAQ9ETJqLWtmz9lrVCrtBMxrSoR50cxPxm8t3tqJ8xQZAyxqb1YaktbUc85+HkscI7a/+3bXpyzL+0LZlCucq3fg5a1aFjwz0LUV+cyfESEg2Qd9BiwjWmKrKyuEydEZ372b03NisjLPd5dUjQkpZ+PGIOoj3RjWsUGYjsBq1CHM4DHJFQbaSHGz013t+l3X+KJFE8Em4A4PjC/MhJ89FvuGNwokVjNAzZImPuWCzke7dPN5+1VXstg+uQYw==","ARC-Authentication-Results":"i=1; lists.ozlabs.org;\n dmarc=pass (p=quarantine dis=none) header.from=kernel.org;\n dkim=pass (2048-bit key;\n unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256\n header.s=k20201202 header.b=m30OgZ4+; dkim-atps=neutral;\n spf=pass (client-ip=172.105.4.254; helo=tor.source.kernel.org;\n envelope-from=mani@kernel.org;\n receiver=lists.ozlabs.org) smtp.mailfrom=kernel.org","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org;\n\ts=k20201202; t=1775546856;\n\tbh=Ke+4cfFIuUyaAYRB85WlhtaSzn9Vsqf7eLubNm2wrC8=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=m30OgZ4+3p1ReP/oAexgtmU1wpHtdCwMpqLsPQ1M0sfld/z/FFzZXpYDh4e3fybYj\n\t /pxqvDO1Tk2YWl7SwXOzZpmgZPuybaqhOnBhM2OusK35qXxa6nPe47jMcbyPObRx9A\n\t LIBBR24hxE+0GEJyrHG5Mor7C5RciZv0K1Y119quGDDpBWRKWuWFCrZtXtkdQBW4P/\n\t 0NSGF1V4uZmeRTPBIIxtlEIQSD1QF0mJS+7VB4QuEvtYAR4nRExM1YEGb6Y1NIFanM\n\t 5dyrIFv4Z+WGfhdSU+6/pMy8q4brJ4lkLN1g/LKq88orVqzymK08dPIflO6p1CiWaw\n\t stDhRXnwwdSEw==","Date":"Tue, 7 Apr 2026 12:57:19 +0530","From":"Manivannan Sadhasivam ","To":"Douglas Anderson ","Cc":"Greg Kroah-Hartman ,\n\t\"Rafael J . Wysocki\" , Danilo Krummrich ,\n\tAlan Stern , Alexey Kardashevskiy ,\n\tJohan Hovold , Eric Dumazet ,\n\tLeon Romanovsky , Christoph Hellwig ,\n\tRobin Murphy , maz@kernel.org,\n Alexander Lobakin ,\n\tSaravana Kannan , Mark Brown ,\n alexander.stein@ew.tq-group.com,\n\tandrew@codeconstruct.com.au, andrew@lunn.ch,\n andriy.shevchenko@linux.intel.com,\n\tastewart@tektelic.com, bhelgaas@google.com, brgl@kernel.org,\n davem@davemloft.net,\n\tdevicetree@vger.kernel.org, driver-core@lists.linux.dev,\n hkallweit1@gmail.com,\n\tjirislaby@kernel.org, joel@jms.id.au, kees@kernel.org, kuba@kernel.org,\n\tlgirdwood@gmail.com, linux-arm-kernel@lists.infradead.org,\n\tlinux-aspeed@lists.ozlabs.org, linux-kernel@vger.kernel.org,\n linux-pci@vger.kernel.org,\n\tlinux-serial@vger.kernel.org, linux-usb@vger.kernel.org,\n linux@armlinux.org.uk,\n\tnetdev@vger.kernel.org, pabeni@redhat.com, robh@kernel.org","Subject":"Re: [PATCH v5 8/9] driver core: Replace dev->of_node_reused with\n dev_of_node_reused()","Message-ID":"","References":"<20260406232444.3117516-1-dianders@chromium.org>\n <20260406162231.v5.8.I806b8636cd3724f6cd1f5e199318ab8694472d90@changeid>","X-Mailing-List":"linux-aspeed@lists.ozlabs.org","List-Id":"","List-Help":"","List-Owner":"","List-Post":"","List-Archive":",\n ","List-Subscribe":",\n ,\n ","List-Unsubscribe":"","Precedence":"list","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Disposition":"inline","Content-Transfer-Encoding":"8bit","In-Reply-To":"\n <20260406162231.v5.8.I806b8636cd3724f6cd1f5e199318ab8694472d90@changeid>","X-Spam-Status":"No, score=-0.2 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,\n\tDKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,SPF_PASS\n\tautolearn=disabled version=4.0.1","X-Spam-Checker-Version":"SpamAssassin 4.0.1 (2024-03-25) on lists.ozlabs.org"}},{"id":3677837,"web_url":"http://patchwork.ozlabs.org/comment/3677837/","msgid":"<177629105858.764226.5689083905556313606.robh@kernel.org>","list_archive_url":null,"date":"2026-04-15T22:10:59","subject":"Re: [PATCH v5 8/9] driver core: Replace dev->of_node_reused with\n dev_of_node_reused()","submitter":{"id":62529,"url":"http://patchwork.ozlabs.org/api/people/62529/","name":"Rob Herring (Arm)","email":"robh@kernel.org"},"content":"On Mon, 06 Apr 2026 16:23:01 -0700, Douglas Anderson wrote:\n> In C, bitfields are not necessarily safe to modify from multiple\n> threads without locking. Switch \"of_node_reused\" over to the \"flags\"\n> field so modifications are safe.\n> \n> Cc: Johan Hovold \n> Acked-by: Mark Brown \n> Reviewed-by: Rafael J. Wysocki (Intel) \n> Reviewed-by: Danilo Krummrich \n> Signed-off-by: Douglas Anderson \n> ---\n> Not fixing any known bugs; problem is theoretical and found by code\n> inspection. Change is done somewhat manually and only lightly tested\n> (mostly compile-time tested).\n> \n> (no changes since v4)\n> \n> Changes in v4:\n> - Use accessor functions for flags\n> \n> Changes in v3:\n> - New\n> \n> drivers/base/core.c | 2 +-\n> drivers/base/pinctrl.c | 2 +-\n> drivers/base/platform.c | 2 +-\n> drivers/net/pcs/pcs-xpcs-plat.c | 2 +-\n> drivers/of/device.c | 6 +++---\n> drivers/pci/of.c | 2 +-\n> drivers/pci/pwrctrl/core.c | 2 +-\n> drivers/regulator/bq257xx-regulator.c | 2 +-\n> drivers/regulator/rk808-regulator.c | 2 +-\n> drivers/tty/serial/serial_base_bus.c | 2 +-\n> drivers/usb/gadget/udc/aspeed-vhub/dev.c | 2 +-\n> include/linux/device.h | 7 ++++---\n> 12 files changed, 17 insertions(+), 16 deletions(-)\n> \n\nReviewed-by: Rob Herring (Arm) ","headers":{"Return-Path":"\n ","X-Original-To":["incoming@patchwork.ozlabs.org","linux-aspeed@lists.ozlabs.org"],"Delivered-To":"patchwork-incoming@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256\n header.s=k20201202 header.b=p0G82jNv;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=lists.ozlabs.org\n (client-ip=112.213.38.117; helo=lists.ozlabs.org;\n envelope-from=linux-aspeed+bounces-3908-incoming=patchwork.ozlabs.org@lists.ozlabs.org;\n receiver=patchwork.ozlabs.org)","lists.ozlabs.org;\n arc=none smtp.remote-ip=\"2600:3c0a:e001:78e:0:1991:8:25\"","lists.ozlabs.org;\n dmarc=pass (p=quarantine dis=none) header.from=kernel.org","lists.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256\n header.s=k20201202 header.b=p0G82jNv;\n\tdkim-atps=neutral","lists.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=kernel.org\n (client-ip=2600:3c0a:e001:78e:0:1991:8:25; helo=sea.source.kernel.org;\n envelope-from=robh@kernel.org; receiver=lists.ozlabs.org)"],"Received":["from lists.ozlabs.org (lists.ozlabs.org [112.213.38.117])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fwwMY5SN5z1yCv\n\tfor ; Thu, 16 Apr 2026 08:11:05 +1000 (AEST)","from boromir.ozlabs.org (localhost [127.0.0.1])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 4fwwMX3XZNz2xlK;\n\tThu, 16 Apr 2026 08:11:04 +1000 (AEST)","from sea.source.kernel.org (sea.source.kernel.org\n [IPv6:2600:3c0a:e001:78e:0:1991:8:25])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519)\n\t(No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 4fwwMW2NPvz2xc8\n\tfor ; Thu, 16 Apr 2026 08:11:03 +1000 (AEST)","from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58])\n\tby sea.source.kernel.org (Postfix) with ESMTP id 33D90415F6;\n\tWed, 15 Apr 2026 22:11:01 +0000 (UTC)","by smtp.kernel.org (Postfix) with ESMTPSA id D9759C2BCB3;\n\tWed, 15 Apr 2026 22:11:00 +0000 (UTC)"],"ARC-Seal":"i=1; a=rsa-sha256; d=lists.ozlabs.org; s=201707; t=1776291064;\n\tcv=none;\n b=ekJx3HL0n7YJU+cDWvhQomLb0JJy/sW+pZgFnOzTZCt07jxo5BesqhSbMoE9QMwsBfvyHgmjOs+wSEnvO7w6wiDZBRrHo381pV2Mkx4CdU7wwEJx4o1cj8euh5gNNORkSGew0VifyeawAkQP6oCZaEpGKQHVNGHdwEfd25bcvjrv4RFORVFPTaWN2tsj/mPpnQhKNSXuJnFYXddBlrriKp2GYY8eVsMIsPslACa4mYOIGwjLG2DrG6kJY5IQhdz4XOrufE6MIre5IbtJX/MRVkZ3oiHmqp4xN49sIcZI/vrLDj0UXfbEM9JNYWvdal+3k8PwVzYxR8I/ylSYdo4+MA==","ARC-Message-Signature":"i=1; a=rsa-sha256; d=lists.ozlabs.org; s=201707;\n\tt=1776291064; c=relaxed/relaxed;\n\tbh=BZdrZIkOSqbzGAWUiUFdmBRnxHfqO64E/VC6vDSmexI=;\n\th=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version:\n\t Content-Type:Content-Disposition:In-Reply-To;\n b=Om/5mtS4rIFmObUXdfqo0rNw4tNf4cBiTo46XjQW1WfJTJC4AmavCrfW+OVIlpfDveor8UgPryqeNTGUFGkuPz9T9GTITe+Ik8BDk8xp+aqf12I9hwRCh9jlXDX1OKu6v7Dv5Uql44tWKqgDOVfiGm4lAcUpMdGuDTydI5G6eQyjnx0bi+Ju3brOq3mBfzOI7LPkuN3GUGYbsvzS56JxXMRTybNVqtmLOs9KZSUFsvUWd4NgqxRzh1pZFjwqNm1tfHILB5LlfXw41YFF/44XBRVAT8mNwZuAkfmolP5s/CfSkMmlOKKJak11Bov0I5n7Nw2hjsIVltg9teUrYgIbfQ==","ARC-Authentication-Results":"i=1; lists.ozlabs.org;\n dmarc=pass (p=quarantine dis=none) header.from=kernel.org;\n dkim=pass (2048-bit key;\n unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256\n header.s=k20201202 header.b=p0G82jNv; dkim-atps=neutral;\n spf=pass (client-ip=2600:3c0a:e001:78e:0:1991:8:25;\n helo=sea.source.kernel.org; envelope-from=robh@kernel.org;\n receiver=lists.ozlabs.org) smtp.mailfrom=kernel.org","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org;\n\ts=k20201202; t=1776291061;\n\tbh=4MjkOgWu+H/4IbKFLTcEqaA80AYv/7LFnNzAZ1Q9/eY=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=p0G82jNvv+WdUrzCg0FXEUgFE67KoF8lsRoHLR1Eixg/LXzBOmBt72ny96ygZ1Yh7\n\t 5pjtAjRmscQPZ3vzTHEJVK2X3F5iY9VBGqR24yPawyJQ/gAIlyZBVqDlxU6RxtLQOu\n\t GIOuEzZ4IIVyp1JVp1Bd0caHsANeP3Y9FjSY0dKokZFCp73u2B9/2SZh9VaeNP9dKw\n\t bDqy1aSrrHxSChU/DDEkPqQ/9mXmyBZFULTx7CeyP1LoiLkGahr6y7zWTmj5JAf+iF\n\t LSKM2823g0Z3h0jTPv9V+xvD6mu4GBgiebxCOKq/m8zV5ubcEy7dHDB71HDaXfInaG\n\t zk3lxaVX52GYQ==","Date":"Wed, 15 Apr 2026 17:10:59 -0500","From":"\"Rob Herring (Arm)\" ","To":"Douglas Anderson ","Cc":"astewart@tektelic.com, linux-arm-kernel@lists.infradead.org,\n\tMark Brown , bhelgaas@google.com,\n\tmaz@kernel.org, linux@armlinux.org.uk, kees@kernel.org,\n\tAlan Stern ,\n\tSaravana Kannan , netdev@vger.kernel.org,\n\tlinux-serial@vger.kernel.org, davem@davemloft.net, andrew@lunn.ch,\n\tGreg Kroah-Hartman , brgl@kernel.org,\n\tjirislaby@kernel.org, mani@kernel.org,\n\tJohan Hovold , linux-aspeed@lists.ozlabs.org,\n\tlinux-pci@vger.kernel.org, kuba@kernel.org,\n\tAlexander Lobakin ,\n\tLeon Romanovsky ,\n\tandriy.shevchenko@linux.intel.com,\n\t\"Rafael J . Wysocki\" ,\n\tAlexey Kardashevskiy , lgirdwood@gmail.com,\n\tandrew@codeconstruct.com.au, hkallweit1@gmail.com,\n\tlinux-kernel@vger.kernel.org, Danilo Krummrich ,\n\tEric Dumazet , linux-usb@vger.kernel.org,\n\talexander.stein@ew.tq-group.com,\n\tRobin Murphy , pabeni@redhat.com,\n\tdevicetree@vger.kernel.org, driver-core@lists.linux.dev,\n\tjoel@jms.id.au, Christoph Hellwig ","Subject":"Re: [PATCH v5 8/9] driver core: Replace dev->of_node_reused with\n dev_of_node_reused()","Message-ID":"<177629105858.764226.5689083905556313606.robh@kernel.org>","References":"<20260406232444.3117516-1-dianders@chromium.org>\n <20260406162231.v5.8.I806b8636cd3724f6cd1f5e199318ab8694472d90@changeid>","X-Mailing-List":"linux-aspeed@lists.ozlabs.org","List-Id":"","List-Help":"","List-Owner":"","List-Post":"","List-Archive":",\n ","List-Subscribe":",\n ,\n ","List-Unsubscribe":"","Precedence":"list","MIME-Version":"1.0","Content-Type":"text/plain; charset=us-ascii","Content-Disposition":"inline","In-Reply-To":"\n <20260406162231.v5.8.I806b8636cd3724f6cd1f5e199318ab8694472d90@changeid>","X-Spam-Status":"No, score=-0.2 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,\n\tDKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,SPF_PASS\n\tautolearn=disabled version=4.0.1","X-Spam-Checker-Version":"SpamAssassin 4.0.1 (2024-03-25) on lists.ozlabs.org"}}]