[PATCH 1/1] clk: ICS8N3QV01 remove superfluous code

[Lukasz Majewski <lukma@denx.de>]

On Sat, 15 Feb 2020 21:27:38 +0100
Heinrich Schuchardt <xypron.glpk@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@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@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>