backlight: Separate backlight properties from backlight ops pointers
[firefly-linux-kernel-4.4.55.git] / include / linux / lcd.h
index 46970af2ca89fc8e9c8a000883a993b54e02b2dc..598793c0745b3c97ec593f1329b351b5743c3b1d 100644 (file)
@@ -14,7 +14,7 @@
 
 /* Notes on locking:
  *
- * lcd_device->props_lock is an internal backlight lock protecting the props
+ * lcd_device->ops_lock is an internal backlight lock protecting the ops
  * field and no code outside the core should need to touch it.
  *
  * Access to set_power() is serialised by the update_lock mutex since
 struct lcd_device;
 struct fb_info;
 
-/* This structure defines all the properties of a LCD flat panel. */
 struct lcd_properties {
+       /* The maximum value for contrast (read-only) */
+       int max_contrast;
+};
+
+struct lcd_ops {
        /* Get the LCD panel power status (0: full on, 1..3: controller
           power on, flat panel power off, 4: full off), see FB_BLANK_XXX */
        int (*get_power)(struct lcd_device *);
        /* Enable or disable power to the LCD (0: on; 4: off, see FB_BLANK_XXX) */
        int (*set_power)(struct lcd_device *, int power);
-       /* The maximum value for contrast (read-only) */
-       int max_contrast;
        /* Get the current contrast setting (0-max_contrast) */
        int (*get_contrast)(struct lcd_device *);
        /* Set LCD panel contrast */
@@ -49,12 +51,13 @@ struct lcd_properties {
 };
 
 struct lcd_device {
-       /* This protects the 'props' field. If 'props' is NULL, the driver that
+       struct lcd_properties props;
+       /* This protects the 'ops' field. If 'ops' is NULL, the driver that
           registered this device has been unloaded, and if class_get_devdata()
           points to something in the body of that driver, it is also invalid. */
-       struct mutex props_lock;
+       struct mutex ops_lock;
        /* If this is NULL, the backing module is unloaded */
-       struct lcd_properties *props;
+       struct lcd_ops *ops;
        /* Serialise access to set_power method */
        struct mutex update_lock;
        /* The framebuffer notifier block */
@@ -66,13 +69,13 @@ struct lcd_device {
 static inline void lcd_set_power(struct lcd_device *ld, int power)
 {
        mutex_lock(&ld->update_lock);
-       if (ld->props && ld->props->set_power)
-               ld->props->set_power(ld, power);
+       if (ld->ops && ld->ops->set_power)
+               ld->ops->set_power(ld, power);
        mutex_unlock(&ld->update_lock);
 }
 
 extern struct lcd_device *lcd_device_register(const char *name,
-       void *devdata, struct lcd_properties *lp);
+       void *devdata, struct lcd_ops *ops);
 extern void lcd_device_unregister(struct lcd_device *ld);
 
 #define to_lcd_device(obj) container_of(obj, struct lcd_device, class_dev)