* [PATCH] omap3: ehci: remove chargepump hack
@ 2009-09-24 12:06 Anand Gadiyar
2009-09-24 12:10 ` Felipe Balbi
2009-09-30 18:29 ` [APPLIED] " Tony Lindgren
0 siblings, 2 replies; 6+ messages in thread
From: Anand Gadiyar @ 2009-09-24 12:06 UTC (permalink / raw)
To: linux-omap; +Cc: Anand Gadiyar
omap3: ehci: remove chargepump hack
This hack was for a very early development board that is no longer
in use. Remove it. No board should require it.
Signed-off-by: Anand Gadiyar <gadiyar@ti.com>
---
diff --git a/arch/arm/mach-omap2/board-3430sdp.c b/arch/arm/mach-omap2/board-3430sdp.c
index 5f913c3..5571f44 100644
--- a/arch/arm/mach-omap2/board-3430sdp.c
+++ b/arch/arm/mach-omap2/board-3430sdp.c
@@ -486,7 +486,6 @@ static struct ehci_hcd_omap_platform_data ehci_pdata __initconst = {
.port_mode[1] = EHCI_HCD_OMAP_MODE_PHY,
.port_mode[2] = EHCI_HCD_OMAP_MODE_UNKNOWN,
- .chargepump = true,
.phy_reset = true,
.reset_gpio_port[0] = 57,
.reset_gpio_port[1] = 61,
diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c
index aa403e6..f81141f 100644
--- a/arch/arm/mach-omap2/board-omap3beagle.c
+++ b/arch/arm/mach-omap2/board-omap3beagle.c
@@ -406,7 +406,6 @@ static struct ehci_hcd_omap_platform_data ehci_pdata __initconst = {
.port_mode[1] = EHCI_HCD_OMAP_MODE_PHY,
.port_mode[2] = EHCI_HCD_OMAP_MODE_UNKNOWN,
- .chargepump = false,
.phy_reset = true,
.reset_gpio_port[0] = -EINVAL,
.reset_gpio_port[1] = 147,
diff --git a/arch/arm/mach-omap2/board-omap3evm.c b/arch/arm/mach-omap2/board-omap3evm.c
index 62a6f32..a2fcd06 100644
--- a/arch/arm/mach-omap2/board-omap3evm.c
+++ b/arch/arm/mach-omap2/board-omap3evm.c
@@ -299,7 +299,6 @@ static struct ehci_hcd_omap_platform_data ehci_pdata __initconst = {
.port_mode[1] = EHCI_HCD_OMAP_MODE_PHY,
.port_mode[2] = EHCI_HCD_OMAP_MODE_UNKNOWN,
- .chargepump = false,
.phy_reset = true,
.reset_gpio_port[0] = -EINVAL,
.reset_gpio_port[1] = 135,
diff --git a/arch/arm/mach-omap2/board-omap3pandora.c b/arch/arm/mach-omap2/board-omap3pandora.c
index 9f115dc..fd4a951 100644
--- a/arch/arm/mach-omap2/board-omap3pandora.c
+++ b/arch/arm/mach-omap2/board-omap3pandora.c
@@ -389,7 +389,6 @@ static struct ehci_hcd_omap_platform_data ehci_pdata __initconst = {
.port_mode[1] = EHCI_HCD_OMAP_MODE_UNKNOWN,
.port_mode[2] = EHCI_HCD_OMAP_MODE_UNKNOWN,
- .chargepump = false,
.phy_reset = true,
.reset_gpio_port[0] = 16,
.reset_gpio_port[1] = -EINVAL,
diff --git a/arch/arm/mach-omap2/board-overo.c b/arch/arm/mach-omap2/board-overo.c
index d16172d..46e376f 100644
--- a/arch/arm/mach-omap2/board-overo.c
+++ b/arch/arm/mach-omap2/board-overo.c
@@ -389,7 +389,6 @@ static struct ehci_hcd_omap_platform_data ehci_pdata __initconst = {
.port_mode[1] = EHCI_HCD_OMAP_MODE_PHY,
.port_mode[2] = EHCI_HCD_OMAP_MODE_UNKNOWN,
- .chargepump = false,
.phy_reset = true,
.reset_gpio_port[0] = -EINVAL,
.reset_gpio_port[1] = OVERO_GPIO_USBH_NRESET,
diff --git a/arch/arm/plat-omap/include/mach/usb.h b/arch/arm/plat-omap/include/mach/usb.h
index d1b14cc..a4068a4 100644
--- a/arch/arm/plat-omap/include/mach/usb.h
+++ b/arch/arm/plat-omap/include/mach/usb.h
@@ -14,7 +14,6 @@ enum ehci_hcd_omap_mode {
struct ehci_hcd_omap_platform_data {
enum ehci_hcd_omap_mode port_mode[OMAP3_HS_USB_PORTS];
- unsigned chargepump:1;
unsigned phy_reset:1;
/* have to be valid if phy_reset is true and portx is in phy mode */
diff --git a/drivers/usb/host/ehci-omap.c b/drivers/usb/host/ehci-omap.c
index 36fb183..f2771cf 100644
--- a/drivers/usb/host/ehci-omap.c
+++ b/drivers/usb/host/ehci-omap.c
@@ -169,9 +169,6 @@ struct ehci_hcd_omap {
/* phy reset workaround */
int phy_reset;
- /* vbus internal chargepump workaround */
- int chargepump;
-
/* desired phy_mode: TLL, PHY */
enum ehci_hcd_omap_mode port_mode[OMAP3_HS_USB_PORTS];
@@ -423,25 +420,6 @@ static int omap_start_ehc(struct ehci_hcd_omap *omap, struct usb_hcd *hcd)
gpio_set_value(omap->reset_gpio_port[1], 1);
}
- if (omap->chargepump) {
-
- /* Refer ISSUE2: LINK assumes external charge pump */
-
- /* use Port1 VBUS to charge externally Port2:
- * So for PHY mode operation use Port2 only
- */
- ehci_omap_writel(omap->ehci_base, EHCI_INSNREG05_ULPI,
- /* OTG ctrl reg*/
- (0xa << EHCI_INSNREG05_ULPI_REGADD_SHIFT) |
- /* Write */
- (2 << EHCI_INSNREG05_ULPI_OPSEL_SHIFT) |
- /* Port1 */
- (1 << EHCI_INSNREG05_ULPI_PORTSEL_SHIFT) |
- /* Start */
- (1 << EHCI_INSNREG05_ULPI_CONTROL_SHIFT) |
- (0x26));
- }
-
return 0;
err_sys_status:
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] omap3: ehci: remove chargepump hack
2009-09-24 12:06 [PATCH] omap3: ehci: remove chargepump hack Anand Gadiyar
@ 2009-09-24 12:10 ` Felipe Balbi
2009-09-24 12:19 ` Gadiyar, Anand
2009-09-30 18:29 ` [APPLIED] " Tony Lindgren
1 sibling, 1 reply; 6+ messages in thread
From: Felipe Balbi @ 2009-09-24 12:10 UTC (permalink / raw)
To: ext Anand Gadiyar; +Cc: linux-omap@vger.kernel.org
On Thu, Sep 24, 2009 at 02:06:57PM +0200, ext Anand Gadiyar wrote:
> omap3: ehci: remove chargepump hack
>
> This hack was for a very early development board that is no longer
> in use. Remove it. No board should require it.
do we have anyone using it besides development boards ?
--
balbi
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH] omap3: ehci: remove chargepump hack
2009-09-24 12:10 ` Felipe Balbi
@ 2009-09-24 12:19 ` Gadiyar, Anand
2009-09-24 12:45 ` Felipe Balbi
0 siblings, 1 reply; 6+ messages in thread
From: Gadiyar, Anand @ 2009-09-24 12:19 UTC (permalink / raw)
To: felipe.balbi@nokia.com; +Cc: linux-omap@vger.kernel.org
> On Thu, Sep 24, 2009 at 02:06:57PM +0200, ext Anand Gadiyar wrote:
> > omap3: ehci: remove chargepump hack
> >
> > This hack was for a very early development board that is no longer
> > in use. Remove it. No board should require it.
>
> do we have anyone using it besides development boards ?
>
Not sure I understand the question. If you meant "is anyone other than
development boards using the chargepump hack", then the short answer is no.
The chargepump hack was only for one particular reference design board - the
very first version of it. Only 6-odd boards were made which needed that hack.
It's long gone from internal codebases, as are the boards.
As far as I know, no board in the wild will need it. If you look at the
patch, only the 3430 SDP board set this flag to true, and all others
were false. I confirm that all later versions of the expansion board in use
with the 3430SDP do not require this hack.
So there's no point having it around.
- Anand
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] omap3: ehci: remove chargepump hack
2009-09-24 12:19 ` Gadiyar, Anand
@ 2009-09-24 12:45 ` Felipe Balbi
2009-09-30 1:19 ` Gadiyar, Anand
0 siblings, 1 reply; 6+ messages in thread
From: Felipe Balbi @ 2009-09-24 12:45 UTC (permalink / raw)
To: ext Gadiyar, Anand
Cc: Balbi Felipe (Nokia-D/Helsinki), linux-omap@vger.kernel.org
On Thu, Sep 24, 2009 at 02:19:52PM +0200, ext Gadiyar, Anand wrote:
> > On Thu, Sep 24, 2009 at 02:06:57PM +0200, ext Anand Gadiyar wrote:
> > > omap3: ehci: remove chargepump hack
> > >
> > > This hack was for a very early development board that is no longer
> > > in use. Remove it. No board should require it.
> >
> > do we have anyone using it besides development boards ?
> >
>
> Not sure I understand the question. If you meant "is anyone other than
> development boards using the chargepump hack", then the short answer is no.
>
> The chargepump hack was only for one particular reference design board - the
> very first version of it. Only 6-odd boards were made which needed that hack.
> It's long gone from internal codebases, as are the boards.
>
> As far as I know, no board in the wild will need it. If you look at the
> patch, only the 3430 SDP board set this flag to true, and all others
> were false. I confirm that all later versions of the expansion board in use
> with the 3430SDP do not require this hack.
>
> So there's no point having it around.
good, so:
Acked-by: Felipe Balbi <felipe.balbi@nokia.com>
--
balbi
^ permalink raw reply [flat|nested] 6+ messages in thread
* [APPLIED] [PATCH] omap3: ehci: remove chargepump hack
2009-09-24 12:06 [PATCH] omap3: ehci: remove chargepump hack Anand Gadiyar
2009-09-24 12:10 ` Felipe Balbi
@ 2009-09-30 18:29 ` Tony Lindgren
1 sibling, 0 replies; 6+ messages in thread
From: Tony Lindgren @ 2009-09-30 18:29 UTC (permalink / raw)
To: linux-omap
This patch has been applied to the linux-omap
by youw fwiendly patch wobot.
Branch in linux-omap: ehci
Initial commit ID (Likely to change): 8aab384eab253c463f3eafae1c19bc44f744cdca
PatchWorks
http://patchwork.kernel.org/patch/49837/
Git (Likely to change, and takes a while to get mirrored)
http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap-2.6.git;a=commit;h=8aab384eab253c463f3eafae1c19bc44f744cdca
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2009-09-30 18:29 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-09-24 12:06 [PATCH] omap3: ehci: remove chargepump hack Anand Gadiyar
2009-09-24 12:10 ` Felipe Balbi
2009-09-24 12:19 ` Gadiyar, Anand
2009-09-24 12:45 ` Felipe Balbi
2009-09-30 1:19 ` Gadiyar, Anand
2009-09-30 18:29 ` [APPLIED] " Tony Lindgren
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox