* [U-Boot] [PATCH] arm, usb, davinci: make USBPHY_CTL register configurable @ 2011-08-02 6:20 Heiko Schocher 2011-08-08 20:30 ` Remy Bohmer 2011-10-05 14:27 ` [U-Boot] [PATCH v2 01/10] " Heiko Schocher 0 siblings, 2 replies; 7+ messages in thread From: Heiko Schocher @ 2011-08-02 6:20 UTC (permalink / raw) To: u-boot Define CONFIG_DV_USBPHY_CTL for setting the USB PHY control register. Signed-off-by: Heiko Schocher <hs@denx.de> cc: Sandeep Paulraj <s-paulraj@ti.com> cc: Remy Bohmer <linux@bohmer.net> --- drivers/usb/musb/davinci.c | 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/drivers/usb/musb/davinci.c b/drivers/usb/musb/davinci.c index f56f2df..98c2c62 100644 --- a/drivers/usb/musb/davinci.c +++ b/drivers/usb/musb/davinci.c @@ -26,6 +26,10 @@ #include "davinci.h" #include <asm/arch/hardware.h> +#if !defined(CONFIG_DV_USBPHY_CTL) +#define CONFIG_DV_USBPHY_CTL (USBPHY_SESNDEN | USBPHY_VBDTCTEN) +#endif + /* MUSB platform configuration */ struct musb_config musb_cfg = { .regs = (struct musb_regs *)MENTOR_USB0_BASE, @@ -50,7 +54,7 @@ static u8 phy_on(void) writel(USBPHY_PHY24MHZ | USBPHY_SESNDEN | USBPHY_VBDTCTEN, USBPHY_CTL_PADDR); #else - writel(USBPHY_SESNDEN | USBPHY_VBDTCTEN, USBPHY_CTL_PADDR); + writel(CONFIG_DV_USBPHY_CTL, USBPHY_CTL_PADDR); #endif timeout = musb_cfg.timeout; -- 1.7.6 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] arm, usb, davinci: make USBPHY_CTL register configurable 2011-08-02 6:20 [U-Boot] [PATCH] arm, usb, davinci: make USBPHY_CTL register configurable Heiko Schocher @ 2011-08-08 20:30 ` Remy Bohmer 2011-08-09 5:05 ` Heiko Schocher 2011-10-05 14:27 ` [U-Boot] [PATCH v2 01/10] " Heiko Schocher 1 sibling, 1 reply; 7+ messages in thread From: Remy Bohmer @ 2011-08-08 20:30 UTC (permalink / raw) To: u-boot Hi, 2011/8/2 Heiko Schocher <hs@denx.de>: > Define CONFIG_DV_USBPHY_CTL for setting the USB PHY control > register. > Signed-off-by: Heiko Schocher <hs@denx.de> > cc: Sandeep Paulraj <s-paulraj@ti.com> > cc: Remy Bohmer <linux@bohmer.net> > --- > ?drivers/usb/musb/davinci.c | ? ?6 +++++- > ?1 files changed, 5 insertions(+), 1 deletions(-) > > diff --git a/drivers/usb/musb/davinci.c b/drivers/usb/musb/davinci.c > index f56f2df..98c2c62 100644 > --- a/drivers/usb/musb/davinci.c > +++ b/drivers/usb/musb/davinci.c > @@ -26,6 +26,10 @@ > ?#include "davinci.h" > ?#include <asm/arch/hardware.h> > > +#if !defined(CONFIG_DV_USBPHY_CTL) > +#define CONFIG_DV_USBPHY_CTL (USBPHY_SESNDEN | USBPHY_VBDTCTEN) > +#endif > + > ?/* MUSB platform configuration */ > ?struct musb_config musb_cfg = { > ? ? ? ?.regs ? ? ? ? ? = (struct musb_regs *)MENTOR_USB0_BASE, > @@ -50,7 +54,7 @@ static u8 phy_on(void) > ? ? ? ?writel(USBPHY_PHY24MHZ | USBPHY_SESNDEN | > ? ? ? ? ? ? ? ? ? ? ? ?USBPHY_VBDTCTEN, USBPHY_CTL_PADDR); > ?#else > - ? ? ? writel(USBPHY_SESNDEN | USBPHY_VBDTCTEN, USBPHY_CTL_PADDR); > + ? ? ? writel(CONFIG_DV_USBPHY_CTL, USBPHY_CTL_PADDR); > ?#endif > ? ? ? ?timeout = musb_cfg.timeout; What does it fix, why do you want this? Kind regards, Remy ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] arm, usb, davinci: make USBPHY_CTL register configurable 2011-08-08 20:30 ` Remy Bohmer @ 2011-08-09 5:05 ` Heiko Schocher 2011-09-15 16:05 ` Albert ARIBAUD 0 siblings, 1 reply; 7+ messages in thread From: Heiko Schocher @ 2011-08-09 5:05 UTC (permalink / raw) To: u-boot Hello Remy, Remy Bohmer wrote: > Hi, > > 2011/8/2 Heiko Schocher <hs@denx.de>: >> Define CONFIG_DV_USBPHY_CTL for setting the USB PHY control >> register. >> Signed-off-by: Heiko Schocher <hs@denx.de> >> cc: Sandeep Paulraj <s-paulraj@ti.com> >> cc: Remy Bohmer <linux@bohmer.net> >> --- >> drivers/usb/musb/davinci.c | 6 +++++- >> 1 files changed, 5 insertions(+), 1 deletions(-) >> >> diff --git a/drivers/usb/musb/davinci.c b/drivers/usb/musb/davinci.c >> index f56f2df..98c2c62 100644 >> --- a/drivers/usb/musb/davinci.c >> +++ b/drivers/usb/musb/davinci.c >> @@ -26,6 +26,10 @@ >> #include "davinci.h" >> #include <asm/arch/hardware.h> >> >> +#if !defined(CONFIG_DV_USBPHY_CTL) >> +#define CONFIG_DV_USBPHY_CTL (USBPHY_SESNDEN | USBPHY_VBDTCTEN) >> +#endif >> + >> /* MUSB platform configuration */ >> struct musb_config musb_cfg = { >> .regs = (struct musb_regs *)MENTOR_USB0_BASE, >> @@ -50,7 +54,7 @@ static u8 phy_on(void) >> writel(USBPHY_PHY24MHZ | USBPHY_SESNDEN | >> USBPHY_VBDTCTEN, USBPHY_CTL_PADDR); >> #else >> - writel(USBPHY_SESNDEN | USBPHY_VBDTCTEN, USBPHY_CTL_PADDR); >> + writel(CONFIG_DV_USBPHY_CTL, USBPHY_CTL_PADDR); >> #endif >> timeout = musb_cfg.timeout; > > What does it fix, why do you want this? I posted the cam_enc_4xx board support. Now adding USB support, and I have to configure this register as: #define CONFIG_DV_USBPHY_CTL (USBPHY_SESNDEN | USBPHY_VBDTCTEN | \ USBPHY_PHY24MHZ) so I need a possibility to configure this register ... and I could not use DAVINCI_DM365EVM! (BTW: This define (so it seems to me) hides board specific code, which should be cleaned up ... Sandeep?) bye, Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] arm, usb, davinci: make USBPHY_CTL register configurable 2011-08-09 5:05 ` Heiko Schocher @ 2011-09-15 16:05 ` Albert ARIBAUD 2011-09-16 5:41 ` Heiko Schocher 0 siblings, 1 reply; 7+ messages in thread From: Albert ARIBAUD @ 2011-09-15 16:05 UTC (permalink / raw) To: u-boot Hi all, Le 09/08/2011 07:05, Heiko Schocher a ?crit : > Hello Remy, > > Remy Bohmer wrote: >> Hi, >> >> 2011/8/2 Heiko Schocher<hs@denx.de>: >>> Define CONFIG_DV_USBPHY_CTL for setting the USB PHY control >>> register. >>> Signed-off-by: Heiko Schocher<hs@denx.de> >>> cc: Sandeep Paulraj<s-paulraj@ti.com> >>> cc: Remy Bohmer<linux@bohmer.net> >>> --- >>> drivers/usb/musb/davinci.c | 6 +++++- >>> 1 files changed, 5 insertions(+), 1 deletions(-) >>> >>> diff --git a/drivers/usb/musb/davinci.c b/drivers/usb/musb/davinci.c >>> index f56f2df..98c2c62 100644 >>> --- a/drivers/usb/musb/davinci.c >>> +++ b/drivers/usb/musb/davinci.c >>> @@ -26,6 +26,10 @@ >>> #include "davinci.h" >>> #include<asm/arch/hardware.h> >>> >>> +#if !defined(CONFIG_DV_USBPHY_CTL) >>> +#define CONFIG_DV_USBPHY_CTL (USBPHY_SESNDEN | USBPHY_VBDTCTEN) >>> +#endif >>> + >>> /* MUSB platform configuration */ >>> struct musb_config musb_cfg = { >>> .regs = (struct musb_regs *)MENTOR_USB0_BASE, >>> @@ -50,7 +54,7 @@ static u8 phy_on(void) >>> writel(USBPHY_PHY24MHZ | USBPHY_SESNDEN | >>> USBPHY_VBDTCTEN, USBPHY_CTL_PADDR); >>> #else >>> - writel(USBPHY_SESNDEN | USBPHY_VBDTCTEN, USBPHY_CTL_PADDR); >>> + writel(CONFIG_DV_USBPHY_CTL, USBPHY_CTL_PADDR); >>> #endif >>> timeout = musb_cfg.timeout; >> >> What does it fix, why do you want this? > > I posted the cam_enc_4xx board support. Now adding USB support, and I > have to configure this register as: > > #define CONFIG_DV_USBPHY_CTL (USBPHY_SESNDEN | USBPHY_VBDTCTEN | \ > USBPHY_PHY24MHZ) > > so I need a possibility to configure this register ... and I could not > use DAVINCI_DM365EVM! (BTW: This define (so it seems to me) hides > board specific code, which should be cleaned up ... Sandeep?) > > bye, > Heiko No new reply on this since early august... Personally, I'd rather this patch came in a patchset where some other patch uses it. Amicalement, -- Albert. ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] arm, usb, davinci: make USBPHY_CTL register configurable 2011-09-15 16:05 ` Albert ARIBAUD @ 2011-09-16 5:41 ` Heiko Schocher 2011-09-24 16:06 ` Remy Bohmer 0 siblings, 1 reply; 7+ messages in thread From: Heiko Schocher @ 2011-09-16 5:41 UTC (permalink / raw) To: u-boot Hello Albert, Albert ARIBAUD wrote: > Hi all, > > Le 09/08/2011 07:05, Heiko Schocher a ?crit : >> Hello Remy, >> >> Remy Bohmer wrote: >>> Hi, >>> >>> 2011/8/2 Heiko Schocher<hs@denx.de>: >>>> Define CONFIG_DV_USBPHY_CTL for setting the USB PHY control >>>> register. >>>> Signed-off-by: Heiko Schocher<hs@denx.de> >>>> cc: Sandeep Paulraj<s-paulraj@ti.com> >>>> cc: Remy Bohmer<linux@bohmer.net> >>>> --- >>>> drivers/usb/musb/davinci.c | 6 +++++- >>>> 1 files changed, 5 insertions(+), 1 deletions(-) >>>> >>>> diff --git a/drivers/usb/musb/davinci.c b/drivers/usb/musb/davinci.c >>>> index f56f2df..98c2c62 100644 >>>> --- a/drivers/usb/musb/davinci.c >>>> +++ b/drivers/usb/musb/davinci.c >>>> @@ -26,6 +26,10 @@ >>>> #include "davinci.h" >>>> #include<asm/arch/hardware.h> >>>> >>>> +#if !defined(CONFIG_DV_USBPHY_CTL) >>>> +#define CONFIG_DV_USBPHY_CTL (USBPHY_SESNDEN | USBPHY_VBDTCTEN) >>>> +#endif >>>> + >>>> /* MUSB platform configuration */ >>>> struct musb_config musb_cfg = { >>>> .regs = (struct musb_regs *)MENTOR_USB0_BASE, >>>> @@ -50,7 +54,7 @@ static u8 phy_on(void) >>>> writel(USBPHY_PHY24MHZ | USBPHY_SESNDEN | >>>> USBPHY_VBDTCTEN, USBPHY_CTL_PADDR); >>>> #else >>>> - writel(USBPHY_SESNDEN | USBPHY_VBDTCTEN, USBPHY_CTL_PADDR); >>>> + writel(CONFIG_DV_USBPHY_CTL, USBPHY_CTL_PADDR); >>>> #endif >>>> timeout = musb_cfg.timeout; >>> >>> What does it fix, why do you want this? >> >> I posted the cam_enc_4xx board support. Now adding USB support, and I >> have to configure this register as: >> >> #define CONFIG_DV_USBPHY_CTL (USBPHY_SESNDEN | USBPHY_VBDTCTEN | \ >> USBPHY_PHY24MHZ) >> >> so I need a possibility to configure this register ... and I could not >> use DAVINCI_DM365EVM! (BTW: This define (so it seems to me) hides >> board specific code, which should be cleaned up ... Sandeep?) >> >> bye, >> Heiko > > No new reply on this since early august... Personally, I'd rather this > patch came in a patchset where some other patch uses it. Hmm.. Anyway I have to resend the cam_enc_4xx board port, as now the new SPL Framework is in mainline, and my post used the old one. Also I added in the meantime USB support for it, for which I needed this patch. But actually I am wating for a new HW, so I can't test it ... and then I post this patchset again with this patch in it! Can I add then your "Acked-by" for this patch? Thanks Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] arm, usb, davinci: make USBPHY_CTL register configurable 2011-09-16 5:41 ` Heiko Schocher @ 2011-09-24 16:06 ` Remy Bohmer 0 siblings, 0 replies; 7+ messages in thread From: Remy Bohmer @ 2011-09-24 16:06 UTC (permalink / raw) To: u-boot Hi, > Hmm.. Anyway I have to resend the cam_enc_4xx board port, as now the new > SPL Framework is in mainline, and my post used the old one. Also > I added in the meantime USB support for it, for which I needed this patch. > But actually I am wating for a new HW, so I can't test it ... and then > I post this patchset again with this patch in it! Can I add then your > "Acked-by" for this patch? Acked-by: Remy Bohmer <linux@bohmer.net> Kind regards, Remy ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v2 01/10] arm, usb, davinci: make USBPHY_CTL register configurable 2011-08-02 6:20 [U-Boot] [PATCH] arm, usb, davinci: make USBPHY_CTL register configurable Heiko Schocher 2011-08-08 20:30 ` Remy Bohmer @ 2011-10-05 14:27 ` Heiko Schocher 1 sibling, 0 replies; 7+ messages in thread From: Heiko Schocher @ 2011-10-05 14:27 UTC (permalink / raw) To: u-boot Define CONFIG_DV_USBPHY_CTL for setting the USB PHY control register. Signed-off-by: Heiko Schocher <hs@denx.de> Acked-by: Remy Bohmer <linux@bohmer.net> cc: Sandeep Paulraj <s-paulraj@ti.com> cc: Remy Bohmer <linux@bohmer.net> --- drivers/usb/musb/davinci.c | 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/drivers/usb/musb/davinci.c b/drivers/usb/musb/davinci.c index f56f2df..98c2c62 100644 --- a/drivers/usb/musb/davinci.c +++ b/drivers/usb/musb/davinci.c @@ -26,6 +26,10 @@ #include "davinci.h" #include <asm/arch/hardware.h> +#if !defined(CONFIG_DV_USBPHY_CTL) +#define CONFIG_DV_USBPHY_CTL (USBPHY_SESNDEN | USBPHY_VBDTCTEN) +#endif + /* MUSB platform configuration */ struct musb_config musb_cfg = { .regs = (struct musb_regs *)MENTOR_USB0_BASE, @@ -50,7 +54,7 @@ static u8 phy_on(void) writel(USBPHY_PHY24MHZ | USBPHY_SESNDEN | USBPHY_VBDTCTEN, USBPHY_CTL_PADDR); #else - writel(USBPHY_SESNDEN | USBPHY_VBDTCTEN, USBPHY_CTL_PADDR); + writel(CONFIG_DV_USBPHY_CTL, USBPHY_CTL_PADDR); #endif timeout = musb_cfg.timeout; -- 1.7.6.2 ^ permalink raw reply related [flat|nested] 7+ messages in thread
end of thread, other threads:[~2011-10-05 14:27 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2011-08-02 6:20 [U-Boot] [PATCH] arm, usb, davinci: make USBPHY_CTL register configurable Heiko Schocher 2011-08-08 20:30 ` Remy Bohmer 2011-08-09 5:05 ` Heiko Schocher 2011-09-15 16:05 ` Albert ARIBAUD 2011-09-16 5:41 ` Heiko Schocher 2011-09-24 16:06 ` Remy Bohmer 2011-10-05 14:27 ` [U-Boot] [PATCH v2 01/10] " Heiko Schocher
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox