| Message ID | 20191210215831.6199-1-tony@atomide.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [1/2] ARM: dts: Configure interconnect target module for am4 qspi | expand |
Hi Tony, On Tue, Dec 10, 2019 at 10:58 PM Tony Lindgren <tony@atomide.com> wrote: > > We can now probe devices with device tree only configuration using > ti-sysc interconnect target module driver. Let's configure the > module, but keep the legacy "ti,hwmods" peroperty to avoid new boot > time warnings. The legacy property will be removed in later patches > together with the legacy platform data. > > Cc: Jean Pihet <jean.pihet@newoldbits.com> > Signed-off-by: Tony Lindgren <tony@atomide.com> Tested OK on AM4376 QSPI. Acked-by: Jean Pihet <jean.pihet@newoldbits.com> Thanks! Jean > --- > arch/arm/boot/dts/am4372.dtsi | 38 ++++++++++++++++++++++++++--------- > 1 file changed, 28 insertions(+), 10 deletions(-) > > diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi > --- a/arch/arm/boot/dts/am4372.dtsi > +++ b/arch/arm/boot/dts/am4372.dtsi > @@ -305,17 +305,35 @@ gpmc: gpmc@50000000 { > status = "disabled"; > }; > > - qspi: spi@47900000 { > - compatible = "ti,am4372-qspi"; > - reg = <0x47900000 0x100>, > - <0x30000000 0x4000000>; > - reg-names = "qspi_base", "qspi_mmap"; > - #address-cells = <1>; > - #size-cells = <0>; > + target-module@47900000 { > + compatible = "ti,sysc-omap4", "ti,sysc"; > ti,hwmods = "qspi"; > - interrupts = <0 138 0x4>; > - num-cs = <4>; > - status = "disabled"; > + reg = <0x47900000 0x4>, > + <0x47900010 0x4>; > + reg-names = "rev", "sysc"; > + ti,sysc-sidle = <SYSC_IDLE_FORCE>, > + <SYSC_IDLE_NO>, > + <SYSC_IDLE_SMART>, > + <SYSC_IDLE_SMART_WKUP>; > + clocks = <&l3s_clkctrl AM4_L3S_QSPI_CLKCTRL 0>; > + clock-names = "fck"; > + #address-cells = <1>; > + #size-cells = <1>; > + ranges = <0x0 0x47900000 0x1000>, > + <0x30000000 0x30000000 0x4000000>; > + > + qspi: spi@0 { > + compatible = "ti,am4372-qspi"; > + reg = <0 0x100>, > + <0x30000000 0x4000000>; > + reg-names = "qspi_base", "qspi_mmap"; > + clocks = <&dpll_per_m2_div4_ck>; > + clock-names = "fck"; > + #address-cells = <1>; > + #size-cells = <0>; > + interrupts = <0 138 0x4>; > + num-cs = <4>; > + }; > }; > > dss: dss@4832a000 { > -- > 2.24.0
On 11/12/19 3:28 am, Tony Lindgren wrote: > We can now probe devices with device tree only configuration using > ti-sysc interconnect target module driver. Let's configure the > module, but keep the legacy "ti,hwmods" peroperty to avoid new boot > time warnings. The legacy property will be removed in later patches > together with the legacy platform data. Tested-by: Keerthy <j-keerthy@ti.com> > > Cc: Jean Pihet <jean.pihet@newoldbits.com> > Signed-off-by: Tony Lindgren <tony@atomide.com> > --- > arch/arm/boot/dts/am4372.dtsi | 38 ++++++++++++++++++++++++++--------- > 1 file changed, 28 insertions(+), 10 deletions(-) > > diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi > --- a/arch/arm/boot/dts/am4372.dtsi > +++ b/arch/arm/boot/dts/am4372.dtsi > @@ -305,17 +305,35 @@ gpmc: gpmc@50000000 { > status = "disabled"; > }; > > - qspi: spi@47900000 { > - compatible = "ti,am4372-qspi"; > - reg = <0x47900000 0x100>, > - <0x30000000 0x4000000>; > - reg-names = "qspi_base", "qspi_mmap"; > - #address-cells = <1>; > - #size-cells = <0>; > + target-module@47900000 { > + compatible = "ti,sysc-omap4", "ti,sysc"; > ti,hwmods = "qspi"; > - interrupts = <0 138 0x4>; > - num-cs = <4>; > - status = "disabled"; > + reg = <0x47900000 0x4>, > + <0x47900010 0x4>; > + reg-names = "rev", "sysc"; > + ti,sysc-sidle = <SYSC_IDLE_FORCE>, > + <SYSC_IDLE_NO>, > + <SYSC_IDLE_SMART>, > + <SYSC_IDLE_SMART_WKUP>; > + clocks = <&l3s_clkctrl AM4_L3S_QSPI_CLKCTRL 0>; > + clock-names = "fck"; > + #address-cells = <1>; > + #size-cells = <1>; > + ranges = <0x0 0x47900000 0x1000>, > + <0x30000000 0x30000000 0x4000000>; > + > + qspi: spi@0 { > + compatible = "ti,am4372-qspi"; > + reg = <0 0x100>, > + <0x30000000 0x4000000>; > + reg-names = "qspi_base", "qspi_mmap"; > + clocks = <&dpll_per_m2_div4_ck>; > + clock-names = "fck"; > + #address-cells = <1>; > + #size-cells = <0>; > + interrupts = <0 138 0x4>; > + num-cs = <4>; > + }; > }; > > dss: dss@4832a000 { >
diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi --- a/arch/arm/boot/dts/am4372.dtsi +++ b/arch/arm/boot/dts/am4372.dtsi @@ -305,17 +305,35 @@ gpmc: gpmc@50000000 { status = "disabled"; }; - qspi: spi@47900000 { - compatible = "ti,am4372-qspi"; - reg = <0x47900000 0x100>, - <0x30000000 0x4000000>; - reg-names = "qspi_base", "qspi_mmap"; - #address-cells = <1>; - #size-cells = <0>; + target-module@47900000 { + compatible = "ti,sysc-omap4", "ti,sysc"; ti,hwmods = "qspi"; - interrupts = <0 138 0x4>; - num-cs = <4>; - status = "disabled"; + reg = <0x47900000 0x4>, + <0x47900010 0x4>; + reg-names = "rev", "sysc"; + ti,sysc-sidle = <SYSC_IDLE_FORCE>, + <SYSC_IDLE_NO>, + <SYSC_IDLE_SMART>, + <SYSC_IDLE_SMART_WKUP>; + clocks = <&l3s_clkctrl AM4_L3S_QSPI_CLKCTRL 0>; + clock-names = "fck"; + #address-cells = <1>; + #size-cells = <1>; + ranges = <0x0 0x47900000 0x1000>, + <0x30000000 0x30000000 0x4000000>; + + qspi: spi@0 { + compatible = "ti,am4372-qspi"; + reg = <0 0x100>, + <0x30000000 0x4000000>; + reg-names = "qspi_base", "qspi_mmap"; + clocks = <&dpll_per_m2_div4_ck>; + clock-names = "fck"; + #address-cells = <1>; + #size-cells = <0>; + interrupts = <0 138 0x4>; + num-cs = <4>; + }; }; dss: dss@4832a000 {
We can now probe devices with device tree only configuration using ti-sysc interconnect target module driver. Let's configure the module, but keep the legacy "ti,hwmods" peroperty to avoid new boot time warnings. The legacy property will be removed in later patches together with the legacy platform data. Cc: Jean Pihet <jean.pihet@newoldbits.com> Signed-off-by: Tony Lindgren <tony@atomide.com> --- arch/arm/boot/dts/am4372.dtsi | 38 ++++++++++++++++++++++++++--------- 1 file changed, 28 insertions(+), 10 deletions(-)