Am 2020-06-05 10:49, schrieb Lee Jones:
[..]

quoted

+static inline struct sl28cpld_pwm *to_sl28cpld_pwm(struct pwm_chip 
*chip)
+{
+	return container_of(chip, struct sl28cpld_pwm, pwm_chip);
+}

Why not save yourself the trouble and just:

  struct sl28cpld_pwm *pwm = dev_get_drvdata(chip->dev);

Is there a reason why not a single pwm driver uses something like that?

-michael