[adding Arnd to Cc, as this was his suggestion] On Fri, Oct 17, 2014 at 10:54:21AM +0100, Chunyan Zhang wrote: > The file of-serial.txt was only for 8250 compatible UART implementations, > so renamed it to 8250.txt to avoid confusing other persons. > > Signed-off-by: Chunyan Zhang <chunyan.zhang@xxxxxxxxxxxxxx> This makes sense to me, so: Acked-by: Mark Rutland <mark.rutland@xxxxxxx> In future please use git format-patch -C -M when moving things around. It should detect that this is a rename and generate a trivial diff, rather than featuring every line of the file twice. Thanks, Mark. > --- > Documentation/devicetree/bindings/serial/8250.txt | 50 ++++++++++++++++++++ > .../devicetree/bindings/serial/of-serial.txt | 50 -------------------- > 2 files changed, 50 insertions(+), 50 deletions(-) > create mode 100644 Documentation/devicetree/bindings/serial/8250.txt > delete mode 100644 Documentation/devicetree/bindings/serial/of-serial.txt > > diff --git a/Documentation/devicetree/bindings/serial/8250.txt b/Documentation/devicetree/bindings/serial/8250.txt > new file mode 100644 > index 0000000..7705477 > --- /dev/null > +++ b/Documentation/devicetree/bindings/serial/8250.txt > @@ -0,0 +1,50 @@ > +* UART (Universal Asynchronous Receiver/Transmitter) > + > +Required properties: > +- compatible : one of: > + - "ns8250" > + - "ns16450" > + - "ns16550a" > + - "ns16550" > + - "ns16750" > + - "ns16850" > + - "nvidia,tegra20-uart" > + - "nxp,lpc3220-uart" > + - "ibm,qpace-nwp-serial" > + - "altr,16550-FIFO32" > + - "altr,16550-FIFO64" > + - "altr,16550-FIFO128" > + - "serial" if the port type is unknown. > +- reg : offset and length of the register set for the device. > +- interrupts : should contain uart interrupt. > +- clock-frequency : the input clock frequency for the UART > + or > + clocks phandle to refer to the clk used as per Documentation/devicetree > + /bindings/clock/clock-bindings.txt > + > +Optional properties: > +- current-speed : the current active speed of the UART. > +- reg-offset : offset to apply to the mapbase from the start of the registers. > +- reg-shift : quantity to shift the register offsets by. > +- reg-io-width : the size (in bytes) of the IO accesses that should be > + performed on the device. There are some systems that require 32-bit > + accesses to the UART (e.g. TI davinci). > +- used-by-rtas : set to indicate that the port is in use by the OpenFirmware > + RTAS and should not be registered. > +- no-loopback-test: set to indicate that the port does not implements loopback > + test mode > +- fifo-size: the fifo size of the UART. > +- auto-flow-control: one way to enable automatic flow control support. The > + driver is allowed to detect support for the capability even without this > + property. > +- has-hw-flow-control: the hardware has flow control capability. > + > +Example: > + > + uart@80230000 { > + compatible = "ns8250"; > + reg = <0x80230000 0x100>; > + clock-frequency = <3686400>; > + interrupts = <10>; > + reg-shift = <2>; > + }; > diff --git a/Documentation/devicetree/bindings/serial/of-serial.txt b/Documentation/devicetree/bindings/serial/of-serial.txt > deleted file mode 100644 > index 7705477..0000000 > --- a/Documentation/devicetree/bindings/serial/of-serial.txt > +++ /dev/null > @@ -1,50 +0,0 @@ > -* UART (Universal Asynchronous Receiver/Transmitter) > - > -Required properties: > -- compatible : one of: > - - "ns8250" > - - "ns16450" > - - "ns16550a" > - - "ns16550" > - - "ns16750" > - - "ns16850" > - - "nvidia,tegra20-uart" > - - "nxp,lpc3220-uart" > - - "ibm,qpace-nwp-serial" > - - "altr,16550-FIFO32" > - - "altr,16550-FIFO64" > - - "altr,16550-FIFO128" > - - "serial" if the port type is unknown. > -- reg : offset and length of the register set for the device. > -- interrupts : should contain uart interrupt. > -- clock-frequency : the input clock frequency for the UART > - or > - clocks phandle to refer to the clk used as per Documentation/devicetree > - /bindings/clock/clock-bindings.txt > - > -Optional properties: > -- current-speed : the current active speed of the UART. > -- reg-offset : offset to apply to the mapbase from the start of the registers. > -- reg-shift : quantity to shift the register offsets by. > -- reg-io-width : the size (in bytes) of the IO accesses that should be > - performed on the device. There are some systems that require 32-bit > - accesses to the UART (e.g. TI davinci). > -- used-by-rtas : set to indicate that the port is in use by the OpenFirmware > - RTAS and should not be registered. > -- no-loopback-test: set to indicate that the port does not implements loopback > - test mode > -- fifo-size: the fifo size of the UART. > -- auto-flow-control: one way to enable automatic flow control support. The > - driver is allowed to detect support for the capability even without this > - property. > -- has-hw-flow-control: the hardware has flow control capability. > - > -Example: > - > - uart@80230000 { > - compatible = "ns8250"; > - reg = <0x80230000 0x100>; > - clock-frequency = <3686400>; > - interrupts = <10>; > - reg-shift = <2>; > - }; > -- > 1.7.9.5 > > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html