On 2020-06-29 9:45 a.m., Florian Fainelli wrote:
On 6/29/2020 5:47 AM, Lee Jones wrote:
'duty' here is an unsigned int, thus checking for <0 will always
evaluate to false.
Fixes the following W=1 warning:
drivers/pwm/pwm-bcm-iproc.c:147:12: warning: comparison of unsigned expression
< 0 is always false [-Wtype-limits]
Cc: Ray Jui <[email protected]>
Cc: Scott Branden <[email protected]>
Cc: Yendapally Reddy Dhananjaya Reddy <[email protected]>
Cc: [email protected]
Cc: [email protected]
Signed-off-by: Lee Jones <[email protected]>
And IPROC_PWM_DUTY_CYCLE_MIN is unused after remove this single use of
the define, if you also remove it you can add:
Reviewed-by: Florian Fainelli <[email protected]>
We actually made this same change internally recently but hadn't
upstreamed it yet.
Tested-by: Scott Branden <[email protected]>
