Re: [PATCH v3 1/3] dt-bindings: timer: rockchip: Drop superfluous rk3288 compatible

From: Krzysztof Kozlowski
Date: Fri Apr 21 2023 - 03:24:01 EST


On 19/04/2023 20:13, Cristian Ciocaltea wrote:
> The compatible string for Rockchip RK3288 is wrongly provided in the
> 'enum' item, in addition to the subsequent 'const', which allows the
> usage of an incorrect specification:
>
> compatible = "rockchip,rk3288-timer", "rockchip,rk3288-timer";
>
> As the rk3288 string is also specified in the top-most 'const' item, the
> binding already allows the usage of the correct variant:
>
> compatible = "rockchip,rk3288-timer";
>
> Drop the unwanted rk3288 entry from the enum.
>
> Fixes: faa186adbd06 ("dt-bindings: timer: convert rockchip,rk-timer.txt to YAML")
> Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@xxxxxxxxxxxxx>
> Reviewed-by: Heiko Stuebner <heiko@xxxxxxxxx>
> ---

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>

Best regards,
Krzysztof