>> +free_irq: >> + free_irq(client->irq, info); >> +free_buf: >> + kfree(info->buf); >> +free_info: >> + kfree(info); >> + >> + dev_info(&client->dev, "Not probed."); >> + return ret; >> +} > >When respinning, you could consider using managed devices >(devm_*); this error path could completely go then. Well, to be honest I did't know about that 'feature' :), but now I see it's pretty nice. Will start to use it. Thanks, Waldek -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html