2015-10-20 21:41 GMT+09:00 Andrzej Hajda <a.hajda@xxxxxxxxxxx>: > On 10/20/2015 02:24 PM, Krzysztof Kozlowski wrote: >> W dniu 20.10.2015 o 18:22, Andrzej Hajda pisze: >>> DECON IP requires this clock to access configuration registers. >>> >>> Signed-off-by: Andrzej Hajda <a.hajda@xxxxxxxxxxx> >>> --- >>> Documentation/devicetree/bindings/video/exynos5433-decon.txt | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/Documentation/devicetree/bindings/video/exynos5433-decon.txt b/Documentation/devicetree/bindings/video/exynos5433-decon.txt >>> index 377afbf..3dff78b 100644 >>> --- a/Documentation/devicetree/bindings/video/exynos5433-decon.txt >>> +++ b/Documentation/devicetree/bindings/video/exynos5433-decon.txt >>> @@ -16,7 +16,7 @@ Required properties: >>> - clocks: must include clock specifiers corresponding to entries in the >>> clock-names property. >>> - clock-names: list of clock names sorted in the same order as the clocks >>> - property. Must contain "aclk_decon", "aclk_smmu_decon0x", >>> + property. Must contain "pclk", "aclk_decon", "aclk_smmu_decon0x", >> I assume that old DTB wouldn't work at all, so there is no point in >> maintaining ABI compatibility? > > Yes, Exynos 5433 has not yet landed in mainline. No, no. I was asking if something using these compatibles would work previously or not? The driver provided a compatible, an ABI. Now you require additional field for the same compatible, so the ABI changes. Best regards, Krzysztof -- 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