Hi Thierry, For the PWM patches, whether we let the first patch land in, that is PWM APB and function clocks are different in rk3328 and rv1108 SOCs. If the first patch was merged before 4.14, it will guarantee that the PWM is working, and the lock function you might review it later. ? 2017/8/8 23:38, David Wu ??: > There are two features of rk3328 pwm module. > - PWM APB and function clocks are different. > - Add pwm atomic hardware update > > David Wu (7): > pwm: rockchip: Add APB and function both clocks support > pwm: rockchip: Remove the judge from return value of pwm_config > pwm: rockchip: Use pwm_apply instead of the pwm_enable > pwm: rockchip: Move the configuration of polarity from > rockchip_pwm_set_enable() to rockchip_pwm_config() > pwm: rockchip: Use same pwm ops for each IP > pwm: rockchip: Add rk3328 pwm support > arm64: dts: rockchip: Add pwm nodes for rk3328 > > .../devicetree/bindings/pwm/pwm-rockchip.txt | 8 +- > arch/arm64/boot/dts/rockchip/rk3328.dtsi | 45 ++++ > drivers/pwm/pwm-rockchip.c | 281 +++++++++++---------- > 3 files changed, 199 insertions(+), 135 deletions(-) >