linux-kernel - Re: [PATCH] staging: iio: adis16203: Add of_device

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Sun, 19 May 2019 11:42:26 +0100
From:   Jonathan Cameron <jic23@...nel.org>
To:     Thiago Estrela <thiagestrela@...il.com>
Cc:     Lars-Peter Clausen <lars@...afoo.de>,
        Michael Hennerich <Michael.Hennerich@...log.com>,
        Stefan Popa <stefan.popa@...log.com>,
        Hartmut Knaack <knaack.h@....de>,
        Peter Meerwald-Stadler <pmeerw@...erw.net>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        linux-iio@...r.kernel.org, devel@...verdev.osuosl.org,
        linux-kernel@...r.kernel.org,
        Tiago Napoli <napoli.tiago96@...il.com>,
        Pedro Sousa <pedroteosousa@...il.com>
Subject: Re: [PATCH] staging: iio: adis16203: Add of_device_id table

On Sat, 18 May 2019 19:42:13 -0300
Thiago Estrela <thiagestrela@...il.com> wrote:

> Accomplish device tree compatibility to driver ADIS16203
> by adding of_device_id table and making a subsequent call to
> MODULE_DEVICE_TABLE.
> 
> Signed-off-by: Thiago Estrela <thiagestrela@...il.com>
> Signed-off-by: Tiago Napoli <napoli.tiago96@...il.com>
> Co-developed-by: Tiago Napoli <napoli.tiago96@...il.com>
> Signed-off-by: Pedro Sousa <pedroteosousa@...il.com>
> Co-developed-by: Pedro Sousa <pedroteosousa@...il.com>
> Reviewed-by: Matheus Tavares <matheus.bernardino@....br>
> Reviewed-by: Marcelo Schmitt <marcelo.schmitt1@...il.com>
Another nice patch.  Certainly seems like the dev day was
successful and welcome to so many new people.

Applied to the togreg branch of iio.git and pushed out as
testing for the autobuilders to see if we missed anything.

Thanks,

Jonathan

> ---
>  drivers/staging/iio/accel/adis16203.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/drivers/staging/iio/accel/adis16203.c b/drivers/staging/iio/accel/adis16203.c
> index 70381756a64a..a5d974ac2e3b 100644
> --- a/drivers/staging/iio/accel/adis16203.c
> +++ b/drivers/staging/iio/accel/adis16203.c
> @@ -311,9 +311,17 @@ static int adis16203_remove(struct spi_device *spi)
>  	return 0;
>  }
>  
> +static const struct of_device_id adis16203_of_match[] = {
> +	{ .compatible = "adi,adis16203" },
> +	{ },
> +};
> +
> +MODULE_DEVICE_TABLE(of, adis16203_of_match);
> +
>  static struct spi_driver adis16203_driver = {
>  	.driver = {
>  		.name = "adis16203",
> +		.of_match_table = adis16203_of_match,
>  	},
>  	.probe = adis16203_probe,
>  	.remove = adis16203_remove,