Yo, On Wed, Sep 20, 2023 at 09:52:58PM -0300, Fabio Estevam wrote: > imx28.dtsi uses the following description: > > compatible = "fsl,imx28-auart", "fsl,imx23-auart"; > > Make it valid so that the following schema warning could be avoided: Dumb question maybe, but it is not mentioned here - they are actually compatible devices, right? I see that they have some different match data in the driver (relating to some DMA erratum on the 23 seemingly), so it's not immediately obvious to me. Thanks, Conor. > > imx28-apx4devkit.dtb: serial@8006a000: compatible: ['fsl,imx28-auart', 'fsl,imx23-auart'] is too long > from schema $id: http://devicetree.org/schemas/serial/fsl-mxs-auart.yaml# > > Signed-off-by: Fabio Estevam <festevam@xxxxxxxxx> > --- > .../devicetree/bindings/serial/fsl-mxs-auart.yaml | 13 ++++++++----- > 1 file changed, 8 insertions(+), 5 deletions(-) > > diff --git a/Documentation/devicetree/bindings/serial/fsl-mxs-auart.yaml b/Documentation/devicetree/bindings/serial/fsl-mxs-auart.yaml > index 6a400a5e6fc7..da032effffe6 100644 > --- a/Documentation/devicetree/bindings/serial/fsl-mxs-auart.yaml > +++ b/Documentation/devicetree/bindings/serial/fsl-mxs-auart.yaml > @@ -14,10 +14,13 @@ allOf: > > properties: > compatible: > - enum: > - - fsl,imx23-auart > - - fsl,imx28-auart > - - alphascale,asm9260-auart > + oneOf: > + - const: fsl,imx23-auart > + - const: alphascale,asm9260-auart > + - items: > + - enum: > + - fsl,imx28-auart > + - const: fsl,imx23-auart > > reg: > maxItems: 1 > @@ -82,7 +85,7 @@ examples: > }; > > auart0: serial@8006a000 { > - compatible = "fsl,imx28-auart"; > + compatible = "fsl,imx28-auart", "fsl,imx23-auart"; > reg = <0x8006a000 0x2000>; > interrupts = <112>; > dmas = <&dma_apbx 8>, <&dma_apbx 9>; > -- > 2.34.1 >
Attachment:
signature.asc
Description: PGP signature