From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jyri Sarha Subject: Re: [PATCH v2 1/8] clk: ti: add gpio controlled clock Date: Tue, 19 Aug 2014 15:23:12 +0300 Message-ID: <53F341B0.3090904@ti.com> References: <20140819113200.GJ3302@leverpostej> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20140819113200.GJ3302@leverpostej> Sender: linux-omap-owner@vger.kernel.org To: Mark Rutland Cc: "dri-devel@lists.freedesktop.org" , "airlied@linux.ie" , "linux-omap@vger.kernel.org" , "devicetree@vger.kernel.org" , "bcousson@baylibre.com" , "alsa-devel@alsa-project.org" , "tony@atomide.com" , "broonie@kernel.org" , "liam.r.girdwood@linux.intel.com" , "peter.ujfalusi@ti.com" , "detheridge@ti.com" , "ezequiel.garcia@free-electrons.com" , "t-kristo@ti.com" List-Id: devicetree@vger.kernel.org On 08/19/2014 02:32 PM, Mark Rutland wrote: > On Mon, Aug 18, 2014 at 10:46:39PM +0100, Jyri Sarha wrote: >> 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