[U-Boot] [PATCH] arm, usb, davinci: make USBPHY_CTL register configurable

[Heiko Schocher]

Hello Remy,

Remy Bohmer wrote:
> Hi,
> 
> 2011/8/2 Heiko Schocher <hs at denx.de>:
>> Define CONFIG_DV_USBPHY_CTL for setting the USB PHY control
>> register.
>> Signed-off-by: Heiko Schocher <hs at denx.de>
>> cc: Sandeep Paulraj <s-paulraj at ti.com>
>> cc: Remy Bohmer <linux at 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