pwm: Stop referencing pwm->chip

thierryreding · thierryreding · commit 80943bbdcfa8 · 2023-12-20T16:04:14.000+01:00
Drivers have access to the chip via a function argument already, so
there is no need to reference it via the PWM device.

Reviewed-by: Uwe Kleine-König &lt;u.kleine-koenig@pengutronix.de&gt;
Signed-off-by: Thierry Reding &lt;thierry.reding@gmail.com&gt;
diff --git a/drivers/pwm/pwm-bcm-kona.c b/drivers/pwm/pwm-bcm-kona.c
@@ -260,7 +260,7 @@ static int kona_pwmc_apply(struct pwm_chip *chip, struct pwm_device *pwm,
 			return err;
 	}
 
-	err = kona_pwmc_config(pwm->chip, pwm, state->duty_cycle, state->period);
+	err = kona_pwmc_config(chip, pwm, state->duty_cycle, state->period);
 	if (err && !pwm->state.enabled)
 		clk_disable_unprepare(kp->clk);
 
diff --git a/drivers/pwm/pwm-img.c b/drivers/pwm/pwm-img.c
@@ -196,7 +196,7 @@ static int img_pwm_apply(struct pwm_chip *chip, struct pwm_device *pwm,
 		return 0;
 	}
 
-	err = img_pwm_config(pwm->chip, pwm, state->duty_cycle, state->period);
+	err = img_pwm_config(chip, pwm, state->duty_cycle, state->period);
 	if (err)
 		return err;
 
diff --git a/drivers/pwm/pwm-jz4740.c b/drivers/pwm/pwm-jz4740.c
@@ -123,7 +123,7 @@ static void jz4740_pwm_disable(struct pwm_chip *chip, struct pwm_device *pwm)
 static int jz4740_pwm_apply(struct pwm_chip *chip, struct pwm_device *pwm,
 			    const struct pwm_state *state)
 {
-	struct jz4740_pwm_chip *jz = to_jz4740(pwm->chip);
+	struct jz4740_pwm_chip *jz = to_jz4740(chip);
 	unsigned long long tmp = 0xffffull * NSEC_PER_SEC;
 	struct clk *clk = jz->clk[pwm->hwpwm];
 	unsigned long period, duty;
diff --git a/drivers/pwm/pwm-lpc18xx-sct.c b/drivers/pwm/pwm-lpc18xx-sct.c
@@ -328,7 +328,7 @@ static int lpc18xx_pwm_apply(struct pwm_chip *chip, struct pwm_device *pwm,
 		return 0;
 	}
 
-	err = lpc18xx_pwm_config(pwm->chip, pwm, state->duty_cycle, state->period);
+	err = lpc18xx_pwm_config(chip, pwm, state->duty_cycle, state->period);
 	if (err)
 		return err;
 
diff --git a/drivers/pwm/pwm-lpc32xx.c b/drivers/pwm/pwm-lpc32xx.c
@@ -103,7 +103,7 @@ static int lpc32xx_pwm_apply(struct pwm_chip *chip, struct pwm_device *pwm,
 		return 0;
 	}
 
-	err = lpc32xx_pwm_config(pwm->chip, pwm, state->duty_cycle, state->period);
+	err = lpc32xx_pwm_config(chip, pwm, state->duty_cycle, state->period);
 	if (err)
 		return err;
 
diff --git a/drivers/pwm/pwm-mediatek.c b/drivers/pwm/pwm-mediatek.c
@@ -217,7 +217,7 @@ static int pwm_mediatek_apply(struct pwm_chip *chip, struct pwm_device *pwm,
 		return 0;
 	}
 
-	err = pwm_mediatek_config(pwm->chip, pwm, state->duty_cycle, state->period);
+	err = pwm_mediatek_config(chip, pwm, state->duty_cycle, state->period);
 	if (err)
 		return err;
 
diff --git a/drivers/pwm/pwm-renesas-tpu.c b/drivers/pwm/pwm-renesas-tpu.c
@@ -416,7 +416,7 @@ static int tpu_pwm_apply(struct pwm_chip *chip, struct pwm_device *pwm,
 		return 0;
 	}
 
-	err = tpu_pwm_config(pwm->chip, pwm,
+	err = tpu_pwm_config(chip, pwm,
 			     state->duty_cycle, state->period, enabled);
 	if (err)
 		return err;
diff --git a/drivers/pwm/pwm-sti.c b/drivers/pwm/pwm-sti.c
@@ -407,7 +407,7 @@ static int sti_pwm_apply(struct pwm_chip *chip, struct pwm_device *pwm,
 		return 0;
 	}
 
-	err = sti_pwm_config(pwm->chip, pwm, state->duty_cycle, state->period);
+	err = sti_pwm_config(chip, pwm, state->duty_cycle, state->period);
 	if (err)
 		return err;
 
diff --git a/drivers/pwm/pwm-stmpe.c b/drivers/pwm/pwm-stmpe.c
@@ -275,7 +275,7 @@ static int stmpe_24xx_pwm_apply(struct pwm_chip *chip, struct pwm_device *pwm,
 		return 0;
 	}
 
-	err = stmpe_24xx_pwm_config(pwm->chip, pwm, state->duty_cycle, state->period);
+	err = stmpe_24xx_pwm_config(chip, pwm, state->duty_cycle, state->period);
 	if (err)
 		return err;
 
diff --git a/drivers/pwm/pwm-tegra.c b/drivers/pwm/pwm-tegra.c
@@ -256,7 +256,7 @@ static int tegra_pwm_apply(struct pwm_chip *chip, struct pwm_device *pwm,
 		return 0;
 	}
 
-	err = tegra_pwm_config(pwm->chip, pwm, state->duty_cycle, state->period);
+	err = tegra_pwm_config(chip, pwm, state->duty_cycle, state->period);
 	if (err)
 		return err;
 
diff --git a/drivers/pwm/pwm-twl-led.c b/drivers/pwm/pwm-twl-led.c
@@ -175,7 +175,7 @@ static int twl4030_pwmled_apply(struct pwm_chip *chip, struct pwm_device *pwm,
 	 * pwm_apply_state because of !state->enabled and so the two values in
 	 * pwm->state might not be configured in hardware.
 	 */
-	ret = twl4030_pwmled_config(pwm->chip, pwm,
+	ret = twl4030_pwmled_config(chip, pwm,
 				    state->duty_cycle, state->period);
 	if (ret)
 		return ret;
@@ -275,7 +275,7 @@ static int twl6030_pwmled_apply(struct pwm_chip *chip, struct pwm_device *pwm,
 		return 0;
 	}
 
-	err = twl6030_pwmled_config(pwm->chip, pwm,
+	err = twl6030_pwmled_config(chip, pwm,
 				    state->duty_cycle, state->period);
 	if (err)
 		return err;
diff --git a/drivers/pwm/pwm-twl.c b/drivers/pwm/pwm-twl.c
@@ -294,7 +294,7 @@ static int twl4030_pwm_apply(struct pwm_chip *chip, struct pwm_device *pwm,
 		return 0;
 	}
 
-	err = twl_pwm_config(pwm->chip, pwm, state->duty_cycle, state->period);
+	err = twl_pwm_config(chip, pwm, state->duty_cycle, state->period);
 	if (err)
 		return err;
 
@@ -319,7 +319,7 @@ static int twl6030_pwm_apply(struct pwm_chip *chip, struct pwm_device *pwm,
 		return 0;
 	}
 
-	err = twl_pwm_config(pwm->chip, pwm, state->duty_cycle, state->period);
+	err = twl_pwm_config(chip, pwm, state->duty_cycle, state->period);
 	if (err)
 		return err;
 
diff --git a/drivers/pwm/pwm-vt8500.c b/drivers/pwm/pwm-vt8500.c
@@ -209,7 +209,7 @@ static int vt8500_pwm_apply(struct pwm_chip *chip, struct pwm_device *pwm,
 	 * pwm_apply_state because of !state->enabled and so the two values in
 	 * pwm->state might not be configured in hardware.
 	 */
-	err = vt8500_pwm_config(pwm->chip, pwm, state->duty_cycle, state->period);
+	err = vt8500_pwm_config(chip, pwm, state->duty_cycle, state->period);
 	if (err)
 		return err;
 

Original file line number	Diff line number	Diff line change
`@@ -260,7 +260,7 @@ static int kona_pwmc_apply(struct pwm_chip chip, struct pwm_device pwm,`
`260`	`260`	`return err;`
`261`	`261`	`}`
`262`	`262`
`263`		`- err = kona_pwmc_config(pwm->chip, pwm, state->duty_cycle, state->period);`
	`263`	`+ err = kona_pwmc_config(chip, pwm, state->duty_cycle, state->period);`
`264`	`264`	`if (err && !pwm->state.enabled)`
`265`	`265`	`clk_disable_unprepare(kp->clk);`
`266`	`266`
Original file line number	Diff line number	Diff line change
`@@ -196,7 +196,7 @@ static int img_pwm_apply(struct pwm_chip chip, struct pwm_device pwm,`
`196`	`196`	`return 0;`
`197`	`197`	`}`
`198`	`198`
`199`		`- err = img_pwm_config(pwm->chip, pwm, state->duty_cycle, state->period);`
	`199`	`+ err = img_pwm_config(chip, pwm, state->duty_cycle, state->period);`
`200`	`200`	`if (err)`
`201`	`201`	`return err;`
`202`	`202`
Original file line number	Diff line number	Diff line change
`@@ -123,7 +123,7 @@ static void jz4740_pwm_disable(struct pwm_chip chip, struct pwm_device pwm)`
`123`	`123`	`static int jz4740_pwm_apply(struct pwm_chip chip, struct pwm_device pwm,`
`124`	`124`	`const struct pwm_state *state)`
`125`	`125`	`{`
`126`		`- struct jz4740_pwm_chip *jz = to_jz4740(pwm->chip);`
	`126`	`+ struct jz4740_pwm_chip *jz = to_jz4740(chip);`
`127`	`127`	`unsigned long long tmp = 0xffffull * NSEC_PER_SEC;`
`128`	`128`	`struct clk *clk = jz->clk[pwm->hwpwm];`
`129`	`129`	`unsigned long period, duty;`
Original file line number	Diff line number	Diff line change
`@@ -328,7 +328,7 @@ static int lpc18xx_pwm_apply(struct pwm_chip chip, struct pwm_device pwm,`
`328`	`328`	`return 0;`
`329`	`329`	`}`
`330`	`330`
`331`		`- err = lpc18xx_pwm_config(pwm->chip, pwm, state->duty_cycle, state->period);`
	`331`	`+ err = lpc18xx_pwm_config(chip, pwm, state->duty_cycle, state->period);`
`332`	`332`	`if (err)`
`333`	`333`	`return err;`
`334`	`334`
Original file line number	Diff line number	Diff line change
`@@ -103,7 +103,7 @@ static int lpc32xx_pwm_apply(struct pwm_chip chip, struct pwm_device pwm,`
`103`	`103`	`return 0;`
`104`	`104`	`}`
`105`	`105`
`106`		`- err = lpc32xx_pwm_config(pwm->chip, pwm, state->duty_cycle, state->period);`
	`106`	`+ err = lpc32xx_pwm_config(chip, pwm, state->duty_cycle, state->period);`
`107`	`107`	`if (err)`
`108`	`108`	`return err;`
`109`	`109`
Original file line number	Diff line number	Diff line change
`@@ -217,7 +217,7 @@ static int pwm_mediatek_apply(struct pwm_chip chip, struct pwm_device pwm,`
`217`	`217`	`return 0;`
`218`	`218`	`}`
`219`	`219`
`220`		`- err = pwm_mediatek_config(pwm->chip, pwm, state->duty_cycle, state->period);`
	`220`	`+ err = pwm_mediatek_config(chip, pwm, state->duty_cycle, state->period);`
`221`	`221`	`if (err)`
`222`	`222`	`return err;`
`223`	`223`
Original file line number	Diff line number	Diff line change
`@@ -416,7 +416,7 @@ static int tpu_pwm_apply(struct pwm_chip chip, struct pwm_device pwm,`
`416`	`416`	`return 0;`
`417`	`417`	`}`
`418`	`418`
`419`		`- err = tpu_pwm_config(pwm->chip, pwm,`
	`419`	`+ err = tpu_pwm_config(chip, pwm,`
`420`	`420`	`state->duty_cycle, state->period, enabled);`
`421`	`421`	`if (err)`
`422`	`422`	`return err;`
Original file line number	Diff line number	Diff line change
`@@ -407,7 +407,7 @@ static int sti_pwm_apply(struct pwm_chip chip, struct pwm_device pwm,`
`407`	`407`	`return 0;`
`408`	`408`	`}`
`409`	`409`
`410`		`- err = sti_pwm_config(pwm->chip, pwm, state->duty_cycle, state->period);`
	`410`	`+ err = sti_pwm_config(chip, pwm, state->duty_cycle, state->period);`
`411`	`411`	`if (err)`
`412`	`412`	`return err;`
`413`	`413`
Original file line number	Diff line number	Diff line change
`@@ -275,7 +275,7 @@ static int stmpe_24xx_pwm_apply(struct pwm_chip chip, struct pwm_device pwm,`
`275`	`275`	`return 0;`
`276`	`276`	`}`
`277`	`277`
`278`		`- err = stmpe_24xx_pwm_config(pwm->chip, pwm, state->duty_cycle, state->period);`
	`278`	`+ err = stmpe_24xx_pwm_config(chip, pwm, state->duty_cycle, state->period);`
`279`	`279`	`if (err)`
`280`	`280`	`return err;`
`281`	`281`
Original file line number	Diff line number	Diff line change
`@@ -256,7 +256,7 @@ static int tegra_pwm_apply(struct pwm_chip chip, struct pwm_device pwm,`
`256`	`256`	`return 0;`
`257`	`257`	`}`
`258`	`258`
`259`		`- err = tegra_pwm_config(pwm->chip, pwm, state->duty_cycle, state->period);`
	`259`	`+ err = tegra_pwm_config(chip, pwm, state->duty_cycle, state->period);`
`260`	`260`	`if (err)`
`261`	`261`	`return err;`
`262`	`262`