pwm-bl: change some warning
author许盛飞 <xsf@rock-chips.com>
Tue, 21 Jan 2014 10:47:26 +0000 (18:47 +0800)
committer许盛飞 <xsf@rock-chips.com>
Tue, 21 Jan 2014 10:47:26 +0000 (18:47 +0800)
arch/arm/boot/dts/rk3188-pinctrl.dtsi
arch/arm/boot/dts/rk3188-tb.dts
arch/arm/boot/dts/rk3188.dtsi
drivers/video/backlight/pwm_bl.c
include/linux/pwm_backlight.h

index 4a6819fb7c7aaa13666e740b25de2725542bae20..8a3579dbfb550d971bdc2b83cab9856f4434b793 100755 (executable)
                };
 
                gpio3_pwm {
-                       pwm0:pwm0 {
+                       pwm0_pin:pwm0 {
                                rockchip,pins = <PWM0>;
                                rockchip,pull = <VALUE_PULL_DISABLE>;
                                //rockchip,voltage = <VALUE_VOL_DEFAULT>;
                        };
 
 
-                       pwm1:pwm1 {
+                       pwm1_pin:pwm1 {
                                rockchip,pins = <PWM1>;
                                rockchip,pull = <VALUE_PULL_DISABLE>;
                                //rockchip,voltage = <VALUE_VOL_DEFAULT>;
                        };
 
 
-                       pwm2:pwm2 {
+                       pwm2_pin:pwm2 {
                                rockchip,pins = <PWM2>;
                                rockchip,pull = <VALUE_PULL_DISABLE>;
                                //rockchip,voltage = <VALUE_VOL_DEFAULT>;
                        };
 
 
-                       pwm3:pwm3 {
+                       pwm3_pin:pwm3 {
                                rockchip,pins = <PWM3>;
                                rockchip,pull = <VALUE_PULL_DISABLE>;
                                //rockchip,voltage = <VALUE_VOL_DEFAULT>;
index 261ad7d205536f45451586b084af4937111bdbe9..34bf60416b221d158801e38ce7fba1e060aba886 100644 (file)
@@ -57,7 +57,7 @@
 &lcdc1 {
        status = "okay";
 };
-&rk_pwm3 {
+&pwm3 {
         status = "okay";
 };
 
index 92c1ded932fb794a1700d1532486afd6990ecc05..3896aeab3bf73736ff1a9e65bacdef881a1c28c6 100755 (executable)
                i2c4 = &i2c4;
                lcdc0 = &lcdc0;
                lcdc1 = &lcdc1;
-               pwm0 = &rk_pwm0;
-               pwm1 = &rk_pwm1;
-               pmw2 = &rk_pwm2;
-               pwm3 = &rk_pwm3;
        };
 
        cpus {
                status = "disabled";
         };
 
-       rk_pwm0: pwm@20030000{
+       pwm0: pwm@20030000{
                 compatible = "rockchip,pwm";
                 reg = <0x20030000 0x10>; /*0x20030000*/
                 #pwm-cells = <2>;
                 pinctrl-names = "default";
-                pinctrl-0 = <&pwm0>;
+                pinctrl-0 = <&pwm0_pin>;
                 status = "disabled";
 
         };
 
-        rk_pwm1: pwm@20030010{
+        pwm1: pwm@20030010{
                 compatible = "rockchip,pwm";
                 reg = <0x20030010 0x10>; /*0x20030000*/
                 #pwm-cells = <2>;
                 pinctrl-names = "default";
-                pinctrl-0 = <&pwm1>;
+                pinctrl-0 = <&pwm1_pin>;
                 status = "disabled";
 
         };
-       rk_pwm2: pwm@20050020{
+       pwm2: pwm@20050020{
                 compatible = "rockchip,pwm";
                 reg = <0x20050020 0x10>; /*0x20030000*/
                 #pwm-cells = <2>;
                 pinctrl-names = "default";
-                pinctrl-0 = <&pwm2>;
+                pinctrl-0 = <&pwm2_pin>;
                 status = "disabled";
 
         };
 
-        rk_pwm3: pwm@20050030{
+        pwm3: pwm@20050030{
                 compatible = "rockchip,pwm";
                 reg = <0x20050030 0x10>; /*0x20030000*/
                 #pwm-cells = <2>;
                 pinctrl-names = "default";
-                pinctrl-0 = <&pwm3>;
+                pinctrl-0 = <&pwm3_pin>;
                 status = "disabled";
 
         };
 
         backlight {
                 compatible = "pwm-backlight";
-                pwms = <&rk_pwm3 0 5000000>;
+                pwms = <&pwm3 0 5000000>;
                 brightness-levels = <0 4 8 16 32 64 128 255>;
                 default-brightness-level = <6>;
                 gpios = <&gpio0 GPIO_A2 GPIO_ACTIVE_LOW>;
index e87db838c86ef905abca5c078091604b994e9c37..07cb9aa119e447c43b557ddebc723b91b4bb40ab 100644 (file)
@@ -103,6 +103,8 @@ static int pwm_backlight_parse_dt(struct device *dev,
        int length;
        u32 value;
        int gpio,ret;
+        enum of_gpio_flags flags;
+
 
        if (!node)
                return -ENODEV;
@@ -134,20 +136,20 @@ static int pwm_backlight_parse_dt(struct device *dev,
                                           &value);
                if (ret < 0)
                        return ret;
-               data->gpios = devm_kzalloc(dev,
+               data->gpio = devm_kzalloc(dev,
                                sizeof(struct gpio) * 1,
                                GFP_KERNEL);
-               gpio = of_get_named_gpio(node, "gpios", 0);
-               data->gpios->gpio = gpio;
+               gpio = of_get_named_gpio_flags(node, "gpios", 0, &flags);
+               data->gpio->gpio = gpio;
 
 
                if (gpio != -1) {
                        ret = devm_gpio_request(dev, gpio, "gpio_pwm_bl_en");
                        if(ret){
-                               data->gpios->gpio = -1;
+                               data->gpio->gpio = -1;
                                goto err_free_pwm;
                        }
-                       ret = gpio_direction_output(data->gpios->gpio , 1);
+                       ret = gpio_direction_output(data->gpio->gpio , flags);
                        if(ret){
                                goto err_free_pwm;
                        }
index 710a5c1b3a3fe7195e2a8ec4b959d5366672b936..0e80289481f58ee277d069f3795930b1006dcc86 100644 (file)
@@ -14,7 +14,7 @@ struct platform_pwm_backlight_data {
        unsigned int pwm_period_ns;
        unsigned int *levels;
 
-       struct gpio *gpios;
+       struct gpio *gpio;
        
        int (*init)(struct device *dev);
        int (*notify)(struct device *dev, int brightness);