linux-kernel - RE: [PATCH 2/2] iio:magnetometer: Support for ST magnetic sensors

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]

Message-ID: <DB7PR06MB4571380406FB0CECCDB9D1F5D7909@DB7PR06MB4571.eurprd06.prod.outlook.com>
Date:   Thu, 11 Mar 2021 07:46:29 +0000
From:   LI Qingwu <qing-wu.li@...ca-geosystems.com.cn>
To:     Andy Shevchenko <andy.shevchenko@...il.com>,
        Jonathan Cameron <jic23@...nel.org>
CC:     Lars-Peter Clausen <lars@...afoo.de>,
        Peter Meerwald <pmeerw@...erw.net>,
        Rob Herring <robh+dt@...nel.org>,
        Denis Ciocca <denis.ciocca@...com>,
        linux-iio <linux-iio@...r.kernel.org>,
        devicetree <devicetree@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        TERTYCHNYI Grygorii <grygorii.tertychnyi@...ca-geosystems.com>,
        ZHIZHIKIN Andrey <andrey.zhizhikin@...ca-geosystems.com>,
        Lorenzo Bianconi <lorenzo.bianconi@...hat.com>
Subject: RE: [PATCH 2/2] iio:magnetometer: Support for ST magnetic sensors

Hi Andy,

Thanks a lot for your input, I just extend the existing driver in patch V2 for the sersor.



Best Regards

**********************************************
Li Qingwu (Terry)
Senior Embedded Software Engineer 
Leica Geosystems(Shanghai)Co.,Limited
(Tel): +86 21 61061036
(FAX): +86 21 61061008
(Mobile): +86 187 0185 9600
E-mail: Qing-wu.Li@...ca-geosystems.com.cn
Http: www.leica-geosystems.com.cn
**********************************************

-----Original Message-----
From: Andy Shevchenko <andy.shevchenko@...il.com> 
Sent: Tuesday, March 9, 2021 7:01 PM
To: Jonathan Cameron <jic23@...nel.org>
Cc: LI Qingwu <qing-wu.li@...ca-geosystems.com.cn>; Lars-Peter Clausen <lars@...afoo.de>; Peter Meerwald <pmeerw@...erw.net>; Rob Herring <robh+dt@...nel.org>; Denis Ciocca <denis.ciocca@...com>; linux-iio <linux-iio@...r.kernel.org>; devicetree <devicetree@...r.kernel.org>; Linux Kernel Mailing List <linux-kernel@...r.kernel.org>; TERTYCHNYI Grygorii <grygorii.tertychnyi@...ca-geosystems.com>; ZHIZHIKIN Andrey <andrey.zhizhikin@...ca-geosystems.com>; Lorenzo Bianconi <lorenzo.bianconi@...hat.com>
Subject: Re: [PATCH 2/2] iio:magnetometer: Support for ST magnetic sensors

This email is not from Hexagon’s Office 365 instance. Please be careful while clicking links, opening attachments, or replying to this email.


On Sat, Mar 6, 2021 at 6:44 PM Jonathan Cameron <jic23@...nel.org> wrote:
> On Fri,  5 Mar 2021 07:05:36 +0000
> LI Qingwu <Qing-wu.Li@...ca-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.Li@...ca-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