* Re: usb: host: Faraday fotg210-hcd driver [not found] <20130903185912.A59316601F4@gitolite.kernel.org> @ 2013-09-03 21:15 ` Dave Jones 2013-09-03 21:23 ` Greg KH 0 siblings, 1 reply; 6+ messages in thread From: Dave Jones @ 2013-09-03 21:15 UTC (permalink / raw) To: Linux Kernel Mailing List; +Cc: yhchen, gregkh On Tue, Sep 03, 2013 at 06:59:12PM +0000, Linux Kernel wrote: > Gitweb: http://git.kernel.org/linus/;a=commit;h=1dd3d123239179fad5de5dc00a6e0014a1918fde > Commit: 1dd3d123239179fad5de5dc00a6e0014a1918fde > Parent: 10e232c597ac757e7f8600649f7e872e86de190f > Author: Yuan-Hsin Chen <yhchen@faraday-tech.com> > AuthorDate: Wed Jun 19 19:53:04 2013 +0000 > Committer: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > CommitDate: Wed Jul 24 16:04:53 2013 -0700 > > usb: host: Faraday fotg210-hcd driver > > FOTG210 is an OTG controller which can be configured as an > USB2.0 host. FOTG210 host is an ehci-like controller with > some differences. First, register layout of FOTG210 is > incompatible with EHCI. Furthermore, FOTG210 is lack of > siTDs which means iTDs are used for both HS and FS ISO > transfer. > > Signed-off-by: Yuan-Hsin Chen <yhchen@faraday-tech.com> > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > +config USB_FOTG210_HCD > + tristate "FOTG210 HCD support" > + depends on USB > + default N > + ---help--- > + Faraday FOTG210 is an OTG controller which can be configured as > + an USB2.0 host. It is designed to meet USB2.0 EHCI specification > + with minor modification. > + > + To compile this driver as a module, choose M here: the > + module will be called fotg210-hcd. > + Shouldn't this depend on USB_OTG ? Seems odd to have OTG related options presented to me when I have that unset. Dave ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: usb: host: Faraday fotg210-hcd driver 2013-09-03 21:15 ` usb: host: Faraday fotg210-hcd driver Dave Jones @ 2013-09-03 21:23 ` Greg KH 2013-09-03 21:32 ` Dave Jones 0 siblings, 1 reply; 6+ messages in thread From: Greg KH @ 2013-09-03 21:23 UTC (permalink / raw) To: Dave Jones, Linux Kernel Mailing List, yhchen On Tue, Sep 03, 2013 at 05:15:44PM -0400, Dave Jones wrote: > On Tue, Sep 03, 2013 at 06:59:12PM +0000, Linux Kernel wrote: > > Gitweb: http://git.kernel.org/linus/;a=commit;h=1dd3d123239179fad5de5dc00a6e0014a1918fde > > Commit: 1dd3d123239179fad5de5dc00a6e0014a1918fde > > Parent: 10e232c597ac757e7f8600649f7e872e86de190f > > Author: Yuan-Hsin Chen <yhchen@faraday-tech.com> > > AuthorDate: Wed Jun 19 19:53:04 2013 +0000 > > Committer: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > CommitDate: Wed Jul 24 16:04:53 2013 -0700 > > > > usb: host: Faraday fotg210-hcd driver > > > > FOTG210 is an OTG controller which can be configured as an > > USB2.0 host. FOTG210 host is an ehci-like controller with > > some differences. First, register layout of FOTG210 is > > incompatible with EHCI. Furthermore, FOTG210 is lack of > > siTDs which means iTDs are used for both HS and FS ISO > > transfer. > > > > Signed-off-by: Yuan-Hsin Chen <yhchen@faraday-tech.com> > > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > > +config USB_FOTG210_HCD > > + tristate "FOTG210 HCD support" > > + depends on USB > > + default N > > + ---help--- > > + Faraday FOTG210 is an OTG controller which can be configured as > > + an USB2.0 host. It is designed to meet USB2.0 EHCI specification > > + with minor modification. > > + > > + To compile this driver as a module, choose M here: the > > + module will be called fotg210-hcd. > > + > > Shouldn't this depend on USB_OTG ? > Seems odd to have OTG related options presented to me when I have that unset. It's a USB Host controller, so it should be built if you have that option set. The fact that it can do OTG isn't an issue here, from what I can tell. thanks, greg k-h ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: usb: host: Faraday fotg210-hcd driver 2013-09-03 21:23 ` Greg KH @ 2013-09-03 21:32 ` Dave Jones 2013-09-03 21:40 ` Greg KH 0 siblings, 1 reply; 6+ messages in thread From: Dave Jones @ 2013-09-03 21:32 UTC (permalink / raw) To: Greg KH; +Cc: Linux Kernel Mailing List, john453 On Tue, Sep 03, 2013 at 02:23:55PM -0700, Greg KH wrote: > > > +config USB_FOTG210_HCD > > > + tristate "FOTG210 HCD support" > > > + depends on USB > > > + default N > > > + ---help--- > > > + Faraday FOTG210 is an OTG controller which can be configured as > > > + an USB2.0 host. It is designed to meet USB2.0 EHCI specification > > > + with minor modification. > > > + > > > + To compile this driver as a module, choose M here: the > > > + module will be called fotg210-hcd. > > > + > > > > Shouldn't this depend on USB_OTG ? > > Seems odd to have OTG related options presented to me when I have that unset. > > It's a USB Host controller, so it should be built if you have that > option set. The fact that it can do OTG isn't an issue here, from what > I can tell. Ok. It still likely needs a 'depends on $ARCH (arm?) | COMPILE_TEST' though. Dave PS: What happened with this driver ? This patch landed twice in your tree, once in 1dd3d123239179fad5de5dc00a6e0014a1918fde from Yuan-Hsin Chen <yhchen@faraday-tech.com> and then in 7d50195f6c5005d6ae7a789d9a7f0a94d104ee96 from Feng-Hsin Chiang <john453@faraday-tech.com> Given the former address seems to be broken, perhaps all those mentions of it in the faraday sources need updating ? ----- The following addresses had permanent fatal errors ----- <yhchen@faraday-tech.com> (reason: 553 5.1.3 <yhchen@faraday-tech.com>... Check FTC account failure!<2>) ----- Transcript of session follows ----- ... while talking to mailhost1.faraday-tech.com.: >>> DATA <<< 553 5.1.3 <yhchen@faraday-tech.com>... Check FTC account failure!<2> 550 5.1.1 <yhchen@faraday-tech.com>... User unknown <<< 503 5.0.0 Need RCPT (recipient) ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: usb: host: Faraday fotg210-hcd driver 2013-09-03 21:32 ` Dave Jones @ 2013-09-03 21:40 ` Greg KH 2013-09-03 21:46 ` Dave Jones 0 siblings, 1 reply; 6+ messages in thread From: Greg KH @ 2013-09-03 21:40 UTC (permalink / raw) To: Dave Jones, Linux Kernel Mailing List, john453 On Tue, Sep 03, 2013 at 05:32:05PM -0400, Dave Jones wrote: > On Tue, Sep 03, 2013 at 02:23:55PM -0700, Greg KH wrote: > > > > > +config USB_FOTG210_HCD > > > > + tristate "FOTG210 HCD support" > > > > + depends on USB > > > > + default N > > > > + ---help--- > > > > + Faraday FOTG210 is an OTG controller which can be configured as > > > > + an USB2.0 host. It is designed to meet USB2.0 EHCI specification > > > > + with minor modification. > > > > + > > > > + To compile this driver as a module, choose M here: the > > > > + module will be called fotg210-hcd. > > > > + > > > > > > Shouldn't this depend on USB_OTG ? > > > Seems odd to have OTG related options presented to me when I have that unset. > > > > It's a USB Host controller, so it should be built if you have that > > option set. The fact that it can do OTG isn't an issue here, from what > > I can tell. > > Ok. It still likely needs a 'depends on $ARCH (arm?) | COMPILE_TEST' though. Probably, yes. > PS: What happened with this driver ? > > This patch landed twice in your tree, once in 1dd3d123239179fad5de5dc00a6e0014a1918fde from Yuan-Hsin Chen <yhchen@faraday-tech.com> > and then in 7d50195f6c5005d6ae7a789d9a7f0a94d104ee96 from Feng-Hsin Chiang <john453@faraday-tech.com> > > Given the former address seems to be broken, perhaps all those mentions of it > in the faraday sources need updating ? I thought they had all been fixed up. The first version was dropped because the maintainer no longer worked for the company (as you found out), so I needed a new developer to step up and support it. thanks, greg k-h ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: usb: host: Faraday fotg210-hcd driver 2013-09-03 21:40 ` Greg KH @ 2013-09-03 21:46 ` Dave Jones 2013-09-03 22:02 ` Greg KH 0 siblings, 1 reply; 6+ messages in thread From: Dave Jones @ 2013-09-03 21:46 UTC (permalink / raw) To: Greg KH; +Cc: Linux Kernel Mailing List, john453 On Tue, Sep 03, 2013 at 02:40:09PM -0700, Greg KH wrote: > On Tue, Sep 03, 2013 at 05:32:05PM -0400, Dave Jones wrote: > > PS: What happened with this driver ? > > > > This patch landed twice in your tree, once in 1dd3d123239179fad5de5dc00a6e0014a1918fde from Yuan-Hsin Chen <yhchen@faraday-tech.com> > > and then in 7d50195f6c5005d6ae7a789d9a7f0a94d104ee96 from Feng-Hsin Chiang <john453@faraday-tech.com> > > > > Given the former address seems to be broken, perhaps all those mentions of it > > in the faraday sources need updating ? > > I thought they had all been fixed up. The first version was dropped > because the maintainer no longer worked for the company (as you found > out), so I needed a new developer to step up and support it. He's still mentioned in a bunch of MODULE_AUTHOR's. While it seems sort of rude to delete mention of someones work, having a dead email address seems bad form too. Perhaps the patch below ? Dave Update the MODULE_AUTHOR field of the Faraday OTG drivers to reflect current maintainers email address. Signed-off-by: Dave Jones <davej@fedoraproject.org> diff --git a/drivers/usb/gadget/fotg210-udc.c b/drivers/usb/gadget/fotg210-udc.c index 32db2ee..bbbfd19 100644 --- a/drivers/usb/gadget/fotg210-udc.c +++ b/drivers/usb/gadget/fotg210-udc.c @@ -1214,6 +1214,6 @@ static struct platform_driver fotg210_driver = { module_platform_driver(fotg210_driver); -MODULE_AUTHOR("Yuan-Hsin Chen <yhchen@faraday-tech.com>"); +MODULE_AUTHOR("Yuan-Hsin Chen, Feng-Hsin Chiang <john453@faraday-tech.com>"); MODULE_LICENSE("GPL"); MODULE_DESCRIPTION(DRIVER_DESC); diff --git a/drivers/usb/gadget/fusb300_udc.c b/drivers/usb/gadget/fusb300_udc.c index f1dd6da..b278abe 100644 --- a/drivers/usb/gadget/fusb300_udc.c +++ b/drivers/usb/gadget/fusb300_udc.c @@ -22,7 +22,7 @@ MODULE_DESCRIPTION("FUSB300 USB gadget driver"); MODULE_LICENSE("GPL"); -MODULE_AUTHOR("Yuan Hsin Chen <yhchen@faraday-tech.com>"); +MODULE_AUTHOR("Yuan-Hsin Chen, Feng-Hsin Chiang <john453@faraday-tech.com>"); MODULE_ALIAS("platform:fusb300_udc"); #define DRIVER_VERSION "20 October 2010" ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: usb: host: Faraday fotg210-hcd driver 2013-09-03 21:46 ` Dave Jones @ 2013-09-03 22:02 ` Greg KH 0 siblings, 0 replies; 6+ messages in thread From: Greg KH @ 2013-09-03 22:02 UTC (permalink / raw) To: Dave Jones, Linux Kernel Mailing List, john453 On Tue, Sep 03, 2013 at 05:46:26PM -0400, Dave Jones wrote: > On Tue, Sep 03, 2013 at 02:40:09PM -0700, Greg KH wrote: > > On Tue, Sep 03, 2013 at 05:32:05PM -0400, Dave Jones wrote: > > > > PS: What happened with this driver ? > > > > > > This patch landed twice in your tree, once in 1dd3d123239179fad5de5dc00a6e0014a1918fde from Yuan-Hsin Chen <yhchen@faraday-tech.com> > > > and then in 7d50195f6c5005d6ae7a789d9a7f0a94d104ee96 from Feng-Hsin Chiang <john453@faraday-tech.com> > > > > > > Given the former address seems to be broken, perhaps all those mentions of it > > > in the faraday sources need updating ? > > > > I thought they had all been fixed up. The first version was dropped > > because the maintainer no longer worked for the company (as you found > > out), so I needed a new developer to step up and support it. > > He's still mentioned in a bunch of MODULE_AUTHOR's. > While it seems sort of rude to delete mention of someones work, having a dead email > address seems bad form too. Perhaps the patch below ? That looks good, I'll queue it up, thanks. greg k-h ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-09-03 22:02 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20130903185912.A59316601F4@gitolite.kernel.org>
2013-09-03 21:15 ` usb: host: Faraday fotg210-hcd driver Dave Jones
2013-09-03 21:23 ` Greg KH
2013-09-03 21:32 ` Dave Jones
2013-09-03 21:40 ` Greg KH
2013-09-03 21:46 ` Dave Jones
2013-09-03 22:02 ` Greg KH
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox