Re: [RESUBMIT PATCH v7 4/9] devicetree: backlight: Add new SKY81452 backlight binding
From: Lee Jones <hidden>
Date: 2014-11-17 12:23:44
Also in:
lkml
On Mon, 17 Nov 2014, gyungoh-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org wrote:
quoted hunk ↗ jump to hunk
From: Gyungoh Yoo <redacted> Signed-off-by: Gyungoh Yoo <redacted> --- Changes v7: Nothing Changes v6: Nothing Changes v5: Nothing Changes v4: Nothing Changes v3: Nothing Changes v2: Added reg attribute for I2C slave address .../video/backlight/sky81452-backlight.txt | 24 ++++++++++++++++++++++ 1 file changed, 24 insertions(+) create mode 100644 Documentation/devicetree/bindings/video/backlight/sky81452-backlight.txtdiff --git a/Documentation/devicetree/bindings/video/backlight/sky81452-backlight.txt b/Documentation/devicetree/bindings/video/backlight/sky81452-backlight.txt new file mode 100644 index 0000000..2c2d947 --- /dev/null +++ b/Documentation/devicetree/bindings/video/backlight/sky81452-backlight.txt@@ -0,0 +1,24 @@ +SKY81452-backlight bindings + +Required properties: +- compatible : Must be "skyworks,sky81452-backlight" + +Optional properties: +- name : Name of backlight device. Default is 'lcd-backlight'. +- gpio-enable : GPIO to use to EN pin. +- enable : Enable mask for current sink channel 1 to 6. +- ignore-pwm : Ignore both PWM input +- dpwm-mode : Enable DPWM dimming mode, otherwise Analog dimming mode +- phase-shift : Enable phase shift mode +- ovp-level : Over-voltage protection level. Should be between 14 or 28V. +- short-detection-threshold : It should be one of 4, 5, 6 and 7V. +- boost-current-limit : It should be one of 800, 1100 and 1500mA.
These are going to need some serious reviewing from the DT guys. Each property should either have a vendor prefix or be made generic and placed in the generic DT documentation for the subsystem. The name 'enable' is far to generic we don't tend to put masks in DT. There is already a generic property for GPIO enable pins.
+Example:
+
+ backlight {
+ compatible = "skyworks,sky81452-backlight";
+ name = "pwm-backlight";
+ enable = <0x3F>;Lower case.
+ ignore-pwm;
You should probably demonstrate more of your properties here.
+ };
-- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html