Re: [RESEND PATCH 3/9] arm64: dts: bcmbca: update BCM4908 board dts files

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

 





On 07/20/2022 11:15 PM, Rafał Miłecki wrote:
On 2022-07-21 02:07, William Zhang wrote:
Append "brcm,bcmbca" to compatible strings based on the new bcmbca
binding rule for BCM4908 family based boards. This will break drivers
that use the old compatible string for binding. Fortunately there is no
such usage in linux and u-boot.

Why should an extra "compatible" value break anything? I don't think it
will happen unless some driver does some really crazy stuff (like
checking full list of "compatible" values).

Yes anything along those lines will break. But there is no such craziness in linux and u-boot(wouldn't be accepted in first place). But Krzysztof suggested to add that to the commit message. I would prefer not to add that either.


Signed-off-by: William Zhang <william.zhang@xxxxxxxxxxxx>

Other than confusing commit message:

Acked-by: Rafał Miłecki <rafal@xxxxxxxxxx>


---

 arch/arm64/boot/dts/broadcom/bcm4908/bcm4906-netgear-r8000p.dts | 2 +-
 .../dts/broadcom/bcm4908/bcm4906-tplink-archer-c2300-v1.dts     | 2 +-
 arch/arm64/boot/dts/broadcom/bcm4908/bcm4908-asus-gt-ac5300.dts | 2 +-
 .../arm64/boot/dts/broadcom/bcm4908/bcm4908-netgear-raxe500.dts | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git
a/arch/arm64/boot/dts/broadcom/bcm4908/bcm4906-netgear-r8000p.dts
b/arch/arm64/boot/dts/broadcom/bcm4908/bcm4906-netgear-r8000p.dts
index 2dd028438c22..d8b60575eb4f 100644
--- a/arch/arm64/boot/dts/broadcom/bcm4908/bcm4906-netgear-r8000p.dts
+++ b/arch/arm64/boot/dts/broadcom/bcm4908/bcm4906-netgear-r8000p.dts
@@ -7,7 +7,7 @@
 #include "bcm4906.dtsi"

 / {
-    compatible = "netgear,r8000p", "brcm,bcm4906", "brcm,bcm4908";
+    compatible = "netgear,r8000p", "brcm,bcm4906", "brcm,bcm4908", "brcm,bcmbca";
     model = "Netgear R8000P";

     memory@0 {
diff --git
a/arch/arm64/boot/dts/broadcom/bcm4908/bcm4906-tplink-archer-c2300-v1.dts
b/arch/arm64/boot/dts/broadcom/bcm4908/bcm4906-tplink-archer-c2300-v1.dts
index 064f7f549665..296393d4aaab 100644
--- a/arch/arm64/boot/dts/broadcom/bcm4908/bcm4906-tplink-archer-c2300-v1.dts +++ b/arch/arm64/boot/dts/broadcom/bcm4908/bcm4906-tplink-archer-c2300-v1.dts
@@ -7,7 +7,7 @@
 #include "bcm4906.dtsi"

 / {
-    compatible = "tplink,archer-c2300-v1", "brcm,bcm4906", "brcm,bcm4908";
+    compatible = "tplink,archer-c2300-v1", "brcm,bcm4906",
"brcm,bcm4908", "brcm,bcmbca";
     model = "TP-Link Archer C2300 V1";

     memory@0 {
diff --git
a/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908-asus-gt-ac5300.dts
b/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908-asus-gt-ac5300.dts
index 04f8524b5335..787c7ddf9102 100644
--- a/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908-asus-gt-ac5300.dts
+++ b/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908-asus-gt-ac5300.dts
@@ -6,7 +6,7 @@
 #include "bcm4908.dtsi"

 / {
-    compatible = "asus,gt-ac5300", "brcm,bcm4908";
+    compatible = "asus,gt-ac5300", "brcm,bcm4908", "brcm,bcmbca";
     model = "Asus GT-AC5300";

     memory@0 {
diff --git
a/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908-netgear-raxe500.dts
b/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908-netgear-raxe500.dts
index 3c2cf2d238b6..23b96c663239 100644
--- a/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908-netgear-raxe500.dts
+++ b/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908-netgear-raxe500.dts
@@ -3,7 +3,7 @@
 #include "bcm4908.dtsi"

 / {
-    compatible = "netgear,raxe500", "brcm,bcm4908";
+    compatible = "netgear,raxe500", "brcm,bcm4908", "brcm,bcmbca";
     model = "Netgear RAXE500";

     memory@0 {

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature


[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