Hi, Sorry, made a little mistake in version 2 with "rockchip,rk3036-pwm", "rockchip,rk2928-pwm". Please trash. Will send version 3. By the change of schema for clocks and clock-names I add "rockchip,rk3328-pwm" to the "if:", so strictly speaking v1 and (v2) v3 will not be the same. Johan On 4/12/21 5:05 PM, Rob Herring wrote: > On Sun, 11 Apr 2021 15:10:03 +0200, Johan Jonker wrote: >> The compatible strings below are already in use in the Rockchip >> dtsi files, but were somehow never added to a document, so add >> >> "rockchip,rk3328-pwm" >> >> "rockchip,rk3036-pwm", "rockchip,rk2928-pwm" >> >> "rockchip,rk3368-pwm", "rockchip,rk3288-pwm" >> "rockchip,rk3399-pwm", "rockchip,rk3288-pwm" >> >> "rockchip,px30-pwm", "rockchip,rk3328-pwm" >> "rockchip,rk3308-pwm", "rockchip,rk3328-pwm" >> >> for pwm nodes to pwm-rockchip.yaml. >> >> Signed-off-by: Johan Jonker <jbx6244@xxxxxxxxx> >> --- >> Changed V2: >> changed schema for clocks and clock-names >> --- >> Documentation/devicetree/bindings/pwm/pwm-rockchip.yaml | 9 +++++++++ >> 1 file changed, 9 insertions(+) >> > > > Please add Acked-by/Reviewed-by tags when posting new versions. However, > there's no need to repost patches *only* to add the tags. The upstream > maintainer will do that for acks received on the version they apply. > > If a tag was not added on purpose, please state why and what changed. >