lkml.org 
[lkml]   [2014]   [Sep]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] clk-rcar-gen2: RCAN clock support
Date
Hi Sergei,

On Friday 05 September 2014 17:33:28 Sergei Shtylyov wrote:
> On 09/05/2014 01:03 PM, Geert Uytterhoeven wrote:
> >>> Add RCAN clock support to the R-Car generation 2 CPG driver. This clock
> >>> gets derived from the USB_EXTAL clock by dividing it by 6. The layout of
> >>> RCANCKCR register is close to those of the clocks supported by the 'clk-
> >>> div6' driver but has no divider field, and so can't be supported by that
> >>> driver...
> >>>
> >>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> >>>
> >>> ---
> >>> The patch is against the 'clk-next' branch of Mike Turquette's
> >>> 'linux.git' repo.
> >>>
> >>> drivers/clk/shmobile/clk-rcar-gen2.c | 99 +++++++++++++++++++++++++++
> >>> 1 file changed, 99 insertions(+)
> >>>
> >> More than a months has passed, there hasn't been any feedback, patch
> >>
> >> hasn't been applied... what's wrong with it?
> >
> > Sorry for missing this, it went under my radar.
> >
> > I'm not a clock expert (pulling in Laurent), but it looks fine to me.
> > I'm just wondering whether you can simplify the code by using clk-gate?
>
> The gated clocks inherit their clock rate from the parent, while the RCAN
> clock has a fixed divisor (6). I'm gonna look into composite clocks instead.

The composite clock looks like it would indeed simplify the code. If that's
the case I would prefer that solution.

Please also remember to update the CPG DT bindings documentation to add the
rcan clock and the USB input clock. On that subject, do you know what other
clocks derive from the USB input clock ? The documentation isn't clear.

I'm also a bit puzzled by V2H that, according to the datasheet, lacks the
RANCKCR register, but has a RCAN clock with a dedicated RCAN divider in the
CPG diagram. I think Geert and Morimoto-san are investigating this.

--
Regards,

Laurent Pinchart



\
 
 \ /
  Last update: 2014-09-10 13:01    [W:0.205 / U:1.028 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site