On Thu, Jan 25, 2024 at 11:32:47AM -0800, Florian Fainelli wrote: > On 1/25/24 04:08, Uwe Kleine-König wrote: > > Currently a pwm_chip stores in its struct device *dev member a pointer > > to the parent device. Preparing a change that embeds a full struct > > device in struct pwm_chip, this accessor macro should be used in all > > drivers directly accessing chip->dev now. This way struct pwm_chip and > > this macro can be changed without having to touch all drivers in the > > same change set. > > > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> > > Nit: this is not a macro but an inline function. Oh right, it used to be a macro, but I changed that. I made the commit log read: pwm: Provide an inline function to get the parent device of a given chip Currently a pwm_chip stores in its struct device *dev member a pointer to the parent device. Preparing a change that embeds a full struct device in struct pwm_chip, this accessor function should be used in all drivers directly accessing chip->dev now. This way struct pwm_chip and this new function can be changed without having to touch all drivers in the same change set. Thanks, Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | https://www.pengutronix.de/ |
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ greybus-dev mailing list -- greybus-dev@xxxxxxxxxxxxxxxx To unsubscribe send an email to greybus-dev-leave@xxxxxxxxxxxxxxxx