HI! During the discussion of some DSA rework [1] I just realized that names for root swnodes are not required and moreover should be unique in the system. Looking into the INT33FE driver I'm now asking myself why we are still using names there (except just convenience for us)? Note, theoretically it might be possible that one can somehow connect another i2c device to those Cherry Trail machines (let's say via USB-to-I2C adapter). And somehow try to register swnode from maybe out-of-tree driver, however it's not an issue to the official kernel. That said, maybe better just to remove them where we don't rely on their names? [1]: https://lore.kernel.org/all/YtWzWdkFVMg0Hyvf@xxxxxxxxxxxxxxxxxx/ -- With Best Regards, Andy Shevchenko