[PATCH v2 2/3] pwm: rockchip: Eliminate potential race condition when probing
Simon South
simon at simonsouth.net
Tue Dec 22 11:26:59 EST 2020
Uwe Kleine-König <u.kleine-koenig at pengutronix.de> writes:
> I wonder if the commit log should be more detailed about this, after
> reading it I thought the effect of the bug would be that the PWM stops
> even though it should oscillate.
Your understanding is correct; this was the result of confusion on my
part. I'll write a new commit message.
>> Also refactor the code slightly to eliminate goto targets as the
>> error handlers no longer share any recovery steps.
>
> This however makes it hard to review the patch. Maybe this refactoring
> can be split out?
I'll give this another shot, too.
--
Simon South
simon at simonsouth.net
More information about the Linux-rockchip
mailing list