Re: [PATCH v2 1/2] clk: rockchip: add bindings for the mmc clock phases

2014-11-17 Thread Alexandru Stan
Sorry for my previous mail, I sent it as HTML and also top posted. Here it goes again: On Mon, Nov 17, 2014 at 11:39 AM, Mike Turquette wrote: > It looks like you are adding new clocks to handle the phase requirement. > Is that the right thing to do? Don't these clks already exist (e.g. >

Re: [PATCH v2 1/2] clk: rockchip: add bindings for the mmc clock phases

2014-11-17 Thread Mike Turquette
Quoting Alexandru M Stan (2014-11-14 16:00:03) > This will be used in a later patch for clock phase tuning. > > Suggested-by: Heiko Stuebner > Signed-off-by: Alexandru M Stan > --- > Changes in v2: None > > include/dt-bindings/clock/rk3288-cru.h | 10 ++ > 1 file changed, 10

Re: [PATCH v2 1/2] clk: rockchip: add bindings for the mmc clock phases

2014-11-17 Thread Mike Turquette
Quoting Alexandru M Stan (2014-11-14 16:00:03) This will be used in a later patch for clock phase tuning. Suggested-by: Heiko Stuebner he...@sntech.de Signed-off-by: Alexandru M Stan ams...@chromium.org --- Changes in v2: None include/dt-bindings/clock/rk3288-cru.h | 10 ++ 1

Re: [PATCH v2 1/2] clk: rockchip: add bindings for the mmc clock phases

2014-11-17 Thread Alexandru Stan
Sorry for my previous mail, I sent it as HTML and also top posted. Here it goes again: On Mon, Nov 17, 2014 at 11:39 AM, Mike Turquette mturque...@linaro.org wrote: It looks like you are adding new clocks to handle the phase requirement. Is that the right thing to do? Don't these clks already

[PATCH v2 1/2] clk: rockchip: add bindings for the mmc clock phases

2014-11-14 Thread Alexandru M Stan
This will be used in a later patch for clock phase tuning. Suggested-by: Heiko Stuebner Signed-off-by: Alexandru M Stan --- Changes in v2: None include/dt-bindings/clock/rk3288-cru.h | 10 ++ 1 file changed, 10 insertions(+) diff --git a/include/dt-bindings/clock/rk3288-cru.h

[PATCH v2 1/2] clk: rockchip: add bindings for the mmc clock phases

2014-11-14 Thread Alexandru M Stan
This will be used in a later patch for clock phase tuning. Suggested-by: Heiko Stuebner he...@sntech.de Signed-off-by: Alexandru M Stan ams...@chromium.org --- Changes in v2: None include/dt-bindings/clock/rk3288-cru.h | 10 ++ 1 file changed, 10 insertions(+) diff --git