Re: [PATCH 02/11] dts: hisilicon: Update coresight bindings for hardware ports

From: Suzuki K Poulose
Date: Wed Sep 12 2018 - 08:42:15 EST


Hi Leo,

On 12/09/18 11:47, leo.yan@xxxxxxxxxx wrote:
On Tue, Sep 11, 2018 at 11:17:03AM +0100, Suzuki K Poulose wrote:
Switch to updated coresight bindings for hw ports.

As Shawn suggested, please change subject as "arm64: dts: hi6220:
...."


Sure, will do.

Cc: xuwei5@xxxxxxxxxxxxx
Cc: lipengcheng8@xxxxxxxxxx
Cc: Mathieu Poirier <mathieu.poirier@xxxxxxxxxx>
Signed-off-by: Suzuki K Poulose <suzuki.poulose@xxxxxxx>
---
.../arm64/boot/dts/hisilicon/hi6220-coresight.dtsi | 147 ++++++++++++---------
1 file changed, 85 insertions(+), 62 deletions(-)

diff --git a/arch/arm64/boot/dts/hisilicon/hi6220-coresight.dtsi b/arch/arm64/boot/dts/hisilicon/hi6220-coresight.dtsi
index 7afee5d..2202816 100644
--- a/arch/arm64/boot/dts/hisilicon/hi6220-coresight.dtsi
+++ b/arch/arm64/boot/dts/hisilicon/hi6220-coresight.dtsi

[...]

@@ -302,10 +315,12 @@
cpu = <&cpu3>;
- port {
- etm3_out: endpoint {
- remote-endpoint =
- <&acpu_funnel_in3>;
+ out-ports {
+ port {
+ etm3_out: endpoint {
+ remote-endpoint =
+ <&acpu_funnel_in3>;
+ };
};
};
};
@@ -319,10 +334,12 @@
cpu = <&cpu4>;
- port {
+ out-ports {
+ port {
etm4_out: endpoint {

Indent?

Yes, I have fixed this locally for v2.


After applied this patch and tested on Hikey board; FWIW:

Tested-by: Leo Yan <leo.yan@xxxxxxxxxx>

Thanks for the testing !

Cheers
Suzuki