On Sat, 15 Feb 2020 21:27:38 +0100 Heinrich Schuchardt <xypron.g...@gmx.de> wrote:
> Do not calculate a unused value of n which is overwritten in both > branches of the subsequent if statement. > > Identified by cppcheck. > > Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de> > --- > drivers/clk/ics8n3qv01.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/clk/ics8n3qv01.c b/drivers/clk/ics8n3qv01.c > index e85ab1f9a0..984379bb12 100644 > --- a/drivers/clk/ics8n3qv01.c > +++ b/drivers/clk/ics8n3qv01.c > @@ -81,7 +81,6 @@ static int ics8n3qv01_calc_parameters(uint fout, > uint *_mint, uint *_mfrac, uint n, foutiic, fvcoiic, mint; > u64 mfrac; > > - n = (2215000000U + fout / 2) / fout; > if (fout < 417000000U) > n = 2 * ((2215000000U / 2 + fout / 2) / fout); > else > -- > 2.25.0 > Reviewed-by: Lukasz Majewski <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
pgpMuyocIT5vp.pgp
Description: OpenPGP digital signature
