lkml.org 
[lkml]   [2017]   [Jul]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 2/2] dt: Add bindings for IDT VersaClock 5P49V5925
From
Date
On 07/07/2017 03:57 AM, Vladimir Barinov wrote:
> From: Vladimir Barinov <vladimir.barinov+renesas@cogentembedded.com>
>
> IDT VersaClock 5 5P49V5925 has 4 clock outputs

In 1/2 you said it has 4 FODs and 5 outputs (and it does have 5 outputs,
4 from FODs and 1 I2C_OUTB).

>, 4 fractional dividers.
> Input clock source can be taken only from external reference clock.
>
> Signed-off-by: Vladimir Barinov <vladimir.barinov+renesas@cogentembedded.com>

You might want to rebase this on top of and retest against [1], there
are 8 patches in total fixing some stuff and adding VC6 support.

[1] https://patchwork.kernel.org/patch/9831797/

> ---
> Documentation/devicetree/bindings/clock/idt,versaclock5.txt | 11 +++++++----
> 1 file changed, 7 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/clock/idt,versaclock5.txt b/Documentation/devicetree/bindings/clock/idt,versaclock5.txt
> index 53d7e50..a1ad9e0 100644
> --- a/Documentation/devicetree/bindings/clock/idt,versaclock5.txt
> +++ b/Documentation/devicetree/bindings/clock/idt,versaclock5.txt
> @@ -6,19 +6,21 @@ from 3 to 12 output clocks.
> ==I2C device node==
>
> Required properties:
> -- compatible: shall be one of "idt,5p49v5923" , "idt,5p49v5933" ,
> - "idt,5p49v5935".
> +- compatible: shall be one of "idt,5p49v5923" , "idt,5p49v5925" ,
> + "idt,5p49v5933", "idt,5p49v5935".
> - reg: i2c device address, shall be 0x68 or 0x6a.
> - #clock-cells: from common clock binding; shall be set to 1.
> - clocks: from common clock binding; list of parent clock handles,
> - - 5p49v5923: (required) either or both of XTAL or CLKIN
> + - 5p49v5923 and
> + - 5p49v5925: (required) either or both of XTAL or CLKIN
> reference clock.
> - 5p49v5933 and
> - 5p49v5935: (optional) property not present (internal
> Xtal used) or CLKIN reference
> clock.
> - clock-names: from common clock binding; clock input names, can be
> - - 5p49v5923: (required) either or both of "xin", "clkin".
> + - 5p49v5923 and
> + - 5p49v5925: (required) either or both of "xin", "clkin".
> - 5p49v5933 and
> - 5p49v5935: (optional) property not present or "clkin".
>
> @@ -37,6 +39,7 @@ clock specifier, the following mapping applies:
> 1 -- OUT1
> 2 -- OUT4
>
> +5P49V5925 and
> 5P49V5935:
> 0 -- OUT0_SEL_I2CB
> 1 -- OUT1
>


--
Best regards,
Marek Vasut

\
 
 \ /
  Last update: 2017-07-09 15:39    [W:1.022 / U:0.408 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site