On Tue, 7 May 2019 13:25:00 +0000 Peng Fan <peng....@nxp.com> wrote: > > Subject: Re: [i.MX8MM+CCF 08/41] clk: export mux/divider ops > > > > On Tue, 30 Apr 2019 10:17:58 +0000 > > Peng Fan <peng....@nxp.com> wrote: > > > > > Export mux/divider ops for composite usage > > > > > > Signed-off-by: Peng Fan <peng....@nxp.com> > > > --- > > > include/linux/clk-provider.h | 1 + > > > 1 file changed, 1 insertion(+) > > > > > > diff --git a/include/linux/clk-provider.h > > > b/include/linux/clk-provider.h index 216095d28c..a60cf6e833 100644 > > > --- a/include/linux/clk-provider.h > > > +++ b/include/linux/clk-provider.h > > > @@ -83,6 +83,7 @@ struct clk_divider { > > > #define CLK_DIVIDER_ROUND_CLOSEST BIT(4) > > > #define CLK_DIVIDER_READ_ONLY BIT(5) > > > #define CLK_DIVIDER_MAX_AT_ZERO BIT(6) > > > +extern const struct clk_ops clk_divider_ops; > > > > > > > Why do we need to export clk_divider_ops? Why iMX8MM requires it? > > > The composite driver will use that to set ops for divider clk inside > composite clk.
Please then write a detailed rationale for this change in commit message or a comment in the header file. The one from the original message seems to be not enough. > The Linux CCF also exports it. > > You could check the clk-composite-imx8m and clk-composite driver in > this patchset. > > Regards, > Peng. > > > > > > struct clk_fixed_factor { > > > struct clk clk; > > > > > > > > > > Best regards, > > > > Lukasz Majewski > > > > -- > > > > DENX Software Engineering GmbH, Managing Director: Wolfgang > > Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, > > Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: > > lu...@denx.de Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lu...@denx.de
pgpGZNo2Ydz6T.pgp
Description: OpenPGP digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot