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

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

[Heinrich Schuchardt]

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

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

[Lukasz Majewski]

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>

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

[Heinrich Schuchardt]

On 2/16/20 9:50 AM, Lukasz Majewski wrote:
> 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(-)
>>
<snip />
> Reviewed-by: Lukasz Majewski <lukma@denx.de>
>

Hello Lukasz,

this patch reviewed by you in February is still not merged. Will you
create a pull request for CLOCK, please?

You can download the patch from

https://patchwork.ozlabs.org/project/uboot/patch/20200215202738.15469-1-xypron.glpk at gmx.de/

Best regards

Heinrich