Re: [PATCH] arm64: dts: broadcom: bcmbca: bcm4908: drop invalid "nand" interrupt name

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 9.11.2022 19:37, Florian Fainelli wrote:
On 11/3/22 03:41, Rafał Miłecki wrote:
From: Rafał Miłecki <rafal@xxxxxxxxxx>

String "nand" was never a valid interrupt name. It was never documented
and never used in Linux or U-Boot driver. This mistake was propagated
from the bcm63138.dtsi.

The whole "interrupt-names" property is optional and can be skipped.

Fixes: 2961f69f151c ("arm64: dts: broadcom: add BCM4908 and Asus GT-AC5300 early DTS files")
Signed-off-by: Rafał Miłecki <rafal@xxxxxxxxxx>

Do you still want me to apply this patch, or are we happy with renaming that interrupt to "nand_ctlrdy" instead?

I don't have any real preference.

Since we already have this patch, we may just apply it. If you think
having "interrupt-names" with just "nand_ctlrdy" is cleaner, I can send
V2.



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux