Re: [PATCH 06/40] dt-bindings: mfd: ti,am3359-tscadc: Describe am4372 MFD compatible

Jonathan Cameron
2021-08-30T12:31:29Z

On Wed, 25 Aug 2021 17:24:44 +0200
Miquel Raynal <miquel.raynal@bootlin.com> wrote:

> A more recent version of the am3359 ADC/Touchscreen controller is
> available on certain SoCs of the am437x family. This IP has evolved a
> little bit and deserves its own compatible.
>
> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
> ---
> .../devicetree/bindings/mfd/ti,am3359-tscadc.yaml | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml b/Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml
> index 96b329508d8a..2240299dcf2f 100644
> --- a/Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml
> +++ b/Documentation/devicetree/bindings/mfd/ti,am3359-tscadc.yaml
> @@ -11,7 +11,9 @@ maintainers:
>
> properties:
> compatible:
> - const: ti,am3359-tscadc
> + oneOf:
> + - const: ti,am3359-tscadc
> + - const: ti,am4372-magadc
enum:
- ti,am3359-tscadc
- ti,am4372-magadc

is a little bit cleaner and means the same thing.
>
> reg:
> maxItems: 1
> @@ -42,6 +44,9 @@ patternProperties:
> "^tsc$":
> description: Touchscreen controller
>
> + "^mag$":
> + description: Magnetic reader
> +
> required:
> - compatible
> - reg