On Tue 2016-03-01 22:28:00, Heiner Kallweit wrote: > Extend brightness sysfs property handling to deal with monochrome > and color mode as well. > > Signed-off-by: Heiner Kallweit <hkallweit1@xxxxxxxxx> > --- > v2: > - split from patch 1 > v3: > - moved one change (led_is_off) to patch 1 > v4: > - changed printf format string to %#.6x > v5: > - no changes > --- > drivers/leds/led-class.c | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/drivers/leds/led-class.c b/drivers/leds/led-class.c > index 007a5b3..8a3748a 100644 > --- a/drivers/leds/led-class.c > +++ b/drivers/leds/led-class.c > @@ -32,7 +32,10 @@ static ssize_t brightness_show(struct device *dev, > /* no lock needed for this */ > led_update_brightness(led_cdev); > > - return sprintf(buf, "%u\n", led_cdev->brightness); > + if (led_cdev->brightness > LED_FULL) > + return sprintf(buf, "%#.6x\n", led_cdev->brightness); > + else > + return sprintf(buf, "%u\n", led_cdev->brightness); > } > > static ssize_t brightness_store(struct device *dev, No... I don't think you should change interface for existing file like this. Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-leds" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
