Re: [PATCHv5 4/5] dt-bindings: add documentation for Exynos3250 clock controller

[Tomasz Figa <tomasz.figa@gmail.com>]

On 14.05.2014 21:46, Kukjin Kim wrote:
> On 05/15/14 04:35, Tomasz Figa wrote:
>> Hi,
>>
>> On 13.05.2014 15:05, Chanwoo Choi wrote:
>>> The Exynos3250 clocks are statically listed and registered using the
>>> Samsung specific common clock helper functions. Both device tree based
>>> clock lookup and clkdev based clock lookups are supported.
>>>
>>> Signed-off-by: Chanwoo Choi<cw00.choi@samsung.com>
>>> Signed-off-by: Tomasz Figa<t.figa@samsung.com>
>>> Acked-by: Kyungmin Park<kyungmin.park@samsung.com>
>>> Cc: Mike Turquette<mturquette@linaro.org>
>>> Cc: Kukjin Kim<kgene.kim@samsung.com>
>>> Cc: Rob Herring<robh+dt@kernel.org>
>>> Cc: Pawel Moll<pawel.moll@arm.com>
>>> Cc: Mark Rutland<mark.rutland@arm.com>
>>> Cc: Ian Campbell<ijc+devicetree@hellion.org.uk>
>>> Cc: Kumar Gala<galak@codeaurora.org>
>>> Cc: Randy Dunlap<rdunlap@infradead.org>
>>> Cc: Tomasz Figa<t.figa@samsung.com>
>>> ---
>>>   .../devicetree/bindings/clock/exynos3250-clock.txt | 41
>>> ++++++++++++++++++++++
>>>   1 file changed, 41 insertions(+)
>>>   create mode 100644
>>> Documentation/devicetree/bindings/clock/exynos3250-clock.txt
>>>
>>> diff --git
>>> a/Documentation/devicetree/bindings/clock/exynos3250-clock.txt
>>> b/Documentation/devicetree/bindings/clock/exynos3250-clock.txt
>>> new file mode 100644
>>> index 0000000..aadc9c5
>>> --- /dev/null
>>> +++ b/Documentation/devicetree/bindings/clock/exynos3250-clock.txt
>>> @@ -0,0 +1,41 @@
>>> +* Samsung Exynos3250 Clock Controller
>>> +
>>> +The Exynos3250 clock controller generates and supplies clock to various
>>> +controllers within the Exynos3250 SoC.
>>> +
>>> +Required Properties:
>>> +
>>> +- compatible: should be one of the following.
>>> +  - "samsung,exynos3250-cmu" - controller compatible with Exynos3250
>>> SoC.
>>> +
>>> +- reg: physical base address of the controller and length of memory
>>> mapped
>>> +  region.
>>> +
>>> +- #clock-cells: should be 1.
>>> +
>>> +Each clock is assigned an identifier and client nodes can use this
>>> identifier
>>> +to specify the clock which they consume.
>>> +
>>> +All available clocks are defined as preprocessor macros in
>>> +dt-bindings/clock/exynos3250.h header and can be used in device
>>> +tree sources.
>>> +
>>> +Example 1: An example of a clock controller node is listed below.
>>> +
>>> +    cmu: clock-controller@10030000 {
>>> +        compatible = "samsung,exynos3250-cmu";
>>> +        reg =<0x10030000 0x20000>;
>>> +        #clock-cells =<1>;
>>> +    };
>>> +
>>> +Example 2: UART controller node that consumes the clock generated by
>>> the clock
>>> +       controller. Refer to the standard clock bindings for information
>>> +       about 'clocks' and 'clock-names' property.
>>> +
>>> +    serial@13800000 {
>>> +        compatible = "samsung,exynos4210-uart";
>>> +        reg =<0x13800000 0x100>;
>>> +        interrupts =<0 109 0>;
>>> +        clocks =<&cmu CLK_UART0>,<&cmu CLK_SCLK_UART0>;
>>> +        clock-names = "uart", "clk_uart_baud0";
>>> +    };
>>>
>>
>> Oops, I've forgotten to apply this one. Since it doesn't have any
>> dependencies, Kukjin, could you apply it on top of my pull requests with
>> my Ack? Thanks in advance.
>>
>> Acked-by: Tomasz Figa<t.figa@samsung.com>
>>
> Sure, I will do it on your 4/4 pull-request after pulling it into
> samsung tree.

Thanks!

Best regards,
Tomasz