public inbox for devicetree@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH -next] ARM: dts: BCM5301X: Drop extra NAND controller compatible
@ 2026-02-04  9:15 Miquel Raynal
  2026-02-04 16:00 ` Rob Herring
                   ` (2 more replies)
  0 siblings, 3 replies; 8+ messages in thread
From: Miquel Raynal @ 2026-02-04  9:15 UTC (permalink / raw)
  To: Florian Fainelli, Hauke Mehrtens, zajec5
  Cc: bcm-kernel-feedback-list, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, William Zhang, linux-arm-kernel, devicetree,
	Thomas Petazzoni, Miquel Raynal

Fix the dtbs_check warning introduced when the brcm,brcmnand fallback
compatible got removed for iProc machines.

Fixes: 4db35366d6dc ("dt-bindings: mtd: brcm,brcmnand: Drop "brcm,brcmnand" compatible for iProc")
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
---

The above commit is in mtd/next and will be part of the opening merge window.

---
 arch/arm/boot/dts/broadcom/bcm-ns.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/broadcom/bcm-ns.dtsi b/arch/arm/boot/dts/broadcom/bcm-ns.dtsi
index d0d5f7e52a91..46b650abdb90 100644
--- a/arch/arm/boot/dts/broadcom/bcm-ns.dtsi
+++ b/arch/arm/boot/dts/broadcom/bcm-ns.dtsi
@@ -479,7 +479,7 @@ thermal: thermal@2c0 {
 	};
 
 	nand_controller: nand-controller@18028000 {
-		compatible = "brcm,nand-iproc", "brcm,brcmnand-v6.1", "brcm,brcmnand";
+		compatible = "brcm,nand-iproc", "brcm,brcmnand-v6.1";
 		reg = <0x18028000 0x600>, <0x1811a408 0x600>, <0x18028f00 0x20>;
 		reg-names = "nand", "iproc-idm", "iproc-ext";
 		interrupts = <GIC_SPI 68 IRQ_TYPE_LEVEL_HIGH>;
-- 
2.51.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH -next] ARM: dts: BCM5301X: Drop extra NAND controller compatible
  2026-02-04  9:15 [PATCH -next] ARM: dts: BCM5301X: Drop extra NAND controller compatible Miquel Raynal
@ 2026-02-04 16:00 ` Rob Herring
  2026-02-04 17:36 ` William Zhang
  2026-02-04 18:26 ` Florian Fainelli
  2 siblings, 0 replies; 8+ messages in thread
From: Rob Herring @ 2026-02-04 16:00 UTC (permalink / raw)
  To: Miquel Raynal
  Cc: Florian Fainelli, Hauke Mehrtens, zajec5,
	bcm-kernel-feedback-list, Krzysztof Kozlowski, Conor Dooley,
	William Zhang, linux-arm-kernel, devicetree, Thomas Petazzoni

On Wed, Feb 4, 2026 at 3:15 AM Miquel Raynal <miquel.raynal@bootlin.com> wrote:
>
> Fix the dtbs_check warning introduced when the brcm,brcmnand fallback
> compatible got removed for iProc machines.
>
> Fixes: 4db35366d6dc ("dt-bindings: mtd: brcm,brcmnand: Drop "brcm,brcmnand" compatible for iProc")
> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
> ---

Acked-by: Rob Herring (Arm) <robh@kernel.org>

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH -next] ARM: dts: BCM5301X: Drop extra NAND controller compatible
  2026-02-04  9:15 [PATCH -next] ARM: dts: BCM5301X: Drop extra NAND controller compatible Miquel Raynal
  2026-02-04 16:00 ` Rob Herring
@ 2026-02-04 17:36 ` William Zhang
  2026-02-04 20:36   ` Miquel Raynal
  2026-02-04 18:26 ` Florian Fainelli
  2 siblings, 1 reply; 8+ messages in thread
From: William Zhang @ 2026-02-04 17:36 UTC (permalink / raw)
  To: Miquel Raynal
  Cc: Florian Fainelli, Hauke Mehrtens, zajec5,
	bcm-kernel-feedback-list, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, linux-arm-kernel, devicetree, Thomas Petazzoni

[-- Attachment #1: Type: text/plain, Size: 1380 bytes --]

On Wed, Feb 4, 2026 at 1:15 AM Miquel Raynal <miquel.raynal@bootlin.com> wrote:
>
> Fix the dtbs_check warning introduced when the brcm,brcmnand fallback
> compatible got removed for iProc machines.
>
> Fixes: 4db35366d6dc ("dt-bindings: mtd: brcm,brcmnand: Drop "brcm,brcmnand" compatible for iProc")
> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
> ---
>
> The above commit is in mtd/next and will be part of the opening merge window.
>
> ---
>  arch/arm/boot/dts/broadcom/bcm-ns.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/broadcom/bcm-ns.dtsi b/arch/arm/boot/dts/broadcom/bcm-ns.dtsi
> index d0d5f7e52a91..46b650abdb90 100644
> --- a/arch/arm/boot/dts/broadcom/bcm-ns.dtsi
> +++ b/arch/arm/boot/dts/broadcom/bcm-ns.dtsi
> @@ -479,7 +479,7 @@ thermal: thermal@2c0 {
>         };
>
>         nand_controller: nand-controller@18028000 {
> -               compatible = "brcm,nand-iproc", "brcm,brcmnand-v6.1", "brcm,brcmnand";
> +               compatible = "brcm,nand-iproc", "brcm,brcmnand-v6.1";
>                 reg = <0x18028000 0x600>, <0x1811a408 0x600>, <0x18028f00 0x20>;
>                 reg-names = "nand", "iproc-idm", "iproc-ext";
>                 interrupts = <GIC_SPI 68 IRQ_TYPE_LEVEL_HIGH>;

Acked-by: William Zhang <william.zhang@broadcom.com>
> --
> 2.51.1
>

[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 5473 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH -next] ARM: dts: BCM5301X: Drop extra NAND controller compatible
  2026-02-04  9:15 [PATCH -next] ARM: dts: BCM5301X: Drop extra NAND controller compatible Miquel Raynal
  2026-02-04 16:00 ` Rob Herring
  2026-02-04 17:36 ` William Zhang
@ 2026-02-04 18:26 ` Florian Fainelli
  2 siblings, 0 replies; 8+ messages in thread
From: Florian Fainelli @ 2026-02-04 18:26 UTC (permalink / raw)
  To: bcm-kernel-feedback-list, Miquel Raynal, Florian Fainelli,
	Hauke Mehrtens, zajec5
  Cc: Florian Fainelli, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	William Zhang, linux-arm-kernel, devicetree, Thomas Petazzoni

From: Florian Fainelli <f.fainelli@gmail.com>

On Wed,  4 Feb 2026 10:15:30 +0100, Miquel Raynal <miquel.raynal@bootlin.com> wrote:
> Fix the dtbs_check warning introduced when the brcm,brcmnand fallback
> compatible got removed for iProc machines.
> 
> Fixes: 4db35366d6dc ("dt-bindings: mtd: brcm,brcmnand: Drop "brcm,brcmnand" compatible for iProc")
> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
> ---

Applied to https://github.com/Broadcom/stblinux/commits/devicetree/next, thanks!
--
Florian

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH -next] ARM: dts: BCM5301X: Drop extra NAND controller compatible
  2026-02-04 17:36 ` William Zhang
@ 2026-02-04 20:36   ` Miquel Raynal
  2026-02-04 21:21     ` Florian Fainelli
  2026-02-04 21:56     ` William Zhang
  0 siblings, 2 replies; 8+ messages in thread
From: Miquel Raynal @ 2026-02-04 20:36 UTC (permalink / raw)
  To: William Zhang
  Cc: Florian Fainelli, Hauke Mehrtens, zajec5,
	bcm-kernel-feedback-list, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, linux-arm-kernel, devicetree, Thomas Petazzoni

Hello William,

