| Message ID | 1413539665-11484-2-git-send-email-chunyan.zhang@spreadtrum.com |
|---|---|
| State | Superseded, archived |
| Headers | show |
[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@spreadtrum.com> This makes sense to me, so: Acked-by: Mark Rutland <mark.rutland@arm.com> 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Friday 17 October 2014 17:54:21 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@spreadtrum.com> > --- > Documentation/devicetree/bindings/serial/8250.txt | 50 ++++++++++++++++++++ > .../devicetree/bindings/serial/of-serial.txt | 50 -------------------- > 2 files changed, 50 insertions(+), 50 deletions(-) > Patch looks good, but please use 'git format-patch -M' when generating it, to avoid having the whole add/remove in the patch, for easier review. Arnd -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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>; - };
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@spreadtrum.com> --- 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