[lkml]   [2008]   [Jun]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] ata: ahci: power off unused ports
    Kristen Carlson Accardi wrote:
    > On Mon, 02 Jun 2008 14:15:27 -0400
    > Jeff Garzik <> wrote:
    >> Kristen Carlson Accardi wrote:
    >>> Here you can see that it would be very easy for another driver
    >>> to just add code to set the NO_HOTPLUG flag and then this code
    >>> will work for them as well, since we power off the phy using
    >>> DET which is specified by SATA.
    >>> here's that code:
    >> Agreed. The main discussion in this sub-thread is more about user
    >> interface. The user interface in this case, a module option, is
    >> specific to AHCI.
    >> Surely you can see how it is a bit silly to force each driver writer to
    >> create the same user interface in each driver, to support a generic concept.
    >> Jeff
    > Not all drivers will need a user interface to turn off hotplug
    > I would think. At any rate - I would think it'd be better to let
    > driver writers decide how they want their drivers to behave wrt
    > hotplug and power instead of forcing a generic policy on everyone.
    > This patch would provide users of AHCI controllers a way to save
    > power now, while you work on the grand scheme for polling/turning on off
    > hotplug via sysfs. It's an interim solution that impacts nobody but
    > ahci users and is can be easily integrated into whatever solution you
    > eventually work out.
    I like the patch - it seems that it will help a lot of users out near
    term in a very positive way while we iterate on the broader solution,


     \ /
      Last update: 2008-06-02 20:33    [W:0.024 / U:8.912 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site