From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukasz Majewski Date: Mon, 6 May 2019 23:41:22 +0200 Subject: [U-Boot] [i.MX8MM+CCF 02/41] clk: fixed-factor: fix get clk_fixed_factor In-Reply-To: <20190430103056.32537-3-peng.fan@nxp.com> References: <20190430103056.32537-1-peng.fan@nxp.com> <20190430103056.32537-3-peng.fan@nxp.com> Message-ID: <20190506234122.49c7e634@jawa> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Tue, 30 Apr 2019 10:17:36 +0000 Peng Fan wrote: > driver data is not clk, not clk_fixed_factor, fix it. This description is a bit misleading, IMHO. The dev->driver_data holds pointer to corresponding struct clk clk element (as described [1]). This patch resembles changes from PATCH [1/42]. > > Signed-off-by: Peng Fan > --- > drivers/clk/clk-fixed-factor.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/clk/clk-fixed-factor.c > b/drivers/clk/clk-fixed-factor.c index acbc0909b4..c535aadc48 100644 > --- a/drivers/clk/clk-fixed-factor.c > +++ b/drivers/clk/clk-fixed-factor.c > @@ -19,7 +19,7 @@ > static ulong clk_factor_recalc_rate(struct clk *clk) > { > struct clk_fixed_factor *fix = > - (struct clk_fixed_factor > *)dev_get_driver_data(clk->dev); > + to_clk_fixed_factor((struct clk > *)dev_get_driver_data(clk->dev)); unsigned long parent_rate = > clk_get_parent_rate(clk); unsigned long long int rate; > 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: lukma at denx.de -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 488 bytes Desc: OpenPGP digital signature URL: