[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <e32c59d7-62bd-4a8d-b2ae-0d73d2f6d90c@broadcom.com>
Date: Thu, 22 Feb 2024 08:21:10 -0800
From: William Zhang <william.zhang@...adcom.com>
To: Rafał Miłecki <rafal@...ecki.pl>
Cc: Linux MTD List <linux-mtd@...ts.infradead.org>,
Linux ARM List <linux-arm-kernel@...ts.infradead.org>,
Broadcom Kernel List <bcm-kernel-feedback-list@...adcom.com>,
f.fainelli@...il.com, kursad.oney@...adcom.com, joel.peshkin@...adcom.com,
anand.gore@...adcom.com, dregan@...l.com, kamal.dasu@...adcom.com,
tomer.yacoby@...adcom.com, dan.beygelman@...adcom.com,
David Regan <dregan@...adcom.com>, devicetree@...r.kernel.org,
Conor Dooley <conor+dt@...nel.org>, linux-kernel@...r.kernel.org,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Rob Herring <robh+dt@...nel.org>,
Florian Fainelli <florian.fainelli@...adcom.com>
Subject: Re: [PATCH v5 05/12] arm64: dts: broadcom: bcmbca: Add NAND
controller node
Hi Rafal,
On 2/22/24 00:27, Rafał Miłecki wrote:
> On 2024-02-07 21:22, William Zhang wrote:
>> diff --git a/arch/arm64/boot/dts/broadcom/bcmbca/bcm4908.dtsi
>> b/arch/arm64/boot/dts/broadcom/bcmbca/bcm4908.dtsi
>> index 2f124b027bbf..336016e334d9 100644
>> --- a/arch/arm64/boot/dts/broadcom/bcmbca/bcm4908.dtsi
>> +++ b/arch/arm64/boot/dts/broadcom/bcmbca/bcm4908.dtsi
>> @@ -589,7 +589,7 @@ hsspi: spi@...0 {
>> status = "disabled";
>> };
>>
>> - nand-controller@...0 {
>> + nand_controller: nand-controller@...0 {
>> #address-cells = <1>;
>> #size-cells = <0>;
>> compatible = "brcm,nand-bcm63138", "brcm,brcmnand-v7.1",
>> "brcm,brcmnand";
>> @@ -597,7 +597,7 @@ nand-controller@...0 {
>> reg-names = "nand", "nand-int-base";
>> interrupts = <GIC_SPI 37 IRQ_TYPE_LEVEL_HIGH>;
>> interrupt-names = "nand_ctlrdy";
>> - status = "okay";
>> + status = "disabled";
>>
>> nandcs: nand@0 {
>> compatible = "brcm,nandcs";
>
> You disable NAND controller here breaking support for all dependant
> devices.
>
> I see you fix it afterwards in PATCH 06/12 but it's still a poor way of
> handling such change and this issue can be hit e.g. during git bisect.
>
> Also: given that 99% of BCM4908 devices use NAND controller can we just
> save ourselves efforts and keep it enabled in .dtsi?
>
I was trying to fix this according to the dts convention per
maintainers' requirement(only enable the node in the board dts, not soc
include). We have 100% of BCMBCA devices use strap for ecc but we still
have to put the property in board dts.
I understand if someone try to bisect the commits, it may hit the
problem but this will be very obvious that controller is not even shown
in the boot up so hopefully it is easy to figure out. Anyway if
maintainer(Minque and/or anyone from device tree side) is okay with your
request, I can change it back to enable in the dtsi.
Download attachment "smime.p7s" of type "application/pkcs7-signature" (4212 bytes)
Powered by blists - more mailing lists