diff mbox

[3/3] usb: host: delete chipidea dependency

Message ID 1375432662-32018-4-git-send-email-peter.chen@freescale.com (mailing list archive)
State New, archived
Headers show

Commit Message

Peter Chen Aug. 2, 2013, 8:37 a.m. UTC 
Now, chipidea host has already depended on USB_EHCI_HCD

Signed-off-by: Peter Chen <peter.chen@freescale.com>
---
 drivers/usb/host/Kconfig |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

Comments

Alexander Shishkin Aug. 9, 2013, 10:51 a.m. UTC | #1 
Peter Chen <peter.chen@freescale.com> writes:

> Now, chipidea host has already depended on USB_EHCI_HCD
>
> Signed-off-by: Peter Chen <peter.chen@freescale.com>

Alan, what do you think?

> ---
>  drivers/usb/host/Kconfig |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/usb/host/Kconfig b/drivers/usb/host/Kconfig
> index cf521d6..b548554 100644
> --- a/drivers/usb/host/Kconfig
> +++ b/drivers/usb/host/Kconfig
> @@ -63,7 +63,7 @@ config USB_EHCI_HCD
>  
>  config USB_EHCI_ROOT_HUB_TT
>  	bool "Root Hub Transaction Translators"
> -	depends on USB_EHCI_HCD || USB_CHIPIDEA_HOST
> +	depends on USB_EHCI_HCD
>  	---help---
>  	  Some EHCI chips have vendor-specific extensions to integrate
>  	  transaction translators, so that no OHCI or UHCI companion
> @@ -75,7 +75,7 @@ config USB_EHCI_ROOT_HUB_TT
>  
>  config USB_EHCI_TT_NEWSCHED
>  	bool "Improved Transaction Translator scheduling"
> -	depends on USB_EHCI_HCD || USB_CHIPIDEA_HOST
> +	depends on USB_EHCI_HCD
>  	default y
>  	---help---
>  	  This changes the periodic scheduling code to fill more of the low
> -- 
> 1.7.0.4
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-usb" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
Alan Stern Aug. 9, 2013, 2:34 p.m. UTC | #2 
On Fri, 9 Aug 2013, Alexander Shishkin wrote:

> Peter Chen <peter.chen@freescale.com> writes:
> 
> > Now, chipidea host has already depended on USB_EHCI_HCD
> >
> > Signed-off-by: Peter Chen <peter.chen@freescale.com>
> 
> Alan, what do you think?
> 
> > ---
> >  drivers/usb/host/Kconfig |    4 ++--
> >  1 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/usb/host/Kconfig b/drivers/usb/host/Kconfig
> > index cf521d6..b548554 100644
> > --- a/drivers/usb/host/Kconfig
> > +++ b/drivers/usb/host/Kconfig
> > @@ -63,7 +63,7 @@ config USB_EHCI_HCD
> >  
> >  config USB_EHCI_ROOT_HUB_TT
> >  	bool "Root Hub Transaction Translators"
> > -	depends on USB_EHCI_HCD || USB_CHIPIDEA_HOST
> > +	depends on USB_EHCI_HCD
> >  	---help---
> >  	  Some EHCI chips have vendor-specific extensions to integrate
> >  	  transaction translators, so that no OHCI or UHCI companion
> > @@ -75,7 +75,7 @@ config USB_EHCI_ROOT_HUB_TT
> >  
> >  config USB_EHCI_TT_NEWSCHED
> >  	bool "Improved Transaction Translator scheduling"
> > -	depends on USB_EHCI_HCD || USB_CHIPIDEA_HOST
> > +	depends on USB_EHCI_HCD
> >  	default y
> >  	---help---
> >  	  This changes the periodic scheduling code to fill more of the low
> > -- 
> > 1.7.0.4

I approve.  Getting rid of dependencies is good.

Alan Stern
Alexander Shishkin Aug. 9, 2013, 2:42 p.m. UTC | #3 
Alan Stern <stern@rowland.harvard.edu> writes:

