On Thu, Jan 10, 2019 at 04:20:17PM -0600, Rob Herring wrote: > The renesas,rzn1-uart binding only differs in compatible string from the > snps-dw-apb-uart binding. Move it there, converting it to json-schema in > the process. This confused me slightly as this patch removes renesas,rzn1-uart.txt but doesn't add the renesas bindings elsewhere. I now see that is done in patch 1/2, which is fine. But it might be worth rewording the changelog. That notwithstanding: Reviewed-by: Simon Horman <horms+renesas@xxxxxxxxxxxx> > Cc: Phil Edworthy <phil.edworthy@xxxxxxxxxxx> > Cc: Simon Horman <horms+renesas@xxxxxxxxxxxx> > Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > Cc: linux-serial@xxxxxxxxxxxxxxx > Signed-off-by: Rob Herring <robh@xxxxxxxxxx> > --- > .../devicetree/bindings/serial/renesas,rzn1-uart.txt | 10 ---------- > 1 file changed, 10 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/serial/renesas,rzn1-uart.txt > > diff --git a/Documentation/devicetree/bindings/serial/renesas,rzn1-uart.txt b/Documentation/devicetree/bindings/serial/renesas,rzn1-uart.txt > deleted file mode 100644 > index 8b9e0d4dc2e4..000000000000 > --- a/Documentation/devicetree/bindings/serial/renesas,rzn1-uart.txt > +++ /dev/null > @@ -1,10 +0,0 @@ > -Renesas RZ/N1 UART > - > -This controller is based on the Synopsys DesignWare ABP UART and inherits all > -properties defined in snps-dw-apb-uart.txt except for the compatible property. > - > -Required properties: > -- compatible : The device specific string followed by the generic RZ/N1 string. > - Therefore it must be one of: > - "renesas,r9a06g032-uart", "renesas,rzn1-uart" > - "renesas,r9a06g033-uart", "renesas,rzn1-uart" > -- > 2.19.1 >
