Re: [PATCH v6 1/7] of/documentation: update with clock information for exynos hdmi subsystem

[Tomasz Figa <t.figa@samsung.com>]

On Thursday 29 of August 2013 11:10:21 Rahul Sharma wrote:
> Adding information about clocks to the binding documentation
> for exynos mixer and hdmi.
> 
> Signed-off-by: Rahul Sharma <rahul.sharma@samsung.com>
> ---
>  Documentation/devicetree/bindings/video/exynos_hdmi.txt |   15
> ++++++++++++++- .../devicetree/bindings/video/exynos_mixer.txt         
> |    4 ++++ 2 files changed, 18 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/video/exynos_hdmi.txt
> b/Documentation/devicetree/bindings/video/exynos_hdmi.txt index
> 323983b..1878418 100644
> --- a/Documentation/devicetree/bindings/video/exynos_hdmi.txt
> +++ b/Documentation/devicetree/bindings/video/exynos_hdmi.txt
> @@ -12,7 +12,20 @@ Required properties:
>  	a) phandle of the gpio controller node.
>  	b) pin number within the gpio controller.
>  	c) optional flags and pull up/down.
> -
> +- clocks: list of clock IDs from SoC clock driver.
> +	a) hdmi: It is required for gate operation on aclk_200_disp1 clock
> +		which clocks the display1 block.

The "aclk_200_disp1" name is valid only on Exynos 5, but this binding is 
generic for all Exynos SoCs and probably also previous S5P SoCs. I believe 
that "Gate of HDMI IP bus clock." would be appropriate here.

Otherwise, this patch looks good to me.

Best regards,
Tomasz