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

[Kukjin Kim]

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@xxxxxxxxxxx>
> > Signed-off-by: Tomasz Figa<t.figa@xxxxxxxxxxx>
> > Acked-by: Kyungmin Park<kyungmin.park@xxxxxxxxxxx>
> > Cc: Mike Turquette<mturquette@xxxxxxxxxx>
> > Cc: Kukjin Kim<kgene.kim@xxxxxxxxxxx>
> > Cc: Rob Herring<robh+dt@xxxxxxxxxx>
> > Cc: Pawel Moll<pawel.moll@xxxxxxx>
> > Cc: Mark Rutland<mark.rutland@xxxxxxx>
> > Cc: Ian Campbell<ijc+devicetree@xxxxxxxxxxxxxx>
> > Cc: Kumar Gala<galak@xxxxxxxxxxxxxx>
> > Cc: Randy Dunlap<rdunlap@xxxxxxxxxxxxx>
> > Cc: Tomasz Figa<t.figa@xxxxxxxxxxx>
> > ---
> >   .../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@xxxxxxxxxxx>
Sure, I will do it on your 4/4 pull-request after pulling it into 
samsung tree.

Thanks,
Kukjin
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/