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 Thanks Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | https://www.pengutronix.de/ |