2015-06-14 18:18 GMT+03:00 Russell King - ARM Linux [off-list ref]:

On Mon, Jun 08, 2015 at 11:56:38PM +0300, Dmitry Eremin-Solenikov wrote:

quoted

+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