Re: [PATCH 0/1] Add new driver for AlienFX control

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2014-02-05 at 10:34 -0600, Mario Limonciello wrote:
> On 02/05/2014 10:24 AM, Matthew Garrett wrote:
> > On Wed, 2014-02-05 at 10:22 -0600, Mario Limonciello wrote:
> >
> >> After digging further in, there actually is control for setting up the LEDs to
> >> perform different tasks in S0 and S3 power states which becomes more difficult
> >> to represent in the LED class.  So instead I've created a series of sysfs nodes
> >> to represent the different colors in the different zones.
> > Have you discussed the best way to represent this with the LED class
> > maintainer?
> No I haven't yet.  I initially coded it using the LED class, but I kept finding more details for it that didn't seem to match what the LED class provided properly.
> 
> The LED class is intended to represent the "brightness" of an individual LED.

Sure, that's the current state. But it makes sense to work on extending
an existing class rather than implementing a custom interface.

-- 
Matthew Garrett <matthew.garrett@xxxxxxxxxx>
��.n��������+%������w��{.n������_���v��z����n�r������&��z�ޗ�zf���h���~����������_��+v���)ߣ�


[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux