On Sun, Apr 16, 2017 at 10:27:12PM +0200, Daniel Lezcano wrote: > From: Alexander Kochetkov <al.kochet at gmail.com> > > Make all properties description in form '"rockchip,<chip>-timer", > "rockchip,rk3288-timer"' for all chips found in linux kernel. > > Suggested-by: Heiko St?bner <heiko at sntech.de> > Signed-off-by: Alexander Kochetkov <al.kochet at gmail.com> > Acked-by: Rob Herring <robh at kernel.org> > Reviewed-by: Heiko Stuebner <heiko at sntech.de> > Signed-off-by: Daniel Lezcano <daniel.lezcano at linaro.org> > --- > .../devicetree/bindings/timer/rockchip,rk-timer.txt | 12 +++++++++--- > 1 file changed, 9 insertions(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/timer/rockchip,rk-timer.txt b/Documentation/devicetree/bindings/timer/rockchip,rk-timer.txt > index a41b184..16a5f45 100644 > --- a/Documentation/devicetree/bindings/timer/rockchip,rk-timer.txt > +++ b/Documentation/devicetree/bindings/timer/rockchip,rk-timer.txt > @@ -1,9 +1,15 @@ > Rockchip rk timer > > Required properties: > -- compatible: shall be one of: > - "rockchip,rk3288-timer" - for rk3066, rk3036, rk3188, rk322x, rk3288, rk3368 > - "rockchip,rk3399-timer" - for rk3399 > +- compatible: should be: should be one of: > + "rockchip,rk3036-timer", "rockchip,rk3288-timer": for Rockchip RK3036 > + "rockchip,rk3066-timer", "rockchip,rk3288-timer": for Rockchip RK3066 > + "rockchip,rk3188-timer", "rockchip,rk3288-timer": for Rockchip RK3188 > + "rockchip,rk3228-timer", "rockchip,rk3288-timer": for Rockchip RK3228 > + "rockchip,rk3229-timer", "rockchip,rk3288-timer": for Rockchip RK3229 > + "rockchip,rk3288-timer": for Rockchip RK3288 > + "rockchip,rk3368-timer", "rockchip,rk3288-timer": for Rockchip RK3368 > + "rockchip,rk3399-timer": for Rockchip RK3399 I find the "for Rockchip RKxxxx" a bit redundant, but it's fine with or without. Otherwise, Acked-by: Rob Herring <robh at kernel.org>