diff mbox

ARM: omap2plus_defconfig: Enable OHCI & EHCI HCD support

Message ID 1421701046-17367-1-git-send-email-sjoerd.simons@collabora.co.uk (mailing list archive)
State New, archived
Headers show

Commit Message

Sjoerd Simons Jan. 19, 2015, 8:57 p.m. UTC
Enable CONFIG_USB_EHCI_HCD and CONFIG_USB_OHCI_HCD to get USB supports
with omap2plus_defconfig.

Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
---
 arch/arm/configs/omap2plus_defconfig | 2 ++
 1 file changed, 2 insertions(+)

Comments

Felipe Balbi Jan. 19, 2015, 9:03 p.m. UTC | #1
On Mon, Jan 19, 2015 at 09:57:26PM +0100, Sjoerd Simons wrote:
> Enable CONFIG_USB_EHCI_HCD and CONFIG_USB_OHCI_HCD to get USB supports
> with omap2plus_defconfig.
> 
> Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
> ---
>  arch/arm/configs/omap2plus_defconfig | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig
> index c2c3a85..295af70 100644
> --- a/arch/arm/configs/omap2plus_defconfig
> +++ b/arch/arm/configs/omap2plus_defconfig
> @@ -159,6 +159,8 @@ CONFIG_USB_NET_SMSC95XX=y
>  CONFIG_USB_ALI_M5632=y
>  CONFIG_USB_AN2720=y
>  CONFIG_USB_EPSON2888=y
> +CONFIG_USB_EHCI_HCD=y
> +CONFIG_USB_OHCI_HCD=y

should be modules as they break PM.
Felipe Balbi Jan. 19, 2015, 9:08 p.m. UTC | #2
On Mon, Jan 19, 2015 at 03:03:41PM -0600, Felipe Balbi wrote:
> On Mon, Jan 19, 2015 at 09:57:26PM +0100, Sjoerd Simons wrote:
> > Enable CONFIG_USB_EHCI_HCD and CONFIG_USB_OHCI_HCD to get USB supports
> > with omap2plus_defconfig.
> > 
> > Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
> > ---
> >  arch/arm/configs/omap2plus_defconfig | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig
> > index c2c3a85..295af70 100644
> > --- a/arch/arm/configs/omap2plus_defconfig
> > +++ b/arch/arm/configs/omap2plus_defconfig
> > @@ -159,6 +159,8 @@ CONFIG_USB_NET_SMSC95XX=y
> >  CONFIG_USB_ALI_M5632=y
> >  CONFIG_USB_AN2720=y
> >  CONFIG_USB_EPSON2888=y
> > +CONFIG_USB_EHCI_HCD=y
> > +CONFIG_USB_OHCI_HCD=y
> 
> should be modules as they break PM.

oh, and when resending, please add linux-omap to Cc, that's listed in
MAINTAINERS.
Sjoerd Simons Jan. 19, 2015, 9:21 p.m. UTC | #3
On Mon, 2015-01-19 at 15:08 -0600, Felipe Balbi wrote:
> On Mon, Jan 19, 2015 at 03:03:41PM -0600, Felipe Balbi wrote:
> > On Mon, Jan 19, 2015 at 09:57:26PM +0100, Sjoerd Simons wrote:
> > > Enable CONFIG_USB_EHCI_HCD and CONFIG_USB_OHCI_HCD to get USB supports
> > > with omap2plus_defconfig.
> > > 
> > > Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
> > > ---
> > >  arch/arm/configs/omap2plus_defconfig | 2 ++
> > >  1 file changed, 2 insertions(+)
> > > 
> > > diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig
> > > index c2c3a85..295af70 100644
> > > --- a/arch/arm/configs/omap2plus_defconfig
> > > +++ b/arch/arm/configs/omap2plus_defconfig
> > > @@ -159,6 +159,8 @@ CONFIG_USB_NET_SMSC95XX=y
> > >  CONFIG_USB_ALI_M5632=y
> > >  CONFIG_USB_AN2720=y
> > >  CONFIG_USB_EPSON2888=y
> > > +CONFIG_USB_EHCI_HCD=y
> > > +CONFIG_USB_OHCI_HCD=y
> > 
> > should be modules as they break PM.
> 
> oh, and when resending, please add linux-omap to Cc, that's listed in
> MAINTAINERS.

Hrm,

Seems like MAINTAINERS is missing an entry to make get_maintainer.pl
recognize linux-omap should be cc'd:

$ ./scripts/get_maintainer.pl -f arch/arm/configs/omap2plus_defconfig
Russell King <linux@arm.linux.org.uk> (maintainer:ARM PORT)
Tony Lindgren <tony@atomide.com>
(commit_signer:23/24=96%,authored:10/24=42%,added_lines:74/107=69%,removed_lines:21/24=88%)
Felipe Balbi <balbi@ti.com>
(commit_signer:7/24=29%,authored:5/24=21%,added_lines:22/107=21%)
Arnd Bergmann <arnd@arndb.de> (commit_signer:6/24=25%)
Nishanth Menon <nm@ti.com> (commit_signer:2/24=8%)
linux-arm-kernel@lists.infradead.org (moderated list:ARM PORT)
linux-kernel@vger.kernel.org (open list)
Felipe Balbi Jan. 19, 2015, 9:41 p.m. UTC | #4
Hi,

On Mon, Jan 19, 2015 at 10:21:15PM +0100, Sjoerd Simons wrote:
> On Mon, 2015-01-19 at 15:08 -0600, Felipe Balbi wrote:
> > On Mon, Jan 19, 2015 at 03:03:41PM -0600, Felipe Balbi wrote:
> > > On Mon, Jan 19, 2015 at 09:57:26PM +0100, Sjoerd Simons wrote:
> > > > Enable CONFIG_USB_EHCI_HCD and CONFIG_USB_OHCI_HCD to get USB supports
> > > > with omap2plus_defconfig.
> > > > 
> > > > Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
> > > > ---
> > > >  arch/arm/configs/omap2plus_defconfig | 2 ++
> > > >  1 file changed, 2 insertions(+)
> > > > 
> > > > diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig
> > > > index c2c3a85..295af70 100644
> > > > --- a/arch/arm/configs/omap2plus_defconfig
> > > > +++ b/arch/arm/configs/omap2plus_defconfig
> > > > @@ -159,6 +159,8 @@ CONFIG_USB_NET_SMSC95XX=y
> > > >  CONFIG_USB_ALI_M5632=y
> > > >  CONFIG_USB_AN2720=y
> > > >  CONFIG_USB_EPSON2888=y
> > > > +CONFIG_USB_EHCI_HCD=y
> > > > +CONFIG_USB_OHCI_HCD=y
> > > 
> > > should be modules as they break PM.
> > 
> > oh, and when resending, please add linux-omap to Cc, that's listed in
> > MAINTAINERS.
> 
> Hrm,
> 
> Seems like MAINTAINERS is missing an entry to make get_maintainer.pl
> recognize linux-omap should be cc'd:
> 
> $ ./scripts/get_maintainer.pl -f arch/arm/configs/omap2plus_defconfig
> Russell King <linux@arm.linux.org.uk> (maintainer:ARM PORT)
> Tony Lindgren <tony@atomide.com>
> (commit_signer:23/24=96%,authored:10/24=42%,added_lines:74/107=69%,removed_lines:21/24=88%)
> Felipe Balbi <balbi@ti.com>
> (commit_signer:7/24=29%,authored:5/24=21%,added_lines:22/107=21%)
> Arnd Bergmann <arnd@arndb.de> (commit_signer:6/24=25%)
> Nishanth Menon <nm@ti.com> (commit_signer:2/24=8%)
> linux-arm-kernel@lists.infradead.org (moderated list:ARM PORT)
> linux-kernel@vger.kernel.org (open list)

