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

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

[Chris Packham]

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;
-- 
2.14.1

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

[Jagan Teki]

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>

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

[Stefan Roese]

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