From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Turquette Subject: Re: [PATCH] clk: palmas: add clock driver for palmas Date: Fri, 04 Oct 2013 12:35:45 -0700 Message-ID: <20131004193545.4343.15596@quantum> References: <1380803407-15196-1-git-send-email-ldewangan@nvidia.com> <20131004164139.GR27287@sirena.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8BIT Return-path: In-Reply-To: <20131004164139.GR27287@sirena.org.uk> Sender: linux-doc-owner@vger.kernel.org To: Mark Brown , Laxman Dewangan Cc: mark.rutland@arm.com, devicetree@vger.kernel.org, pawel.moll@arm.com, ijc+devicetree@hellion.org.uk, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, rob.herring@calxeda.com, rob@landley.net, grant.likely@linaro.org, linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org Quoting Mark Brown (2013-10-04 09:41:39) > On Thu, Oct 03, 2013 at 06:00:07PM +0530, Laxman Dewangan wrote: > > > + Optional subnode properties: > > + ti,clock-boot-enable: Enable clock at the time of booting. > > Shouldn't this be a generic clock subsystem thing? It seems like > something that might be desired for nearly any clock. The best solution is to have a driver claim the clock with clk_get and enable it with clk_prepare_enable. I gave more detail in my reply to Laxman. Regards, Mike