Hello Simon, On Wed, Dec 23, 2020 at 11:01:05AM -0500, Simon South wrote: > Clarify the Rockchip PWM driver's error messages by referring to the clock > that operates a PWM device as the "PWM" clock, rather than the "bus" > clock (which is especially misleading in the case of devices that also use > a separate clock for bus access). Maybe mention that this is the clock that is named "pwm" in the devicetree and so matches to that, too. Other than that the patch is fine. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | https://www.pengutronix.de/ |