[U-Boot] [PATCH] ARM: mvebu: add "spi-flash" compatible string

U-boots spi-nor support is currently considered a work in progress. For now to avoid issues it is necessary to add a "spi-flash" compatible string. Eventually the "jedec,spi-nor" will be sufficient when the core U-boot code is updated to support it.
Signed-off-by: Chris Packham judge.packham@gmail.com --- This is an alternative to https://lists.denx.de/pipermail/u-boot/2017-August/301816.html. Using the u-boot specific "spi-flash" is Jagan's preference while the spi-nor support is still a WIP and works for getting the db-88f6820-amc working for me so I'm happy.
arch/arm/dts/armada-385-amc.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/dts/armada-385-amc.dts b/arch/arm/dts/armada-385-amc.dts index 4649c910c7b0..5e1588d57438 100644 --- a/arch/arm/dts/armada-385-amc.dts +++ b/arch/arm/dts/armada-385-amc.dts @@ -154,7 +154,7 @@ u-boot,dm-pre-reloc; #address-cells = <1>; #size-cells = <1>; - compatible = "st,m25p128", "jedec,spi-nor"; + compatible = "st,m25p128", "jedec,spi-nor", "spi-flash"; reg = <0>; /* Chip select 0 */ spi-max-frequency = <50000000>; m25p,fast-read;

On Tue, Aug 29, 2017 at 2:07 PM, Chris Packham judge.packham@gmail.com wrote:
U-boots spi-nor support is currently considered a work in progress. For now to avoid issues it is necessary to add a "spi-flash" compatible string. Eventually the "jedec,spi-nor" will be sufficient when the core U-boot code is updated to support it.
Signed-off-by: Chris Packham judge.packham@gmail.com
This is an alternative to https://lists.denx.de/pipermail/u-boot/2017-August/301816.html. Using the u-boot specific "spi-flash" is Jagan's preference while the spi-nor support is still a WIP and works for getting the db-88f6820-amc working for me so I'm happy.
arch/arm/dts/armada-385-amc.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/dts/armada-385-amc.dts b/arch/arm/dts/armada-385-amc.dts index 4649c910c7b0..5e1588d57438 100644 --- a/arch/arm/dts/armada-385-amc.dts +++ b/arch/arm/dts/armada-385-amc.dts @@ -154,7 +154,7 @@ u-boot,dm-pre-reloc; #address-cells = <1>; #size-cells = <1>;
compatible = "st,m25p128", "jedec,spi-nor";
compatible = "st,m25p128", "jedec,spi-nor", "spi-flash";
Reviewed-by: Jagan Teki jagan@openedev.com

On 30.08.2017 08:37, Jagan Teki wrote:
On Tue, Aug 29, 2017 at 2:07 PM, Chris Packham judge.packham@gmail.com wrote:
U-boots spi-nor support is currently considered a work in progress. For now to avoid issues it is necessary to add a "spi-flash" compatible string. Eventually the "jedec,spi-nor" will be sufficient when the core U-boot code is updated to support it.
Signed-off-by: Chris Packham judge.packham@gmail.com
This is an alternative to https://lists.denx.de/pipermail/u-boot/2017-August/301816.html. Using the u-boot specific "spi-flash" is Jagan's preference while the spi-nor support is still a WIP and works for getting the db-88f6820-amc working for me so I'm happy.
arch/arm/dts/armada-385-amc.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/dts/armada-385-amc.dts b/arch/arm/dts/armada-385-amc.dts index 4649c910c7b0..5e1588d57438 100644 --- a/arch/arm/dts/armada-385-amc.dts +++ b/arch/arm/dts/armada-385-amc.dts @@ -154,7 +154,7 @@ u-boot,dm-pre-reloc; #address-cells = <1>; #size-cells = <1>;
compatible = "st,m25p128", "jedec,spi-nor";
compatible = "st,m25p128", "jedec,spi-nor", "spi-flash";
Reviewed-by: Jagan Teki jagan@openedev.com
Applied to u-boot-marvell/master.
Thanks, Stefan
participants (3)
-
Chris Packham
-
Jagan Teki
-
Stefan Roese