On Thu, Apr 25, 2019 at 04:19:47PM +0200, Wolfram Sang wrote: > There are two problems with WARN_ON() here. One: It is not ratelimited. > Two: We don't see which adapter was used when trying to transfer > something when already suspended. Implement a custom ratelimit once per > adapter and use dev_WARN there. This fixes both issues. Drawback is that > we don't see if multiple drivers are trying to transfer with the same > adapter while suspended. They need to be discovered one after the other > now. This is better than a high CPU load because a really broken driver > might try to resend endlessly. > > Fixes: 9ac6cb5fbb17 ("i2c: add suspended flag and accessors for i2c adapters") > Signed-off-by: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> Reviewed-by: Simon Horman <horms+renesas@xxxxxxxxxxxx>