>> diff --git a/arch/arm/boot/dts/broadcom/bcm-ns.dtsi b/arch/arm/boot/dts/broadcom/bcm-ns.dtsi
>> index d0d5f7e52a91..46b650abdb90 100644
>> --- a/arch/arm/boot/dts/broadcom/bcm-ns.dtsi
>> +++ b/arch/arm/boot/dts/broadcom/bcm-ns.dtsi
>> @@ -479,7 +479,7 @@ thermal: thermal@2c0 {
>>         };
>>
>>         nand_controller: nand-controller@18028000 {
>> -               compatible = "brcm,nand-iproc", "brcm,brcmnand-v6.1", "brcm,brcmnand";
>> +               compatible = "brcm,nand-iproc", "brcm,brcmnand-v6.1";
>>                 reg = <0x18028000 0x600>, <0x1811a408 0x600>, <0x18028f00 0x20>;
>>                 reg-names = "nand", "iproc-idm", "iproc-ext";
>>                 interrupts = <GIC_SPI 68 IRQ_TYPE_LEVEL_HIGH>;
>
> Acked-by: William Zhang <william.zhang@broadcom.com>

Do you mind if I take it on top of the commit it fixes?

Thanks,
Miquèl


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH -next] ARM: dts: BCM5301X: Drop extra NAND controller compatible
  2026-02-04 20:36   ` Miquel Raynal
@ 2026-02-04 21:21     ` Florian Fainelli
  2026-02-05  8:49       ` Miquel Raynal
  2026-02-04 21:56     ` William Zhang
  1 sibling, 1 reply; 8+ messages in thread
From: Florian Fainelli @ 2026-02-04 21:21 UTC (permalink / raw)
  To: Miquel Raynal, William Zhang
  Cc: Hauke Mehrtens, zajec5, bcm-kernel-feedback-list, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, linux-arm-kernel, devicetree,
	Thomas Petazzoni

On 2/4/26 12:36, Miquel Raynal wrote:
> Hello William,
> 
>>> diff --git a/arch/arm/boot/dts/broadcom/bcm-ns.dtsi b/arch/arm/boot/dts/broadcom/bcm-ns.dtsi
>>> index d0d5f7e52a91..46b650abdb90 100644
>>> --- a/arch/arm/boot/dts/broadcom/bcm-ns.dtsi
>>> +++ b/arch/arm/boot/dts/broadcom/bcm-ns.dtsi
>>> @@ -479,7 +479,7 @@ thermal: thermal@2c0 {
>>>          };
>>>
>>>          nand_controller: nand-controller@18028000 {
>>> -               compatible = "brcm,nand-iproc", "brcm,brcmnand-v6.1", "brcm,brcmnand";
>>> +               compatible = "brcm,nand-iproc", "brcm,brcmnand-v6.1";
>>>                  reg = <0x18028000 0x600>, <0x1811a408 0x600>, <0x18028f00 0x20>;
>>>                  reg-names = "nand", "iproc-idm", "iproc-ext";
>>>                  interrupts = <GIC_SPI 68 IRQ_TYPE_LEVEL_HIGH>;
>>
>> Acked-by: William Zhang <william.zhang@broadcom.com>
> 
> Do you mind if I take it on top of the commit it fixes?

I already applied this change to the Broadcom ARM SoC tree. I don't 
expect conflicts so if you prefer to take it, I can drop it.
-- 
Florian

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH -next] ARM: dts: BCM5301X: Drop extra NAND controller compatible
  2026-02-04 20:36   ` Miquel Raynal
  2026-02-04 21:21     ` Florian Fainelli
