diff mbox series

arm64: dts: meson-g12-common: specify full DMC range

Message ID 3f4a7735-5327-df75-79ac-66209dbffa37@free.fr (mailing list archive)
State New, archived
Headers show
Series arm64: dts: meson-g12-common: specify full DMC range | expand

Commit Message

Marc Gonzalez March 23, 2023, 4:12 p.m. UTC
According to S905X2 Datasheet - Revision 07:
DRAM Memory Controller (DMC) register area spans ff638000-ff63a000.

According to DeviceTree Specification - Release v0.4-rc1:
simple-bus nodes do not require reg property.

Fixes: 1499218c80c9 ("arm64: dts: move common G12A & G12B modes to meson-g12-common.dtsi")
Signed-off-by: Marc Gonzalez <marc.w.gonzalez@free.fr>
---
 arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Martin Blumenstingl March 25, 2023, 8:43 p.m. UTC | #1
On Thu, Mar 23, 2023 at 5:12 PM Marc Gonzalez <marc.w.gonzalez@free.fr> wrote:
>
> According to S905X2 Datasheet - Revision 07:
> DRAM Memory Controller (DMC) register area spans ff638000-ff63a000.
>
> According to DeviceTree Specification - Release v0.4-rc1:
> simple-bus nodes do not require reg property.
Up until this point I thought that the reg property is mandatory even
for a simple-bus. Turns out this is not the case so we probably have
to clean up more meson.dtsi - but that's out of scope of this patch of
course.

> Fixes: 1499218c80c9 ("arm64: dts: move common G12A & G12B modes to meson-g12-common.dtsi")
> Signed-off-by: Marc Gonzalez <marc.w.gonzalez@free.fr>
Reviewed-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi b/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi
index 534178eaaf373..a5653ab1f0b43 100644
--- a/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi
+++ b/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi
@@ -1577,10 +1577,9 @@  usb2_phy0: phy@36000 {
 
 			dmc: bus@38000 {
 				compatible = "simple-bus";
-				reg = <0x0 0x38000 0x0 0x400>;
 				#address-cells = <2>;
 				#size-cells = <2>;
-				ranges = <0x0 0x0 0x0 0x38000 0x0 0x400>;
+				ranges = <0x0 0x0 0x0 0x38000 0x0 0x2000>;
 
 				canvas: video-lut@48 {
 					compatible = "amlogic,canvas";