On Tue, Feb 25, 2020 at 10:32:04AM +0530, Lokesh Vutla wrote: > Hi Uwe, > > On 24/02/20 1:23 PM, Uwe Kleine-König wrote: > > Hello, > > > > On Mon, Feb 24, 2020 at 10:51:32AM +0530, Lokesh Vutla wrote: > >> @@ -190,9 +190,8 @@ static int pwm_omap_dmtimer_config(struct pwm_chip *chip, > >> load_value, load_value, match_value, match_value); > >> > >> omap->pdata->set_pwm(omap->dm_timer, > >> - pwm_get_polarity(pwm) == PWM_POLARITY_INVERSED, > >> - true, > >> - PWM_OMAP_DMTIMER_TRIGGER_OVERFLOW_AND_COMPARE); > >> + pwm_get_polarity(pwm) == PWM_POLARITY_INVERSED, > >> + true, OMAP_TIMER_TRIGGER_OVERFLOW_AND_COMPARE); > > > > This is unrelated. > > PWM_OMAP_DMTIMER_TRIGGER_OVERFLOW_AND_COMPARE is deleted along with header file > and used OMAP_TIMER_TRIGGER_OVERFLOW_AND_COMPARE. I should have mentioned this > in my commit description. Will fix it in v2. Ah, indeed I missed that the register name was changed in that hunk and considered that an indention only change. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | https://www.pengutronix.de/ |