I'll send a patch (unless Tony wants to do it), thanks for reporting.
Tony Lindgren Jan. 19, 2015, 9:50 p.m. UTC | #5
* Felipe Balbi <balbi@ti.com> [150119 13:45]:
> Hi,
> 
> On Mon, Jan 19, 2015 at 10:21:15PM +0100, Sjoerd Simons wrote:
> > On Mon, 2015-01-19 at 15:08 -0600, Felipe Balbi wrote:
> > > On Mon, Jan 19, 2015 at 03:03:41PM -0600, Felipe Balbi wrote:
> > > > On Mon, Jan 19, 2015 at 09:57:26PM +0100, Sjoerd Simons wrote:
> > > > > Enable CONFIG_USB_EHCI_HCD and CONFIG_USB_OHCI_HCD to get USB supports
> > > > > with omap2plus_defconfig.
> > > > > 
> > > > > Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
> > > > > ---
> > > > >  arch/arm/configs/omap2plus_defconfig | 2 ++
> > > > >  1 file changed, 2 insertions(+)
> > > > > 
> > > > > diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig
> > > > > index c2c3a85..295af70 100644
> > > > > --- a/arch/arm/configs/omap2plus_defconfig
> > > > > +++ b/arch/arm/configs/omap2plus_defconfig
> > > > > @@ -159,6 +159,8 @@ CONFIG_USB_NET_SMSC95XX=y
> > > > >  CONFIG_USB_ALI_M5632=y
> > > > >  CONFIG_USB_AN2720=y
> > > > >  CONFIG_USB_EPSON2888=y
> > > > > +CONFIG_USB_EHCI_HCD=y
> > > > > +CONFIG_USB_OHCI_HCD=y
> > > > 
> > > > should be modules as they break PM.
> > > 
> > > oh, and when resending, please add linux-omap to Cc, that's listed in
> > > MAINTAINERS.
> > 
> > Hrm,
> > 
> > Seems like MAINTAINERS is missing an entry to make get_maintainer.pl
> > recognize linux-omap should be cc'd:
> > 
> > $ ./scripts/get_maintainer.pl -f arch/arm/configs/omap2plus_defconfig
> > Russell King <linux@arm.linux.org.uk> (maintainer:ARM PORT)
> > Tony Lindgren <tony@atomide.com>
> > (commit_signer:23/24=96%,authored:10/24=42%,added_lines:74/107=69%,removed_lines:21/24=88%)
> > Felipe Balbi <balbi@ti.com>
> > (commit_signer:7/24=29%,authored:5/24=21%,added_lines:22/107=21%)
> > Arnd Bergmann <arnd@arndb.de> (commit_signer:6/24=25%)
> > Nishanth Menon <nm@ti.com> (commit_signer:2/24=8%)
> > linux-arm-kernel@lists.infradead.org (moderated list:ARM PORT)
> > linux-kernel@vger.kernel.org (open list)
> 
> I'll send a patch (unless Tony wants to do it), thanks for reporting.

Thanks for resending, I'll have a bit hard time tracking patches unless
linux-omap is in Cc.

Tony
diff mbox

Patch

diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig
index c2c3a85..295af70 100644
--- a/arch/arm/configs/omap2plus_defconfig
+++ b/arch/arm/configs/omap2plus_defconfig
@@ -159,6 +159,8 @@  CONFIG_USB_NET_SMSC95XX=y
 CONFIG_USB_ALI_M5632=y
 CONFIG_USB_AN2720=y
 CONFIG_USB_EPSON2888=y
+CONFIG_USB_EHCI_HCD=y
+CONFIG_USB_OHCI_HCD=y
 CONFIG_USB_KC2190=y
 CONFIG_LIBERTAS=m
 CONFIG_LIBERTAS_USB=m