> On Fri, 9 Aug 2013, Alexander Shishkin wrote:
>
>> Peter Chen <peter.chen@freescale.com> writes:
>> 
>> > Now, chipidea host has already depended on USB_EHCI_HCD
>> >
>> > Signed-off-by: Peter Chen <peter.chen@freescale.com>
>> 
>> Alan, what do you think?
>> 
>> > ---
>> >  drivers/usb/host/Kconfig |    4 ++--
>> >  1 files changed, 2 insertions(+), 2 deletions(-)
>> >
>> > diff --git a/drivers/usb/host/Kconfig b/drivers/usb/host/Kconfig
>> > index cf521d6..b548554 100644
>> > --- a/drivers/usb/host/Kconfig
>> > +++ b/drivers/usb/host/Kconfig
>> > @@ -63,7 +63,7 @@ config USB_EHCI_HCD
>> >  
>> >  config USB_EHCI_ROOT_HUB_TT
>> >  	bool "Root Hub Transaction Translators"
>> > -	depends on USB_EHCI_HCD || USB_CHIPIDEA_HOST
>> > +	depends on USB_EHCI_HCD
>> >  	---help---
>> >  	  Some EHCI chips have vendor-specific extensions to integrate
>> >  	  transaction translators, so that no OHCI or UHCI companion
>> > @@ -75,7 +75,7 @@ config USB_EHCI_ROOT_HUB_TT
>> >  
>> >  config USB_EHCI_TT_NEWSCHED
>> >  	bool "Improved Transaction Translator scheduling"
>> > -	depends on USB_EHCI_HCD || USB_CHIPIDEA_HOST
>> > +	depends on USB_EHCI_HCD
>> >  	default y
>> >  	---help---
>> >  	  This changes the periodic scheduling code to fill more of the low
>> > -- 
>> > 1.7.0.4
>
> I approve.  Getting rid of dependencies is good.

Can I have you acked-by and take this patch to my tree?

Thanks,
--
Alex
Alan Stern Aug. 9, 2013, 2:52 p.m. UTC | #4 
On Fri, 9 Aug 2013, Alexander Shishkin wrote:

> Alan Stern <stern@rowland.harvard.edu> writes:
> 
> > On Fri, 9 Aug 2013, Alexander Shishkin wrote:
> >
> >> Peter Chen <peter.chen@freescale.com> writes:
> >> 
> >> > Now, chipidea host has already depended on USB_EHCI_HCD
> >> >
> >> > Signed-off-by: Peter Chen <peter.chen@freescale.com>
> >> 
> >> Alan, what do you think?
> >> 
> >> > ---
> >> >  drivers/usb/host/Kconfig |    4 ++--
> >> >  1 files changed, 2 insertions(+), 2 deletions(-)
> >> >
> >> > diff --git a/drivers/usb/host/Kconfig b/drivers/usb/host/Kconfig
> >> > index cf521d6..b548554 100644
> >> > --- a/drivers/usb/host/Kconfig
> >> > +++ b/drivers/usb/host/Kconfig
> >> > @@ -63,7 +63,7 @@ config USB_EHCI_HCD
> >> >  
> >> >  config USB_EHCI_ROOT_HUB_TT
> >> >  	bool "Root Hub Transaction Translators"
> >> > -	depends on USB_EHCI_HCD || USB_CHIPIDEA_HOST
> >> > +	depends on USB_EHCI_HCD
> >> >  	---help---
> >> >  	  Some EHCI chips have vendor-specific extensions to integrate
> >> >  	  transaction translators, so that no OHCI or UHCI companion
> >> > @@ -75,7 +75,7 @@ config USB_EHCI_ROOT_HUB_TT
> >> >  
> >> >  config USB_EHCI_TT_NEWSCHED
> >> >  	bool "Improved Transaction Translator scheduling"
> >> > -	depends on USB_EHCI_HCD || USB_CHIPIDEA_HOST
> >> > +	depends on USB_EHCI_HCD
> >> >  	default y
> >> >  	---help---
> >> >  	  This changes the periodic scheduling code to fill more of the low
> >> > -- 
> >> > 1.7.0.4
> >
> > I approve.  Getting rid of dependencies is good.
> 
> Can I have you acked-by and take this patch to my tree?

Yes.

Acked-by: Alan Stern <stern@rowland.harvard.edu>
diff mbox

Patch

diff --git a/drivers/usb/host/Kconfig b/drivers/usb/host/Kconfig
index cf521d6..b548554 100644
--- a/drivers/usb/host/Kconfig
+++ b/drivers/usb/host/Kconfig
@@ -63,7 +63,7 @@  config USB_EHCI_HCD
 
 config USB_EHCI_ROOT_HUB_TT
 	bool "Root Hub Transaction Translators"
-	depends on USB_EHCI_HCD || USB_CHIPIDEA_HOST
+	depends on USB_EHCI_HCD
 	---help---
 	  Some EHCI chips have vendor-specific extensions to integrate
 	  transaction translators, so that no OHCI or UHCI companion
@@ -75,7 +75,7 @@  config USB_EHCI_ROOT_HUB_TT
 
 config USB_EHCI_TT_NEWSCHED
 	bool "Improved Transaction Translator scheduling"
-	depends on USB_EHCI_HCD || USB_CHIPIDEA_HOST
+	depends on USB_EHCI_HCD
 	default y
 	---help---
 	  This changes the periodic scheduling code to fill more of the low