| Message ID | cover.1414558424.git.horms+renesas@verge.net.au |
|---|---|
| State | New |
| Headers | show |
On Wed, Oct 29, 2014 at 01:58:17PM +0900, Simon Horman wrote: > Hi Olof, Hi Kevin, Hi Arnd, > > Please consider these Renesas ARM based SoC clocks updates for v3.19. > > > The following changes since commit f114040e3ea6e07372334ade75d1ee0775c355e1: > > Linux 3.18-rc1 (2014-10-19 18:08:38 -0700) > > are available in the git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-clocks-for-v3.19 > > for you to fetch changes up to 2b02b8020fad6ed505ff9cfb1b418c4b59f102a5: > > ARM: shmobile: r8a7740 legacy: Add missing INTCA clock for irqpin module (2014-10-24 10:30:50 +0900) > > ---------------------------------------------------------------- > Renesas ARM Based SoC Clocks Updates for v3.19 > > * Add missing INTCA clock for irqpin module on r8a7740 > > ---------------------------------------------------------------- > Geert Uytterhoeven (1): > ARM: shmobile: r8a7740 legacy: Add missing INTCA clock for irqpin module Hi, I'm trying to categorize this patch and it's hard. Is it a fix for a bug? Support for new firmware that doesn't enable this clock? How does the breakage show itself, and in what circumstances? Essentially, the patch description doesn't describe why it's needed and what it fixes. Which means I don't know where to merge it to. -Olof
On Tue, Nov 04, 2014 at 02:30:54PM -0800, Olof Johansson wrote: > On Wed, Oct 29, 2014 at 01:58:17PM +0900, Simon Horman wrote: > > Hi Olof, Hi Kevin, Hi Arnd, > > > > Please consider these Renesas ARM based SoC clocks updates for v3.19. > > > > > > The following changes since commit f114040e3ea6e07372334ade75d1ee0775c355e1: > > > > Linux 3.18-rc1 (2014-10-19 18:08:38 -0700) > > > > are available in the git repository at: > > > > git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-clocks-for-v3.19 > > > > for you to fetch changes up to 2b02b8020fad6ed505ff9cfb1b418c4b59f102a5: > > > > ARM: shmobile: r8a7740 legacy: Add missing INTCA clock for irqpin module (2014-10-24 10:30:50 +0900) > > > > ---------------------------------------------------------------- > > Renesas ARM Based SoC Clocks Updates for v3.19 > > > > * Add missing INTCA clock for irqpin module on r8a7740 > > > > ---------------------------------------------------------------- > > Geert Uytterhoeven (1): > > ARM: shmobile: r8a7740 legacy: Add missing INTCA clock for irqpin module > > Hi, > > I'm trying to categorize this patch and it's hard. Is it a fix for a > bug? Support for new firmware that doesn't enable this clock? How does > the breakage show itself, and in what circumstances? Essentially, the > patch description doesn't describe why it's needed and what it fixes. > > Which means I don't know where to merge it to. I believe that this fixes a problem that doesn't manifest. It ensures that the clock required for the irqpin controller is available. But as it is in practice the boot loader appears to make the clock available and its been working due to that for several years. And I'm not aware of anyone for whom it is not working without this patch. My personal feeling is that you should just queue it up as a regular non-bug-fix patch. Sorry for not making that clearer in the first place.
Hi Olof, Simon, On Wed, Nov 5, 2014 at 5:04 AM, Simon Horman <horms@verge.net.au> wrote: >> > Renesas ARM Based SoC Clocks Updates for v3.19 >> > >> > * Add missing INTCA clock for irqpin module on r8a7740 >> > >> > ---------------------------------------------------------------- >> > Geert Uytterhoeven (1): >> > ARM: shmobile: r8a7740 legacy: Add missing INTCA clock for irqpin module >> >> Hi, >> >> I'm trying to categorize this patch and it's hard. Is it a fix for a >> bug? Support for new firmware that doesn't enable this clock? How does >> the breakage show itself, and in what circumstances? Essentially, the >> patch description doesn't describe why it's needed and what it fixes. >> >> Which means I don't know where to merge it to. > > I believe that this fixes a problem that doesn't manifest. > > It ensures that the clock required for the irqpin controller is available. > But as it is in practice the boot loader appears to make the clock > available and its been working due to that for several years. And I'm not > aware of anyone for whom it is not working without this patch. That was the first part. There's a second part (which I indeed forgot to mention in the commit message, sorry for that). If the system doesn't know about the clock, it cannot disable it when the affected devices are not used (system suspend, or runtime PM), thus wasting power. 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 Tue, Nov 4, 2014 at 11:30 PM, Olof Johansson <olof@lixom.net> wrote: > On Wed, Oct 29, 2014 at 01:58:17PM +0900, Simon Horman wrote: >> Hi Olof, Hi Kevin, Hi Arnd, >> >> Please consider these Renesas ARM based SoC clocks updates for v3.19. >> >> >> The following changes since commit f114040e3ea6e07372334ade75d1ee0775c355e1: >> >> Linux 3.18-rc1 (2014-10-19 18:08:38 -0700) >> >> are available in the git repository at: >> >> git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-clocks-for-v3.19 >> >> for you to fetch changes up to 2b02b8020fad6ed505ff9cfb1b418c4b59f102a5: >> >> ARM: shmobile: r8a7740 legacy: Add missing INTCA clock for irqpin module (2014-10-24 10:30:50 +0900) >> >> ---------------------------------------------------------------- >> Renesas ARM Based SoC Clocks Updates for v3.19 >> >> * Add missing INTCA clock for irqpin module on r8a7740 >> >> ---------------------------------------------------------------- >> Geert Uytterhoeven (1): >> ARM: shmobile: r8a7740 legacy: Add missing INTCA clock for irqpin module > > Hi, > > I'm trying to categorize this patch and it's hard. Is it a fix for a > bug? Support for new firmware that doesn't enable this clock? How does > the breakage show itself, and in what circumstances? Essentially, the > patch description doesn't describe why it's needed and what it fixes. > > Which means I don't know where to merge it to. Unfortunately I've just discovered a stupid typo in that patch. I've sent a fix ("[PATCH] ARM: shmobile: r8a7740 legacy: Fix INTCA clock parent"). If you prefer a complete new patch including the fix, just let me know. Sorry for the inconvenience. 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 Wed, Nov 5, 2014 at 1:11 AM, Geert Uytterhoeven <geert@linux-m68k.org> wrote: > On Tue, Nov 4, 2014 at 11:30 PM, Olof Johansson <olof@lixom.net> wrote: >> On Wed, Oct 29, 2014 at 01:58:17PM +0900, Simon Horman wrote: >>> Hi Olof, Hi Kevin, Hi Arnd, >>> >>> Please consider these Renesas ARM based SoC clocks updates for v3.19. >>> >>> >>> The following changes since commit f114040e3ea6e07372334ade75d1ee0775c355e1: >>> >>> Linux 3.18-rc1 (2014-10-19 18:08:38 -0700) >>> >>> are available in the git repository at: >>> >>> git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-clocks-for-v3.19 >>> >>> for you to fetch changes up to 2b02b8020fad6ed505ff9cfb1b418c4b59f102a5: >>> >>> ARM: shmobile: r8a7740 legacy: Add missing INTCA clock for irqpin module (2014-10-24 10:30:50 +0900) >>> >>> ---------------------------------------------------------------- >>> Renesas ARM Based SoC Clocks Updates for v3.19 >>> >>> * Add missing INTCA clock for irqpin module on r8a7740 >>> >>> ---------------------------------------------------------------- >>> Geert Uytterhoeven (1): >>> ARM: shmobile: r8a7740 legacy: Add missing INTCA clock for irqpin module >> >> Hi, >> >> I'm trying to categorize this patch and it's hard. Is it a fix for a >> bug? Support for new firmware that doesn't enable this clock? How does >> the breakage show itself, and in what circumstances? Essentially, the >> patch description doesn't describe why it's needed and what it fixes. >> >> Which means I don't know where to merge it to. > > Unfortunately I've just discovered a stupid typo in that patch. > I've sent a fix ("[PATCH] ARM: shmobile: r8a7740 legacy: Fix INTCA > clock parent"). > > If you prefer a complete new patch including the fix, just let me know. Since I haven't applied it yet, feel free to squash it in and send a fresh pull request. That way you can revisit the commit message too! -Olof
Hi Olof, On Wed, Nov 5, 2014 at 8:14 PM, Olof Johansson <olof@lixom.net> wrote: >>>> Geert Uytterhoeven (1): >>>> ARM: shmobile: r8a7740 legacy: Add missing INTCA clock for irqpin module >>> >>> I'm trying to categorize this patch and it's hard. Is it a fix for a >>> bug? Support for new firmware that doesn't enable this clock? How does >>> the breakage show itself, and in what circumstances? Essentially, the >>> patch description doesn't describe why it's needed and what it fixes. >>> >>> Which means I don't know where to merge it to. >> >> Unfortunately I've just discovered a stupid typo in that patch. >> I've sent a fix ("[PATCH] ARM: shmobile: r8a7740 legacy: Fix INTCA >> clock parent"). >> >> If you prefer a complete new patch including the fix, just let me know. > > Since I haven't applied it yet, feel free to squash it in and send a > fresh pull request. That way you can revisit the commit message too! Sent v2. 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
Hi Olof, Hi Kevin, Hi Arnd, Please consider these Renesas ARM based SoC clocks updates for v3.19. The following changes since commit f114040e3ea6e07372334ade75d1ee0775c355e1: Linux 3.18-rc1 (2014-10-19 18:08:38 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-clocks-for-v3.19 for you to fetch changes up to 2b02b8020fad6ed505ff9cfb1b418c4b59f102a5: ARM: shmobile: r8a7740 legacy: Add missing INTCA clock for irqpin module (2014-10-24 10:30:50 +0900) ---------------------------------------------------------------- Renesas ARM Based SoC Clocks Updates for v3.19 * Add missing INTCA clock for irqpin module on r8a7740 ---------------------------------------------------------------- Geert Uytterhoeven (1): ARM: shmobile: r8a7740 legacy: Add missing INTCA clock for irqpin module arch/arm/mach-shmobile/clock-r8a7740.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-)