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? Other than the nit, this looks good to me: Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx>