On Fri, Aug 06, 2021 at 12:38:36PM +0100, Lee Jones wrote: > On Fri, 06 Aug 2021, Alistair Francis wrote: > > + i2c_set_clientdata(i2c, regmap); > > + > No need to store this here. > Just do this in the child device: > dev_get_regmap(pdev->dev.parent, NULL); Note that dev_get_regmap() is relatively expensive compared to a simple driver data reference since it needs to go search for the device to find the regmap, it's not the end of the world especially in the context of doing I2C I/O but storing it isn't a terrible idea.
Attachment:
signature.asc
Description: PGP signature
