Date: Tue, 3 Sep 2013 14:40:09 -0700
From: Greg KH <gregkh@linuxfoundation.org>
To: Dave Jones <davej@redhat.com>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        john453@faraday-tech.com
Subject: Re: usb: host: Faraday fotg210-hcd driver
Message-ID: <20130903214009.GA16463@kroah.com>
References: <20130903185912.A59316601F4@gitolite.kernel.org>
 <20130903211544.GB2247@redhat.com>
 <20130903212355.GA12978@kroah.com>
 <20130903213205.GA7433@redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20130903213205.GA7433@redhat.com>
User-Agent: Mutt/1.5.21 (2010-09-15)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1885
Lines: 47

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/