On Mon, Jun 29, 2020 at 09:45:31AM -0700, Florian Fainelli wrote: > > > On 6/29/2020 5:47 AM, Lee Jones wrote: > > 'dc' here is an unsigned long, thus checking for <0 will always > > evaluate to false. > > > > Fixes the following W=1 warning: > > > > drivers/pwm/pwm-bcm-kona.c:141:35: warning: comparison of unsigned > > expression < 0 is always false [-Wtype-limits] > > > > Cc: Florian Fainelli <[email protected]> > > Cc: Ray Jui <[email protected]> > > Cc: Scott Branden <[email protected]> > > Cc: [email protected] > > Cc: [email protected] > > Signed-off-by: Lee Jones <[email protected]> > > And similar to patch #1, DUTY_CYCLE_HIGH_MIN now gets unused, so if you > remove it as well, you can add: > > Reviewed-by: Florian Fainelli <[email protected]>
Even without removing DUTY_CYCLE_HIGH_MIN (which might still serve as documentation): Acked-by: Uwe Kleine-König <[email protected]> Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | https://www.pengutronix.de/ |
signature.asc
Description: PGP signature
