* [PATCH] arm64: renesas: r8a77990-ebisu: remove clkout-lr-synchronous from rcar_sound
@ 2019-12-16 2:08 ` Kuninori Morimoto
0 siblings, 0 replies; 7+ messages in thread
From: Kuninori Morimoto @ 2019-12-16 2:08 UTC (permalink / raw)
To: Rob Herring, Mark Rutland, Geert Uytterhoeven
Cc: Patrice Chotard, Linux-Renesas, Linux-DT, Linux-ARM
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
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH] arm64: renesas: r8a77990-ebisu: remove clkout-lr-synchronous from rcar_sound
@ 2019-12-16 2:08 ` Kuninori Morimoto
0 siblings, 0 replies; 7+ messages in thread
From: Kuninori Morimoto @ 2019-12-16 2:08 UTC (permalink / raw)
To: Rob Herring, Mark Rutland, Geert Uytterhoeven
Cc: Linux-Renesas, Linux-DT, Patrice Chotard, Linux-ARM
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
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] arm64: renesas: r8a77990-ebisu: remove clkout-lr-synchronous from rcar_sound
2019-12-16 2:08 ` Kuninori Morimoto
@ 2019-12-16 8:10 ` Geert Uytterhoeven
-1 siblings, 0 replies; 7+ messages in thread
From: Geert Uytterhoeven @ 2019-12-16 8:10 UTC (permalink / raw)
To: Kuninori Morimoto
Cc: Rob Herring, Mark Rutland, Geert Uytterhoeven, Patrice Chotard,
Linux-Renesas, Linux-DT, Linux-ARM
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
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] arm64: renesas: r8a77990-ebisu: remove clkout-lr-synchronous from rcar_sound
@ 2019-12-16 8:10 ` Geert Uytterhoeven
0 siblings, 0 replies; 7+ messages in thread
From: Geert Uytterhoeven @ 2019-12-16 8:10 UTC (permalink / raw)
To: Kuninori Morimoto
Cc: Mark Rutland, Linux-DT, Geert Uytterhoeven, Patrice Chotard,
Linux-Renesas, Rob Herring, Linux-ARM
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
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] arm64: renesas: r8a77990-ebisu: remove clkout-lr-synchronous from rcar_sound
2019-12-16 8:10 ` Geert Uytterhoeven
@ 2019-12-16 8:19 ` Kuninori Morimoto
-1 siblings, 0 replies; 7+ messages in thread
From: Kuninori Morimoto @ 2019-12-16 8:19 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Rob Herring, Mark Rutland, Geert Uytterhoeven, Patrice Chotard,
Linux-Renesas, Linux-DT, Linux-ARM
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
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] arm64: renesas: r8a77990-ebisu: remove clkout-lr-synchronous from rcar_sound
@ 2019-12-16 8:19 ` Kuninori Morimoto
0 siblings, 0 replies; 7+ messages in thread
From: Kuninori Morimoto @ 2019-12-16 8:19 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Mark Rutland, Linux-DT, Geert Uytterhoeven, Patrice Chotard,
Linux-Renesas, Rob Herring, Linux-ARM
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
^ permalink raw reply [flat|nested] 7+ messages in thread
* Patchwork summary for: linux-renesas-soc
2019-12-16 2:08 ` Kuninori Morimoto
(?)
(?)
@ 2019-12-20 17:40 ` patchwork-bot+linux-renesas-soc
-1 siblings, 0 replies; 7+ messages in thread
From: patchwork-bot+linux-renesas-soc @ 2019-12-20 17:40 UTC (permalink / raw)
To: 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
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2019-12-20 17:40 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-16 2:08 [PATCH] arm64: renesas: r8a77990-ebisu: remove clkout-lr-synchronous from rcar_sound Kuninori Morimoto
2019-12-16 2:08 ` Kuninori Morimoto
2019-12-16 8:10 ` Geert Uytterhoeven
2019-12-16 8:10 ` Geert Uytterhoeven
2019-12-16 8:19 ` Kuninori Morimoto
2019-12-16 8:19 ` Kuninori Morimoto
2019-12-20 17:40 ` Patchwork summary for: linux-renesas-soc patchwork-bot+linux-renesas-soc
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.