On Fri, 05 Jun 2020, Michael Walle wrote:
> Am 2020-06-05 10:49, schrieb Lee Jones:
> [..]
> > > +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?
Copy/paste? Habit?
--
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
