Re: [PATCH] usb: remove CONFIG_USB_MUSB_HOST etc
On 10/23/2012 6:09 PM, Felipe Balbi wrote: > Hi, > > On Tue, Oct 23, 2012 at 06:04:53PM +0530, Sekhar Nori wrote: >> On 10/8/2012 6:47 PM, Constantine Shulyupin wrote: >>> From: Constantine Shulyupin >>> >>> Remove USB configuration in arch/arm/mach-davinci/usb.c accordingly >>> CONFIG_USB_MUSB_OTG CONFIG_USB_MUSB_PERIPHERAL CONFIG_USB_MUSB_HOST >>> and set MUSB_OTG configuration by default >>> because this configuration options are removed from Kconfig. >>> >>> Signed-off-by: Constantine Shulyupin >> >> Queuing this patch for v3.8. Since the config options are removed there >> is no use having code which refers to them. The patch has been tested on >> DM644x and DM365 in both host and gadget mode (I will add this >> information to commit text while committing). Without this patch .mode >> seems to be defaulting to MUSB_UNDEFINED which I think is definitely wrong. > > sorry for the delay, this looks ok: > > Acked-by: Felipe Balbi Thanks Felipe, I added your ack. Constantine, Patches touching arch/arm/* should be prefixed with 'ARM:' and those touching mach-davinci should be prefixed with 'davinci:'. I added these two while committing the patch this time. Please take care next time on. Thanks, Sekhar -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] usb: remove CONFIG_USB_MUSB_HOST etc
On 10/23/2012 6:09 PM, Felipe Balbi wrote: Hi, On Tue, Oct 23, 2012 at 06:04:53PM +0530, Sekhar Nori wrote: On 10/8/2012 6:47 PM, Constantine Shulyupin wrote: From: Constantine Shulyupin co...@makelinux.com Remove USB configuration in arch/arm/mach-davinci/usb.c accordingly CONFIG_USB_MUSB_OTG CONFIG_USB_MUSB_PERIPHERAL CONFIG_USB_MUSB_HOST and set MUSB_OTG configuration by default because this configuration options are removed from Kconfig. Signed-off-by: Constantine Shulyupin co...@makelinux.com Queuing this patch for v3.8. Since the config options are removed there is no use having code which refers to them. The patch has been tested on DM644x and DM365 in both host and gadget mode (I will add this information to commit text while committing). Without this patch .mode seems to be defaulting to MUSB_UNDEFINED which I think is definitely wrong. sorry for the delay, this looks ok: Acked-by: Felipe Balbi ba...@ti.com Thanks Felipe, I added your ack. Constantine, Patches touching arch/arm/* should be prefixed with 'ARM:' and those touching mach-davinci should be prefixed with 'davinci:'. I added these two while committing the patch this time. Please take care next time on. Thanks, Sekhar -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] usb: remove CONFIG_USB_MUSB_HOST etc
Hi, On Tue, Oct 23, 2012 at 06:04:53PM +0530, Sekhar Nori wrote: > On 10/8/2012 6:47 PM, Constantine Shulyupin wrote: > > From: Constantine Shulyupin > > > > Remove USB configuration in arch/arm/mach-davinci/usb.c accordingly > > CONFIG_USB_MUSB_OTG CONFIG_USB_MUSB_PERIPHERAL CONFIG_USB_MUSB_HOST > > and set MUSB_OTG configuration by default > > because this configuration options are removed from Kconfig. > > > > Signed-off-by: Constantine Shulyupin > > Queuing this patch for v3.8. Since the config options are removed there > is no use having code which refers to them. The patch has been tested on > DM644x and DM365 in both host and gadget mode (I will add this > information to commit text while committing). Without this patch .mode > seems to be defaulting to MUSB_UNDEFINED which I think is definitely wrong. sorry for the delay, this looks ok: Acked-by: Felipe Balbi > > Thanks, > Sekhar > > > > > --- > > arch/arm/mach-davinci/usb.c |6 -- > > 1 file changed, 6 deletions(-) > > > > diff --git a/arch/arm/mach-davinci/usb.c b/arch/arm/mach-davinci/usb.c > > index f77b953..34509ff 100644 > > --- a/arch/arm/mach-davinci/usb.c > > +++ b/arch/arm/mach-davinci/usb.c > > @@ -42,14 +42,8 @@ static struct musb_hdrc_config musb_config = { > > }; > > > > static struct musb_hdrc_platform_data usb_data = { > > -#if defined(CONFIG_USB_MUSB_OTG) > > /* OTG requires a Mini-AB connector */ > > .mode = MUSB_OTG, > > -#elif defined(CONFIG_USB_MUSB_PERIPHERAL) > > - .mode = MUSB_PERIPHERAL, > > -#elif defined(CONFIG_USB_MUSB_HOST) > > - .mode = MUSB_HOST, > > -#endif > > .clock = "usb", > > .config = _config, > > }; > > -- balbi signature.asc Description: Digital signature
Re: [PATCH] usb: remove CONFIG_USB_MUSB_HOST etc
On 10/8/2012 6:47 PM, Constantine Shulyupin wrote: > From: Constantine Shulyupin > > Remove USB configuration in arch/arm/mach-davinci/usb.c accordingly > CONFIG_USB_MUSB_OTG CONFIG_USB_MUSB_PERIPHERAL CONFIG_USB_MUSB_HOST > and set MUSB_OTG configuration by default > because this configuration options are removed from Kconfig. > > Signed-off-by: Constantine Shulyupin Queuing this patch for v3.8. Since the config options are removed there is no use having code which refers to them. The patch has been tested on DM644x and DM365 in both host and gadget mode (I will add this information to commit text while committing). Without this patch .mode seems to be defaulting to MUSB_UNDEFINED which I think is definitely wrong. Thanks, Sekhar > > --- > arch/arm/mach-davinci/usb.c |6 -- > 1 file changed, 6 deletions(-) > > diff --git a/arch/arm/mach-davinci/usb.c b/arch/arm/mach-davinci/usb.c > index f77b953..34509ff 100644 > --- a/arch/arm/mach-davinci/usb.c > +++ b/arch/arm/mach-davinci/usb.c > @@ -42,14 +42,8 @@ static struct musb_hdrc_config musb_config = { > }; > > static struct musb_hdrc_platform_data usb_data = { > -#if defined(CONFIG_USB_MUSB_OTG) > /* OTG requires a Mini-AB connector */ > .mode = MUSB_OTG, > -#elif defined(CONFIG_USB_MUSB_PERIPHERAL) > - .mode = MUSB_PERIPHERAL, > -#elif defined(CONFIG_USB_MUSB_HOST) > - .mode = MUSB_HOST, > -#endif > .clock = "usb", > .config = _config, > }; > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] usb: remove CONFIG_USB_MUSB_HOST etc
On 10/8/2012 6:47 PM, Constantine Shulyupin wrote: From: Constantine Shulyupin co...@makelinux.com Remove USB configuration in arch/arm/mach-davinci/usb.c accordingly CONFIG_USB_MUSB_OTG CONFIG_USB_MUSB_PERIPHERAL CONFIG_USB_MUSB_HOST and set MUSB_OTG configuration by default because this configuration options are removed from Kconfig. Signed-off-by: Constantine Shulyupin co...@makelinux.com Queuing this patch for v3.8. Since the config options are removed there is no use having code which refers to them. The patch has been tested on DM644x and DM365 in both host and gadget mode (I will add this information to commit text while committing). Without this patch .mode seems to be defaulting to MUSB_UNDEFINED which I think is definitely wrong. Thanks, Sekhar --- arch/arm/mach-davinci/usb.c |6 -- 1 file changed, 6 deletions(-) diff --git a/arch/arm/mach-davinci/usb.c b/arch/arm/mach-davinci/usb.c index f77b953..34509ff 100644 --- a/arch/arm/mach-davinci/usb.c +++ b/arch/arm/mach-davinci/usb.c @@ -42,14 +42,8 @@ static struct musb_hdrc_config musb_config = { }; static struct musb_hdrc_platform_data usb_data = { -#if defined(CONFIG_USB_MUSB_OTG) /* OTG requires a Mini-AB connector */ .mode = MUSB_OTG, -#elif defined(CONFIG_USB_MUSB_PERIPHERAL) - .mode = MUSB_PERIPHERAL, -#elif defined(CONFIG_USB_MUSB_HOST) - .mode = MUSB_HOST, -#endif .clock = usb, .config = musb_config, }; -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] usb: remove CONFIG_USB_MUSB_HOST etc
Hi, On Tue, Oct 23, 2012 at 06:04:53PM +0530, Sekhar Nori wrote: On 10/8/2012 6:47 PM, Constantine Shulyupin wrote: From: Constantine Shulyupin co...@makelinux.com Remove USB configuration in arch/arm/mach-davinci/usb.c accordingly CONFIG_USB_MUSB_OTG CONFIG_USB_MUSB_PERIPHERAL CONFIG_USB_MUSB_HOST and set MUSB_OTG configuration by default because this configuration options are removed from Kconfig. Signed-off-by: Constantine Shulyupin co...@makelinux.com Queuing this patch for v3.8. Since the config options are removed there is no use having code which refers to them. The patch has been tested on DM644x and DM365 in both host and gadget mode (I will add this information to commit text while committing). Without this patch .mode seems to be defaulting to MUSB_UNDEFINED which I think is definitely wrong. sorry for the delay, this looks ok: Acked-by: Felipe Balbi ba...@ti.com Thanks, Sekhar --- arch/arm/mach-davinci/usb.c |6 -- 1 file changed, 6 deletions(-) diff --git a/arch/arm/mach-davinci/usb.c b/arch/arm/mach-davinci/usb.c index f77b953..34509ff 100644 --- a/arch/arm/mach-davinci/usb.c +++ b/arch/arm/mach-davinci/usb.c @@ -42,14 +42,8 @@ static struct musb_hdrc_config musb_config = { }; static struct musb_hdrc_platform_data usb_data = { -#if defined(CONFIG_USB_MUSB_OTG) /* OTG requires a Mini-AB connector */ .mode = MUSB_OTG, -#elif defined(CONFIG_USB_MUSB_PERIPHERAL) - .mode = MUSB_PERIPHERAL, -#elif defined(CONFIG_USB_MUSB_HOST) - .mode = MUSB_HOST, -#endif .clock = usb, .config = musb_config, }; -- balbi signature.asc Description: Digital signature
Re: [PATCH] usb: remove CONFIG_USB_MUSB_HOST etc
I've tested gadget mode on DM365 without HW changes. On Thu, Oct 11, 2012 at 10:50 AM, Sekhar Nori wrote: > On 10/11/2012 12:05 PM, Heiko Schocher wrote: >> Hello Manjunathappa >> >> On 11.10.2012 07:42, Manjunathappa, Prakash wrote: >>> Hi, >>> On Mon, Oct 08, 2012 at 18:47:07, Constantine Shulyupin wrote: From: Constantine Shulyupin Remove USB configuration in arch/arm/mach-davinci/usb.c accordingly CONFIG_USB_MUSB_OTG CONFIG_USB_MUSB_PERIPHERAL CONFIG_USB_MUSB_HOST and set MUSB_OTG configuration by default because this configuration options are removed from Kconfig. Signed-off-by: Constantine Shulyupin --- arch/arm/mach-davinci/usb.c |6 -- 1 file changed, 6 deletions(-) diff --git a/arch/arm/mach-davinci/usb.c b/arch/arm/mach-davinci/usb.c index f77b953..34509ff 100644 --- a/arch/arm/mach-davinci/usb.c +++ b/arch/arm/mach-davinci/usb.c @@ -42,14 +42,8 @@ static struct musb_hdrc_config musb_config = { }; static struct musb_hdrc_platform_data usb_data = { -#if defined(CONFIG_USB_MUSB_OTG) /* OTG requires a Mini-AB connector */ .mode = MUSB_OTG, -#elif defined(CONFIG_USB_MUSB_PERIPHERAL) -.mode = MUSB_PERIPHERAL, -#elif defined(CONFIG_USB_MUSB_HOST) -.mode = MUSB_HOST, -#endif .clock= "usb", .config=_config, }; >>> >>> Tested it on DM6446-EVM for host mode with MSC thumb drive and gadget >>> mode with g-ether. It works. >>> >>> Acked-by: Manjunathappa, Prakash >> >> I sent a similiar patch here: >> >> http://comments.gmane.org/gmane.linux.usb.general/54512 >> >> If the issues, mentioned from Sergei for my patch, nullified I add my: > > The last outstanding issue from Sergei seems to be additional comments > describing why MUSB_OTG is OK to use. > > Prakash/Constantine, > > Did you have to make any hardware changes when testing host/gadget on > DM6446 EVM? Or change in kernel configuration? > > It appears that there is no way to choose any of the config option > affecting the mode setting. Right now it seems to be just defaulting to > MUSB_UNDEFINED. Setting it to MUSB_OTG would be better than that. > > Thanks, > Sekhar -- Constantine Shulyupin http://www.MakeLinux.com/ Embedded Linux Systems, Device Drivers, TI DaVinci -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] usb: remove CONFIG_USB_MUSB_HOST etc
On 10/11/2012 12:05 PM, Heiko Schocher wrote: > Hello Manjunathappa > > On 11.10.2012 07:42, Manjunathappa, Prakash wrote: >> Hi, >> On Mon, Oct 08, 2012 at 18:47:07, Constantine Shulyupin wrote: >>> From: Constantine Shulyupin >>> >>> Remove USB configuration in arch/arm/mach-davinci/usb.c accordingly >>> CONFIG_USB_MUSB_OTG CONFIG_USB_MUSB_PERIPHERAL CONFIG_USB_MUSB_HOST >>> and set MUSB_OTG configuration by default >>> because this configuration options are removed from Kconfig. >>> >>> Signed-off-by: Constantine Shulyupin >>> >>> --- >>> arch/arm/mach-davinci/usb.c |6 -- >>> 1 file changed, 6 deletions(-) >>> >>> diff --git a/arch/arm/mach-davinci/usb.c b/arch/arm/mach-davinci/usb.c >>> index f77b953..34509ff 100644 >>> --- a/arch/arm/mach-davinci/usb.c >>> +++ b/arch/arm/mach-davinci/usb.c >>> @@ -42,14 +42,8 @@ static struct musb_hdrc_config musb_config = { >>> }; >>> >>> static struct musb_hdrc_platform_data usb_data = { >>> -#if defined(CONFIG_USB_MUSB_OTG) >>> /* OTG requires a Mini-AB connector */ >>> .mode = MUSB_OTG, >>> -#elif defined(CONFIG_USB_MUSB_PERIPHERAL) >>> -.mode = MUSB_PERIPHERAL, >>> -#elif defined(CONFIG_USB_MUSB_HOST) >>> -.mode = MUSB_HOST, >>> -#endif >>> .clock= "usb", >>> .config=_config, >>> }; >> >> Tested it on DM6446-EVM for host mode with MSC thumb drive and gadget >> mode with g-ether. It works. >> >> Acked-by: Manjunathappa, Prakash > > I sent a similiar patch here: > > http://comments.gmane.org/gmane.linux.usb.general/54512 > > If the issues, mentioned from Sergei for my patch, nullified I add my: The last outstanding issue from Sergei seems to be additional comments describing why MUSB_OTG is OK to use. Prakash/Constantine, Did you have to make any hardware changes when testing host/gadget on DM6446 EVM? Or change in kernel configuration? It appears that there is no way to choose any of the config option affecting the mode setting. Right now it seems to be just defaulting to MUSB_UNDEFINED. Setting it to MUSB_OTG would be better than that. Thanks, Sekhar -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: RE: [PATCH] usb: remove CONFIG_USB_MUSB_HOST etc
Hello Manjunathappa On 11.10.2012 07:42, Manjunathappa, Prakash wrote: Hi, On Mon, Oct 08, 2012 at 18:47:07, Constantine Shulyupin wrote: From: Constantine Shulyupin Remove USB configuration in arch/arm/mach-davinci/usb.c accordingly CONFIG_USB_MUSB_OTG CONFIG_USB_MUSB_PERIPHERAL CONFIG_USB_MUSB_HOST and set MUSB_OTG configuration by default because this configuration options are removed from Kconfig. Signed-off-by: Constantine Shulyupin --- arch/arm/mach-davinci/usb.c |6 -- 1 file changed, 6 deletions(-) diff --git a/arch/arm/mach-davinci/usb.c b/arch/arm/mach-davinci/usb.c index f77b953..34509ff 100644 --- a/arch/arm/mach-davinci/usb.c +++ b/arch/arm/mach-davinci/usb.c @@ -42,14 +42,8 @@ static struct musb_hdrc_config musb_config = { }; static struct musb_hdrc_platform_data usb_data = { -#if defined(CONFIG_USB_MUSB_OTG) /* OTG requires a Mini-AB connector */ .mode = MUSB_OTG, -#elif defined(CONFIG_USB_MUSB_PERIPHERAL) - .mode = MUSB_PERIPHERAL, -#elif defined(CONFIG_USB_MUSB_HOST) - .mode = MUSB_HOST, -#endif .clock = "usb", .config =_config, }; Tested it on DM6446-EVM for host mode with MSC thumb drive and gadget mode with g-ether. It works. Acked-by: Manjunathappa, Prakash I sent a similiar patch here: http://comments.gmane.org/gmane.linux.usb.general/54512 If the issues, mentioned from Sergei for my patch, nullified I add my: Acked-by: Heiko Schocher for this patch. bye, Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: RE: [PATCH] usb: remove CONFIG_USB_MUSB_HOST etc
Hello Manjunathappa On 11.10.2012 07:42, Manjunathappa, Prakash wrote: Hi, On Mon, Oct 08, 2012 at 18:47:07, Constantine Shulyupin wrote: From: Constantine Shulyupinco...@makelinux.com Remove USB configuration in arch/arm/mach-davinci/usb.c accordingly CONFIG_USB_MUSB_OTG CONFIG_USB_MUSB_PERIPHERAL CONFIG_USB_MUSB_HOST and set MUSB_OTG configuration by default because this configuration options are removed from Kconfig. Signed-off-by: Constantine Shulyupinco...@makelinux.com --- arch/arm/mach-davinci/usb.c |6 -- 1 file changed, 6 deletions(-) diff --git a/arch/arm/mach-davinci/usb.c b/arch/arm/mach-davinci/usb.c index f77b953..34509ff 100644 --- a/arch/arm/mach-davinci/usb.c +++ b/arch/arm/mach-davinci/usb.c @@ -42,14 +42,8 @@ static struct musb_hdrc_config musb_config = { }; static struct musb_hdrc_platform_data usb_data = { -#if defined(CONFIG_USB_MUSB_OTG) /* OTG requires a Mini-AB connector */ .mode = MUSB_OTG, -#elif defined(CONFIG_USB_MUSB_PERIPHERAL) - .mode = MUSB_PERIPHERAL, -#elif defined(CONFIG_USB_MUSB_HOST) - .mode = MUSB_HOST, -#endif .clock = usb, .config =musb_config, }; Tested it on DM6446-EVM for host mode with MSC thumb drive and gadget mode with g-ether. It works. Acked-by: Manjunathappa, Prakashprakash...@ti.com I sent a similiar patch here: http://comments.gmane.org/gmane.linux.usb.general/54512 If the issues, mentioned from Sergei for my patch, nullified I add my: Acked-by: Heiko Schocher h...@denx.de for this patch. bye, Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] usb: remove CONFIG_USB_MUSB_HOST etc
On 10/11/2012 12:05 PM, Heiko Schocher wrote: Hello Manjunathappa On 11.10.2012 07:42, Manjunathappa, Prakash wrote: Hi, On Mon, Oct 08, 2012 at 18:47:07, Constantine Shulyupin wrote: From: Constantine Shulyupinco...@makelinux.com Remove USB configuration in arch/arm/mach-davinci/usb.c accordingly CONFIG_USB_MUSB_OTG CONFIG_USB_MUSB_PERIPHERAL CONFIG_USB_MUSB_HOST and set MUSB_OTG configuration by default because this configuration options are removed from Kconfig. Signed-off-by: Constantine Shulyupinco...@makelinux.com --- arch/arm/mach-davinci/usb.c |6 -- 1 file changed, 6 deletions(-) diff --git a/arch/arm/mach-davinci/usb.c b/arch/arm/mach-davinci/usb.c index f77b953..34509ff 100644 --- a/arch/arm/mach-davinci/usb.c +++ b/arch/arm/mach-davinci/usb.c @@ -42,14 +42,8 @@ static struct musb_hdrc_config musb_config = { }; static struct musb_hdrc_platform_data usb_data = { -#if defined(CONFIG_USB_MUSB_OTG) /* OTG requires a Mini-AB connector */ .mode = MUSB_OTG, -#elif defined(CONFIG_USB_MUSB_PERIPHERAL) -.mode = MUSB_PERIPHERAL, -#elif defined(CONFIG_USB_MUSB_HOST) -.mode = MUSB_HOST, -#endif .clock= usb, .config=musb_config, }; Tested it on DM6446-EVM for host mode with MSC thumb drive and gadget mode with g-ether. It works. Acked-by: Manjunathappa, Prakashprakash...@ti.com I sent a similiar patch here: http://comments.gmane.org/gmane.linux.usb.general/54512 If the issues, mentioned from Sergei for my patch, nullified I add my: The last outstanding issue from Sergei seems to be additional comments describing why MUSB_OTG is OK to use. Prakash/Constantine, Did you have to make any hardware changes when testing host/gadget on DM6446 EVM? Or change in kernel configuration? It appears that there is no way to choose any of the config option affecting the mode setting. Right now it seems to be just defaulting to MUSB_UNDEFINED. Setting it to MUSB_OTG would be better than that. Thanks, Sekhar -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] usb: remove CONFIG_USB_MUSB_HOST etc
I've tested gadget mode on DM365 without HW changes. On Thu, Oct 11, 2012 at 10:50 AM, Sekhar Nori nsek...@ti.com wrote: On 10/11/2012 12:05 PM, Heiko Schocher wrote: Hello Manjunathappa On 11.10.2012 07:42, Manjunathappa, Prakash wrote: Hi, On Mon, Oct 08, 2012 at 18:47:07, Constantine Shulyupin wrote: From: Constantine Shulyupinco...@makelinux.com Remove USB configuration in arch/arm/mach-davinci/usb.c accordingly CONFIG_USB_MUSB_OTG CONFIG_USB_MUSB_PERIPHERAL CONFIG_USB_MUSB_HOST and set MUSB_OTG configuration by default because this configuration options are removed from Kconfig. Signed-off-by: Constantine Shulyupinco...@makelinux.com --- arch/arm/mach-davinci/usb.c |6 -- 1 file changed, 6 deletions(-) diff --git a/arch/arm/mach-davinci/usb.c b/arch/arm/mach-davinci/usb.c index f77b953..34509ff 100644 --- a/arch/arm/mach-davinci/usb.c +++ b/arch/arm/mach-davinci/usb.c @@ -42,14 +42,8 @@ static struct musb_hdrc_config musb_config = { }; static struct musb_hdrc_platform_data usb_data = { -#if defined(CONFIG_USB_MUSB_OTG) /* OTG requires a Mini-AB connector */ .mode = MUSB_OTG, -#elif defined(CONFIG_USB_MUSB_PERIPHERAL) -.mode = MUSB_PERIPHERAL, -#elif defined(CONFIG_USB_MUSB_HOST) -.mode = MUSB_HOST, -#endif .clock= usb, .config=musb_config, }; Tested it on DM6446-EVM for host mode with MSC thumb drive and gadget mode with g-ether. It works. Acked-by: Manjunathappa, Prakashprakash...@ti.com I sent a similiar patch here: http://comments.gmane.org/gmane.linux.usb.general/54512 If the issues, mentioned from Sergei for my patch, nullified I add my: The last outstanding issue from Sergei seems to be additional comments describing why MUSB_OTG is OK to use. Prakash/Constantine, Did you have to make any hardware changes when testing host/gadget on DM6446 EVM? Or change in kernel configuration? It appears that there is no way to choose any of the config option affecting the mode setting. Right now it seems to be just defaulting to MUSB_UNDEFINED. Setting it to MUSB_OTG would be better than that. Thanks, Sekhar -- Constantine Shulyupin http://www.MakeLinux.com/ Embedded Linux Systems, Device Drivers, TI DaVinci -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
RE: [PATCH] usb: remove CONFIG_USB_MUSB_HOST etc
Hi, On Mon, Oct 08, 2012 at 18:47:07, Constantine Shulyupin wrote: > From: Constantine Shulyupin > > Remove USB configuration in arch/arm/mach-davinci/usb.c accordingly > CONFIG_USB_MUSB_OTG CONFIG_USB_MUSB_PERIPHERAL CONFIG_USB_MUSB_HOST > and set MUSB_OTG configuration by default > because this configuration options are removed from Kconfig. > > Signed-off-by: Constantine Shulyupin > > --- > arch/arm/mach-davinci/usb.c |6 -- > 1 file changed, 6 deletions(-) > > diff --git a/arch/arm/mach-davinci/usb.c b/arch/arm/mach-davinci/usb.c > index f77b953..34509ff 100644 > --- a/arch/arm/mach-davinci/usb.c > +++ b/arch/arm/mach-davinci/usb.c > @@ -42,14 +42,8 @@ static struct musb_hdrc_config musb_config = { > }; > > static struct musb_hdrc_platform_data usb_data = { > -#if defined(CONFIG_USB_MUSB_OTG) > /* OTG requires a Mini-AB connector */ > .mode = MUSB_OTG, > -#elif defined(CONFIG_USB_MUSB_PERIPHERAL) > - .mode = MUSB_PERIPHERAL, > -#elif defined(CONFIG_USB_MUSB_HOST) > - .mode = MUSB_HOST, > -#endif > .clock = "usb", > .config = _config, > }; Tested it on DM6446-EVM for host mode with MSC thumb drive and gadget mode with g-ether. It works. Acked-by: Manjunathappa, Prakash Thanks, Prakash > -- > 1.7.9.5 > > ___ > Davinci-linux-open-source mailing list > davinci-linux-open-sou...@linux.davincidsp.com > http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
RE: [PATCH] usb: remove CONFIG_USB_MUSB_HOST etc
Hi, On Mon, Oct 08, 2012 at 18:47:07, Constantine Shulyupin wrote: From: Constantine Shulyupin co...@makelinux.com Remove USB configuration in arch/arm/mach-davinci/usb.c accordingly CONFIG_USB_MUSB_OTG CONFIG_USB_MUSB_PERIPHERAL CONFIG_USB_MUSB_HOST and set MUSB_OTG configuration by default because this configuration options are removed from Kconfig. Signed-off-by: Constantine Shulyupin co...@makelinux.com --- arch/arm/mach-davinci/usb.c |6 -- 1 file changed, 6 deletions(-) diff --git a/arch/arm/mach-davinci/usb.c b/arch/arm/mach-davinci/usb.c index f77b953..34509ff 100644 --- a/arch/arm/mach-davinci/usb.c +++ b/arch/arm/mach-davinci/usb.c @@ -42,14 +42,8 @@ static struct musb_hdrc_config musb_config = { }; static struct musb_hdrc_platform_data usb_data = { -#if defined(CONFIG_USB_MUSB_OTG) /* OTG requires a Mini-AB connector */ .mode = MUSB_OTG, -#elif defined(CONFIG_USB_MUSB_PERIPHERAL) - .mode = MUSB_PERIPHERAL, -#elif defined(CONFIG_USB_MUSB_HOST) - .mode = MUSB_HOST, -#endif .clock = usb, .config = musb_config, }; Tested it on DM6446-EVM for host mode with MSC thumb drive and gadget mode with g-ether. It works. Acked-by: Manjunathappa, Prakash prakash...@ti.com Thanks, Prakash -- 1.7.9.5 ___ Davinci-linux-open-source mailing list davinci-linux-open-sou...@linux.davincidsp.com http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] usb: remove CONFIG_USB_MUSB_HOST etc
From: Constantine Shulyupin Remove USB configuration in arch/arm/mach-davinci/usb.c accordingly CONFIG_USB_MUSB_OTG CONFIG_USB_MUSB_PERIPHERAL CONFIG_USB_MUSB_HOST and set MUSB_OTG configuration by default because this configuration options are removed from Kconfig. Signed-off-by: Constantine Shulyupin --- arch/arm/mach-davinci/usb.c |6 -- 1 file changed, 6 deletions(-) diff --git a/arch/arm/mach-davinci/usb.c b/arch/arm/mach-davinci/usb.c index f77b953..34509ff 100644 --- a/arch/arm/mach-davinci/usb.c +++ b/arch/arm/mach-davinci/usb.c @@ -42,14 +42,8 @@ static struct musb_hdrc_config musb_config = { }; static struct musb_hdrc_platform_data usb_data = { -#if defined(CONFIG_USB_MUSB_OTG) /* OTG requires a Mini-AB connector */ .mode = MUSB_OTG, -#elif defined(CONFIG_USB_MUSB_PERIPHERAL) - .mode = MUSB_PERIPHERAL, -#elif defined(CONFIG_USB_MUSB_HOST) - .mode = MUSB_HOST, -#endif .clock = "usb", .config = _config, }; -- 1.7.9.5 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] usb: remove CONFIG_USB_MUSB_HOST etc
From: Constantine Shulyupin co...@makelinux.com Remove USB configuration in arch/arm/mach-davinci/usb.c accordingly CONFIG_USB_MUSB_OTG CONFIG_USB_MUSB_PERIPHERAL CONFIG_USB_MUSB_HOST and set MUSB_OTG configuration by default because this configuration options are removed from Kconfig. Signed-off-by: Constantine Shulyupin co...@makelinux.com --- arch/arm/mach-davinci/usb.c |6 -- 1 file changed, 6 deletions(-) diff --git a/arch/arm/mach-davinci/usb.c b/arch/arm/mach-davinci/usb.c index f77b953..34509ff 100644 --- a/arch/arm/mach-davinci/usb.c +++ b/arch/arm/mach-davinci/usb.c @@ -42,14 +42,8 @@ static struct musb_hdrc_config musb_config = { }; static struct musb_hdrc_platform_data usb_data = { -#if defined(CONFIG_USB_MUSB_OTG) /* OTG requires a Mini-AB connector */ .mode = MUSB_OTG, -#elif defined(CONFIG_USB_MUSB_PERIPHERAL) - .mode = MUSB_PERIPHERAL, -#elif defined(CONFIG_USB_MUSB_HOST) - .mode = MUSB_HOST, -#endif .clock = usb, .config = musb_config, }; -- 1.7.9.5 -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/