Re: [PATCH v2] devicetree: serial: Add documentation for imx serial

Re: [PATCH v2] devicetree: serial: Add documentation for imx serial

[Rob Herring]

On 08/21/2012 12:04 PM, Fabio Estevam wrote:
> Add documentation for imx serial.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
> ---

Applied for 3.7.

Rob

> Changes since v1:
> - Add 'alias' entry
>  .../devicetree/bindings/serial/fsl-imx-uart.txt    |   35 ++++++++++++++++++++
>  1 file changed, 35 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/serial/fsl-imx-uart.txt
> 
> diff --git a/Documentation/devicetree/bindings/serial/fsl-imx-uart.txt b/Documentation/devicetree/bindings/serial/fsl-imx-uart.txt
> new file mode 100644
> index 0000000..c58573b
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/serial/fsl-imx-uart.txt
> @@ -0,0 +1,35 @@
> +* Freescale i.MX UART controller
> +
> +Required properties:
> +- compatible : should be "fsl,imx21-uart"
> +- reg : Address and length of the register set for the device
> +- interrupts : Should contain UART interrupt number
> +
> +Optional properties:
> +- fsl,uart-has-rtscts: indicate that RTS/CTS signals are used
> +
> +Note: Each uart controller should have an alias correctly numbered
> +in "aliases" node.
> +
> +Example:
> +
> +- From imx51.dtsi:
> +aliases {
> +	serial0 = &uart1;
> +	serial1 = &uart2;
> +	serial2 = &uart3;
> +};
> +
> +uart1: serial@73fbc000 {
> +	compatible = "fsl,imx51-uart", "fsl,imx21-uart";
> +	reg = <0x73fbc000 0x4000>;
> +	interrupts = <31>;
> +	status = "disabled";
> +}
> +
> +- From imx51-babbage.dts:
> +uart1: serial@73fbc000 {
> +	fsl,uart-has-rtscts;
> +	status = "okay";
> +};
> +
> 

Re: [PATCH v2] devicetree: serial: Add documentation for imx serial

[Shawn Guo]

On Tue, Aug 21, 2012 at 02:04:59PM -0300, Fabio Estevam wrote:
> Add documentation for imx serial.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
> ---
> Changes since v1:
> - Add 'alias' entry
>  .../devicetree/bindings/serial/fsl-imx-uart.txt    |   35 ++++++++++++++++++++
>  1 file changed, 35 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/serial/fsl-imx-uart.txt

Isn't there already one, Documentation/devicetree/bindings/tty/serial/fsl-imx-uart.txt?

It looks we should merge the following two folders.

- Documentation/devicetree/bindings/serial/
- Documentation/devicetree/bindings/tty/serial/

-- 
Regards,
Shawn