On Sun, Apr 16, 2017 at 10:27:12PM +0200, Daniel Lezcano wrote: > From: Alexander Kochetkov <al.kochet@xxxxxxxxx> > > 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@xxxxxxxxx> > Signed-off-by: Alexander Kochetkov <al.kochet@xxxxxxxxx> > Acked-by: Rob Herring <robh@xxxxxxxxxx> > Reviewed-by: Heiko Stuebner <heiko@xxxxxxxxx> > Signed-off-by: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx> > --- > .../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@xxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