@ 2026-02-04 21:56     ` William Zhang
  1 sibling, 0 replies; 8+ messages in thread
From: William Zhang @ 2026-02-04 21:56 UTC (permalink / raw)
  To: Miquel Raynal
  Cc: Florian Fainelli, Hauke Mehrtens, zajec5,
	bcm-kernel-feedback-list, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, linux-arm-kernel, devicetree, Thomas Petazzoni

[-- Attachment #1: Type: text/plain, Size: 1069 bytes --]

Hi Miquel,

On Wed, Feb 4, 2026 at 12:36 PM Miquel Raynal <miquel.raynal@bootlin.com> wrote:
>
> Hello William,
>
> >> diff --git a/arch/arm/boot/dts/broadcom/bcm-ns.dtsi b/arch/arm/boot/dts/broadcom/bcm-ns.dtsi
> >> index d0d5f7e52a91..46b650abdb90 100644
> >> --- a/arch/arm/boot/dts/broadcom/bcm-ns.dtsi
> >> +++ b/arch/arm/boot/dts/broadcom/bcm-ns.dtsi
> >> @@ -479,7 +479,7 @@ thermal: thermal@2c0 {
> >>         };
> >>
> >>         nand_controller: nand-controller@18028000 {
> >> -               compatible = "brcm,nand-iproc", "brcm,brcmnand-v6.1", "brcm,brcmnand";
> >> +               compatible = "brcm,nand-iproc", "brcm,brcmnand-v6.1";
> >>                 reg = <0x18028000 0x600>, <0x1811a408 0x600>, <0x18028f00 0x20>;
> >>                 reg-names = "nand", "iproc-idm", "iproc-ext";
> >>                 interrupts = <GIC_SPI 68 IRQ_TYPE_LEVEL_HIGH>;
> >
> > Acked-by: William Zhang <william.zhang@broadcom.com>
>
> Do you mind if I take it on top of the commit it fixes?
>
That's fine with me.

> Thanks,
> Miquèl
>

[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 5473 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH -next] ARM: dts: BCM5301X: Drop extra NAND controller compatible
  2026-02-04 21:21     ` Florian Fainelli
@ 2026-02-05  8:49       ` Miquel Raynal
  0 siblings, 0 replies; 8+ messages in thread
From: Miquel Raynal @ 2026-02-05  8:49 UTC (permalink / raw)
  To: Florian Fainelli
  Cc: William Zhang, Hauke Mehrtens, zajec5, bcm-kernel-feedback-list,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, linux-arm-kernel,
	devicetree, Thomas Petazzoni

Hi Florian,

On 04/02/2026 at 13:21:46 -08, Florian Fainelli <florian.fainelli@broadcom.com> wrote:

> On 2/4/26 12:36, Miquel Raynal wrote:
>> Hello William,
>> 
>>>> diff --git a/arch/arm/boot/dts/broadcom/bcm-ns.dtsi b/arch/arm/boot/dts/broadcom/bcm-ns.dtsi
>>>> index d0d5f7e52a91..46b650abdb90 100644
>>>> --- a/arch/arm/boot/dts/broadcom/bcm-ns.dtsi
>>>> +++ b/arch/arm/boot/dts/broadcom/bcm-ns.dtsi
>>>> @@ -479,7 +479,7 @@ thermal: thermal@2c0 {
>>>>          };
>>>>
>>>>          nand_controller: nand-controller@18028000 {
>>>> -               compatible = "brcm,nand-iproc", "brcm,brcmnand-v6.1", "brcm,brcmnand";
>>>> +               compatible = "brcm,nand-iproc", "brcm,brcmnand-v6.1";
>>>>                  reg = <0x18028000 0x600>, <0x1811a408 0x600>, <0x18028f00 0x20>;
>>>>                  reg-names = "nand", "iproc-idm", "iproc-ext";
>>>>                  interrupts = <GIC_SPI 68 IRQ_TYPE_LEVEL_HIGH>;
>>>
>>> Acked-by: William Zhang <william.zhang@broadcom.com>
>> Do you mind if I take it on top of the commit it fixes?
>
> I already applied this change to the Broadcom ARM SoC tree. I don't
> expect conflicts so if you prefer to take it, I can drop it.

It's okay, this will just generate a dtbs_check warning if it reaches
Linus' tree before my branch, but honestly I don't think it matters
much.

Please keep it.

Thanks!
Miquèl

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2026-02-05  8:49 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-02-04  9:15 [PATCH -next] ARM: dts: BCM5301X: Drop extra NAND controller compatible Miquel Raynal
2026-02-04 16:00 ` Rob Herring
2026-02-04 17:36 ` William Zhang
2026-02-04 20:36   ` Miquel Raynal
2026-02-04 21:21     ` Florian Fainelli
2026-02-05  8:49       ` Miquel Raynal
2026-02-04 21:56     ` William Zhang
2026-02-04 18:26 ` Florian Fainelli

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox