Hi Parthiban

On Sun, 2019-04-07 at 20:32 +0200, Parthiban Nallathambi wrote:
> u-boot,dm-spl property is specific to U-Boot, so created one
> for i.MX6ULL platforms.

What exactly makes you so sure others do want your exact configuration
thereof as well? Shouldn't you make this board specific instead like
e.g. done elsewhere [1]?

[1] 
https://git.denx.de/?p=u-boot.git;a=blob;f=arch/arm/dts/am335x-chiliboard-u-boot.dtsi;h=4f9d3080394be001f79731b736c5eac934c07ebf;hb=HEAD

Cheers

Marcel

> Signed-off-by: Parthiban Nallathambi <[email protected]>
> ---
>  arch/arm/dts/imx6ull-u-boot.dtsi | 34
> ++++++++++++++++++++++++++++++++
>  board/phytec/pcl063/MAINTAINERS  |  1 +
>  2 files changed, 35 insertions(+)
>  create mode 100644 arch/arm/dts/imx6ull-u-boot.dtsi
> 
> diff --git a/arch/arm/dts/imx6ull-u-boot.dtsi b/arch/arm/dts/imx6ull-
> u-boot.dtsi
> new file mode 100644
> index 0000000000..74ca95fa2c
> --- /dev/null
> +++ b/arch/arm/dts/imx6ull-u-boot.dtsi
> @@ -0,0 +1,34 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +/*
> + * Copyright (C) 2019 Parthiban Nallathambi <[email protected]>
> + */
> +
> +/ {
> +     soc {
> +             u-boot,dm-spl;
> +     };
> +};
> +
> +&aips1 {
> +     u-boot,dm-spl;
> +};
> +
> +&aips2 {
> +     u-boot,dm-spl;
> +};
> +
> +&aips3 {
> +     u-boot,dm-spl;
> +};
> +
> +&gpio1 {
> +     u-boot,dm-spl;
> +};
> +
> +&gpio4 {
> +     u-boot,dm-spl;
> +};
> +
> +&iomuxc {
> +     u-boot,dm-spl;
> +};
> diff --git a/board/phytec/pcl063/MAINTAINERS
> b/board/phytec/pcl063/MAINTAINERS
> index 70e03cfe71..7d9978c339 100644
> --- a/board/phytec/pcl063/MAINTAINERS
> +++ b/board/phytec/pcl063/MAINTAINERS
> @@ -3,6 +3,7 @@ M:    Martyn Welch <[email protected]>
>  M:   Parthiban Nallathambi <[email protected]>
>  S:   Maintained
>  F:   arch/arm/dts/imx6ul-phycore-segin.dts
> +F:   arch/arm/dts/imx6ull-u-boot.dtsi
>  F:   arch/arm/dts/imx6ull-phycore-segin.dts
>  F:   arch/arm/dts/pcl063-common.dtsi
>  F:   board/phytec/pcl063/
_______________________________________________
U-Boot mailing list
[email protected]
https://lists.denx.de/listinfo/u-boot

Reply via email to