Re: [PATCH] i2c: introduce device_i2c_driver() macro

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Sam,

On 18-10-18 17:42, Sam Ravnborg wrote:
> Hi Marco
> 
> > diff --git a/drivers/video/mtl017.c b/drivers/video/mtl017.c
> > index 1a1f686223..085ea110ba 100644
> > --- a/drivers/video/mtl017.c
> > +++ b/drivers/video/mtl017.c
> > @@ -16,7 +16,6 @@
> >   *
> >   */
> >  #include <common.h>
> > -#include <init.h>
> >  #include <driver.h>
> >  #include <xfuncs.h>
> >  #include <errno.h>
> > @@ -270,11 +269,4 @@ static struct driver_d twl_driver = {
> >  	.name  = "mtl017",
> >  	.probe = mtl017_probe,
> >  };
> > -
> > -static int mtl017_init(void)
> > -{
> > -	i2c_driver_register(&twl_driver);
> > -	return 0;
> > -}
> > -
> > -device_initcall(mtl017_init);
> > +device_i2c_driver(twl_driver);
> 
> Not related to your patch, but it looks strange that
> the mtl017 file has a twl_driver.

You're right.

> It looks like a copy-paste issue from the past.

Yes it does and I copy-paste the issue too *argh*
We should fix this by a follow up patch.

Regards,
Marco

_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox



[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux