On Fri, 22 Sept 2023 at 14:39, Jian Zhang <zhangjian.3032@xxxxxxxxxxxxx> wrote: > > > > Tommy has submitted a similar fix: > > > > https://lore.kernel.org/linux-i2c/20230906004910.4157305-1-tommy_huang@xxxxxxxxxxxxxx/ > > > > His change is very heavy handed; it reinitialises the bus including > > re-parsing the device tree (!). > > > > Should we have merged this fix instead? If not, are you able to > > confirm that his change fixes your issue? > > I feel it's for solving the same issue, but I think this patch is > missing the action > `bus->slave_state = ASPEED_I2C_SLAVE_INACTIVE;`, > which means it can't resolve my problem. @Tommy, can you help confirm this? You're right, it doesn't change the slave_state at all. Unfortunately, despite no acks from the maintainers, this patch has now been merged and backported to stable. We should complete the fix, or revert it asap.
