Re: [PATCH v2] clk: add userspace clock consumer

[Stephen Boyd <sboyd@codeaurora.org>]

On 02/23, Rob Herring wrote:
> On Mon, Feb 22, 2016 at 08:48:39PM +0900, Akinobu Mita wrote:
> > This adds userspace consumer for common clock.
> > 
> > This driver is inspired from Userspace regulator consumer
> > (REGULATOR_USERSPACE_CONSUMER) and it is useful for test purposes and
> > some classes of devices that are controlled entirely from user space.
> 
> Whether the kernel handles a clock or userspace does should have nothing 
> to do with DT. Plus DT does not expose many of the clocks, so any 
> debug feature needs to expose all clocks and be independent of DT.
> 
> Beyond debug, I don't think we should be exposing clock control to 
> userspace.

Agreed. Plus, the userspace regulator consumer doesn't have any
DT binding for it, so I don't see why we would have a DT binding
for the clk one.

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project