| Message ID | 1492478242-16146-2-git-send-email-zhangfei.gao@linaro.org |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [1/2] clk: hi6220: add acpu clock | expand |
Hi Zhangfei, On 2017/4/18 2:17, Zhangfei Gao wrote: > Add acpu_sctrl clock node > > Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org> > Signed-off-by: Li Pengcheng <lipengcheng8@huawei.com> Fine to me. Thanks! Acked-by: Wei Xu <xuwei5@hisilicon.com> BR, Wei > --- > arch/arm64/boot/dts/hisilicon/hi6220.dtsi | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi > index 470461d..710cc34 100644 > --- a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi > +++ b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi > @@ -262,6 +262,12 @@ > #clock-cells = <1>; > }; > > + acpu_sctrl: acpu_sctrl@f7032000 { > + compatible = "hisilicon,hi6220-acpu-sctrl", "syscon"; > + reg = <0x0 0xf6504000 0x0 0x1000>; > + #clock-cells = <1>; > + }; > + > medianoc_ade: medianoc_ade@f4520000 { > compatible = "syscon"; > reg = <0x0 0xf4520000 0x0 0x4000>; > -- 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
Hi Zhangfei, On 2017/4/18 2:17, Zhangfei Gao wrote: > Add acpu_sctrl clock node > > Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org> > Signed-off-by: Li Pengcheng <lipengcheng8@huawei.com> > --- > arch/arm64/boot/dts/hisilicon/hi6220.dtsi | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi > index 470461d..710cc34 100644 > --- a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi > +++ b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi > @@ -262,6 +262,12 @@ > #clock-cells = <1>; > }; > > + acpu_sctrl: acpu_sctrl@f7032000 { > + compatible = "hisilicon,hi6220-acpu-sctrl", "syscon"; > + reg = <0x0 0xf6504000 0x0 0x1000>; > + #clock-cells = <1>; > + }; > + > medianoc_ade: medianoc_ade@f4520000 { > compatible = "syscon"; > reg = <0x0 0xf4520000 0x0 0x4000>; > Thanks! Applied to the hisilicon dt tree since the driver part is in the mainline now. Best Regards, Wei -- 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, Aug 15, 2017 at 02:32:43PM +0100, Wei Xu wrote: > Hi Zhangfei, > > On 2017/4/18 2:17, Zhangfei Gao wrote: > > Add acpu_sctrl clock node > > > > Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org> > > Signed-off-by: Li Pengcheng <lipengcheng8@huawei.com> > > --- > > arch/arm64/boot/dts/hisilicon/hi6220.dtsi | 6 ++++++ > > 1 file changed, 6 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi > > index 470461d..710cc34 100644 > > --- a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi > > +++ b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi > > @@ -262,6 +262,12 @@ > > #clock-cells = <1>; > > }; > > > > + acpu_sctrl: acpu_sctrl@f7032000 { I think the unit address is incorrect here (this value has been copied from the pm controller while it should match the 1st one the reg). > > + compatible = "hisilicon,hi6220-acpu-sctrl", "syscon"; > > + reg = <0x0 0xf6504000 0x0 0x1000>; > > + #clock-cells = <1>; > > + }; > > + > > medianoc_ade: medianoc_ade@f4520000 { > > compatible = "syscon"; > > reg = <0x0 0xf4520000 0x0 0x4000>; > > Also I'm sorry I'm going to ask stupid questions: - why 's' in "sctrl" while all the other node controllers declared around seems to use "ctrl"? - is it possible to have access to the specs where those addresses can be found? The only public doc I could find is the "Hi6220V100 Multi-Mode Application Processor" one where this information doesn't seem present. - is there any work pending adding the pmu counters node (interrupt GCI_SPI 99) ? AFAIK the ACPU controller added here can be used to enable the counters. > > Thanks! > Applied to the hisilicon dt tree since the driver part is in the mainline now. > Is this refering to https://github.com/hisilicon/linux-hisi/commits/next/dt64 ? [...] -- Clément B.
On Tue, Aug 15, 2017 at 11:01 PM, Clément Bœsch <u@pkh.me> wrote: > On Tue, Aug 15, 2017 at 02:32:43PM +0100, Wei Xu wrote: >> Hi Zhangfei, >> >> On 2017/4/18 2:17, Zhangfei Gao wrote: >> > Add acpu_sctrl clock node >> > >> > Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org> >> > Signed-off-by: Li Pengcheng <lipengcheng8@huawei.com> >> > --- >> > arch/arm64/boot/dts/hisilicon/hi6220.dtsi | 6 ++++++ >> > 1 file changed, 6 insertions(+) >> > >> > diff --git a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi >> > index 470461d..710cc34 100644 >> > --- a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi >> > +++ b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi >> > @@ -262,6 +262,12 @@ >> > #clock-cells = <1>; >> > }; >> > >> > + acpu_sctrl: acpu_sctrl@f7032000 { > > I think the unit address is incorrect here (this value has been copied > from the pm controller while it should match the 1st one the reg). > >> > + compatible = "hisilicon,hi6220-acpu-sctrl", "syscon"; >> > + reg = <0x0 0xf6504000 0x0 0x1000>; >> > + #clock-cells = <1>; >> > + }; >> > + >> > medianoc_ade: medianoc_ade@f4520000 { >> > compatible = "syscon"; >> > reg = <0x0 0xf4520000 0x0 0x4000>; >> > > > Also I'm sorry I'm going to ask stupid questions: > > - why 's' in "sctrl" while all the other node controllers declared around > seems to use "ctrl"? > > - is it possible to have access to the specs where those addresses can be > found? The only public doc I could find is the "Hi6220V100 Multi-Mode > Application Processor" one where this information doesn't seem present. > That's the only public doc. But if you need, welcome to discuss with me off-line about referencing kernels for hikey. > - is there any work pending adding the pmu counters node (interrupt > GCI_SPI 99) ? AFAIK the ACPU controller added here can be used to enable > the counters. > For hikey PMU, please have a look of this patchset: - https://github.com/96boards/linux/pull/86/commits Discussion is here: - https://discuss.96boards.org/t/performance-monitoring-unit-pmu/293/4 Due to its implementation is not compatible with upstream, this patchset was not upstreamed. If you are interested, welcome to adding them LKML. Appreciate your effort. -Guodong >> >> Thanks! >> Applied to the hisilicon dt tree since the driver part is in the mainline now. >> > > Is this refering to https://github.com/hisilicon/linux-hisi/commits/next/dt64 ? > > [...] > > -- > Clément B. -- 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 2017年08月15日 23:01, Clément Bœsch wrote: > On Tue, Aug 15, 2017 at 02:32:43PM +0100, Wei Xu wrote: >> Hi Zhangfei, >> >> On 2017/4/18 2:17, Zhangfei Gao wrote: >>> Add acpu_sctrl clock node >>> >>> Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org> >>> Signed-off-by: Li Pengcheng <lipengcheng8@huawei.com> >>> --- >>> arch/arm64/boot/dts/hisilicon/hi6220.dtsi | 6 ++++++ >>> 1 file changed, 6 insertions(+) >>> >>> diff --git a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi >>> index 470461d..710cc34 100644 >>> --- a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi >>> +++ b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi >>> @@ -262,6 +262,12 @@ >>> #clock-cells = <1>; >>> }; >>> >>> + acpu_sctrl: acpu_sctrl@f7032000 { > I think the unit address is incorrect here (this value has been copied > from the pm controller while it should match the 1st one the reg). oops, Yes, you are right, will update. >>> + compatible = "hisilicon,hi6220-acpu-sctrl", "syscon"; >>> + reg = <0x0 0xf6504000 0x0 0x1000>; >>> + #clock-cells = <1>; >>> + }; >>> + >>> medianoc_ade: medianoc_ade@f4520000 { >>> compatible = "syscon"; >>> reg = <0x0 0xf4520000 0x0 0x4000>; >>> > Also I'm sorry I'm going to ask stupid questions: > > - why 's' in "sctrl" while all the other node controllers declared around > seems to use "ctrl"? Just from hisilicon convention, sctrl means system control. acpu_ctrl seems better, more match spec. However clk part using this name already been merged, so will keep use this. > > - is it possible to have access to the specs where those addresses can be > found? The only public doc I could find is the "Hi6220V100 Multi-Mode > Application Processor" one where this information doesn't seem present. https://github.com/96boards/documentation/tree/master/ConsumerEdition/HiKey/HardwareDocs Looks only this part. > > - is there any work pending adding the pmu counters node (interrupt > GCI_SPI 99) ? AFAIK the ACPU controller added here can be used to enable > the counters. This patch is part of enabling core-sight, not touching pmu. >> Thanks! >> Applied to the hisilicon dt tree since the driver part is in the mainline now. >> > Is this refering to https://github.com/hisilicon/linux-hisi/commits/next/dt64 ? Yes. > > [...] > -- 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/arm64/boot/dts/hisilicon/hi6220.dtsi b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi index 470461d..710cc34 100644 --- a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi +++ b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi @@ -262,6 +262,12 @@ #clock-cells = <1>; }; + acpu_sctrl: acpu_sctrl@f7032000 { + compatible = "hisilicon,hi6220-acpu-sctrl", "syscon"; + reg = <0x0 0xf6504000 0x0 0x1000>; + #clock-cells = <1>; + }; + medianoc_ade: medianoc_ade@f4520000 { compatible = "syscon"; reg = <0x0 0xf4520000 0x0 0x4000>;