2022-05-17 14:54:28

by Conor Paxton

[permalink] [raw]
Subject: [PATCH] riscv: dts: microchip: fix gpio1 reg property typo

Fix reg address typo in the gpio1 stanza.

Signed-off-by: Conor Paxton <[email protected]>
---
arch/riscv/boot/dts/microchip/microchip-mpfs.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/riscv/boot/dts/microchip/microchip-mpfs.dtsi b/arch/riscv/boot/dts/microchip/microchip-mpfs.dtsi
index 746c4d4e7686..cf2f55e1dcb6 100644
--- a/arch/riscv/boot/dts/microchip/microchip-mpfs.dtsi
+++ b/arch/riscv/boot/dts/microchip/microchip-mpfs.dtsi
@@ -366,7 +366,7 @@ gpio0: gpio@20120000 {

gpio1: gpio@20121000 {
compatible = "microchip,mpfs-gpio";
- reg = <000 0x20121000 0x0 0x1000>;
+ reg = <0x0 0x20121000 0x0 0x1000>;
interrupt-parent = <&plic>;
interrupt-controller;
#interrupt-cells = <1>;
--
2.25.1



2022-05-17 18:07:48

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] riscv: dts: microchip: fix gpio1 reg property typo

On 17/05/2022 12:40, Conor Paxton wrote:
> Fix reg address typo in the gpio1 stanza.
>
> Signed-off-by: Conor Paxton <[email protected]>
> ---
> arch/riscv/boot/dts/microchip/microchip-mpfs.dtsi | 2 +-


Reviewed-by: Krzysztof Kozlowski <[email protected]>


Best regards,
Krzysztof

2022-05-18 03:15:38

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH] riscv: dts: microchip: fix gpio1 reg property typo



On 17/05/2022 16:53, Palmer Dabbelt wrote:
> On Tue, 17 May 2022 07:51:32 PDT (-0700), [email protected] wrote:
>> On 17/05/2022 12:40, Conor Paxton wrote:
>>> Fix reg address typo in the gpio1 stanza.
>>>
>>> Signed-off-by: Conor Paxton <[email protected]>
>>> ---
>>>  arch/riscv/boot/dts/microchip/microchip-mpfs.dtsi | 2 +-
>>
>> Pressed send to fast...
>>
>> Please add Fixes tag, unless fixed commit SHA is not considered stable.
>
> Conor might be asleep, and as it's late in the cycle I figured I'd just chime in directly.  This LGTM, I usually add fixes myself if there aren't any.  Unless there's any objections to

It's not even 1700 yet, neither Conor on the mail is asleep!

>
> Fixes: 528a5b1f2556 ("riscv: dts: microchip: add new peripherals to icicle kit device tree")

Saw Krzysztof's mail as I went out the door, I was gonna add an RB & the fixes tag
once I got home but beaten to the punch.
FWIW at this point:

Reviewed-by: Conor Dooley <[email protected]>

>
> I'll put this on fixes.
>
> Thanks!
>
>
>>
>>
>> Best regards,
>> Krzysztof

2022-05-18 03:42:48

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] riscv: dts: microchip: fix gpio1 reg property typo

On 17/05/2022 12:40, Conor Paxton wrote:
> Fix reg address typo in the gpio1 stanza.
>
> Signed-off-by: Conor Paxton <[email protected]>
> ---
> arch/riscv/boot/dts/microchip/microchip-mpfs.dtsi | 2 +-

Pressed send to fast...

Please add Fixes tag, unless fixed commit SHA is not considered stable.


Best regards,
Krzysztof

2022-05-18 04:08:09

by Palmer Dabbelt

[permalink] [raw]
Subject: Re: [PATCH] riscv: dts: microchip: fix gpio1 reg property typo

On Tue, 17 May 2022 07:51:32 PDT (-0700), [email protected] wrote:
> On 17/05/2022 12:40, Conor Paxton wrote:
>> Fix reg address typo in the gpio1 stanza.
>>
>> Signed-off-by: Conor Paxton <[email protected]>
>> ---
>> arch/riscv/boot/dts/microchip/microchip-mpfs.dtsi | 2 +-
>
> Pressed send to fast...
>
> Please add Fixes tag, unless fixed commit SHA is not considered stable.

Conor might be asleep, and as it's late in the cycle I figured I'd just
chime in directly. This LGTM, I usually add fixes myself if there
aren't any. Unless there's any objections to

Fixes: 528a5b1f2556 ("riscv: dts: microchip: add new peripherals to icicle kit device tree")

I'll put this on fixes.

Thanks!


>
>
> Best regards,
> Krzysztof