[U-Boot] [PATCH v4 14/15] tegra: dts: Add serial port details

Stephen Warren swarren at wwwdotorg.org
Fri Aug 15 19:10:44 CEST 2014


On 08/11/2014 09:27 AM, Simon Glass wrote:
> Some Tegra device tree files do not include information about the serial
> ports. Add this and also add information about the input clock speed.
>
> The console alias needs to be set up to indicate which port is used for
> the console.
>
> Also add a binding file since this is missing.

> diff --git a/arch/arm/dts/tegra114-dalmore.dts b/arch/arm/dts/tegra114-dalmore.dts

>   	aliases {
> +		console = &uart_d;

Did the use of "console" get OK'd on the devicetree mailing list then?

I know we have this bug in the kernel DTs, but let's fix it here: 
phandle values should have <> around them:

		console = <&uart_d>;

> diff --git a/arch/arm/dts/tegra114.dtsi b/arch/arm/dts/tegra114.dtsi

> +	uart_a: serial at 70006000 {
> +		compatible = "nvidia,tegra20-uart";
> +		reg = <0x70006000 0x40>;
> +		reg-shift = <2>;
> +		interrupts = < 68 >;
> +	};

In order to prevent the kernel and U-Boot DT files from diverging, can 
you copy the entire DT content from the kernel rather than just a 
subset, i.e.:

         uarta: serial at 70006000 {
                 compatible = "nvidia,tegra114-uart", "nvidia,tegra20-uart";
                 reg = <0x70006000 0x40>;
                 reg-shift = <2>;
                 interrupts = <GIC_SPI 36 IRQ_TYPE_LEVEL_HIGH>;
                 clocks = <&tegra_car TEGRA114_CLK_UARTA>;
                 resets = <&tegra_car 6>;
                 reset-names = "serial";
                 dmas = <&apbdma 8>, <&apbdma 8>;
                 dma-names = "rx", "tx";
                 status = "disabled";
         };


More information about the U-Boot mailing list