Hi Wolfram, On Mon, Apr 08, 2024 at 09:42:11AM +0200, Wolfram Sang wrote: > > Another simple solution would be to move the implementation of > > i2c_root_adapter() from i2c mux to i2c core. It just uses > > i2c_parent_is_i2c_adapter() which is an inline function of > > i2c core. What do you think? > > I have no objections putting i2c_root_adapter() into the core. I think, > however, that this patch makes the code a tad more readable. What is the > downside of the symbol (despite we have way too many of those in > general)? That's the only reason: "having too many of those in general". But i2c_root_adapter(), I believe should belong to the mux, even though it makes life easier moving it to the core. Heiner, I tested your patch and it works as expected. If no objections or suggestions, I will push it tomorrow. Andi
