Re: [PATCH] staging: iio: ad9834: add of_device_id table

[Jonathan Cameron]

On Sat, 18 May 2019 19:47:20 -0300
JoÃo Victor Marques de Oliveira         <joao.marques.oliveira@xxxxxx> wrote:

> Add a of_device_id struct array of_match_table variable and subsequent
> call to MODULE_DEVICE_TABLE macro to device tree support.
> 
> Co-developed-by: Thiago L. A. Miller <tmiller@xxxxxxxxxxxxx>
> Signed-off-by: Thiago L. A. Miller <tmiller@xxxxxxxxxxxxx>
> Co-developed-by: Osvaldo M. Yasuda <omyasuda@xxxxxxxxxxxx>
> Signed-off-by: Osvaldo M. Yasuda <omyasuda@xxxxxxxxxxxx>
> Signed-off-by: JoÃo Victor Marques de Oliveira <joao.marques.oliveira@xxxxxx>
Nice patch.

Applied to the togreg branch of iio.git and pushed out as testing for the
autobuilders to play with it.

Thanks,

j
> ---
>  drivers/staging/iio/frequency/ad9834.c | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/drivers/staging/iio/frequency/ad9834.c b/drivers/staging/iio/frequency/ad9834.c
> index 6de3cd7363d7..038d6732c3fd 100644
> --- a/drivers/staging/iio/frequency/ad9834.c
> +++ b/drivers/staging/iio/frequency/ad9834.c
> @@ -521,9 +521,20 @@ static const struct spi_device_id ad9834_id[] = {
>  };
>  MODULE_DEVICE_TABLE(spi, ad9834_id);
>  
> +static const struct of_device_id ad9834_of_match[] = {
> +	{.compatible = "adi,ad9833"},
> +	{.compatible = "adi,ad9834"},
> +	{.compatible = "adi,ad9837"},
> +	{.compatible = "adi,ad9838"},
> +	{}
> +};
> +
> +MODULE_DEVICE_TABLE(of, ad9834_of_match);
> +
>  static struct spi_driver ad9834_driver = {
>  	.driver = {
>  		.name	= "ad9834",
> +		.of_match_table = ad9834_of_match
>  	},
>  	.probe		= ad9834_probe,
>  	.remove		= ad9834_remove,