On Wed, Aug 19, 2020 at 10:13:58PM +0530, Shubhrajyoti Datta wrote: > Add the arm,xlnx-sbsa-uart compatible. > Xilinx versal uart is similar to sbsa-uart except that it has > termios configurable. Yet still not just a pl011? Sigh. > Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@xxxxxxxxxx> > --- > Documentation/devicetree/bindings/serial/pl011.yaml | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/Documentation/devicetree/bindings/serial/pl011.yaml b/Documentation/devicetree/bindings/serial/pl011.yaml > index c23c93b..6e123b1 100644 > --- a/Documentation/devicetree/bindings/serial/pl011.yaml > +++ b/Documentation/devicetree/bindings/serial/pl011.yaml > @@ -20,6 +20,7 @@ select: > enum: > - arm,pl011 > - zte,zx296702-uart > + - arm,xlnx-sbsa-uart > required: > - compatible > > @@ -32,6 +33,7 @@ properties: > - items: > - const: zte,zx296702-uart > - const: arm,primecell > + - const: arm,xlnx-sbsa-uart You are defining compatible must be: compatible = "zte,zx296702-uart", "arm,primecell", "arm,xlnx-sbsa-uart"; Probably not what you want. And breaks ZTE.