On Thu, May 27, 2021 at 10:26:33PM +0200, Wolfram Sang wrote: > On Wed, May 26, 2021 at 03:43:17PM +0300, Andy Shevchenko wrote: > > There is at least one user that will gain from the > > i2c_acpi_find_client_by_adev() being exported. > > No objections per se. But as the user is in staging, I want to ask if > the use there is really a solution we would also accept outside of > staging? Or is it a hack? The similar OF API is exported for users, although amount of users and their locations are different. The AtomISP driver is not in the best shape, I agree, but for now any possible steps to make it better would be good steps in my opinion. Later we may see if we can do this piece of code differently (IIRC current way is probably the best taking into account legacy platforms support). -- With Best Regards, Andy Shevchenko