Re: [PATCH v11 05/10] cxl/test: Use fw_devlink_set_device()
From: Bartosz Golaszewski
Date: Wed Sep 09 2026 - 05:11:48 EST
On Wed, 9 Sep 2026 10:01:04 +0200, Herve Codina <herve.codina@xxxxxxxxxxx> said:
> The code set directly fwnode.dev field.
>
> Use the dedicated fw_devlink_set_device() helper to perform this
> operation.
>
> Signed-off-by: Herve Codina <herve.codina@xxxxxxxxxxx>
> Reviewed-by: Dave Jiang <dave.jiang@xxxxxxxxx>
> Reviewed-by: Jonathan Cameron <jonathan.cameron@xxxxxxxxxx>
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> Reviewed-by: Richard Cheng <icheng@xxxxxxxxxx>
> ---
> tools/testing/cxl/test/cxl.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/testing/cxl/test/cxl.c b/tools/testing/cxl/test/cxl.c
> index 547ebc97bf59..e21e45fe1572 100644
> --- a/tools/testing/cxl/test/cxl.c
> +++ b/tools/testing/cxl/test/cxl.c
> @@ -1678,7 +1678,7 @@ static void mock_companion(struct acpi_device *adev, struct device *dev)
> device_initialize(&adev->dev);
> fwnode_init(&adev->fwnode, NULL);
> device_set_node(dev, &adev->fwnode);
> - adev->fwnode.dev = dev;
> + fw_devlink_set_device(&adev->fwnode, dev);
> }
>
> #ifndef SZ_64G
> --
> 2.55.0
>
>
Reviewed-by: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxxxxxxxx>