| Message ID | 1440757911-9120-2-git-send-email-lee.jones@linaro.org |
|---|---|
| State | New |
| Headers | show |
Hi Lee, On Fri, 28 Aug 2015, Lee Jones wrote: > Signed-off-by: Ludovic Barre <ludovic.barre@st.com> > Signed-off-by: Lee Jones <lee.jones@linaro.org> > --- > arch/arm/boot/dts/stih407-family.dtsi | 48 +++++++++++++++++++++++++++++++++++ > 1 file changed, 48 insertions(+) > > diff --git a/arch/arm/boot/dts/stih407-family.dtsi b/arch/arm/boot/dts/stih407-family.dtsi > index 33daa49..89a363c 100644 > --- a/arch/arm/boot/dts/stih407-family.dtsi > +++ b/arch/arm/boot/dts/stih407-family.dtsi > @@ -608,5 +608,53 @@ > status = "okay"; > tx-only; > }; > + > + st231_gp0: st231-gp0@0 { > + compatible = "st,st231-rproc"; > + reg = <0x40000000 0x01000000>; > + reg-names = "carveout"; > + resets = <&softreset STIH407_ST231_GP0_SOFTRESET>; Support for these co-processor reset lines does not exist upstream. So this series as it stands won't compile. regards, Peter. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Tue, 01 Sep 2015, Peter Griffin wrote: > On Fri, 28 Aug 2015, Lee Jones wrote: > > > Signed-off-by: Ludovic Barre <ludovic.barre@st.com> > > Signed-off-by: Lee Jones <lee.jones@linaro.org> > > --- > > arch/arm/boot/dts/stih407-family.dtsi | 48 +++++++++++++++++++++++++++++++++++ > > 1 file changed, 48 insertions(+) > > > > diff --git a/arch/arm/boot/dts/stih407-family.dtsi b/arch/arm/boot/dts/stih407-family.dtsi > > index 33daa49..89a363c 100644 > > --- a/arch/arm/boot/dts/stih407-family.dtsi > > +++ b/arch/arm/boot/dts/stih407-family.dtsi > > @@ -608,5 +608,53 @@ > > status = "okay"; > > tx-only; > > }; > > + > > + st231_gp0: st231-gp0@0 { > > + compatible = "st,st231-rproc"; > > + reg = <0x40000000 0x01000000>; > > + reg-names = "carveout"; > > + resets = <&softreset STIH407_ST231_GP0_SOFTRESET>; > > Support for these co-processor reset lines does not exist upstream. So > this series as it stands won't compile. That's true. I have the support as part of some DEBUG patches I have in my branch. Are you aware of a plan for these, or should I upstream them myself as part of this set?
Hi Lee, On Tue, 01 Sep 2015, Lee Jones wrote: > On Tue, 01 Sep 2015, Peter Griffin wrote: > > On Fri, 28 Aug 2015, Lee Jones wrote: > > > > > Signed-off-by: Ludovic Barre <ludovic.barre@st.com> > > > Signed-off-by: Lee Jones <lee.jones@linaro.org> > > > --- > > > arch/arm/boot/dts/stih407-family.dtsi | 48 +++++++++++++++++++++++++++++++++++ > > > 1 file changed, 48 insertions(+) > > > > > > diff --git a/arch/arm/boot/dts/stih407-family.dtsi b/arch/arm/boot/dts/stih407-family.dtsi > > > index 33daa49..89a363c 100644 > > > --- a/arch/arm/boot/dts/stih407-family.dtsi > > > +++ b/arch/arm/boot/dts/stih407-family.dtsi > > > @@ -608,5 +608,53 @@ > > > status = "okay"; > > > tx-only; > > > }; > > > + > > > + st231_gp0: st231-gp0@0 { > > > + compatible = "st,st231-rproc"; > > > + reg = <0x40000000 0x01000000>; > > > + reg-names = "carveout"; > > > + resets = <&softreset STIH407_ST231_GP0_SOFTRESET>; > > > > Support for these co-processor reset lines does not exist upstream. So > > this series as it stands won't compile. > > That's true. I have the support as part of some DEBUG patches I have > in my branch. Are you aware of a plan for these, or should I upstream > them myself as part of this set? I'm not aware of any plan, so I think upstreaming as part of this set would be good. regards, Peter. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/arch/arm/boot/dts/stih407-family.dtsi b/arch/arm/boot/dts/stih407-family.dtsi index 33daa49..89a363c 100644 --- a/arch/arm/boot/dts/stih407-family.dtsi +++ b/arch/arm/boot/dts/stih407-family.dtsi @@ -608,5 +608,53 @@ status = "okay"; tx-only; }; + + st231_gp0: st231-gp0@0 { + compatible = "st,st231-rproc"; + reg = <0x40000000 0x01000000>; + reg-names = "carveout"; + resets = <&softreset STIH407_ST231_GP0_SOFTRESET>; + reset-names = "sw_reset"; + clocks = <&clk_s_c0_flexgen CLK_ST231_GP_0>; + clock-names = "rproc_clk"; + clock-frequency = <600000000>; + st,syscfg-boot = <&syscfg_core 0x22C>; + }; + + st231_gp1: st231-gp1@0 { + compatible = "st,st231-rproc"; + reg = <0x41000000 0x01000000>; + reg-names = "carveout"; + resets = <&softreset STIH407_ST231_GP1_SOFTRESET>; + reset-names = "sw_reset"; + clocks = <&clk_s_c0_flexgen CLK_ST231_GP_1>; + clock-names = "rproc_clk"; + clock-frequency = <600000000>; + st,syscfg-boot = <&syscfg_core 0x220>; + }; + + st231_audio: st231-audio@0 { + compatible = "st,st231-rproc"; + reg = <0x42000000 0x01000000>; + reg-names = "carveout"; + resets = <&softreset STIH407_ST231_AUD_SOFTRESET>; + reset-names = "sw_reset"; + clocks = <&clk_s_c0_flexgen CLK_ST231_AUD_0>; + clock-names = "rproc_clk"; + clock-frequency = <600000000>; + st,syscfg-boot = <&syscfg_core 0x228>; + }; + + st231_dmu: st231-dmu@0 { + compatible = "st,st231-rproc"; + reg = <0x43000000 0x01000000>; + reg-names = "carveout"; + resets = <&softreset STIH407_ST231_DMU_SOFTRESET>; + reset-names = "sw_reset"; + clocks = <&clk_s_c0_flexgen CLK_ST231_DMU>; + clock-names = "rproc_clk"; + clock-frequency = <600000000>; + st,syscfg-boot = <&syscfg_core 0x224>; + }; }; };