On Wed, Oct 16, 2024 at 2:00 AM Doug Anderson <dianders@xxxxxxxxxxxx> wrote: > > Hi, > > On Tue, Oct 8, 2024 at 12:35 AM Chen-Yu Tsai <wenst@xxxxxxxxxxxx> wrote: > > > > +/** > > + * i2c_of_probe_simple_cleanup - Clean up and release resources for I2C OF prober simple helpers > > + * @dev: Pointer to the &struct device of the caller, only used for dev_printk() messages > > + * @data: Pointer to &struct i2c_of_probe_simple_ctx helper context. > > + * > > + * * If a regulator supply was found, disable that regulator and release it. > > nit: was the double "*" starting the line above intentional? Yes. That's kernel-doc syntax. > Other than the nit, this looks good to me: > > Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx> Thanks!