Hi Jonathan,
Am 19.03.23 um 16:42 schrieb Jonathan Cameron:
On Sun, 19 Mar 2023 13:40:49 +0100
Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> wrote:
On 18/03/2023 16:55, Jonathan Cameron wrote:
On Fri, 17 Mar 2023 11:33:19 +0100
Stefan Wahren <stefan.wahren@xxxxxxxxxxxxxx> wrote:
The ST IIS328DQ is an accelerometer sensor, which is compatible with
the already supported ST H3LIS331DL. So add the new compatible
with a fallback to the ST sensor binding.
Link: https://lore.kernel.org/linux-iio/2bac9ecf-9d2e-967e-9020-1c950487d781@xxxxxxxx/
Suggested-by: Jonathan Cameron <jic23@xxxxxxxxxx>
Signed-off-by: Stefan Wahren <stefan.wahren@xxxxxxxxxxxxxx>
---
Documentation/devicetree/bindings/iio/st,st-sensors.yaml | 3 +++
1 file changed, 3 insertions(+)
diff --git a/Documentation/devicetree/bindings/iio/st,st-sensors.yaml b/Documentation/devicetree/bindings/iio/st,st-sensors.yaml
index c6201976378f..5eb71b24a7cb 100644
--- a/Documentation/devicetree/bindings/iio/st,st-sensors.yaml
+++ b/Documentation/devicetree/bindings/iio/st,st-sensors.yaml
@@ -48,6 +48,9 @@ properties:
- st,lsm330d-accel
- st,lsm330dl-accel
- st,lsm330dlc-accel
+ - items:
+ - const: st,iis328dq
+ - const: st,h3lis331dl-accel
Hmm. So this separates the particular 'oneOf' group from the description
associated with the ST accelerometers.
DT experts, is there a better way to do this? Ideally we'd also associate
other similar groups going forwards so it might become less and less obvious
how the groupings work. (bit late for the many groups that exist
already in that list above this point).
If by "grouping" you mean how the oneOf items are organized with
descriptions, I wouldn't put too much attention to it. You can add some
'#' comments to separate the groups, but for me this is still readable.
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
Works for me. Thanks for quick response! Applied this and next patch
to the togreg branch of iio.git
Shawn had some comments to patch #7. Just to make it clear, there is no
need to resend patch 4 and 5 in V3?
Best regards
Thanks,
Jonathan
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
