On Fri, May 28, 2021 at 11:25:51AM +0200, Greg Kroah-Hartman wrote:

On Thu, May 27, 2021 at 10:26:33PM +0200, Wolfram Sang wrote:

quoted

On Wed, May 26, 2021 at 03:43:17PM +0300, Andy Shevchenko wrote:

quoted

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?

staging drivers should be self-contained, do not accept code in the core
kernel that only is used by staging drivers.

So I would not recommend this be accepted at this point in time.

Fair enough.

Andy, work to get the driver out of staging first before doing stuff
like this.

Okay, I'll drop first one and patches related to it in the v2.
It should bring us closer to the mentioned point.

Thanks for clarification!

-- 
With Best Regards,
Andy Shevchenko