On Sat, Mar 6, 2021 at 6:44 PM Jonathan Cameron <ji...@kernel.org> wrote: > On Fri, 5 Mar 2021 07:05:36 +0000 > LI Qingwu <qing-wu...@leica-geosystems.com.cn> wrote: > > > Add support for STMicroelectronics digital magnetic sensors, > > LSM303AH,LSM303AGR,LIS2MDL,ISM303DAC,IIS2MDC. > > > > The patch tested with IIS2MDC instrument. > > > > Signed-off-by: LI Qingwu <qing-wu...@leica-geosystems.com.cn> > > Hi, > > Given that at least two parts in here is supported by the existing > driver in > iio/magnetometers/st_magn_*.c (lsm303agr) can you confirm that it > doesn't make sense to simply extend that driver to support the > other parts? This is particularly true when the WHO AM I register > reads 0x40 for all these parts. > > I've done a fairly superficial review whilst here, but please check > you can't just add the relevant entries to the existing driver.
I even hadn't looked into the code because this one needs a very good justification why it's a new driver rather than extension of the existing one. -- With Best Regards, Andy Shevchenko
