вт, 9 черв. 2026 р. о 22:06 Andy Shevchenko <[email protected]> пише:
>
> On Sat, Jun 06, 2026 at 07:57:28AM +0300, Svyatoslav Ryhel wrote:
> > Instead of passing the entire lm3533 core data structure, only pass the
> > regmap and the light sensor presence flag to child devices.
>
> ...
>
> > struct lm3533_als {
> > - struct lm3533 *lm3533;
> > + struct regmap *regmap;
> > struct platform_device *pdev;
>
> And this pdev is probably not needed. But I haven't checked the whole lot of
> the patches yet.
>
It is needed since it holds childs pdev (dev would be better, but not
in this patchset), you cannot get childs dev from regmap since regmap
holds cores dev.
> > unsigned long flags;
>
> ...
>
> > struct lm3533_ctrlbank {
> > - struct lm3533 *lm3533;
> > + struct regmap *regmap;
> > struct device *dev;
>
> Ditto.
>
Same here, dev holds either LEDs dev or backlight dev.
> > int id;
> > };
>
> --
> With Best Regards,
> Andy Shevchenko
>
>
