2023-01-23 15:16:15

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH] ARM: dts: mstar: align UART node name with bindings

Bindings expect UART/serial node names to be "serial".

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
arch/arm/boot/dts/mstar-v7.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/mstar-v7.dtsi b/arch/arm/boot/dts/mstar-v7.dtsi
index c26ba9b7b6dd..3eeafd8c7121 100644
--- a/arch/arm/boot/dts/mstar-v7.dtsi
+++ b/arch/arm/boot/dts/mstar-v7.dtsi
@@ -174,7 +174,7 @@ gpio: gpio@207800 {
status = "disabled";
};

- pm_uart: uart@221000 {
+ pm_uart: serial@221000 {
compatible = "ns16550a";
reg = <0x221000 0x100>;
reg-shift = <3>;
--
2.34.1



2023-01-25 10:20:21

by Daniel Palmer

[permalink] [raw]
Subject: Re: [PATCH] ARM: dts: mstar: align UART node name with bindings

Hi,

On Tue, 24 Jan 2023 at 00:15, Krzysztof Kozlowski
<[email protected]> wrote:
>
> Bindings expect UART/serial node names to be "serial".

snip

> - pm_uart: uart@221000 {
> + pm_uart: serial@221000 {
> compatible = "ns16550a";
> reg = <0x221000 0x100>;
> reg-shift = <3>;
> --
> 2.34.1
>

I have this change in my changes to use the proper uart driver that I
need to fix up and resend but it doesn't hurt to fix it now.

Reviewed-by: Daniel Palmer <[email protected]>

But we don't have anything queued up in our tree to send a PR for so
maybe this should be merged via soc.

2023-01-25 10:37:51

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] ARM: dts: mstar: align UART node name with bindings

On 25/01/2023 11:20, Daniel Palmer wrote:
> Hi,
>
> On Tue, 24 Jan 2023 at 00:15, Krzysztof Kozlowski
> <[email protected]> wrote:
>>
>> Bindings expect UART/serial node names to be "serial".
>
> snip
>
>> - pm_uart: uart@221000 {
>> + pm_uart: serial@221000 {
>> compatible = "ns16550a";
>> reg = <0x221000 0x100>;
>> reg-shift = <3>;
>> --
>> 2.34.1
>>
>
> I have this change in my changes to use the proper uart driver that I
> need to fix up and resend but it doesn't hurt to fix it now.
>
> Reviewed-by: Daniel Palmer <[email protected]>
>
> But we don't have anything queued up in our tree to send a PR for so
> maybe this should be merged via soc.

I can take it with my other cleanups. Thanks.

Best regards,
Krzysztof


2023-01-25 10:39:32

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] ARM: dts: mstar: align UART node name with bindings

On Mon, 23 Jan 2023 16:15:23 +0100, Krzysztof Kozlowski wrote:
> Bindings expect UART/serial node names to be "serial".
>
>

Applied, thanks!

[1/1] ARM: dts: mstar: align UART node name with bindings
https://git.kernel.org/krzk/linux-dt/c/5e4505d22f2c3c739bdd063e5348bca3ff34dae2

Best regards,
--
Krzysztof Kozlowski <[email protected]>