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:

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.

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

thanks,

greg k-h