Re: [PATCH 1/2] dt-bindings: sound: Convert Allwinner SPDIF binding to YAML

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Apr 15, 2019 at 7:07 AM Maxime Ripard <maxime.ripard@xxxxxxxxxxx> wrote:
>
> The Allwinner SoCs feature an SPDIF controller across multiple SoC
> generations.
>
> However, earlier generations were a bit simpler than the subsequent ones,
> and for example would always have RX and TX capabilities, and no reset
> lines.
>
> In order to express this, let's create two YAML schemas instead of the free
> form text we had before.

The only difference is 'reset' is required in one? Perhaps better to
just not make 'reset' required til we figure out how to handle a
conditional like this. Seems like we discussed this and I probably
said to split things? I guess it's a judgement call depending on how
different things are.

Possibly, we could handle this case like this:

allOf:
  - $ref: allwinner,sun4i-a10-spdif.yaml

properties:
  resets:
    maxItems: 1

required:
  - resets


Plus we'd need the 'allwinner,sun6i-a31-spdif' and other compatibles
in both files. Note that you can't use 'additionalProperties: false'
in either file in this case.

I don't really love this solution though.

Rob



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux