Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1682860 > unrolled thread

[PATCH 0/2] clk: vc5: Add IDT VersaClock 5P49V5925

Started byVladimir Barinov <vladimir.barinov@cogentembedded.com>
First post2017-07-07 04:00 +0200
Last post2017-07-09 19:50 +0200
Articles 5 — 2 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH 0/2] clk: vc5: Add IDT VersaClock 5P49V5925 Vladimir Barinov <vladimir.barinov@cogentembedded.com> - 2017-07-07 04:00 +0200
    [PATCH 2/2] dt: Add bindings for IDT VersaClock 5P49V5925 Vladimir Barinov <vladimir.barinov@cogentembedded.com> - 2017-07-07 04:00 +0200
      Re: [PATCH 2/2] dt: Add bindings for IDT VersaClock 5P49V5925 Marek Vasut <marek.vasut@gmail.com> - 2017-07-09 15:40 +0200
        Re: [PATCH 2/2] dt: Add bindings for IDT VersaClock 5P49V5925 Vladimir Barinov <vladimir.barinov@cogentembedded.com> - 2017-07-09 19:40 +0200
          Re: [PATCH 2/2] dt: Add bindings for IDT VersaClock 5P49V5925 Marek Vasut <marek.vasut@gmail.com> - 2017-07-09 19:50 +0200

#1682860 — [PATCH 0/2] clk: vc5: Add IDT VersaClock 5P49V5925

FromVladimir Barinov <vladimir.barinov@cogentembedded.com>
Date2017-07-07 04:00 +0200
Subject[PATCH 0/2] clk: vc5: Add IDT VersaClock 5P49V5925
Message-ID<u0qIN-5el-5@gated-at.bofh.it>
Hello,

This adds the folowing:
- Update IDT VersaClock 5 driver to support 5P49V5925
- Add bindings for IDT VersaClock 5P49V5925

Vladimir Barinov (2):
[1/2] clk: vc5: Add support for IDT VersaClock 5P49V5925
[2/2] dt: Add bindings for IDT VersaClock 5P49V5925

---
This patchset is against the 'kernel/git/torvalds/linux.git' repo.

 Documentation/devicetree/bindings/clock/idt,versaclock5.txt |   11 +++++++----
 drivers/clk/clk-versaclock5.c                               |   11 +++++++++++
 2 files changed, 18 insertions(+), 4 deletions(-)

[toc] | [next] | [standalone]

#1682865 — [PATCH 2/2] dt: Add bindings for IDT VersaClock 5P49V5925

FromVladimir Barinov <vladimir.barinov@cogentembedded.com>
Date2017-07-07 04:00 +0200
Subject[PATCH 2/2] dt: Add bindings for IDT VersaClock 5P49V5925
Message-ID<u0qIO-5el-25@gated-at.bofh.it>
In reply to#1682860
From: Vladimir Barinov <vladimir.barinov+renesas@cogentembedded.com>

IDT VersaClock 5 5P49V5925 has 4 clock outputs, 4 fractional dividers.
Input clock source can be taken only from external reference clock.

Signed-off-by: Vladimir Barinov <vladimir.barinov+renesas@cogentembedded.com>
---
 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
-- 
1.9.1

[toc] | [prev] | [next] | [standalone]

#1683750 — Re: [PATCH 2/2] dt: Add bindings for IDT VersaClock 5P49V5925

FromMarek Vasut <marek.vasut@gmail.com>
Date2017-07-09 15:40 +0200
SubjectRe: [PATCH 2/2] dt: Add bindings for IDT VersaClock 5P49V5925
Message-ID<u1kBk-8ix-3@gated-at.bofh.it>
In reply to#1682865
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

[toc] | [prev] | [next] | [standalone]

#1683778 — Re: [PATCH 2/2] dt: Add bindings for IDT VersaClock 5P49V5925

FromVladimir Barinov <vladimir.barinov@cogentembedded.com>
Date2017-07-09 19:40 +0200
SubjectRe: [PATCH 2/2] dt: Add bindings for IDT VersaClock 5P49V5925
Message-ID<u1olB-2eX-29@gated-at.bofh.it>
In reply to#1683750
On 09.07.2017 16:31, Marek Vasut wrote:
> 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).
That was a copy typo. Thank you for pointing to this.

>
>> , 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/

I've  tested your patch series with ULCB board and HDMI display.
I will rebase 5P49V5925 patches on top of them.

Regards,
Vladimir

[toc] | [prev] | [next] | [standalone]

#1683781 — Re: [PATCH 2/2] dt: Add bindings for IDT VersaClock 5P49V5925

FromMarek Vasut <marek.vasut@gmail.com>
Date2017-07-09 19:50 +0200
SubjectRe: [PATCH 2/2] dt: Add bindings for IDT VersaClock 5P49V5925
Message-ID<u1ovf-2iE-5@gated-at.bofh.it>
In reply to#1683778
On 07/09/2017 07:38 PM, Vladimir Barinov wrote:
> On 09.07.2017 16:31, Marek Vasut wrote:
>> 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).
> That was a copy typo. Thank you for pointing to this.
> 
>>
>>> , 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/
> 
> I've  tested your patch series with ULCB board and HDMI display.
> I will rebase 5P49V5925 patches on top of them.

Thanks! That makes things easy :)

-- 
Best regards,
Marek Vasut

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web