On Tue, Aug 15, 2017 at 11:01 PM, Clément Bœsch <u@xxxxxx> 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@xxxxxxxxxx>
>> > Signed-off-by: Li Pengcheng <lipengcheng8@xxxxxxxxxx>
>> > ---
>> > 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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
