On Sun, Oct 28, 2018 at 11:30 PM Wolfram Sang <wsa@xxxxxxxxxxxxx> wrote: > > On Fri, Oct 19, 2018 at 09:59:57AM +0100, Charles Keepax wrote: > > irq_create_mapping calls irq_find_mapping internally and will use the > > found mapping if one exists, so there is no need to manually call this > > from i2c_smbus_host_notify_to_irq. > > > > Signed-off-by: Charles Keepax <ckeepax@xxxxxxxxxxxxxxxxxxxxx> > > Adding Benjamin to recipients. He wrote that code. Yep, definitely looking good. For completeness, I just tested it on the t450s, and it works fine. This one is Reviewed-by: Benjamin Tissoires <benjamin.tissoires@xxxxxxxxxx> Cheers, Benjamin > > > --- > > drivers/i2c/i2c-core-base.c | 5 +---- > > 1 file changed, 1 insertion(+), 4 deletions(-) > > > > diff --git a/drivers/i2c/i2c-core-base.c b/drivers/i2c/i2c-core-base.c > > index dc78aa7369def..656f0a6fe3adf 100644 > > --- a/drivers/i2c/i2c-core-base.c > > +++ b/drivers/i2c/i2c-core-base.c > > @@ -306,10 +306,7 @@ static int i2c_smbus_host_notify_to_irq(const struct i2c_client *client) > > if (client->flags & I2C_CLIENT_TEN) > > return -EINVAL; > > > > - irq = irq_find_mapping(adap->host_notify_domain, client->addr); > > - if (!irq) > > - irq = irq_create_mapping(adap->host_notify_domain, > > - client->addr); > > + irq = irq_create_mapping(adap->host_notify_domain, client->addr); > > > > return irq > 0 ? irq : -ENXIO; > > } > > -- > > 2.11.0 > >
