On Sun, 28 Jun 2020 17:52:21 +0200 Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> wrote: > Hallo Jonathan, > > On Sun, Jun 28, 2020 at 01:36:38PM +0100, Jonathan Cameron wrote: > > @@ -96,7 +96,7 @@ MODULE_DEVICE_TABLE(of, ltc2496_of_match); > > static struct spi_driver ltc2496_driver = { > > .driver = { > > .name = "ltc2496", > > - .of_match_table = of_match_ptr(ltc2496_of_match), > > + .of_match_table = ltc2496_of_match, > > The usage of of_match_ptr would only make sense here if ltc2496_of_match > was defined conditionally. As this isn't the case: > > Reviewed-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> Thanks, Applied to the togreg branch of iio.git and pushed out as testing to see if we missed anything. Thanks, Jonathan > > Thanks > Uwe >