dtb: amd: Fix-up for ccn504 and kcs nodes

PATCH Applied Suravee Suthikulpanit 2 participants 3 messages Feb 12, 2016 – Feb 25, 2016

Subsystems (2)

Device tree bindings AMD seattle device tree support

lore

Share & tools

lore

Suravee Suthikulpanit

author maintainer

2016-02-12 00:20 UTC #0 lore raw

PATCH 1/1 dtb: amd: Fix-up for ccn504 and kcs nodes

From: Suravee Suthikulpanit <[email protected]>

This is a fix-up patch based on the review comment from Arnd regarding:
* fix ccn504 address in the node name
* remove kcs interrupt-name

Signed-off-by Brijesh Singh <[email protected]>

Signed-off-by Suravee Suthikulpanit <[email protected]>

Cc Arnd Bergmann <[email protected]>

---

Hi Olof,

Here are the changes from the V3 that I sent earlier. Sorry for confusion. Would you please pull this into the next tree?

Thank you,
Suravee

 arch/arm64/boot/dts/amd/amd-seattle-soc.dtsi | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

arch/arm64/boot/dts/amd/amd-seattle-soc.dtsi +1 -2

index a7fc059..bd3adea 100644
@@ -231,7 +231,7 @@
 		};
 
 		/* Perf CCN504 PMU */
-		ccn: ccn at 0xe8000000 {
+		ccn: ccn at e8000000 {
 			compatible = "arm,ccn-504";
 			reg = <0x0 0xe8000000 0 0x1000000>;
 			interrupts = <0 380 4>;

@@ -243,7 +243,6 @@
 			device_type = "ipmi";
 			reg = <0x0 0xe0010000 0 0x8>;
 			interrupts = <0 389 4>;
-			interrupt-names = "ipmi_kcs";
 			reg-size = <1>;
 			reg-spacing = <4>;
 		};

Signature · 1 line

2.5.0

Olof Johansson

Applied

2016-02-24 22:01 UTC #1 lore raw

Quoted 12 lines · Suravee Suthikulpanit

From: Suravee Suthikulpanit <[email protected]>
This is a fix-up patch based on the review comment from Arnd regarding:
* fix ccn504 address in the node name
* remove kcs interrupt-name
Signed-off-by Brijesh Singh <[email protected]>
Signed-off-by Suravee Suthikulpanit <[email protected]>
Cc Arnd Bergmann <[email protected]>
---

Quoted 7 lines

Hi Olof,
Here are the changes from the V3 that I sent earlier. Sorry for confusion. Would you please pull this into the next tree?
Thank you,
Suravee

Applied.

I also noticed that the patch subjects were nonstandard the last time around. In the future, please use:

arm64: dts: amd: ...

as prefix, instead of just dtb (which is the binary format). I fixed it up for this patch, and the others, well, they're already applied. So let's try to remember it for the future.

-Olof

Suravee Suthikulpanit

author maintainer

2016-02-25 05:23 UTC #2 lore raw

Hi,

Quoted 26 lines · Olof Johansson

Quoted 12 lines · Suravee Suthikulpanit
From: Suravee Suthikulpanit <[email protected]>
This is a fix-up patch based on the review comment from Arnd regarding:
* fix ccn504 address in the node name
* remove kcs interrupt-name
Signed-off-by Brijesh Singh <[email protected]>
Signed-off-by Suravee Suthikulpanit <[email protected]>
Cc Arnd Bergmann <[email protected]>
---
Quoted 7 lines
Hi Olof,
Here are the changes from the V3 that I sent earlier. Sorry for confusion. Would you please pull this into the next tree?
Thank you,
Suravee
Applied.
I also noticed that the patch subjects were nonstandard the last time around. In the future, please use:

Quoted 9 lines

arm64: dts: amd: ...
as prefix, instead of just dtb (which is the binary format). I fixed it up for this patch, and the others, well, they're already applied. So let's try to remember it for the future.
-Olof

Thanks, and sorry for the mis-use of the subject line.

Suravee