On 08/19/2014 02:32 PM, Mark Rutland wrote:

On Mon, Aug 18, 2014 at 10:46:39PM +0100, Jyri Sarha wrote:

quoted

The added ti,gpio-clock is a basic clock that can be enabled and
disabled trough a gpio output. The DT binding document for the clock
is also added. For EPROBE_DEFER handling the registering of the clock
has to be delayed until of_clk_get() call time.

I guess this is basically an AND gate with the GPIO and parent clock as
inputs? Or is this something more complex that we might want to do more
things with later?

If it's the former it sounds like this could be a completely generic
"gpio-gate-clock".

Yes, its completely generic. However, since there has been no response 
to my several earlier attempts to get this patch in I finally gave up 
and moved it under clk/ti.

"gpio-gate-clock" sound like more accurate name for ti. I'll change that.

Do you think I should give this one more try to get it in as generic 
basic clock?

Cheers,
Jyri