[lkml]   [2009]   [Apr]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [RFD] voltage/current regulator consumer interface
    On Mon, Apr 20, 2009 at 05:32:21PM +0300, Mike Rapoport wrote:

    > It is the most simple and straight forward solution. However, I have several
    > questions and I cannot answer them myself:

    Some initial thoughts:

    > Should the driver handle single or several supplies at once?

    I'd expect that it should be able to cope with that - a lot of hardware
    takes multiple supplies.

    > If the driver handles several supplies how to define attributes per-supply?
    > What attributes will be exposed? Would it be simple 'state' that can be
    > enabled or disabled? Or the entire set of micorvolts/microamps etc?

    I'd expect that if you're getting into much more than a simple enable
    for the entire device it'd be getting to the point where it's worth
    considering writing an actual driver for the device. That said, it's a
    fairly natural extension to support more fine grained stuff if someone
    does actually end up wanting it.

    > - Extend regulator framework so that regulator_consumer_supply entities will
    > gain their own representation in sysfs. Depending on machine constraints the
    > supply attributes can be either read-only or writable. I haven't dedicated much
    > thought to it, so a cannot state any pros and cons.

    I'd expect making them writable to be very fragile if there is an active
    consumer driver - I'd expect reference counts to get lost with enable
    and disable and I'd expect drivers managing voltages and currents to get
    confused if their own configuration gets changed under them. The core
    already needs to arbitrate between multiple users so it seems to make
    sense to have userspace represented as other users are to the core.

    Being able to read back the consumer status doesn't seem unreasonable,

     \ /
      Last update: 2009-04-20 16:59    [W:0.020 / U:10.392 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site