Messages in this thread | | | Date | Thu, 4 Jun 2020 16:43:04 +0200 | From | Uwe Kleine-König <> | Subject | Re: [PATCH] pwm: Add missing "CONFIG_" prefix |
| |
On Wed, Jun 03, 2020 at 03:40:56PM -0700, Kees Cook wrote: > The IS_ENABLED() use was missing the CONFIG_ prefix which would have > lead to skipping this code. > > Fixes: 3ad1f3a33286 ("pwm: Implement some checks for lowlevel drivers") > Signed-off-by: Kees Cook <keescook@chromium.org> > --- > drivers/pwm/core.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pwm/core.c b/drivers/pwm/core.c > index 9973c442b455..6b3cbc0490c6 100644 > --- a/drivers/pwm/core.c > +++ b/drivers/pwm/core.c > @@ -121,7 +121,7 @@ static int pwm_device_request(struct pwm_device *pwm, const char *label) > pwm->chip->ops->get_state(pwm->chip, pwm, &pwm->state); > trace_pwm_get(pwm, &pwm->state); > > - if (IS_ENABLED(PWM_DEBUG)) > + if (IS_ENABLED(CONFIG_PWM_DEBUG)) > pwm->last = pwm->state;
Thanks
Reviewed-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Best regards Uwe
-- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | https://www.pengutronix.de/ |
| |