| Message ID | 1994467.CVFvpbaMyF@wasted.cogentembedded.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 059baea1d33a64e752af201140ef122e29f2a19a |
| Delegated to: | Simon Horman |
| Headers | show |
On Sun, Sep 4, 2016 at 9:55 PM, Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> wrote: > When removing the non-existing thermal clock I forgot to remove its > parent from the node's "clocks" property -- this led to a wrong PWM > clock's parent clock. > > Fixes: 2a29f9d6fea8 ("ARM: dts: r8a7794: add MSTP5 clocks") > Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be> 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
On Mon, Sep 05, 2016 at 11:50:41AM +0200, Geert Uytterhoeven wrote: > On Sun, Sep 4, 2016 at 9:55 PM, Sergei Shtylyov > <sergei.shtylyov@cogentembedded.com> wrote: > > When removing the non-existing thermal clock I forgot to remove its > > parent from the node's "clocks" property -- this led to a wrong PWM > > clock's parent clock. > > > > Fixes: 2a29f9d6fea8 ("ARM: dts: r8a7794: add MSTP5 clocks") > > Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> > > Acked-by: Geert Uytterhoeven <geert+renesas@glider.be> Thanks, I have queued this up.
Index: renesas/arch/arm/boot/dts/r8a7794.dtsi =================================================================== --- renesas.orig/arch/arm/boot/dts/r8a7794.dtsi +++ renesas/arch/arm/boot/dts/r8a7794.dtsi @@ -1251,7 +1251,7 @@ mstp5_clks: mstp5_clks@e6150144 { compatible = "renesas,r8a7794-mstp-clocks", "renesas,cpg-mstp-clocks"; reg = <0 0xe6150144 0 4>, <0 0xe615003c 0 4>; - clocks = <&hp_clk>, <&extal_clk>, <&p_clk>; + clocks = <&hp_clk>, <&p_clk>; #clock-cells = <1>; clock-indices = <R8A7794_CLK_AUDIO_DMAC0 R8A7794_CLK_PWM>;
When removing the non-existing thermal clock I forgot to remove its parent from the node's "clocks" property -- this led to a wrong PWM clock's parent clock. Fixes: 2a29f9d6fea8 ("ARM: dts: r8a7794: add MSTP5 clocks") Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> --- The patch is against the 'renesas-devel-20160902-v4.8-rc4' branch of Simon Horman's 'renesas.git' repo... arch/arm/boot/dts/r8a7794.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)