Re: similar files: fusbh200-hcd.c and fotg210-hcd.c

2015-09-25 Thread Peter Senna Tschudin
On Tue, Sep 15, 2015 at 6:50 PM, Felipe Balbi wrote: > Hi, > > On Tue, Sep 15, 2015 at 06:41:55PM +0200, Peter Senna Tschudin wrote: >> On Tue, Sep 15, 2015 at 4:33 PM, Felipe Balbi wrote: >> > On Mon, Sep 14, 2015 at 07:50:02PM +0200, Peter Senna Tschudin wrote: >> >> On Mon, Sep 14, 2015 at 5:0

Re: similar files: fusbh200-hcd.c and fotg210-hcd.c

2015-09-15 Thread Felipe Balbi
Hi, On Tue, Sep 15, 2015 at 06:41:55PM +0200, Peter Senna Tschudin wrote: > On Tue, Sep 15, 2015 at 4:33 PM, Felipe Balbi wrote: > > On Mon, Sep 14, 2015 at 07:50:02PM +0200, Peter Senna Tschudin wrote: > >> On Mon, Sep 14, 2015 at 5:01 PM, Felipe Balbi wrote: > >> > On Sat, Sep 12, 2015 at 03:1

Re: similar files: fusbh200-hcd.c and fotg210-hcd.c

2015-09-15 Thread Peter Senna Tschudin
On Tue, Sep 15, 2015 at 4:33 PM, Felipe Balbi wrote: > On Mon, Sep 14, 2015 at 07:50:02PM +0200, Peter Senna Tschudin wrote: >> On Mon, Sep 14, 2015 at 5:01 PM, Felipe Balbi wrote: >> > On Sat, Sep 12, 2015 at 03:14:50PM +0200, Peter Senna Tschudin wrote: >> >> >> Should these files be consolidat

Re: similar files: fusbh200-hcd.c and fotg210-hcd.c

2015-09-15 Thread Felipe Balbi
On Mon, Sep 14, 2015 at 07:50:02PM +0200, Peter Senna Tschudin wrote: > On Mon, Sep 14, 2015 at 5:01 PM, Felipe Balbi wrote: > > On Sat, Sep 12, 2015 at 03:14:50PM +0200, Peter Senna Tschudin wrote: > >> >> Should these files be consolidated? And if so how? > >> > if you can find an easy way, that

Re: similar files: fusbh200-hcd.c and fotg210-hcd.c

2015-09-14 Thread Peter Senna Tschudin
On Mon, Sep 14, 2015 at 5:01 PM, Felipe Balbi wrote: > On Sat, Sep 12, 2015 at 03:14:50PM +0200, Peter Senna Tschudin wrote: >> >> Should these files be consolidated? And if so how? >> > if you can find an easy way, that would be a very, very welcome patch. >> >> Is the ideal solution to consolida

Re: similar files: fusbh200-hcd.c and fotg210-hcd.c

2015-09-14 Thread Felipe Balbi
On Sat, Sep 12, 2015 at 03:14:50PM +0200, Peter Senna Tschudin wrote: > >> Should these files be consolidated? And if so how? > > if you can find an easy way, that would be a very, very welcome patch. > > Is the ideal solution to consolidate both fusbh200-hcd.c and > fotg210-hcd.c in a single modu

Re: similar files: fusbh200-hcd.c and fotg210-hcd.c

2015-09-12 Thread Peter Senna Tschudin
>> Should these files be consolidated? And if so how? > if you can find an easy way, that would be a very, very welcome patch. Is the ideal solution to consolidate both fusbh200-hcd.c and fotg210-hcd.c in a single module? If this is the case, how to detect at run time which version of the hw is pr

Re: similar files: fusbh200-hcd.c and fotg210-hcd.c

2015-09-08 Thread Felipe Balbi
On Mon, Sep 07, 2015 at 04:47:45PM +0200, Peter Senna Tschudin wrote: > I executed a clone detection tool* on drivers source code and I found > that the files > > drivers/usb/host/fusbh200-hcd.c > > and > > drivers/usb/host/fotg210-hcd.c > > are very similar. The main difference between the two

similar files: fusbh200-hcd.c and fotg210-hcd.c

2015-09-07 Thread Peter Senna Tschudin
I executed a clone detection tool* on drivers source code and I found that the files drivers/usb/host/fusbh200-hcd.c and drivers/usb/host/fotg210-hcd.c are very similar. The main difference between the two files are replacing the string 'USBH20' by 'OTG21' and some white space fixes. Some chang