[U-Boot] [PATCH v5 05/11] exynos: dts: Adjust device tree files for U-Boot

Tom Rini trini at ti.com
Mon Jul 28 22:27:34 CEST 2014


On Mon, Jul 28, 2014 at 06:11:32AM -0600, Simon Glass wrote:

> The pinctrl bindings used by Linux are an incomplete description of the
> hardware. It is possible in most cases to determine the register address
> of each, but not in all cases. By adding an additional property we can
> fix this, and avoid adding a table to U-Boot for every single Exynos
> SOC.

So here's my fear..

[snip]
> @@ -49,7 +57,7 @@
>  	i2c at 12ca0000 {
>  		#address-cells = <1>;
>  		#size-cells = <0>;
> -		compatible = "samsung,s3c2440-i27c";
> +		compatible = "samsung,s3c2440-i2c";
>  		reg = <0x12CA0000 0x100>;
>  		interrupts = <0 60 0>;
>  	};

Except for the above (what's going on? pulling in a typo fix from
upstream?) they're legal "regular" non-U-Boot-prefixed changes.  Are
they going back into the master copy in Linux?

> diff --git a/arch/arm/dts/exynos5420-pinctrl.dtsi b/arch/arm/dts/exynos5420-pinctrl.dtsi
> index b3e63d1..df31f37 100644
> --- a/arch/arm/dts/exynos5420-pinctrl.dtsi
> +++ b/arch/arm/dts/exynos5420-pinctrl.dtsi
> @@ -13,6 +13,18 @@
>  */
>  
>  / {
> +	/* Replicate the ordering of arch/arm/include/asm/arch-exynos/gpio.h */
> +	pinctrl at 14010000 {
> +	};
> +	pinctrl at 13400000 {
> +	};
> +	pinctrl at 13410000 {
> +	};
> +	pinctrl at 14000000 {
> +	};
> +	pinctrl at 03860000 {
> +	};

So this isn't going to head back to Linux, clearly...

Is there some way we can contain our changes under includes perhaps?

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140728/2f308976/attachment.pgp>


More information about the U-Boot mailing list