| Message ID | 20190307151342.7381-2-narmstrong@baylibre.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | arm64: dts: g12a: Add peripherals | expand |
Hi Neil, On Thu, Mar 7, 2019 at 4:14 PM Neil Armstrong <narmstrong@baylibre.com> wrote: > > Add nodes and properties for the AO Clocks and Resets. > > Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> > Signed-off-by: Jerome Brunet <jbrunet@baylibre.com> > --- > arch/arm64/boot/dts/amlogic/meson-g12a.dtsi | 17 +++++++++++++++++ > 1 file changed, 17 insertions(+) > > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12a.dtsi b/arch/arm64/boot/dts/amlogic/meson-g12a.dtsi > index 191d31db9853..0eb5220da82c 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12a.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-g12a.dtsi > @@ -122,6 +122,23 @@ > #size-cells = <2>; > ranges = <0x0 0x0 0x0 0xff800000 0x0 0x100000>; > > + rti: sys-ctrl@0 { > + compatible = "amlogic,meson-gx-ao-sysctrl", > + "simple-mfd", "syscon"; > + reg = <0x0 0x0 0x0 0x1000>; > + #address-cells = <2>; > + #size-cells = <2>; > + ranges = <0x0 0x0 0x0 0x0 0x0 0x1000>; is this supposed to be 0x1000 (GX uses 0x100 here)? my understanding is that G12A has the AO CEC at aobus + 0x100, so using a size of 0x1000 would make it overlap Regards Martin
Hi, Le 15/03/2019 22:07, Martin Blumenstingl a écrit : > Hi Neil, > > On Thu, Mar 7, 2019 at 4:14 PM Neil Armstrong <narmstrong@baylibre.com> wrote: >> >> Add nodes and properties for the AO Clocks and Resets. >> >> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> >> Signed-off-by: Jerome Brunet <jbrunet@baylibre.com> >> --- >> arch/arm64/boot/dts/amlogic/meson-g12a.dtsi | 17 +++++++++++++++++ >> 1 file changed, 17 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/amlogic/meson-g12a.dtsi b/arch/arm64/boot/dts/amlogic/meson-g12a.dtsi >> index 191d31db9853..0eb5220da82c 100644 >> --- a/arch/arm64/boot/dts/amlogic/meson-g12a.dtsi >> +++ b/arch/arm64/boot/dts/amlogic/meson-g12a.dtsi >> @@ -122,6 +122,23 @@ >> #size-cells = <2>; >> ranges = <0x0 0x0 0x0 0xff800000 0x0 0x100000>; >> >> + rti: sys-ctrl@0 { >> + compatible = "amlogic,meson-gx-ao-sysctrl", >> + "simple-mfd", "syscon"; >> + reg = <0x0 0x0 0x0 0x1000>; >> + #address-cells = <2>; >> + #size-cells = <2>; >> + ranges = <0x0 0x0 0x0 0x0 0x0 0x1000>; > is this supposed to be 0x1000 (GX uses 0x100 here)? > my understanding is that G12A has the AO CEC at aobus + 0x100, so > using a size of 0x1000 would make it overlap Hmm, Let me check Neil > > > Regards > Martin >
diff --git a/arch/arm64/boot/dts/amlogic/meson-g12a.dtsi b/arch/arm64/boot/dts/amlogic/meson-g12a.dtsi index 191d31db9853..0eb5220da82c 100644 --- a/arch/arm64/boot/dts/amlogic/meson-g12a.dtsi +++ b/arch/arm64/boot/dts/amlogic/meson-g12a.dtsi @@ -122,6 +122,23 @@ #size-cells = <2>; ranges = <0x0 0x0 0x0 0xff800000 0x0 0x100000>; + rti: sys-ctrl@0 { + compatible = "amlogic,meson-gx-ao-sysctrl", + "simple-mfd", "syscon"; + reg = <0x0 0x0 0x0 0x1000>; + #address-cells = <2>; + #size-cells = <2>; + ranges = <0x0 0x0 0x0 0x0 0x0 0x1000>; + + clkc_AO: clock-controller { + compatible = "amlogic,meson-g12a-aoclkc"; + #clock-cells = <1>; + #reset-cells = <1>; + clocks = <&xtal>, <&clkc CLKID_CLK81>; + clock-names = "xtal", "mpeg-clk"; + }; + }; + uart_AO: serial@3000 { compatible = "amlogic,meson-gx-uart", "amlogic,meson-ao-uart";