Hi, On Thu, Nov 03, 2016 at 01:41:58AM -0400, Anurup M wrote: > From: Tan Xiaojun <tanxiaojun@xxxxxxxxxx> > > 1) Add Hisilicon HiP05/06/07 CPU and ALGSUB system controller dts > bindings. > 2) Add Hisilicon Djtag dts binding. > > Signed-off-by: Tan Xiaojun <tanxiaojun@xxxxxxxxxx> > Signed-off-by: Anurup M <anurup.m@xxxxxxxxxx> > --- > .../bindings/arm/hisilicon/hisilicon.txt | 82 ++++++++++++++++++++++ > 1 file changed, 82 insertions(+) > > diff --git a/Documentation/devicetree/bindings/arm/hisilicon/hisilicon.txt b/Documentation/devicetree/bindings/arm/hisilicon/hisilicon.txt > index 7df79a7..341cbb9 100644 > --- a/Documentation/devicetree/bindings/arm/hisilicon/hisilicon.txt > +++ b/Documentation/devicetree/bindings/arm/hisilicon/hisilicon.txt > @@ -270,3 +270,85 @@ Required Properties: > [1]: bootwrapper size > [2]: relocation physical address > [3]: relocation size > +----------------------------------------------------------------------- > +The Hisilicon Djtag in CPU die is an independent component which connects with > +some other components in the SoC by Debug Bus. This driver can be configured > +to access the registers of connecting components (like L3 cache, l3 cache PMU > + etc.) during real time debugging by sysctrl. These components appear as child > +nodes of djtag. Please put the djtag binding in a new file. It's clearly unrelated to many other things in this file, which should also be split out. > +The Hip05/06/07 CPU system controller(sysctrl) support to manage some important > +components (such as clock, reset, soft reset, secure debugger, etc.). > +The CPU sysctrl registers in hip05/06/07 doesnot use syscon but will be mapped > +by djtag driver for use by connecting components. The djtag driver is irrelvant here. If there's a realtionship between the two, please define that in the binding rather than implicitly assuming it in the driver. > + > +Required properties: > + - compatible : "hisilicon,hip05-cpu-djtag-v1" > + - reg : Register address and size > + > +Hisilicon HiP06 djtag for CPU sysctrl > +Required properties: > +- compatible : "hisilicon,hip06-sysctrl", "syscon", "simple-mfd"; This looks messy. Why is this syscon and a simple-mfd? We should kill off / deprecate the syscon binding. It's completely meaningless. > +- reg : Register address and size > +- djtag : > + - compatible : "hisilicon,hip06-cpu-djtag-v1" > + - reg : Register address and size > + > +Hisilicon HiP07 djtag for CPU sysctrl > +Required properties: > + - compatible : "hisilicon,hip07-cpu-djtag-v2" > + - reg : Register address and size > + > +Example: > + /* for Hisilicon HiP05 djtag for CPU sysctrl */ > + djtag0: djtag@80010000 { > + compatible = "hisilicon,hip05-cpu-djtag-v1"; > + reg = <0x0 0x80010000 0x0 0x10000>; > + > + /* For L3 cache PMU */ > + pmul3c0 { > + compatible = "hisilicon,hisi-pmu-l3c-v1"; > + scl-id = <0x02>; > + num-events = <0x16>; > + num-counters = <0x08>; > + module-id = <0x04>; > + num-banks = <0x04>; > + cfgen-map = <0x02 0x04 0x01 0x08>; > + counter-reg = <0x170>; > + evctrl-reg = <0x04>; > + event-en = <0x1000000>; > + evtype-reg = <0x140>; > + }; This sub-node needs a binding document. These properties are completely opaque > + }; > + > +----------------------------------------------------------------------- > +The Hisilicon HiP05/06/07 ALGSUB system controller(sysctrl) is in IO die > +of SoC. It has a similar function as the Hisilicon HiP05/06/07 CPU system > +controller in CPU die and it manage different components, like RSA, etc. > +The Hisilicon Djtag in IO die has a similar function as in CPU die and maps > +the sysctrl registers for use by connecting components. > +All connecting components shall appear as child nodes of djtag. I don't follow the above. This describes an ALGSUB system controllerm but the documentation below is all about djtag. Thanks, Mark. > +Hisilicon HiP05 djtag for ALGSUB sysctrl > +Required properties: > + - compatible : "hisilicon,hip05-io-djtag-v1" > + - reg : Register address and size > + > +Hisilicon HiP06 djtag for ALGSUB sysctrl > +Required properties: > + - compatible : "hisilicon,hip06-io-djtag-v2" > + - reg : Register address and size > + > +Hisilicon HiP07 djtag for ALGSUB sysctrl > +Required properties: > + - compatible : "hisilicon,hip07-io-djtag-v2" > + - reg : Register address and size > + > +Example: > + /* for Hisilicon HiP05 djtag for alg sysctrl */ > + djtag0: djtag@d0000000 { > + compatible = "hisilicon,hip05-io-djtag-v1"; > + reg = <0x0 0xd0000000 0x0 0x10000>; > + }; > -- > 2.1.4 > -- 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