Re: [PATCH] clk: rockchip: add pclk_cpu to the list of rk3188 critical clocks
From: Heiko Stuebner
Date: Mon Aug 17 2015 - 15:37:44 EST
Hi Michael,
need to pick a bit more on your patch submission :-)
Please take a look for example at
http://lists.infradead.org/pipermail/linux-rockchip/2014-November/001043.html
on how a v2 submission needs to look like.
Specifically:
- In most cases not as a reply to a reply to another mail.
- Mail subject: [PATCH v2] clk: rockchip: add pclk_cpu to the list of rk3188
critical clocks
Am Montag, 17. August 2015, 20:51:14 schrieb Michael Niewoehner:
> Hi Heiko,
>
> I merged yours and mine :-)
>
>
no prose nor empty lines. If necessary such stuff could sit below the "---"
which results in it not making it into the actuall commit message.
> pclk_cpu needs to keep running because it is needed for devices like
> the act8865 regulator but with the recent gpio clock handling this is
> not always the case anymore. So add it to the list of critical clocks.
>
> Signed-off-by: Michael Niewoehner <linux@xxxxxxxxxxxxxx>
> ---
Here any prose-text you like to add.
Also, changelog:
changes in v2:
- adapt commit message
- add Linus Walleij to recipients, as the patch is related to the gpio clock
change in the rockchip pinctrl driver it should go through his tree as well
> drivers/clk/rockchip/clk-rk3188.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/clk/rockchip/clk-rk3188.c
> b/drivers/clk/rockchip/clk-rk3188.c index e4f9d47..1c93229 100644
> --- a/drivers/clk/rockchip/clk-rk3188.c
> +++ b/drivers/clk/rockchip/clk-rk3188.c
> @@ -708,6 +708,7 @@ static const char *const rk3188_critical_clocks[]
> __initconst = { "aclk_cpu",
> "aclk_peri",
> "hclk_peri",
> + "pclk_cpu",
> };
>
> static void __init rk3188_common_clk_init(struct device_node *np)
no quotes of other mails, like the stuff below.
Heiko
>
> > Hi,
> >
> > Am Montag, 17. August 2015, 19:38:22 schrieb Michael Niewoehner:
> >> gpio clock is getting disabled to save power but pclk_cpu is needed for
> >> act8865 regulator
> >
> > Please refine the commit message a bit :-) . Something along
> >
> > pclk_cpu needs to keep running and with the recent gpio clock
> > handling this is not always the case anymore. So add it to the list
> > of critical clocks.
> >
> >
> > and also please add "Linus Walleij <linus.walleij@xxxxxxxxxx>" to the list
> > of recipients. As the gpio clock handling change does go through his
> > tree, the matching critical clock handling should also go through him.
> >
> >
> > Heiko
> >
> >> Signed-off-by: Michael Niewoehner <linux@xxxxxxxxxxxxxx>
> >> ---
> >> drivers/clk/rockchip/clk-rk3188.c | 1 +
> >> 1 file changed, 1 insertion(+)
> >>
> >> diff --git a/drivers/clk/rockchip/clk-rk3188.c
> >> b/drivers/clk/rockchip/clk-rk3188.c index e4f9d47..1c93229 100644
> >> --- a/drivers/clk/rockchip/clk-rk3188.c
> >> +++ b/drivers/clk/rockchip/clk-rk3188.c
> >> @@ -708,6 +708,7 @@ static const char *const rk3188_critical_clocks[]
> >> __initconst = { "aclk_cpu",
> >>
> >> "aclk_peri",
> >> "hclk_peri",
> >>
> >> + "pclk_cpu",
> >> };
> >>
> >> static void __init rk3188_common_clk_init(struct device_node *np)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/