drivers/clk/bcm/clk-iproc-pll.c:280]: (style) Redundant condition

Ray Jui ray.jui at broadcom.com
Wed Apr 5 11:27:36 PDT 2017


Hi David,

Thanks for reporting. I'll send out a patch to address this.

Ray

On 3/27/2017 3:47 AM, David Binderman wrote:
> Hello there,
> 
> 
> linux-4.11-rc4/drivers/clk/bcm/clk-iproc-pll.c:280]: (style) Redundant condition: If 'rate >= 2200000000', the comparison 'rate' is always true.
> 
> Source code is
> 
>     } else if (rate >= VCO_HIGH && rate && rate < VCO_HIGH_HIGH) {
> 
> Suggest new code
> 
>     } else if (rate >= VCO_HIGH  && rate < VCO_HIGH_HIGH) {
> 
> Regards
> 
> David Binderman
> 



More information about the linux-arm-kernel mailing list