[PATCH] arm64: renesas: r8a77990-ebisu: remove clkout-lr-synchronous from rcar_sound

[PATCH] arm64: renesas: r8a77990-ebisu: remove clkout-lr-synchronous from rcar_sound

[Kuninori Morimoto]

From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

rcar_sound doesn't support clkout-lr-synchronous in upstream.
It was supported under out-of-tree rcar_sound.
upstream rcar_sound is supporting
	- clkout-lr-synchronous
	+ clkout-lr-asynchronous

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
index b38f9d4..e6d700f 100644
--- a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
+++ b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
@@ -636,7 +636,6 @@
 	/* audio_clkout0/1/2/3 */
 	#clock-cells = <1>;
 	clock-frequency = <12288000 11289600>;
-	clkout-lr-synchronous;
 
 	status = "okay";
 
-- 
2.7.4

[PATCH] arm64: renesas: r8a77990-ebisu: remove clkout-lr-synchronous from rcar_sound

[Kuninori Morimoto]

From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

rcar_sound doesn't support clkout-lr-synchronous in upstream.
It was supported under out-of-tree rcar_sound.
upstream rcar_sound is supporting
	- clkout-lr-synchronous
	+ clkout-lr-asynchronous

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
index b38f9d4..e6d700f 100644
--- a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
+++ b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
@@ -636,7 +636,6 @@
 	/* audio_clkout0/1/2/3 */
 	#clock-cells = <1>;
 	clock-frequency = <12288000 11289600>;
-	clkout-lr-synchronous;
 
 	status = "okay";
 
-- 
2.7.4


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [PATCH] arm64: renesas: r8a77990-ebisu: remove clkout-lr-synchronous from rcar_sound

[Geert Uytterhoeven]

Hi Morimoto-san,

On Mon, Dec 16, 2019 at 3:08 AM Kuninori Morimoto
<kuninori.morimoto.gx@renesas.com> wrote:
> From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
>
> rcar_sound doesn't support clkout-lr-synchronous in upstream.
> It was supported under out-of-tree rcar_sound.
> upstream rcar_sound is supporting
>         - clkout-lr-synchronous
>         + clkout-lr-asynchronous
>
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

Thanks for your patch!

IIUIC, as no upstream code looks at this property, there is no need to
apply this as a fix for v5.5, and it is safe to queue this for v5.6?

Nevertheless, I think it is good to add
Fixes: 56629fcba94c698d ("arm64: dts: renesas: ebisu: Enable Audio")
No need to resend, I can add that while applying.

Thank you!

> --- a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> @@ -636,7 +636,6 @@
>         /* audio_clkout0/1/2/3 */
>         #clock-cells = <1>;
>         clock-frequency = <12288000 11289600>;
> -       clkout-lr-synchronous;
>
>         status = "okay";

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Re: [PATCH] arm64: renesas: r8a77990-ebisu: remove clkout-lr-synchronous from rcar_sound

[Geert Uytterhoeven]

Hi Morimoto-san,

On Mon, Dec 16, 2019 at 3:08 AM Kuninori Morimoto
<kuninori.morimoto.gx@renesas.com> wrote:
> From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
>
> rcar_sound doesn't support clkout-lr-synchronous in upstream.
> It was supported under out-of-tree rcar_sound.
> upstream rcar_sound is supporting
>         - clkout-lr-synchronous
>         + clkout-lr-asynchronous
>
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

Thanks for your patch!

IIUIC, as no upstream code looks at this property, there is no need to
apply this as a fix for v5.5, and it is safe to queue this for v5.6?

Nevertheless, I think it is good to add
Fixes: 56629fcba94c698d ("arm64: dts: renesas: ebisu: Enable Audio")
No need to resend, I can add that while applying.

Thank you!

> --- a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> @@ -636,7 +636,6 @@
>         /* audio_clkout0/1/2/3 */
>         #clock-cells = <1>;
>         clock-frequency = <12288000 11289600>;
> -       clkout-lr-synchronous;
>
>         status = "okay";

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [PATCH] arm64: renesas: r8a77990-ebisu: remove clkout-lr-synchronous from rcar_sound

[Kuninori Morimoto]

Hi Geert

Thank you for your feedback

> > From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> >
> > rcar_sound doesn't support clkout-lr-synchronous in upstream.
> > It was supported under out-of-tree rcar_sound.
> > upstream rcar_sound is supporting
> >         - clkout-lr-synchronous
> >         + clkout-lr-asynchronous
> >
> > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
(snip)
> IIUIC, as no upstream code looks at this property, there is no need to
> apply this as a fix for v5.5, and it is safe to queue this for v5.6?

Yes, I think so.
Driver never use/check it.
This is just for cleanup.

> Nevertheless, I think it is good to add
> Fixes: 56629fcba94c698d ("arm64: dts: renesas: ebisu: Enable Audio")
> No need to resend, I can add that while applying.

Ah, yes please.
Thank you for your help

Thank you for your help !!
Best regards
---
Kuninori Morimoto

Re: [PATCH] arm64: renesas: r8a77990-ebisu: remove clkout-lr-synchronous from rcar_sound

[Kuninori Morimoto]

Hi Geert

Thank you for your feedback

> > From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> >
> > rcar_sound doesn't support clkout-lr-synchronous in upstream.
> > It was supported under out-of-tree rcar_sound.
> > upstream rcar_sound is supporting
> >         - clkout-lr-synchronous
> >         + clkout-lr-asynchronous
> >
> > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
(snip)
> IIUIC, as no upstream code looks at this property, there is no need to
> apply this as a fix for v5.5, and it is safe to queue this for v5.6?

Yes, I think so.
Driver never use/check it.
This is just for cleanup.

> Nevertheless, I think it is good to add
> Fixes: 56629fcba94c698d ("arm64: dts: renesas: ebisu: Enable Audio")
> No need to resend, I can add that while applying.

Ah, yes please.
Thank you for your help

Thank you for your help !!
Best regards
---
Kuninori Morimoto

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Patchwork summary for: linux-renesas-soc

[patchwork-bot+linux-renesas-soc]

Hello:

The following patches were marked "accepted", because they were applied to
geert/renesas-devel (refs/heads/next):

Patch: arm64: renesas: r8a77990-ebisu: remove clkout-lr-synchronous from rcar_sound
  Submitter: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
  Patchwork: https://patchwork.kernel.org/project/linux-renesas-soc/list/?series=217187

Total patches: 1

-- 
Deet-doot-dot, I am a bot.
https://korg.wiki.kernel.org/userdoc/pwbot