[PATCH v5 07/17] video: lcd: add LoCoMo LCD driver

Dmitry Eremin-Solenikov dbaryshkov at gmail.com
Sun Jun 14 09:28:50 PDT 2015


2015-06-14 18:18 GMT+03:00 Russell King - ARM Linux <linux at arm.linux.org.uk>:
> On Mon, Jun 08, 2015 at 11:56:38PM +0300, Dmitry Eremin-Solenikov wrote:
>> +int locomo_lcd_set_power(struct lcd_device *ldev, int power)
>> +{
>> +     struct locomo_lcd *lcd = lcd_get_data(ldev);
>> +
>> +     dev_dbg(&ldev->dev, "LCD power %d (is %d)\n", power, lcd->power);
>> +
>> +     if (!power && lcd->power)
>> +             locomo_lcd_on(lcd);
>> +
>> +     if (power && !lcd->power)
>> +             locomo_lcd_off(lcd);
>
> Is this correct?  You want to turn the LCD _off_ if power is non-zero, but
> turn it _on_ if power is _zero_ ?

Yes, this is a crazy part of fb/backlight/LCD, where power is a FB_BLANK_*
value, and FB_BLANK_UNBLANK is equal to 0.

-- 
With best wishes
Dmitry



More information about the linux-arm-kernel mailing list