Hello.

On 28-07-2011 12:21, [email protected] wrote:

From: Constantine Shulyupin<[email protected]>

Please in the future add "v2" (or whatever version it's gonna be) after "PATCH" in the subject, and describe the changes that you've done below your signoff and the --- tear line.

Signed-off-by: Constantine Shulyupin<[email protected]>

diff --git a/arch/arm/mach-davinci/board-dm365-evm.c 
b/arch/arm/mach-davinci/board-dm365-evm.c
index 8710614..c62ca53 100644
--- a/arch/arm/mach-davinci/board-dm365-evm.c
+++ b/arch/arm/mach-davinci/board-dm365-evm.c
@@ -39,6 +39,7 @@
  #include<mach/mmc.h>
  #include<mach/nand.h>
  #include<mach/keyscan.h>
+#include<mach/usb.h>

  #include<media/tvp514x.h>

@@ -612,6 +613,7 @@ static __init void dm365_evm_init(void)

        dm365_init_spi0(BIT(0), dm365_evm_spi_info,
                        ARRAY_SIZE(dm365_evm_spi_info));
+       davinci_setup_usb(500, 8);
  }

  MACHINE_START(DAVINCI_DM365_EVM, "DaVinci DM365 EVM")

The patch needs to be split here (and the second part pushed thru the 'linux-usb' mailing list. I also wonder whether we should setup PinMux to activate VBUS, once we dropped the GPIO manipulation.

diff --git a/drivers/usb/musb/davinci.c b/drivers/usb/musb/davinci.c
index 2a2adf6..905a107 100644
--- a/drivers/usb/musb/davinci.c
+++ b/drivers/usb/musb/davinci.c
@@ -72,6 +72,11 @@ static inline void phy_on(void)
        /* power everything up; start the on-chip PHY and its PLL */
        phy_ctrl&= ~(USBPHY_OSCPDWN | USBPHY_OTGPDWN | USBPHY_PHYPDWN);
        phy_ctrl |= USBPHY_SESNDEN | USBPHY_VBDTCTEN | USBPHY_PHYPLLON;
+
+       if (cpu_is_davinci_dm365()) {
+               phy_ctrl |= USBPHY_CLKFREQ_24MHZ;
+       }

I'm still thinking this should belong to the board code. That, and {} are not needed.

+
        __raw_writel(phy_ctrl, USB_PHY_CTRL);

        /* wait for PLL to lock before proceeding */
diff --git a/drivers/usb/musb/davinci.h b/drivers/usb/musb/davinci.h
index 046c844..1bf50e6 100644
--- a/drivers/usb/musb/davinci.h
+++ b/drivers/usb/musb/davinci.h
@@ -17,6 +17,7 @@
  /* Integrated highspeed/otg PHY */
  #define USBPHY_CTL_PADDR      (DAVINCI_SYSTEM_MODULE_BASE + 0x34)
  #define USBPHY_DATAPOL                BIT(11) /* (dm355) switch D+/D- */
+#define USBPHY_CLKFREQ_24MHZ    BIT(13)

   Please indent with tabs, not spaces.

WBR, Sergei
_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to