* [PATCH] ARM: dts: at91: sama5d4_xplained: Mark optional AT25DF321A as disabled
@ 2021-06-22 13:42 Tudor Ambarus
2021-06-22 13:53 ` Nicolas Ferre
0 siblings, 1 reply; 3+ messages in thread
From: Tudor Ambarus @ 2021-06-22 13:42 UTC (permalink / raw)
To: nicolas.ferre, alexandre.belloni, ludovic.desroches
Cc: robh+dt, devicetree, linux-arm-kernel, linux-kernel, Tudor Ambarus
sama5d4_xplained has an optional on-board serial DataFlash (AT25DF321A),
which does not come populated on board by default. Mark it as disabled
to avoid the following probe error:
spi-nor spi0.0: unrecognized JEDEC id bytes: 00 00 00 00 00 00
Signed-off-by: Tudor Ambarus <tudor.ambarus@microchip.com>
---
arch/arm/boot/dts/at91-sama5d4_xplained.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/at91-sama5d4_xplained.dts b/arch/arm/boot/dts/at91-sama5d4_xplained.dts
index 0b3ad1b580b8..f397e2a68e5c 100644
--- a/arch/arm/boot/dts/at91-sama5d4_xplained.dts
+++ b/arch/arm/boot/dts/at91-sama5d4_xplained.dts
@@ -40,7 +40,7 @@ uart0: serial@f8004000 {
spi0: spi@f8010000 {
cs-gpios = <&pioC 3 0>, <0>, <0>, <0>;
- status = "okay";
+ status = "disabled";
m25p80@0 {
compatible = "atmel,at25df321a";
spi-max-frequency = <50000000>;
--
2.25.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] ARM: dts: at91: sama5d4_xplained: Mark optional AT25DF321A as disabled
2021-06-22 13:42 [PATCH] ARM: dts: at91: sama5d4_xplained: Mark optional AT25DF321A as disabled Tudor Ambarus
@ 2021-06-22 13:53 ` Nicolas Ferre
2021-06-24 15:22 ` Tudor.Ambarus
0 siblings, 1 reply; 3+ messages in thread
From: Nicolas Ferre @ 2021-06-22 13:53 UTC (permalink / raw)
To: Tudor Ambarus, alexandre.belloni, ludovic.desroches
Cc: robh+dt, devicetree, linux-arm-kernel, linux-kernel
Hi Tudor,
On 22/06/2021 at 15:42, Tudor Ambarus wrote:
> sama5d4_xplained has an optional on-board serial DataFlash (AT25DF321A),
> which does not come populated on board by default. Mark it as disabled
> to avoid the following probe error:
> spi-nor spi0.0: unrecognized JEDEC id bytes: 00 00 00 00 00 00
>
> Signed-off-by: Tudor Ambarus <tudor.ambarus@microchip.com>
> ---
> arch/arm/boot/dts/at91-sama5d4_xplained.dts | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/at91-sama5d4_xplained.dts b/arch/arm/boot/dts/at91-sama5d4_xplained.dts
> index 0b3ad1b580b8..f397e2a68e5c 100644
> --- a/arch/arm/boot/dts/at91-sama5d4_xplained.dts
> +++ b/arch/arm/boot/dts/at91-sama5d4_xplained.dts
> @@ -40,7 +40,7 @@ uart0: serial@f8004000 {
>
> spi0: spi@f8010000 {
> cs-gpios = <&pioC 3 0>, <0>, <0>, <0>;
> - status = "okay";
> + status = "disabled";
> m25p80@0 {
> compatible = "atmel,at25df321a";
> spi-max-frequency = <50000000>;
What about keeping SPI enabled and just having the status of m25p80 as
disabled: honestly, I don't know if it works and if it's desirable, but
might be closer to reality.
Tell me what you think.
Regards,
Nicolas
--
Nicolas Ferre
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ARM: dts: at91: sama5d4_xplained: Mark optional AT25DF321A as disabled
2021-06-22 13:53 ` Nicolas Ferre
@ 2021-06-24 15:22 ` Tudor.Ambarus
0 siblings, 0 replies; 3+ messages in thread
From: Tudor.Ambarus @ 2021-06-24 15:22 UTC (permalink / raw)
To: Nicolas.Ferre, alexandre.belloni, Ludovic.Desroches
Cc: robh+dt, devicetree, linux-arm-kernel, linux-kernel
On 6/22/21 4:53 PM, Nicolas Ferre wrote:
> Hi Tudor,
>
> On 22/06/2021 at 15:42, Tudor Ambarus wrote:
>> sama5d4_xplained has an optional on-board serial DataFlash (AT25DF321A),
>> which does not come populated on board by default. Mark it as disabled
>> to avoid the following probe error:
>> spi-nor spi0.0: unrecognized JEDEC id bytes: 00 00 00 00 00 00
>>
>> Signed-off-by: Tudor Ambarus <tudor.ambarus@microchip.com>
>> ---
>> arch/arm/boot/dts/at91-sama5d4_xplained.dts | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/boot/dts/at91-sama5d4_xplained.dts b/arch/arm/boot/dts/at91-sama5d4_xplained.dts
>> index 0b3ad1b580b8..f397e2a68e5c 100644
>> --- a/arch/arm/boot/dts/at91-sama5d4_xplained.dts
>> +++ b/arch/arm/boot/dts/at91-sama5d4_xplained.dts
>> @@ -40,7 +40,7 @@ uart0: serial@f8004000 {
>> spi0: spi@f8010000 {
>> cs-gpios = <&pioC 3 0>, <0>, <0>, <0>;
>> - status = "okay";
>> + status = "disabled";
>> m25p80@0 {
>> compatible = "atmel,at25df321a";
>> spi-max-frequency = <50000000>;
>
> What about keeping SPI enabled and just having the status of m25p80 as disabled: honestly, I don't know if it works and if it's desirable, but might be closer to reality.
>
> Tell me what you think.
I think it will work, because child status is checked with __of_device_is_available,
and the child will be skipped when marked as disabled.
But I feel that spi0 was enabled just because at25df321a was assumed to be populated.
Wouldn't it be strange to have spi0 marked as okay, but the only child as disabled?
How about removing spi0 node entirely from sama5d4_xplained dts?
Cheers,
ta
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-06-24 15:24 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-22 13:42 [PATCH] ARM: dts: at91: sama5d4_xplained: Mark optional AT25DF321A as disabled Tudor Ambarus
2021-06-22 13:53 ` Nicolas Ferre
2021-06-24 15:22 ` Tudor.Ambarus
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).