[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.156 / U:1.632 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site