[PATCH V2 4/8] dt-bindings: iio: st-sensors: Add IIS328DQ accelerometer

[Jonathan Cameron]

On Fri, 17 Mar 2023 11:33:19 +0100
Stefan Wahren <stefan.wahren at chargebyte.com> 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@i2se.com/
> Suggested-by: Jonathan Cameron <jic23 at kernel.org>
> Signed-off-by: Stefan Wahren <stefan.wahren at chargebyte.com>	
> ---
>  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 not I'm fine with it being fairly obvious what is going on in the binding file
current and we can rethink it later.

>        - description: Silan Accelerometers
>          enum:
>            - silan,sc7a20