On 17/01/17 13:39, Suzuki K Poulose wrote: > On 17/01/17 12:15, Sudeep Holla wrote: >> From: Mike Leach <mike.leach@xxxxxxxxxx> >> >> This patch adds the missing CoreSight STM component definition to the >> device tree of all the juno variants(r0,r1,r2) >> >> STM component is connected to different funnels depending on Juno >> platform variant. >> >> Reviewed-and-Tested-by: Mathieu Poirier <mathieu.poirier@xxxxxxxxxx> >> Signed-off-by: Mike Leach <mike.leach@xxxxxxxxxx> >> [sudeep.holla@xxxxxxx: minor changelog update and reorganising the STM >> node back into juno-base.dtsi to avoid duplication] >> Signed-off-by: Sudeep Holla <sudeep.holla@xxxxxxx> >> --- >> arch/arm64/boot/dts/arm/juno-base.dtsi | 15 +++++++++++++++ >> arch/arm64/boot/dts/arm/juno-r1.dts | 4 ++++ >> arch/arm64/boot/dts/arm/juno-r2.dts | 4 ++++ >> arch/arm64/boot/dts/arm/juno.dts | 16 ++++++++++++++++ >> 4 files changed, 39 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/arm/juno-base.dtsi >> b/arch/arm64/boot/dts/arm/juno-base.dtsi >> index faedf357db3b..58c1773c3aa4 100644 >> --- a/arch/arm64/boot/dts/arm/juno-base.dtsi >> +++ b/arch/arm64/boot/dts/arm/juno-base.dtsi >> @@ -185,6 +185,21 @@ >> }; >> }; >> >> + stm@20100000 { >> + compatible = "arm,coresight-stm", "arm,primecell"; >> + reg = <0 0x20100000 0 0x1000>, >> + <0 0x28000000 0 0x180000>; > > The size of the stimulus base is 16MB (as per TRM), implies => 0x1000000. > Fixed locally. Mike, Can you confirm ? I know TRM had some issues with size of other coresight components. > with that fixed : > > Reviewed-by: Suzuki K Poulose <suzuki.poulose@xxxxxxx> > Thanks. -- Regards, Sudeep -- 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