| Message ID | 20241202-dt-bcm2712-fixes-v1-4-fac67cc2f98a@raspberrypi.com |
|---|---|
| State | New |
| Headers | show |
| Series | drm/vc4: Fixup DT and DT binding issues from recent patchset | expand |
On 12/2/24 07:20, Krzysztof Kozlowski wrote: > On 02/12/2024 15:31, Dave Stevenson wrote: >> Fixes up errors on HDMI and interrupt controllers that weren't > > What errors? I can't find anything in the commit. Describe the error. > >> noticed before merging. >> >> Fixes: de9bc2dba3db ("arm64: dts: broadcom: Add display pipeline support to BCM2712") >> Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.com> >> --- >> arch/arm64/boot/dts/broadcom/bcm2712.dtsi | 8 ++++---- >> 1 file changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/arch/arm64/boot/dts/broadcom/bcm2712.dtsi b/arch/arm64/boot/dts/broadcom/bcm2712.dtsi >> index 39305e0869ec..f42fad2d8b37 100644 >> --- a/arch/arm64/boot/dts/broadcom/bcm2712.dtsi >> +++ b/arch/arm64/boot/dts/broadcom/bcm2712.dtsi >> @@ -336,7 +336,7 @@ ddc1: i2c@7d508280 { >> #size-cells = <0>; >> }; >> >> - bsc_irq: intc@7d508380 { >> + bsc_irq: interrupt-controller@7d508380 { > > > Do not mix cleanups with bugfixes. > > BTW, do not mix DTS to DRM. DRM has its own development style. DTS > *cannot* be merged there, so combining these series is not welcomed and > leads to issues (like DRM applying DTS!). That is not what was done, I took the DTS patches via the Broadcom ARM SoC tree, as I usually do, unless someone tells me otherwise. We got a number of DTS warnings after I took Dave's patches so I asked for those to be fixed, as it turns out there are binding and DTS fixes. I intend to squash these fixes with their original commit, so all of those commit messages will be discarded, they are only meant to be transitional.
diff --git a/arch/arm64/boot/dts/broadcom/bcm2712.dtsi b/arch/arm64/boot/dts/broadcom/bcm2712.dtsi index 39305e0869ec..f42fad2d8b37 100644 --- a/arch/arm64/boot/dts/broadcom/bcm2712.dtsi +++ b/arch/arm64/boot/dts/broadcom/bcm2712.dtsi @@ -336,7 +336,7 @@ ddc1: i2c@7d508280 { #size-cells = <0>; }; - bsc_irq: intc@7d508380 { + bsc_irq: interrupt-controller@7d508380 { compatible = "brcm,bcm7271-l2-intc"; reg = <0x7d508380 0x10>; interrupts = <GIC_SPI 242 IRQ_TYPE_LEVEL_HIGH>; @@ -344,7 +344,7 @@ bsc_irq: intc@7d508380 { #interrupt-cells = <1>; }; - main_irq: intc@7d508400 { + main_irq: interrupt-controller@7d508400 { compatible = "brcm,bcm7271-l2-intc"; reg = <0x7d508400 0x10>; interrupts = <GIC_SPI 244 IRQ_TYPE_LEVEL_HIGH>; @@ -352,7 +352,7 @@ main_irq: intc@7d508400 { #interrupt-cells = <1>; }; - hdmi0: hdmi@7ef00700 { + hdmi0: hdmi@7c701400 { compatible = "brcm,bcm2712-hdmi0"; reg = <0x7c701400 0x300>, <0x7c701000 0x200>, @@ -381,7 +381,7 @@ hdmi0: hdmi@7ef00700 { ddc = <&ddc0>; }; - hdmi1: hdmi@7ef05700 { + hdmi1: hdmi@7c706400 { compatible = "brcm,bcm2712-hdmi1"; reg = <0x7c706400 0x300>, <0x7c706000 0x200>,
Fixes up errors on HDMI and interrupt controllers that weren't noticed before merging. Fixes: de9bc2dba3db ("arm64: dts: broadcom: Add display pipeline support to BCM2712") Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.com> --- arch/arm64/boot/dts/broadcom/bcm2712.dtsi | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)