2022-07-05 17:05:59

by Ryan.Wanner

[permalink] [raw]
Subject: [PATCH] ARM: dts: at91: sama5d2/trivial: Fixed typo in i2s1 node

From: Ryan Wanner <[email protected]>

Fixed typo in i2s1 node to match i2s0 node

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

diff --git a/arch/arm/boot/dts/sama5d2.dtsi b/arch/arm/boot/dts/sama5d2.dtsi
index 89c71d419f82..659a17fc755c 100644
--- a/arch/arm/boot/dts/sama5d2.dtsi
+++ b/arch/arm/boot/dts/sama5d2.dtsi
@@ -1124,7 +1124,7 @@ AT91_XDMAC_DT_PERID(33))>,
clocks = <&pmc PMC_TYPE_PERIPHERAL 55>, <&pmc PMC_TYPE_GCK 55>;
clock-names = "pclk", "gclk";
assigned-clocks = <&pmc PMC_TYPE_CORE PMC_I2S1_MUX>;
- assigned-parrents = <&pmc PMC_TYPE_GCK 55>;
+ assigned-clock-parents = <&pmc PMC_TYPE_GCK 55>;
status = "disabled";
};

--
2.34.1


2022-07-05 18:02:01

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH] ARM: dts: at91: sama5d2/trivial: Fixed typo in i2s1 node

On 05/07/2022 17:42, [email protected] wrote:
> From: Ryan Wanner <[email protected]>
>
> Fixed typo in i2s1 node to match i2s0 node

Do you know what specfic commit this fixes? is it:
Fixes: 1ca81883c557 ("ARM: dts: at91: sama5d2: add nodes for I2S controllers")

>
> Signed-off-by: Ryan Wanner <[email protected]>
> ---
> arch/arm/boot/dts/sama5d2.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/sama5d2.dtsi b/arch/arm/boot/dts/sama5d2.dtsi
> index 89c71d419f82..659a17fc755c 100644
> --- a/arch/arm/boot/dts/sama5d2.dtsi
> +++ b/arch/arm/boot/dts/sama5d2.dtsi
> @@ -1124,7 +1124,7 @@ AT91_XDMAC_DT_PERID(33))>,
> clocks = <&pmc PMC_TYPE_PERIPHERAL 55>, <&pmc PMC_TYPE_GCK 55>;
> clock-names = "pclk", "gclk";
> assigned-clocks = <&pmc PMC_TYPE_CORE PMC_I2S1_MUX>;
> - assigned-parrents = <&pmc PMC_TYPE_GCK 55>;
> + assigned-clock-parents = <&pmc PMC_TYPE_GCK 55>;
> status = "disabled";
> };
>

2022-07-06 09:52:04

by Claudiu Beznea

[permalink] [raw]
Subject: Re: [PATCH] ARM: dts: at91: sama5d2/trivial: Fixed typo in i2s1 node

Hi, Ryan,

There is no need for trivial in patch title. And as Conor suggested please
add the fixes tag in the next version.

Thank you,
Claudiu Beznea

On 05.07.2022 19:42, [email protected] wrote:
> From: Ryan Wanner <[email protected]>
>
> Fixed typo in i2s1 node to match i2s0 node
>
> Signed-off-by: Ryan Wanner <[email protected]>
> ---
> arch/arm/boot/dts/sama5d2.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/sama5d2.dtsi b/arch/arm/boot/dts/sama5d2.dtsi
> index 89c71d419f82..659a17fc755c 100644
> --- a/arch/arm/boot/dts/sama5d2.dtsi
> +++ b/arch/arm/boot/dts/sama5d2.dtsi
> @@ -1124,7 +1124,7 @@ AT91_XDMAC_DT_PERID(33))>,
> clocks = <&pmc PMC_TYPE_PERIPHERAL 55>, <&pmc PMC_TYPE_GCK 55>;
> clock-names = "pclk", "gclk";
> assigned-clocks = <&pmc PMC_TYPE_CORE PMC_I2S1_MUX>;
> - assigned-parrents = <&pmc PMC_TYPE_GCK 55>;
> + assigned-clock-parents = <&pmc PMC_TYPE_GCK 55>;
> status = "disabled";
> };
>