| Message ID | 20200215202738.15469-1-xypron.glpk@gmx.de |
|---|---|
| State | Accepted |
| Commit | 6befc1f900db638ae54ea1bbab51a42cd1d5a153 |
| Headers | show |
| Series | [1/1] clk: ICS8N3QV01 remove superfluous code | expand |
On Sat, 15 Feb 2020 21:27:38 +0100 Heinrich Schuchardt <xypron.glpk at 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.glpk at 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 <lukma at 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: 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: <https://lists.denx.de/pipermail/u-boot/attachments/20200216/67d79489/attachment.sig>
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
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.glpk at gmx.de> --- drivers/clk/ics8n3qv01.c | 1 - 1 file changed, 1 deletion(-) -- 2.25.0