Message ID | 1573629896-23954-1-git-send-email-peng.fan@nxp.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | clk: imx: pll14xx: introduce imx_clk_hw_pll14xx_flags | expand |
On 13.11.2019 09:26, Peng Fan wrote: > From: Peng Fan <peng.fan@nxp.com> > > Introduce imx_clk_hw_pll14xx_flags, then no need to add new > imx_pll14xx_clk variable for new flags. > > Since the original imx_pll14xx_clk flags is not used, so drop it. > > Signed-off-by: Peng Fan <peng.fan@nxp.com> Reviewed-by: Leonard Crestez <leonard.crestez@nxp.com> Most other imx clock wrappers take flags as an argument so it's nice to be consistent. > V1: > Based on https://patchwork.kernel.org/patch/11217889/ In general if you sent patches on top of patches it makes sense to resend as a series. That clk_hw series is not in yet. > drivers/clk/imx/clk-pll14xx.c | 12 +++++++++++- > drivers/clk/imx/clk.h | 7 ++++++- > 2 files changed, 17 insertions(+), 2 deletions(-) > > diff --git a/drivers/clk/imx/clk-pll14xx.c b/drivers/clk/imx/clk-pll14xx.c > index 2bbcfbf8081a..a8af949f0848 100644 > --- a/drivers/clk/imx/clk-pll14xx.c > +++ b/drivers/clk/imx/clk-pll14xx.c > @@ -379,6 +379,16 @@ struct clk_hw *imx_clk_hw_pll14xx(const char *name, const char *parent_name, > void __iomem *base, > const struct imx_pll14xx_clk *pll_clk) > { > + > + return imx_clk_hw_pll14xx_flags(name, parent_name, base, pll_clk, 0); > +} > + > +struct clk_hw *imx_clk_hw_pll14xx_flags(const char *name, > + const char *parent_name, > + void __iomem *base, > + const struct imx_pll14xx_clk *pll_clk, > + unsigned long flags) > +{ > struct clk_pll14xx *pll; > struct clk_hw *hw; > struct clk_init_data init; > @@ -390,7 +400,7 @@ struct clk_hw *imx_clk_hw_pll14xx(const char *name, const char *parent_name, > return ERR_PTR(-ENOMEM); > > init.name = name; > - init.flags = pll_clk->flags; > + init.flags = flags; > init.parent_names = &parent_name; > init.num_parents = 1; > > diff --git a/drivers/clk/imx/clk.h b/drivers/clk/imx/clk.h > index cd92d9fdccf4..c2851a82b4fd 100644 > --- a/drivers/clk/imx/clk.h > +++ b/drivers/clk/imx/clk.h > @@ -48,7 +48,6 @@ struct imx_pll14xx_clk { > enum imx_pll14xx_type type; > const struct imx_pll14xx_rate_table *rate_table; > int rate_count; > - int flags; > }; > > extern struct imx_pll14xx_clk imx_1416x_pll; > @@ -105,6 +104,12 @@ struct clk_hw *imx_clk_hw_pll14xx(const char *name, const char *parent_name, > void __iomem *base, > const struct imx_pll14xx_clk *pll_clk); > > +struct clk_hw *imx_clk_hw_pll14xx_flags(const char *name, > + const char *parent_name, > + void __iomem *base, > + const struct imx_pll14xx_clk *pll_clk, > + unsigned long flags); > + > struct clk *imx_clk_pllv1(enum imx_pllv1_type type, const char *name, > const char *parent, void __iomem *base);
> Subject: Re: [PATCH] clk: imx: pll14xx: introduce imx_clk_hw_pll14xx_flags > > On 13.11.2019 09:26, Peng Fan wrote: > > From: Peng Fan <peng.fan@nxp.com> > > > > Introduce imx_clk_hw_pll14xx_flags, then no need to add new > > imx_pll14xx_clk variable for new flags. > > > > Since the original imx_pll14xx_clk flags is not used, so drop it. > > > > Signed-off-by: Peng Fan <peng.fan@nxp.com> > > Reviewed-by: Leonard Crestez <leonard.crestez@nxp.com> > > Most other imx clock wrappers take flags as an argument so it's nice to be > consistent. > > > V1: > > Based on https://patchwork.kernel.org/patch/11217889/ > > In general if you sent patches on top of patches it makes sense to resend as a > series. That clk_hw series is not in yet. Thanks for hints, I'll take care in future. Thanks, Peng. > > > drivers/clk/imx/clk-pll14xx.c | 12 +++++++++++- > > drivers/clk/imx/clk.h | 7 ++++++- > > 2 files changed, 17 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/clk/imx/clk-pll14xx.c > > b/drivers/clk/imx/clk-pll14xx.c index 2bbcfbf8081a..a8af949f0848 > > 100644 > > --- a/drivers/clk/imx/clk-pll14xx.c > > +++ b/drivers/clk/imx/clk-pll14xx.c > > @@ -379,6 +379,16 @@ struct clk_hw *imx_clk_hw_pll14xx(const char > *name, const char *parent_name, > > void __iomem *base, > > const struct imx_pll14xx_clk *pll_clk) > > { > > + > > + return imx_clk_hw_pll14xx_flags(name, parent_name, base, pll_clk, > > +0); } > > + > > +struct clk_hw *imx_clk_hw_pll14xx_flags(const char *name, > > + const char *parent_name, > > + void __iomem *base, > > + const struct imx_pll14xx_clk *pll_clk, > > + unsigned long flags) > > +{ > > struct clk_pll14xx *pll; > > struct clk_hw *hw; > > struct clk_init_data init; > > @@ -390,7 +400,7 @@ struct clk_hw *imx_clk_hw_pll14xx(const char > *name, const char *parent_name, > > return ERR_PTR(-ENOMEM); > > > > init.name = name; > > - init.flags = pll_clk->flags; > > + init.flags = flags; > > init.parent_names = &parent_name; > > init.num_parents = 1; > > > > diff --git a/drivers/clk/imx/clk.h b/drivers/clk/imx/clk.h index > > cd92d9fdccf4..c2851a82b4fd 100644 > > --- a/drivers/clk/imx/clk.h > > +++ b/drivers/clk/imx/clk.h > > @@ -48,7 +48,6 @@ struct imx_pll14xx_clk { > > enum imx_pll14xx_type type; > > const struct imx_pll14xx_rate_table *rate_table; > > int rate_count; > > - int flags; > > }; > > > > extern struct imx_pll14xx_clk imx_1416x_pll; @@ -105,6 +104,12 @@ > > struct clk_hw *imx_clk_hw_pll14xx(const char *name, const char > *parent_name, > > void __iomem *base, > > const struct imx_pll14xx_clk *pll_clk); > > > > +struct clk_hw *imx_clk_hw_pll14xx_flags(const char *name, > > + const char *parent_name, > > + void __iomem *base, > > + const struct imx_pll14xx_clk *pll_clk, > > + unsigned long flags); > > + > > struct clk *imx_clk_pllv1(enum imx_pllv1_type type, const char *name, > > const char *parent, void __iomem *base);
diff --git a/drivers/clk/imx/clk-pll14xx.c b/drivers/clk/imx/clk-pll14xx.c index 2bbcfbf8081a..a8af949f0848 100644 --- a/drivers/clk/imx/clk-pll14xx.c +++ b/drivers/clk/imx/clk-pll14xx.c @@ -379,6 +379,16 @@ struct clk_hw *imx_clk_hw_pll14xx(const char *name, const char *parent_name, void __iomem *base, const struct imx_pll14xx_clk *pll_clk) { + + return imx_clk_hw_pll14xx_flags(name, parent_name, base, pll_clk, 0); +} + +struct clk_hw *imx_clk_hw_pll14xx_flags(const char *name, + const char *parent_name, + void __iomem *base, + const struct imx_pll14xx_clk *pll_clk, + unsigned long flags) +{ struct clk_pll14xx *pll; struct clk_hw *hw; struct clk_init_data init; @@ -390,7 +400,7 @@ struct clk_hw *imx_clk_hw_pll14xx(const char *name, const char *parent_name, return ERR_PTR(-ENOMEM); init.name = name; - init.flags = pll_clk->flags; + init.flags = flags; init.parent_names = &parent_name; init.num_parents = 1; diff --git a/drivers/clk/imx/clk.h b/drivers/clk/imx/clk.h index cd92d9fdccf4..c2851a82b4fd 100644 --- a/drivers/clk/imx/clk.h +++ b/drivers/clk/imx/clk.h @@ -48,7 +48,6 @@ struct imx_pll14xx_clk { enum imx_pll14xx_type type; const struct imx_pll14xx_rate_table *rate_table; int rate_count; - int flags; }; extern struct imx_pll14xx_clk imx_1416x_pll; @@ -105,6 +104,12 @@ struct clk_hw *imx_clk_hw_pll14xx(const char *name, const char *parent_name, void __iomem *base, const struct imx_pll14xx_clk *pll_clk); +struct clk_hw *imx_clk_hw_pll14xx_flags(const char *name, + const char *parent_name, + void __iomem *base, + const struct imx_pll14xx_clk *pll_clk, + unsigned long flags); + struct clk *imx_clk_pllv1(enum imx_pllv1_type type, const char *name, const char *parent, void __iomem *base);