[v2,2/3] clk: rockchip: add dclk_vop_frac ids for vop

Commit Message

Chris Zhong Sept. 2, 2016, 3:26 a.m. UTC

From: Yakir Yang <[e-mail]>

Export the dclk_vop_frac out, so we can set the dclk_vop as the
child of dclk_vop_frac, and then we can start to take use of
the fractional dividers.

Signed-off-by: Xing Zheng <[e-mail]>
Signed-off-by: Yakir Yang <[e-mail]>
Signed-off-by: Chris Zhong <[e-mail]>
---

 include/dt-bindings/clock/rk3399-cru.h | 2 ++
 1 file changed, 2 insertions(+)

Comments

Heiko Stuebner Sept. 2, 2016, 8:43 p.m. UTC | #1

Am Donnerstag, 1. September 2016, 20:26:24 schrieb Chris Zhong:
> From: Yakir Yang <[e-mail]>
> 
> Export the dclk_vop_frac out, so we can set the dclk_vop as the
> child of dclk_vop_frac, and then we can start to take use of
> the fractional dividers.
> 
> Signed-off-by: Xing Zheng <[e-mail]>
> Signed-off-by: Yakir Yang <[e-mail]>
> Signed-off-by: Chris Zhong <[e-mail]>

Yakir is listed as author above, what relationship does Xing have to this 
patch and the 3rd one?

Chris Zhong Sept. 2, 2016, 8:58 p.m. UTC | #2

Hi Heiko


On 09/02/2016 01:43 PM, Heiko Stübner wrote:
> Am Donnerstag, 1. September 2016, 20:26:24 schrieb Chris Zhong:
>> From: Yakir Yang <[e-mail]>
>>
>> Export the dclk_vop_frac out, so we can set the dclk_vop as the
>> child of dclk_vop_frac, and then we can start to take use of
>> the fractional dividers.
>>
>> Signed-off-by: Xing Zheng <[e-mail]>
>> Signed-off-by: Yakir Yang <[e-mail]>
>> Signed-off-by: Chris Zhong <[e-mail]>
> Yakir is listed as author above, what relationship does Xing have to this
> patch and the 3rd one?
>
>

Yakir is the author of the [2/3] and [3/3] patch, and Doug is the author 
of [1/3]
The others are committer, and these tags should be removed.

Heiko Stuebner Sept. 2, 2016, 9:14 p.m. UTC | #3

Am Freitag, 2. September 2016, 13:58:19 schrieb Chris Zhong:
> Hi Heiko
> 
> On 09/02/2016 01:43 PM, Heiko Stübner wrote:
> > Am Donnerstag, 1. September 2016, 20:26:24 schrieb Chris Zhong:
> >> From: Yakir Yang <[e-mail]>
> >> 
> >> Export the dclk_vop_frac out, so we can set the dclk_vop as the
> >> child of dclk_vop_frac, and then we can start to take use of
> >> the fractional dividers.
> >> 
> >> Signed-off-by: Xing Zheng <[e-mail]>
> >> Signed-off-by: Yakir Yang <[e-mail]>
> >> Signed-off-by: Chris Zhong <[e-mail]>
> > 
> > Yakir is listed as author above, what relationship does Xing have to this
> > patch and the 3rd one?
> 
> Yakir is the author of the [2/3] and [3/3] patch, and Doug is the author
> of [1/3]
> The others are committer, and these tags should be removed.

not all of them ... i.e. your Signed-off needs to stay in, as you are the one 
sending in this set of patches ... I'll drop the Signed-off from Xing though.


Thanks for clarifying
Heiko

Heiko Stuebner Sept. 4, 2016, 9:50 p.m. UTC | #4

Am Donnerstag, 1. September 2016, 20:26:24 CEST schrieb Chris Zhong:
> From: Yakir Yang <[e-mail]>
> 
> Export the dclk_vop_frac out, so we can set the dclk_vop as the
> child of dclk_vop_frac, and then we can start to take use of
> the fractional dividers.
> 
> Signed-off-by: Xing Zheng <[e-mail]>
> Signed-off-by: Yakir Yang <[e-mail]>
> Signed-off-by: Chris Zhong <[e-mail]>

applied to my clk-branch for 4.9

Patch

diff --git a/include/dt-bindings/clock/rk3399-cru.h b/include/dt-bindings/clock/rk3399-cru.h
index ce5f3e9..220a60f 100644
--- a/include/dt-bindings/clock/rk3399-cru.h
+++ b/include/dt-bindings/clock/rk3399-cru.h
@@ -138,6 +138,8 @@ 
 #define DCLK_VOP0_DIV			182
 #define DCLK_VOP1_DIV			183
 #define DCLK_M0_PERILP			184
+#define DCLK_VOP0_FRAC			185
+#define DCLK_VOP1_FRAC			186
 
 #define FCLK_CM0S			190