Re: [PATCH 1/5] cxl/acpi: Autoload driver for 'cxl_acpi' test devices

["Verma, Vishal L" <vishal.l.verma@intel.com>]

On Fri, 2022-07-22 at 17:55 -0700, Dan Williams wrote:
> In support of CXL unit tests in the ndctl project, arrange for the
> cxl_acpi driver to load in response to the registration of cxl_test
> devices.
> 
> Reported-by: Dave Jiang <dave.jiang@intel.com>
> Tested-by: Dave Jiang <dave.jiang@intel.com>
> Signed-off-by: Dan Williams <dan.j.williams@intel.com>
> ---
>  drivers/cxl/acpi.c |    7 +++++++
>  1 file changed, 7 insertions(+)
> 

Looks good, 

Reviewed-by: Vishal Verma <vishal.l.verma@intel.com>

> diff --git a/drivers/cxl/acpi.c b/drivers/cxl/acpi.c
> index 64004eb672d0..eb436268b92c 100644
> --- a/drivers/cxl/acpi.c
> +++ b/drivers/cxl/acpi.c
> @@ -515,12 +515,19 @@ static const struct acpi_device_id
> cxl_acpi_ids[] = {
>  };
>  MODULE_DEVICE_TABLE(acpi, cxl_acpi_ids);
>  
> +static const struct platform_device_id cxl_test_ids[] = {
> +       { "cxl_acpi" },
> +       { },
> +};
> +MODULE_DEVICE_TABLE(platform, cxl_test_ids);
> +
>  static struct platform_driver cxl_acpi_driver = {
>         .probe = cxl_acpi_probe,
>         .driver = {
>                 .name = KBUILD_MODNAME,
>                 .acpi_match_table = cxl_acpi_ids,
>         },
> +       .id_table = cxl_test_ids,
>  };
>  
>  module_platform_driver(cxl_acpi_driver);
>