[lkml]   [2012]   [Apr]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH V2 4/9] ata/sata_mv: Remove conditional compilation of clk code
On Tue, Apr 24, 2012 at 01:28:20PM +0530, Viresh Kumar wrote:
> On 4/24/2012 1:12 PM, Russell King - ARM Linux wrote:
> > If you don't have the clk API configured, you have no clocks to control.
> > So, why not make clk_get() return NULL, and make the rest of the API
> > calls do nothing? That's what you'll end up codifying in the drivers
> > anyway.
> Ok. We can return NULL from calls that return clk *. What about other
> routines that return integers. Like, clk_enable().
> Is returning 0 correct? Which would mean we were able to enable clk, but
> actually we haven't.

Think about this case: if you don't have the means to control the clock
inputs to a device (for example, you don't support the clk API on your
CPU arch) then for the device to be functional, it must be supplied with
all its necessary clocks. Therefore, the clock is already enabled. It
makes sense for the clk API to stub-out to be completely transparent and
non-error inducing to the driver.

The problem comes with clk_get_rate(). I'd suggest merely returning zero
for that in this case. If the clock rate is really required by a driver,
then the clk API would need to be enabled.

 \ /
  Last update: 2012-04-24 10:29    [from the cache]
©2003-2011 Jasper Spaans