* [RFC PATCH 0/2] clk: renesas: RZ/G2L: Add support for no PM clocks
@ 2022-09-27 20:30 Prabhakar
2022-09-27 20:30 ` [RFC PATCH 1/2] clk: renesas: rzg2l: Don't assume all CPG_MOD clocks support PM Prabhakar
2022-09-27 20:30 ` [RFC PATCH 2/2] clk: renesas: r9a07g044: Mark CRU_SYSCLK and CRU_VCLK as no PM Prabhakar
0 siblings, 2 replies; 7+ messages in thread
From: Prabhakar @ 2022-09-27 20:30 UTC (permalink / raw)
To: Geert Uytterhoeven, Michael Turquette, Stephen Boyd
Cc: Laurent Pinchart, linux-renesas-soc, linux-clk, linux-kernel,
Prabhakar, Biju Das, Lad Prabhakar
From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
Hi All,
This patch series adds support for indicating MOD clocks as no PM (if any).
Patch#1 adds DEF_NO_PM() macro to flagup no PM clock and patch #2 switches
sysclk and vclk clocks to no PM.
Sending it as an RFC as there wasn't any way we could obtain the priv data
due to which I had to create a static global var for rzg2l_cpg_priv.
Cheers,
Prabhakar
Lad Prabhakar (2):
clk: renesas: rzg2l: Don't assume all CPG_MOD clocks support PM
clk: renesas: r9a07g044: Mark CRU_SYSCLK and CRU_VCLK as no PM
drivers/clk/renesas/r9a07g044-cpg.c | 4 ++--
drivers/clk/renesas/rzg2l-cpg.c | 35 +++++++++++++++++++++++++----
drivers/clk/renesas/rzg2l-cpg.h | 12 +++++++---
3 files changed, 42 insertions(+), 9 deletions(-)
--
2.25.1
^ permalink raw reply [flat|nested] 7+ messages in thread
* [RFC PATCH 1/2] clk: renesas: rzg2l: Don't assume all CPG_MOD clocks support PM
2022-09-27 20:30 [RFC PATCH 0/2] clk: renesas: RZ/G2L: Add support for no PM clocks Prabhakar
@ 2022-09-27 20:30 ` Prabhakar
2022-09-28 7:02 ` Biju Das
2022-09-27 20:30 ` [RFC PATCH 2/2] clk: renesas: r9a07g044: Mark CRU_SYSCLK and CRU_VCLK as no PM Prabhakar
1 sibling, 1 reply; 7+ messages in thread
From: Prabhakar @ 2022-09-27 20:30 UTC (permalink / raw)
To: Geert Uytterhoeven, Michael Turquette, Stephen Boyd
Cc: Laurent Pinchart, linux-renesas-soc, linux-clk, linux-kernel,
Prabhakar, Biju Das, Lad Prabhakar
From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
There are cases where not all CPG_MOD clocks should be assumed to support
PM. For example on the CRU block there is a particular sequence that needs
to be followed to initialize the CSI-2 D-PHY in which individual clocks
need to be turned ON/OFF, due to which Runtime PM support wasn't used by
the CRU CSI-2 driver.
This patch adds support to allow indicating if PM is supported by the
CPG_MOD clocks. A new macro is DEF_NO_PM() is added which sets the no_pm
flag in struct rzg2l_mod_clk and when the driver uses Runtime PM support
no_pm flag is checked to see if the clk needs to included as part of
Runtime PM.
CPG_MOD clocks with no_pm flag set need to be individually turned ON/OFF
depending on the requirement of the driver.
Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
---
drivers/clk/renesas/rzg2l-cpg.c | 35 +++++++++++++++++++++++++++++----
drivers/clk/renesas/rzg2l-cpg.h | 12 ++++++++---
2 files changed, 40 insertions(+), 7 deletions(-)
diff --git a/drivers/clk/renesas/rzg2l-cpg.c b/drivers/clk/renesas/rzg2l-cpg.c
index 3ff6ecd61756..d275324909e7 100644
--- a/drivers/clk/renesas/rzg2l-cpg.c
+++ b/drivers/clk/renesas/rzg2l-cpg.c
@@ -114,6 +114,8 @@ struct rzg2l_cpg_priv {
struct rzg2l_pll5_mux_dsi_div_param mux_dsi_div_params;
};
+static struct rzg2l_cpg_priv *rzg2l_cpg_priv;
+
static void rzg2l_cpg_del_clk_provider(void *data)
{
of_clk_del_provider(data);
@@ -1223,18 +1225,42 @@ static int rzg2l_cpg_reset_controller_register(struct rzg2l_cpg_priv *priv)
return devm_reset_controller_register(priv->dev, &priv->rcdev);
}
+static inline const struct rzg2l_mod_clk
+*rzg2l_get_mod_clk(const struct rzg2l_cpg_info *info, int id)
+{
+ unsigned int i;
+
+ id += info->num_total_core_clks;
+ for (i = 0; i < info->num_mod_clks; i++) {
+ if (info->mod_clks[i].id == id)
+ return &info->mod_clks[i];
+ }
+
+ return NULL;
+}
+
static bool rzg2l_cpg_is_pm_clk(const struct of_phandle_args *clkspec)
{
+ struct rzg2l_cpg_priv *priv = rzg2l_cpg_priv;
+ const struct rzg2l_cpg_info *info = priv->info;
+ const struct rzg2l_mod_clk *mod_clk;
+
if (clkspec->args_count != 2)
return false;
- switch (clkspec->args[0]) {
- case CPG_MOD:
- return true;
+ if (clkspec->args[0] != CPG_MOD)
+ return false;
- default:
+ if (clkspec->args[1] >= info->num_hw_mod_clks) {
+ dev_err(priv->dev, "Invalid clk index\n");
return false;
}
+
+ mod_clk = rzg2l_get_mod_clk(info, clkspec->args[1]);
+ if (mod_clk && mod_clk->no_pm)
+ return false;
+
+ return true;
}
static int rzg2l_cpg_attach_dev(struct generic_pm_domain *unused, struct device *dev)
@@ -1348,6 +1374,7 @@ static int __init rzg2l_cpg_probe(struct platform_device *pdev)
if (IS_ERR(priv->base))
return PTR_ERR(priv->base);
+ rzg2l_cpg_priv = priv;
nclks = info->num_total_core_clks + info->num_hw_mod_clks;
clks = devm_kmalloc_array(dev, nclks, sizeof(*clks), GFP_KERNEL);
if (!clks)
diff --git a/drivers/clk/renesas/rzg2l-cpg.h b/drivers/clk/renesas/rzg2l-cpg.h
index cecbdf5e4f93..1d68d3838392 100644
--- a/drivers/clk/renesas/rzg2l-cpg.h
+++ b/drivers/clk/renesas/rzg2l-cpg.h
@@ -176,6 +176,7 @@ enum clk_types {
* @off: register offset
* @bit: ON/MON bit
* @is_coupled: flag to indicate coupled clock
+ * @no_pm: flag to indicate if clock doesn't support PM
*/
struct rzg2l_mod_clk {
const char *name;
@@ -184,9 +185,10 @@ struct rzg2l_mod_clk {
u16 off;
u8 bit;
bool is_coupled;
+ bool no_pm;
};
-#define DEF_MOD_BASE(_name, _id, _parent, _off, _bit, _is_coupled) \
+#define DEF_MOD_BASE(_name, _id, _parent, _off, _bit, _is_coupled, _no_pm) \
{ \
.name = _name, \
.id = MOD_CLK_BASE + (_id), \
@@ -194,13 +196,17 @@ struct rzg2l_mod_clk {
.off = (_off), \
.bit = (_bit), \
.is_coupled = (_is_coupled), \
+ .no_pm = (_no_pm), \
}
#define DEF_MOD(_name, _id, _parent, _off, _bit) \
- DEF_MOD_BASE(_name, _id, _parent, _off, _bit, false)
+ DEF_MOD_BASE(_name, _id, _parent, _off, _bit, false, false)
#define DEF_COUPLED(_name, _id, _parent, _off, _bit) \
- DEF_MOD_BASE(_name, _id, _parent, _off, _bit, true)
+ DEF_MOD_BASE(_name, _id, _parent, _off, _bit, true, false)
+
+#define DEF_NO_PM(_name, _id, _parent, _off, _bit) \
+ DEF_MOD_BASE(_name, _id, _parent, _off, _bit, false, true)
/**
* struct rzg2l_reset - Reset definitions
--
2.25.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [RFC PATCH 2/2] clk: renesas: r9a07g044: Mark CRU_SYSCLK and CRU_VCLK as no PM
2022-09-27 20:30 [RFC PATCH 0/2] clk: renesas: RZ/G2L: Add support for no PM clocks Prabhakar
2022-09-27 20:30 ` [RFC PATCH 1/2] clk: renesas: rzg2l: Don't assume all CPG_MOD clocks support PM Prabhakar
@ 2022-09-27 20:30 ` Prabhakar
1 sibling, 0 replies; 7+ messages in thread
From: Prabhakar @ 2022-09-27 20:30 UTC (permalink / raw)
To: Geert Uytterhoeven, Michael Turquette, Stephen Boyd
Cc: Laurent Pinchart, linux-renesas-soc, linux-clk, linux-kernel,
Prabhakar, Biju Das, Lad Prabhakar
From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
CRU_SYSCLK and CRU_VCLK clocks need to be turned ON/OFF in particular
sequence for the CRU block hence use DEF_NO_PM() to set the no_pm flag.
Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
---
drivers/clk/renesas/r9a07g044-cpg.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/clk/renesas/r9a07g044-cpg.c b/drivers/clk/renesas/r9a07g044-cpg.c
index 6935441f7504..31b1437c57e4 100644
--- a/drivers/clk/renesas/r9a07g044-cpg.c
+++ b/drivers/clk/renesas/r9a07g044-cpg.c
@@ -252,9 +252,9 @@ static const struct {
0x558, 1),
DEF_MOD("gpu_ace_clk", R9A07G044_GPU_ACE_CLK, R9A07G044_CLK_P1,
0x558, 2),
- DEF_MOD("cru_sysclk", R9A07G044_CRU_SYSCLK, CLK_M2_DIV2,
+ DEF_NO_PM("cru_sysclk", R9A07G044_CRU_SYSCLK, CLK_M2_DIV2,
0x564, 0),
- DEF_MOD("cru_vclk", R9A07G044_CRU_VCLK, R9A07G044_CLK_M2,
+ DEF_NO_PM("cru_vclk", R9A07G044_CRU_VCLK, R9A07G044_CLK_M2,
0x564, 1),
DEF_MOD("cru_pclk", R9A07G044_CRU_PCLK, R9A07G044_CLK_ZT,
0x564, 2),
--
2.25.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* RE: [RFC PATCH 1/2] clk: renesas: rzg2l: Don't assume all CPG_MOD clocks support PM
2022-09-27 20:30 ` [RFC PATCH 1/2] clk: renesas: rzg2l: Don't assume all CPG_MOD clocks support PM Prabhakar
@ 2022-09-28 7:02 ` Biju Das
2022-09-28 8:28 ` Lad, Prabhakar
0 siblings, 1 reply; 7+ messages in thread
From: Biju Das @ 2022-09-28 7:02 UTC (permalink / raw)
To: Prabhakar, Geert Uytterhoeven, Michael Turquette, Stephen Boyd
Cc: Laurent Pinchart, linux-renesas-soc, linux-clk, linux-kernel,
Prabhakar Mahadev Lad
Hi Prabhakar,
> Subject: [RFC PATCH 1/2] clk: renesas: rzg2l: Don't assume all CPG_MOD
> clocks support PM
>
> From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
>
> There are cases where not all CPG_MOD clocks should be assumed to
> support PM. For example on the CRU block there is a particular
> sequence that needs to be followed to initialize the CSI-2 D-PHY in
> which individual clocks need to be turned ON/OFF, due to which Runtime
> PM support wasn't used by the CRU CSI-2 driver.
>
> This patch adds support to allow indicating if PM is supported by the
> CPG_MOD clocks. A new macro is DEF_NO_PM() is added which sets the
> no_pm flag in struct rzg2l_mod_clk and when the driver uses Runtime PM
> support no_pm flag is checked to see if the clk needs to included as
> part of Runtime PM.
>
> CPG_MOD clocks with no_pm flag set need to be individually turned
> ON/OFF depending on the requirement of the driver.
>
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> ---
> drivers/clk/renesas/rzg2l-cpg.c | 35 +++++++++++++++++++++++++++++---
> - drivers/clk/renesas/rzg2l-cpg.h | 12 ++++++++---
> 2 files changed, 40 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/clk/renesas/rzg2l-cpg.c
> b/drivers/clk/renesas/rzg2l-cpg.c index 3ff6ecd61756..d275324909e7
> 100644
> --- a/drivers/clk/renesas/rzg2l-cpg.c
> +++ b/drivers/clk/renesas/rzg2l-cpg.c
> @@ -114,6 +114,8 @@ struct rzg2l_cpg_priv {
> struct rzg2l_pll5_mux_dsi_div_param mux_dsi_div_params; };
>
> +static struct rzg2l_cpg_priv *rzg2l_cpg_priv;
> +
> static void rzg2l_cpg_del_clk_provider(void *data) {
> of_clk_del_provider(data);
> @@ -1223,18 +1225,42 @@ static int
> rzg2l_cpg_reset_controller_register(struct rzg2l_cpg_priv *priv)
> return devm_reset_controller_register(priv->dev, &priv->rcdev);
> }
>
> +static inline const struct rzg2l_mod_clk *rzg2l_get_mod_clk(const
> +struct rzg2l_cpg_info *info, int id) {
> + unsigned int i;
> +
> + id += info->num_total_core_clks;
> + for (i = 0; i < info->num_mod_clks; i++) {
> + if (info->mod_clks[i].id == id)
> + return &info->mod_clks[i];
> + }
May be as an optimization add ID and clk to a separate list
and traverse that smaller list for DEF_NO_PM case.
case CPG_MOD:
return rzg2l_cpg_is_pm_mod(clkspec->args[1]);
Cheers,
Biju
> +
> + return NULL;
> +}
> +
> static bool rzg2l_cpg_is_pm_clk(const struct of_phandle_args
> *clkspec) {
> + struct rzg2l_cpg_priv *priv = rzg2l_cpg_priv;
> + const struct rzg2l_cpg_info *info = priv->info;
> + const struct rzg2l_mod_clk *mod_clk;
> +
> if (clkspec->args_count != 2)
> return false;
>
> - switch (clkspec->args[0]) {
> - case CPG_MOD:
> - return true;
> + if (clkspec->args[0] != CPG_MOD)
> + return false;
>
> - default:
> + if (clkspec->args[1] >= info->num_hw_mod_clks) {
> + dev_err(priv->dev, "Invalid clk index\n");
> return false;
> }
> +
> + mod_clk = rzg2l_get_mod_clk(info, clkspec->args[1]);
> + if (mod_clk && mod_clk->no_pm)
> + return false;
> +
> + return true;
> }
>
> static int rzg2l_cpg_attach_dev(struct generic_pm_domain *unused,
> struct device *dev) @@ -1348,6 +1374,7 @@ static int __init
> rzg2l_cpg_probe(struct platform_device *pdev)
> if (IS_ERR(priv->base))
> return PTR_ERR(priv->base);
>
> + rzg2l_cpg_priv = priv;
> nclks = info->num_total_core_clks + info->num_hw_mod_clks;
> clks = devm_kmalloc_array(dev, nclks, sizeof(*clks), GFP_KERNEL);
> if (!clks)
> diff --git a/drivers/clk/renesas/rzg2l-cpg.h
> b/drivers/clk/renesas/rzg2l-cpg.h index cecbdf5e4f93..1d68d3838392
> 100644
> --- a/drivers/clk/renesas/rzg2l-cpg.h
> +++ b/drivers/clk/renesas/rzg2l-cpg.h
> @@ -176,6 +176,7 @@ enum clk_types {
> * @off: register offset
> * @bit: ON/MON bit
> * @is_coupled: flag to indicate coupled clock
> + * @no_pm: flag to indicate if clock doesn't support PM
> */
> struct rzg2l_mod_clk {
> const char *name;
> @@ -184,9 +185,10 @@ struct rzg2l_mod_clk {
> u16 off;
> u8 bit;
> bool is_coupled;
> + bool no_pm;
> };
>
> -#define DEF_MOD_BASE(_name, _id, _parent, _off, _bit, _is_coupled) \
> +#define DEF_MOD_BASE(_name, _id, _parent, _off, _bit, _is_coupled,
> _no_pm) \
> { \
> .name = _name, \
> .id = MOD_CLK_BASE + (_id), \
> @@ -194,13 +196,17 @@ struct rzg2l_mod_clk {
> .off = (_off), \
> .bit = (_bit), \
> .is_coupled = (_is_coupled), \
> + .no_pm = (_no_pm), \
> }
>
> #define DEF_MOD(_name, _id, _parent, _off, _bit) \
> - DEF_MOD_BASE(_name, _id, _parent, _off, _bit, false)
> + DEF_MOD_BASE(_name, _id, _parent, _off, _bit, false, false)
>
> #define DEF_COUPLED(_name, _id, _parent, _off, _bit) \
> - DEF_MOD_BASE(_name, _id, _parent, _off, _bit, true)
> + DEF_MOD_BASE(_name, _id, _parent, _off, _bit, true, false)
> +
> +#define DEF_NO_PM(_name, _id, _parent, _off, _bit) \
> + DEF_MOD_BASE(_name, _id, _parent, _off, _bit, false, true)
>
> /**
> * struct rzg2l_reset - Reset definitions
> --
> 2.25.1
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [RFC PATCH 1/2] clk: renesas: rzg2l: Don't assume all CPG_MOD clocks support PM
2022-09-28 7:02 ` Biju Das
@ 2022-09-28 8:28 ` Lad, Prabhakar
2022-09-28 8:42 ` Biju Das
0 siblings, 1 reply; 7+ messages in thread
From: Lad, Prabhakar @ 2022-09-28 8:28 UTC (permalink / raw)
To: Biju Das
Cc: Geert Uytterhoeven, Michael Turquette, Stephen Boyd,
Laurent Pinchart, linux-renesas-soc, linux-clk, linux-kernel,
Prabhakar Mahadev Lad
Hi Biju,
Thank you for the review.
On Wed, Sep 28, 2022 at 8:02 AM Biju Das <biju.das.jz@bp.renesas.com> wrote:
>
> Hi Prabhakar,
>
> > Subject: [RFC PATCH 1/2] clk: renesas: rzg2l: Don't assume all CPG_MOD
> > clocks support PM
> >
> > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> >
> > There are cases where not all CPG_MOD clocks should be assumed to
> > support PM. For example on the CRU block there is a particular
> > sequence that needs to be followed to initialize the CSI-2 D-PHY in
> > which individual clocks need to be turned ON/OFF, due to which Runtime
> > PM support wasn't used by the CRU CSI-2 driver.
> >
> > This patch adds support to allow indicating if PM is supported by the
> > CPG_MOD clocks. A new macro is DEF_NO_PM() is added which sets the
> > no_pm flag in struct rzg2l_mod_clk and when the driver uses Runtime PM
> > support no_pm flag is checked to see if the clk needs to included as
> > part of Runtime PM.
> >
> > CPG_MOD clocks with no_pm flag set need to be individually turned
> > ON/OFF depending on the requirement of the driver.
> >
> > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> > ---
> > drivers/clk/renesas/rzg2l-cpg.c | 35 +++++++++++++++++++++++++++++---
> > - drivers/clk/renesas/rzg2l-cpg.h | 12 ++++++++---
> > 2 files changed, 40 insertions(+), 7 deletions(-)
> >
> > diff --git a/drivers/clk/renesas/rzg2l-cpg.c
> > b/drivers/clk/renesas/rzg2l-cpg.c index 3ff6ecd61756..d275324909e7
> > 100644
> > --- a/drivers/clk/renesas/rzg2l-cpg.c
> > +++ b/drivers/clk/renesas/rzg2l-cpg.c
> > @@ -114,6 +114,8 @@ struct rzg2l_cpg_priv {
> > struct rzg2l_pll5_mux_dsi_div_param mux_dsi_div_params; };
> >
> > +static struct rzg2l_cpg_priv *rzg2l_cpg_priv;
> > +
> > static void rzg2l_cpg_del_clk_provider(void *data) {
> > of_clk_del_provider(data);
> > @@ -1223,18 +1225,42 @@ static int
> > rzg2l_cpg_reset_controller_register(struct rzg2l_cpg_priv *priv)
> > return devm_reset_controller_register(priv->dev, &priv->rcdev);
> > }
> >
> > +static inline const struct rzg2l_mod_clk *rzg2l_get_mod_clk(const
> > +struct rzg2l_cpg_info *info, int id) {
> > + unsigned int i;
> > +
> > + id += info->num_total_core_clks;
> > + for (i = 0; i < info->num_mod_clks; i++) {
> > + if (info->mod_clks[i].id == id)
> > + return &info->mod_clks[i];
> > + }
>
> May be as an optimization add ID and clk to a separate list
> and traverse that smaller list for DEF_NO_PM case.
>
> case CPG_MOD:
> return rzg2l_cpg_is_pm_mod(clkspec->args[1]);
>
Are you suggesting adding no_pm_mod_clks and no_pm_mod_clks or
building an internal structure in struct rzg2l_cpg_priv while calling
rzg2l_cpg_register_mod_clk() for each mod clock?
Cheers,
Prabhakar
^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [RFC PATCH 1/2] clk: renesas: rzg2l: Don't assume all CPG_MOD clocks support PM
2022-09-28 8:28 ` Lad, Prabhakar
@ 2022-09-28 8:42 ` Biju Das
2022-09-28 9:37 ` Lad, Prabhakar
0 siblings, 1 reply; 7+ messages in thread
From: Biju Das @ 2022-09-28 8:42 UTC (permalink / raw)
To: Lad, Prabhakar
Cc: Geert Uytterhoeven, Michael Turquette, Stephen Boyd,
Laurent Pinchart, linux-renesas-soc, linux-clk, linux-kernel,
Prabhakar Mahadev Lad
> Subject: Re: [RFC PATCH 1/2] clk: renesas: rzg2l: Don't assume all
> CPG_MOD clocks support PM
>
> Hi Biju,
>
> Thank you for the review.
>
> On Wed, Sep 28, 2022 at 8:02 AM Biju Das <biju.das.jz@bp.renesas.com>
> wrote:
> >
> > Hi Prabhakar,
> >
> > > Subject: [RFC PATCH 1/2] clk: renesas: rzg2l: Don't assume all
> > > CPG_MOD clocks support PM
> > >
> > > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> > >
> > > There are cases where not all CPG_MOD clocks should be assumed to
> > > support PM. For example on the CRU block there is a particular
> > > sequence that needs to be followed to initialize the CSI-2 D-PHY
> in
> > > which individual clocks need to be turned ON/OFF, due to which
> > > Runtime PM support wasn't used by the CRU CSI-2 driver.
> > >
> > > This patch adds support to allow indicating if PM is supported by
> > > the CPG_MOD clocks. A new macro is DEF_NO_PM() is added which sets
> > > the no_pm flag in struct rzg2l_mod_clk and when the driver uses
> > > Runtime PM support no_pm flag is checked to see if the clk needs
> to
> > > included as part of Runtime PM.
> > >
> > > CPG_MOD clocks with no_pm flag set need to be individually turned
> > > ON/OFF depending on the requirement of the driver.
> > >
> > > Signed-off-by: Lad Prabhakar
> > > <prabhakar.mahadev-lad.rj@bp.renesas.com>
> > > ---
> > > drivers/clk/renesas/rzg2l-cpg.c | 35
> > > +++++++++++++++++++++++++++++---
> > > - drivers/clk/renesas/rzg2l-cpg.h | 12 ++++++++---
> > > 2 files changed, 40 insertions(+), 7 deletions(-)
> > >
> > > diff --git a/drivers/clk/renesas/rzg2l-cpg.c
> > > b/drivers/clk/renesas/rzg2l-cpg.c index 3ff6ecd61756..d275324909e7
> > > 100644
> > > --- a/drivers/clk/renesas/rzg2l-cpg.c
> > > +++ b/drivers/clk/renesas/rzg2l-cpg.c
> > > @@ -114,6 +114,8 @@ struct rzg2l_cpg_priv {
> > > struct rzg2l_pll5_mux_dsi_div_param mux_dsi_div_params; };
> > >
> > > +static struct rzg2l_cpg_priv *rzg2l_cpg_priv;
> > > +
> > > static void rzg2l_cpg_del_clk_provider(void *data) {
> > > of_clk_del_provider(data);
> > > @@ -1223,18 +1225,42 @@ static int
> > > rzg2l_cpg_reset_controller_register(struct rzg2l_cpg_priv *priv)
> > > return devm_reset_controller_register(priv->dev,
> > > &priv->rcdev); }
> > >
> > > +static inline const struct rzg2l_mod_clk *rzg2l_get_mod_clk(const
> > > +struct rzg2l_cpg_info *info, int id) {
> > > + unsigned int i;
> > > +
> > > + id += info->num_total_core_clks;
> > > + for (i = 0; i < info->num_mod_clks; i++) {
> > > + if (info->mod_clks[i].id == id)
> > > + return &info->mod_clks[i];
> > > + }
> >
> > May be as an optimization add ID and clk to a separate list and
> > traverse that smaller list for DEF_NO_PM case.
> >
> > case CPG_MOD:
> > return rzg2l_cpg_is_pm_mod(clkspec->args[1]);
> >
> Are you suggesting adding no_pm_mod_clks and no_pm_mod_clks or
> building an internal structure in struct rzg2l_cpg_priv while calling
> rzg2l_cpg_register_mod_clk() for each mod clock?
Later one. To have a smaller list of no_pm_mod_clks while registering as MOD
Clocks and later add PM clocks will check for the matching clock ID and
excludes it.
Cheers,
Biju
>
> Cheers,
> Prabhakar
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [RFC PATCH 1/2] clk: renesas: rzg2l: Don't assume all CPG_MOD clocks support PM
2022-09-28 8:42 ` Biju Das
@ 2022-09-28 9:37 ` Lad, Prabhakar
0 siblings, 0 replies; 7+ messages in thread
From: Lad, Prabhakar @ 2022-09-28 9:37 UTC (permalink / raw)
To: Biju Das
Cc: Geert Uytterhoeven, Michael Turquette, Stephen Boyd,
Laurent Pinchart, linux-renesas-soc, linux-clk, linux-kernel,
Prabhakar Mahadev Lad
Hi Biju,
On Wed, Sep 28, 2022 at 9:42 AM Biju Das <biju.das.jz@bp.renesas.com> wrote:
>
> > Subject: Re: [RFC PATCH 1/2] clk: renesas: rzg2l: Don't assume all
> > CPG_MOD clocks support PM
> >
> > Hi Biju,
> >
> > Thank you for the review.
> >
> > On Wed, Sep 28, 2022 at 8:02 AM Biju Das <biju.das.jz@bp.renesas.com>
> > wrote:
> > >
> > > Hi Prabhakar,
> > >
> > > > Subject: [RFC PATCH 1/2] clk: renesas: rzg2l: Don't assume all
> > > > CPG_MOD clocks support PM
> > > >
> > > > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> > > >
> > > > There are cases where not all CPG_MOD clocks should be assumed to
> > > > support PM. For example on the CRU block there is a particular
> > > > sequence that needs to be followed to initialize the CSI-2 D-PHY
> > in
> > > > which individual clocks need to be turned ON/OFF, due to which
> > > > Runtime PM support wasn't used by the CRU CSI-2 driver.
> > > >
> > > > This patch adds support to allow indicating if PM is supported by
> > > > the CPG_MOD clocks. A new macro is DEF_NO_PM() is added which sets
> > > > the no_pm flag in struct rzg2l_mod_clk and when the driver uses
> > > > Runtime PM support no_pm flag is checked to see if the clk needs
> > to
> > > > included as part of Runtime PM.
> > > >
> > > > CPG_MOD clocks with no_pm flag set need to be individually turned
> > > > ON/OFF depending on the requirement of the driver.
> > > >
> > > > Signed-off-by: Lad Prabhakar
> > > > <prabhakar.mahadev-lad.rj@bp.renesas.com>
> > > > ---
> > > > drivers/clk/renesas/rzg2l-cpg.c | 35
> > > > +++++++++++++++++++++++++++++---
> > > > - drivers/clk/renesas/rzg2l-cpg.h | 12 ++++++++---
> > > > 2 files changed, 40 insertions(+), 7 deletions(-)
> > > >
> > > > diff --git a/drivers/clk/renesas/rzg2l-cpg.c
> > > > b/drivers/clk/renesas/rzg2l-cpg.c index 3ff6ecd61756..d275324909e7
> > > > 100644
> > > > --- a/drivers/clk/renesas/rzg2l-cpg.c
> > > > +++ b/drivers/clk/renesas/rzg2l-cpg.c
> > > > @@ -114,6 +114,8 @@ struct rzg2l_cpg_priv {
> > > > struct rzg2l_pll5_mux_dsi_div_param mux_dsi_div_params; };
> > > >
> > > > +static struct rzg2l_cpg_priv *rzg2l_cpg_priv;
> > > > +
> > > > static void rzg2l_cpg_del_clk_provider(void *data) {
> > > > of_clk_del_provider(data);
> > > > @@ -1223,18 +1225,42 @@ static int
> > > > rzg2l_cpg_reset_controller_register(struct rzg2l_cpg_priv *priv)
> > > > return devm_reset_controller_register(priv->dev,
> > > > &priv->rcdev); }
> > > >
> > > > +static inline const struct rzg2l_mod_clk *rzg2l_get_mod_clk(const
> > > > +struct rzg2l_cpg_info *info, int id) {
> > > > + unsigned int i;
> > > > +
> > > > + id += info->num_total_core_clks;
> > > > + for (i = 0; i < info->num_mod_clks; i++) {
> > > > + if (info->mod_clks[i].id == id)
> > > > + return &info->mod_clks[i];
> > > > + }
> > >
> > > May be as an optimization add ID and clk to a separate list and
> > > traverse that smaller list for DEF_NO_PM case.
> > >
> > > case CPG_MOD:
> > > return rzg2l_cpg_is_pm_mod(clkspec->args[1]);
> > >
> > Are you suggesting adding no_pm_mod_clks and no_pm_mod_clks or
> > building an internal structure in struct rzg2l_cpg_priv while calling
> > rzg2l_cpg_register_mod_clk() for each mod clock?
>
> Later one. To have a smaller list of no_pm_mod_clks while registering as MOD
> Clocks and later add PM clocks will check for the matching clock ID and
> excludes it.
>
Agreed.
Cheers,
Prabhakar
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-09-28 9:39 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-27 20:30 [RFC PATCH 0/2] clk: renesas: RZ/G2L: Add support for no PM clocks Prabhakar
2022-09-27 20:30 ` [RFC PATCH 1/2] clk: renesas: rzg2l: Don't assume all CPG_MOD clocks support PM Prabhakar
2022-09-28 7:02 ` Biju Das
2022-09-28 8:28 ` Lad, Prabhakar
2022-09-28 8:42 ` Biju Das
2022-09-28 9:37 ` Lad, Prabhakar
2022-09-27 20:30 ` [RFC PATCH 2/2] clk: renesas: r9a07g044: Mark CRU_SYSCLK and CRU_VCLK as no PM Prabhakar
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).