| Message ID | 1470742059-12341-2-git-send-email-wsa-dev@sang-engineering.com |
|---|---|
| State | Awaiting Upstream |
| Headers | show |
On 09/08/16 14:27, Wolfram Sang wrote: > The core will do this for us now. > > Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com> > --- > drivers/video/fbdev/mb862xx/mb862xx-i2c.c | 7 +------ > 1 file changed, 1 insertion(+), 6 deletions(-) Thanks, queued for v4.9. One comment: the patch description should be "stand-alone" text, not a continuation of the subject. I fixed this. Tomi
> One comment: the patch description should be "stand-alone" text, not a > continuation of the subject. I fixed this. I see. Will keep that in mind for the future. Thanks!
diff --git a/drivers/video/fbdev/mb862xx/mb862xx-i2c.c b/drivers/video/fbdev/mb862xx/mb862xx-i2c.c index c87e17afb3e2c9..76d206857e3e2c 100644 --- a/drivers/video/fbdev/mb862xx/mb862xx-i2c.c +++ b/drivers/video/fbdev/mb862xx/mb862xx-i2c.c @@ -162,12 +162,7 @@ int mb862xx_i2c_init(struct mb862xxfb_par *par) mb862xx_i2c_adapter.algo_data = par; par->adap = &mb862xx_i2c_adapter; - ret = i2c_add_adapter(par->adap); - if (ret < 0) { - dev_err(par->dev, "failed to add %s\n", - mb862xx_i2c_adapter.name); - } - return ret; + return i2c_add_adapter(par->adap); } void mb862xx_i2c_exit(struct mb862xxfb_par *par)
The core will do this for us now. Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com> --- drivers/video/fbdev/mb862xx/mb862xx-i2c.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-)