| Message ID | 20200221122133.32024-2-geert+renesas@glider.be |
|---|---|
| State | New |
| Headers | show |
| Series | [1/3] staging: pi433: overlay: Fix Broadcom vendor prefix | expand |
diff --git a/drivers/staging/pi433/Documentation/devicetree/pi433-overlay.dts b/drivers/staging/pi433/Documentation/devicetree/pi433-overlay.dts index 61fad96818c28c5d..dfc9f974ac24ecc1 100644 --- a/drivers/staging/pi433/Documentation/devicetree/pi433-overlay.dts +++ b/drivers/staging/pi433/Documentation/devicetree/pi433-overlay.dts @@ -3,7 +3,7 @@ /plugin/; / { - compatible = "bcm,bcm2835", "bcm,bcm2708", "bcm,bcm2709"; + compatible = "brcm,bcm2835", "brcm,bcm2708", "brcm,bcm2709"; fragment@0 { target = <&spi0>;
checkpatch.pl says: WARNING: DT compatible string "bcm,bcm2708" appears un-documented -- check ./Documentation/devicetree/bindings/ The vendor prefix of Broadcom Corporation is "brcm", not "bcm". Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> --- Why do you need these 3 compatible values at the root of the overlay, not referencing the actual root of the machine's DT? Does the Raspbian overlay manager use this to check if the overlay is applicable to the machine it is running on? --- .../staging/pi433/Documentation/devicetree/pi433-overlay.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)