[linux-yocto] [PATCH 06/10] UPSTREAM: backlight: lm3630a_bl: Stop messing with the pwm->period field

California Sullivan california.l.sullivan at intel.com
Thu Jul 21 16:47:06 PDT 2016


From: Boris BREZILLON <boris.brezillon at free-electrons.com>

pwm->period field is not supposed to be changed by PWM users. The only
ones authorized to change it are the PWM core and PWM drivers.

Signed-off-by: Boris Brezillon <boris.brezillon at free-electrons.com>
Acked-by: Lee Jones <lee.jones at linaro.org>
Signed-off-by: Thierry Reding <thierry.reding at gmail.com>
(cherry picked from 4ff66efd59102280a71d432c9f0e89cdaf66b849)
Signed-off-by: California Sullivan <california.l.sullivan at intel.com>
---
 drivers/video/backlight/lm3630a_bl.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/video/backlight/lm3630a_bl.c b/drivers/video/backlight/lm3630a_bl.c
index 35fe482..3d16bd6 100644
--- a/drivers/video/backlight/lm3630a_bl.c
+++ b/drivers/video/backlight/lm3630a_bl.c
@@ -162,7 +162,7 @@ static int lm3630a_intr_config(struct lm3630a_chip *pchip)
 
 static void lm3630a_pwm_ctrl(struct lm3630a_chip *pchip, int br, int br_max)
 {
-	unsigned int period = pwm_get_period(pchip->pwmd);
+	unsigned int period = pchip->pdata->pwm_period;
 	unsigned int duty = br * period / br_max;
 
 	pwm_config(pchip->pwmd, duty, period);
@@ -425,7 +425,6 @@ static int lm3630a_probe(struct i2c_client *client,
 			return PTR_ERR(pchip->pwmd);
 		}
 	}
-	pchip->pwmd->period = pdata->pwm_period;
 
 	/* interrupt enable  : irq 0 is not allowed */
 	pchip->irq = client->irq;
-- 
2.5.5



More information about the linux-yocto mailing list