diff mbox series

[v2,6/6] dt-bindings: iio: st-sensors: Add LSM303D accelerometer+magnetometer

Message ID 20230413024013.450165-7-mail@mariushoch.de (mailing list archive)
State Changes Requested
Headers show
Series iio: st_sensors: Add lsm303d support | expand

Commit Message

Marius Hoch April 13, 2023, 2:40 a.m. UTC
Same as the lsm9ds0, except that the lsm303d doesn't
feature a gyroscope.

Signed-off-by: Marius Hoch <mail@mariushoch.de>
---
 Documentation/devicetree/bindings/iio/st,st-sensors.yaml | 1 +
 1 file changed, 1 insertion(+)

Comments

Krzysztof Kozlowski April 13, 2023, 8:07 a.m. UTC | #1
On 13/04/2023 04:40, Marius Hoch wrote:
> Same as the lsm9ds0, except that the lsm303d doesn't
> feature a gyroscope.
> 
> Signed-off-by: Marius Hoch <mail@mariushoch.de>

Please use scripts/get_maintainers.pl to get a list of necessary people
and lists to CC.  It might happen, that command when run on an older
kernel, gives you outdated entries.  Therefore please be sure you base
your patches on recent Linux kernel.

Best regards,
Krzysztof
Jonathan Cameron April 15, 2023, 4:51 p.m. UTC | #2
On Thu, 13 Apr 2023 10:07:04 +0200
Krzysztof Kozlowski <krzk@kernel.org> wrote:

> On 13/04/2023 04:40, Marius Hoch wrote:
> > Same as the lsm9ds0, except that the lsm303d doesn't
> > feature a gyroscope.
> > 
> > Signed-off-by: Marius Hoch <mail@mariushoch.de>  
> 
> Please use scripts/get_maintainers.pl to get a list of necessary people
> and lists to CC.  It might happen, that command when run on an older
> kernel, gives you outdated entries.  Therefore please be sure you base
> your patches on recent Linux kernel.

Just to avoid any confusion.  Point here is that the dt-binding
maintainers and list should be cc'd on series that touch bindings.

Jonathan

> 
> Best regards,
> Krzysztof
>
Marius Hoch April 15, 2023, 11:19 p.m. UTC | #3
On 15/04/2023 18:51, Jonathan Cameron wrote:
> On Thu, 13 Apr 2023 10:07:04 +0200
> Krzysztof Kozlowski <krzk@kernel.org> wrote:
>
>> On 13/04/2023 04:40, Marius Hoch wrote:
>>> Same as the lsm9ds0, except that the lsm303d doesn't
>>> feature a gyroscope.
>>>
>>> Signed-off-by: Marius Hoch <mail@mariushoch.de>
>> Please use scripts/get_maintainers.pl to get a list of necessary people
>> and lists to CC.  It might happen, that command when run on an older
>> kernel, gives you outdated entries.  Therefore please be sure you base
>> your patches on recent Linux kernel.
> Just to avoid any confusion.  Point here is that the dt-binding
> maintainers and list should be cc'd on series that touch bindings.
>
> Jonathan
Thank you for your replies. Not sure what went wrong here initially, I 
probably only ran get_maintainer.pl on some of the patches.

For v3 I used a complete and up to date list of recipients.

Cheers,
Marius
>
>> Best regards,
>> Krzysztof
>>
Krzysztof Kozlowski April 16, 2023, 7:14 a.m. UTC | #4
On 15/04/2023 18:51, Jonathan Cameron wrote:
> On Thu, 13 Apr 2023 10:07:04 +0200
> Krzysztof Kozlowski <krzk@kernel.org> wrote:
> 
>> On 13/04/2023 04:40, Marius Hoch wrote:
>>> Same as the lsm9ds0, except that the lsm303d doesn't
>>> feature a gyroscope.
>>>
>>> Signed-off-by: Marius Hoch <mail@mariushoch.de>  
>>
>> Please use scripts/get_maintainers.pl to get a list of necessary people
>> and lists to CC.  It might happen, that command when run on an older
>> kernel, gives you outdated entries.  Therefore please be sure you base
>> your patches on recent Linux kernel.
> 
> Just to avoid any confusion.  Point here is that the dt-binding
> maintainers and list should be cc'd on series that touch bindings.
> 

Maybe not only there, maybe more people were missed.

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/iio/st,st-sensors.yaml b/Documentation/devicetree/bindings/iio/st,st-sensors.yaml
index c6201976378f..194aca5542c2 100644
--- a/Documentation/devicetree/bindings/iio/st,st-sensors.yaml
+++ b/Documentation/devicetree/bindings/iio/st,st-sensors.yaml
@@ -85,6 +85,7 @@  properties:
       - description: IMUs
         enum:
           - st,lsm9ds0-imu
+          - st,lsm303d-imu
       - description: Deprecated bindings
         enum:
           - st,lis302dl-spi