[U-Boot] [PATCH 1/6] arm64: mvebu: Modify the A8K SPI and I2C config in DTS

[Kostya Porotchkin]

Hi, Stefan,

The A0 SoC is not for production, it is engineering samples only.
Marvell has no plans for supporting this SoC in the future.
I believe that once you get a replacement for your board there will be no other customers using A0 SoC.

Regard
Kosta

________________________________________
From: Stefan Roese <sr at denx.de>
Sent: Thursday, November 24, 2016 11:02
To: Kostya Porotchkin; u-boot at lists.denx.de
Cc: Nadav Haklai; Neta Zur Hershkovits; Omri Itach; Igal Liberman; Haim Boot; Hanna Hawa
Subject: Re: [PATCH 1/6] arm64: mvebu: Modify the A8K SPI and I2C config in DTS

Hi Kosta,

On 20.11.2016 16:38, kostap at marvell.com wrote:
> From: Konstantin Porotchkin <kostap at marvell.com>
>
> Align the Armada-8040-db SPI and I2C DTS settings with latest
> A8040 DB settings:
> - disable i2c0 and spi0 on AP (pins are reserved for eMMC)
> - disable cps_i2c0 on CP1
> - enable spi1 on CP1 (the new location of the boot flash)

Thanks. I understand that the current SPI / I2C settings are
still valid for boards with earlier SoC revisions. Is this
correct? Would it make sense to move the old version into
a new file then, perhaps:

arch/arm/dts/armada-8040-db-revA.dts

?

This would be handy for users of this version at least for a
short period of time. This new file can be removed once its
not needed any more in a few months.

If you think this is a good idea, could you please add this
new file in a new patch to this patchset in v2?

> Change-Id: I54698ce4dc8dbe6a2af14099f5bcc3ca3b21d7e1
> Signed-off-by: Konstantin Porotchkin <kostap at marvell.com>
> Cc: Stefan Roese <sr at denx.de>
> Cc: Nadav Haklai <nadavh at marvell.com>
> Cc: Neta Zur Hershkovits <neta at marvell.com>
> Cc: Omri Itach <omrii at marvell.com>
> Cc: Igal Liberman <igall at marvell.com>
> Cc: Haim Boot <hayim at marvell.com>
> Cc: Hanna Hawa <hannah at marvell.com>
> ---
>  arch/arm/dts/armada-8040-db.dts | 60 +++++++++++++++++------------------------
>  1 file changed, 25 insertions(+), 35 deletions(-)
>
> diff --git a/arch/arm/dts/armada-8040-db.dts b/arch/arm/dts/armada-8040-db.dts
> index 7fb674b..86666a1 100644
> --- a/arch/arm/dts/armada-8040-db.dts
> +++ b/arch/arm/dts/armada-8040-db.dts
> @@ -57,7 +57,7 @@
>
>       aliases {
>               i2c0 = &cpm_i2c0;
> -             spi0 = &spi0;
> +             spi0 = &cps_spi1;
>       };
>
>       memory at 00000000 {
> @@ -66,38 +66,6 @@
>       };
>  };
>
> -&i2c0 {
> -     status = "okay";
> -     clock-frequency = <100000>;
> -};
> -
> -&spi0 {
> -     status = "okay";
> -
> -     spi-flash at 0 {
> -             #address-cells = <1>;
> -             #size-cells = <1>;
> -             compatible = "jedec,spi-nor";
> -             reg = <0>;
> -             spi-max-frequency = <10000000>;
> -
> -             partitions {
> -                     compatible = "fixed-partitions";
> -                     #address-cells = <1>;
> -                     #size-cells = <1>;
> -
> -                     partition at 0 {
> -                             label = "U-Boot";
> -                             reg = <0 0x200000>;
> -                     };
> -                     partition at 400000 {
> -                             label = "Filesystem";
> -                             reg = <0x200000 0xce0000>;
> -                     };
> -             };
> -     };
> -};
> -
>  /* Accessible over the mini-USB CON9 connector on the main board */
>  &uart0 {
>       status = "okay";
> @@ -134,9 +102,31 @@
>       status = "okay";
>  };
>
> -&cps_i2c0 {
> +&cps_spi1 {
>       status = "okay";
> -     clock-frequency = <100000>;
> +
> +     spi-flash at 0 {
> +             #address-cells = <1>;
> +             #size-cells = <1>;
> +             compatible = "jedec,spi-nor";
> +             reg = <0>;
> +             spi-max-frequency = <10000000>;
> +
> +             partitions {
> +                     compatible = "fixed-partitions";
> +                     #address-cells = <1>;
> +                     #size-cells = <1>;
> +
> +                     partition at 0 {
> +                             label = "U-Boot";
> +                             reg = <0 0x200000>;
> +                     };
> +                     partition at 400000 {
> +                             label = "Filesystem";
> +                             reg = <0x200000 0xce0000>;
> +                     };
> +             };
> +     };
>  };
>
>  /* CON4 on CP1 expansion */
>

Thanks,
Stefan