Hello Jaehoon, On Fri, Oct 17, 2014 at 1:52 AM, Jaehoon Chung <jh80.ch...@samsung.com> wrote: > Hi, Suriyan. > > This patch can be separated. >
OK, I shall separate out the power.c/power.h changes for enabling and disabling the usbhost phy. > On 10/02/2014 10:45 PM, Suriyan Ramasami wrote: >> This change adds support for enabling the USB host features of the board. >> This includes the USB3503A hub and the SMC LAN9730 ethernet controller >> as well. >> >> Credit goes to Tushar Behera (Linaro) for the function set_usb_ethaddr(). >> >> Signed-off-by: Suriyan Ramasami <suriya...@gmail.com> >> --- >> arch/arm/cpu/armv7/exynos/power.c | 26 +++++++++++++++ >> arch/arm/dts/exynos4412-odroid.dts | 11 +++++++ >> arch/arm/include/asm/arch-exynos/cpu.h | 2 ++ >> arch/arm/include/asm/arch-exynos/ehci.h | 13 ++++++++ >> arch/arm/include/asm/arch-exynos/power.h | 7 ++++ >> board/samsung/odroid/odroid.c | 55 >> ++++++++++++++++++++++++++++++++ >> drivers/usb/host/ehci-exynos.c | 52 +++++++++++++++++++++++++----- >> include/configs/odroid.h | 13 ++++++++ >> 8 files changed, 171 insertions(+), 8 deletions(-) >> >> diff --git a/arch/arm/cpu/armv7/exynos/power.c >> b/arch/arm/cpu/armv7/exynos/power.c >> index e1ab3d6..6578a07 100644 >> --- a/arch/arm/cpu/armv7/exynos/power.c >> +++ b/arch/arm/cpu/armv7/exynos/power.c >> @@ -53,10 +53,36 @@ void exynos5_set_usbhost_phy_ctrl(unsigned int enable) >> } >> } >> >> +void exynos4412_set_usbhost_phy_ctrl(unsigned int enable) >> +{ >> + struct exynos4412_power *power = >> + (struct exynos4412_power *)samsung_get_base_power(); >> + >> + if (enable) { >> + /* Enabling USBHOST_PHY */ >> + setbits_le32(&power->usbhost_phy_control, >> + POWER_USB_HOST_PHY_CTRL_EN); >> + setbits_le32(&power->hsic1_phy_control, >> + POWER_USB_HOST_PHY_CTRL_EN); >> + setbits_le32(&power->hsic2_phy_control, >> + POWER_USB_HOST_PHY_CTRL_EN); >> + } else { >> + /* Disabling USBHOST_PHY */ >> + clrbits_le32(&power->usbhost_phy_control, >> + POWER_USB_HOST_PHY_CTRL_EN); >> + clrbits_le32(&power->hsic1_phy_control, >> + POWER_USB_HOST_PHY_CTRL_EN); >> + clrbits_le32(&power->hsic2_phy_control, >> + POWER_USB_HOST_PHY_CTRL_EN); >> + } >> +} >> + >> void set_usbhost_phy_ctrl(unsigned int enable) >> { >> if (cpu_is_exynos5()) >> exynos5_set_usbhost_phy_ctrl(enable); >> + else if (proid_is_exynos4412()) >> + exynos4412_set_usbhost_phy_ctrl(enable); >> } >> >> static void exynos5_set_usbdrd_phy_ctrl(unsigned int enable) >> diff --git a/arch/arm/dts/exynos4412-odroid.dts >> b/arch/arm/dts/exynos4412-odroid.dts >> index 24d0bf1..8da8568 100644 >> --- a/arch/arm/dts/exynos4412-odroid.dts >> +++ b/arch/arm/dts/exynos4412-odroid.dts >> @@ -67,4 +67,15 @@ >> div = <0x3>; >> index = <4>; >> }; >> + >> + ehci@12580000 { >> + compatible = "samsung,exynos-ehci"; >> + reg = <0x12580000 0x100>; >> + #address-cells = <1>; >> + #size-cells = <1>; >> + phy { >> + compatible = "samsung,exynos-usb-phy"; >> + reg = <0x125B0000 0x100>; >> + }; >> + }; > > Fix the indentation. > Shall do that. Thanks - Suriyan > Best Regards, > Jaehoon Chung > >> }; >> diff --git a/arch/arm/include/asm/arch-exynos/cpu.h >> b/arch/arm/include/asm/arch-exynos/cpu.h >> index ba71714..fda21fb 100644 >> --- a/arch/arm/include/asm/arch-exynos/cpu.h >> +++ b/arch/arm/include/asm/arch-exynos/cpu.h >> @@ -18,6 +18,8 @@ >> >> #define EXYNOS4_GPIO_PART3_BASE 0x03860000 >> #define EXYNOS4_PRO_ID 0x10000000 >> +#define EXYNOS4_GUID_LOW 0x10000014 >> +#define EXYNOS4_GUID_HIGH 0x10000018 >> #define EXYNOS4_SYSREG_BASE 0x10010000 >> #define EXYNOS4_POWER_BASE 0x10020000 >> #define EXYNOS4_SWRESET 0x10020400 >> diff --git a/arch/arm/include/asm/arch-exynos/ehci.h >> b/arch/arm/include/asm/arch-exynos/ehci.h >> index d2d70bd..3800fa9 100644 >> --- a/arch/arm/include/asm/arch-exynos/ehci.h >> +++ b/arch/arm/include/asm/arch-exynos/ehci.h >> @@ -12,6 +12,13 @@ >> >> #define CLK_24MHZ 5 >> >> +#define PHYPWR_NORMAL_MASK_PHY0 (0x39 << 0) >> +#define PHYPWR_NORMAL_MASK_PHY1 (0x7 << 6) >> +#define PHYPWR_NORMAL_MASK_HSIC0 (0x7 << 9) >> +#define PHYPWR_NORMAL_MASK_HSIC1 (0x7 << 12) >> +#define RSTCON_HOSTPHY_SWRST (0xf << 3) >> +#define RSTCON_SWRST (0x1 << 0) >> + >> #define HOST_CTRL0_PHYSWRSTALL (1 << 31) >> #define HOST_CTRL0_COMMONON_N (1 << 9) >> #define HOST_CTRL0_SIDDQ (1 << 6) >> @@ -61,6 +68,12 @@ struct exynos_usb_phy { >> unsigned int usbotgtune; >> }; >> >> +struct exynos4412_usb_phy { >> + unsigned int usbphyctrl; >> + unsigned int usbphyclk; >> + unsigned int usbphyrstcon; >> +}; >> + >> /* Switch on the VBUS power. */ >> int board_usb_vbus_init(void); >> >> diff --git a/arch/arm/include/asm/arch-exynos/power.h >> b/arch/arm/include/asm/arch-exynos/power.h >> index e8a98a5..3f97b31 100644 >> --- a/arch/arm/include/asm/arch-exynos/power.h >> +++ b/arch/arm/include/asm/arch-exynos/power.h >> @@ -210,6 +210,13 @@ struct exynos4_power { >> unsigned int gps_alive_option; >> }; >> >> +struct exynos4412_power { >> + unsigned char res1[0x0704]; >> + unsigned int usbhost_phy_control; >> + unsigned int hsic1_phy_control; >> + unsigned int hsic2_phy_control; >> +}; >> + >> struct exynos5_power { >> unsigned int om_stat; >> unsigned char res1[0x18]; >> diff --git a/board/samsung/odroid/odroid.c b/board/samsung/odroid/odroid.c >> index fd5d2d2..4764a71 100644 >> --- a/board/samsung/odroid/odroid.c >> +++ b/board/samsung/odroid/odroid.c >> @@ -453,9 +453,64 @@ struct s3c_plat_otg_data s5pc210_otg_data = { >> .usb_phy_ctrl = EXYNOS4X12_USBPHY_CONTROL, >> .usb_flags = PHY0_SLEEP, >> }; >> +#endif >> + >> +#if defined(CONFIG_USB_GADGET) || defined(CONFIG_CMD_USB) >> + >> +#ifdef CONFIG_CMD_USB >> +static void set_usb_ethaddr(void) >> +{ >> + int i; >> + uchar mac[6]; >> + unsigned int guid_high = readl(EXYNOS4_GUID_HIGH); >> + unsigned int guid_low = readl(EXYNOS4_GUID_LOW); >> + >> + for (i = 0; i < 2; i++) >> + mac[i] = (guid_high >> (8 * (1 - i))) & 0xFF; >> + >> + for (i = 0; i < 4; i++) >> + mac[i+2] = (guid_low >> (8 * (3 - i))) & 0xFF; >> + >> + /* mark it as not multicast and outside official 80211 MAC namespace */ >> + mac[0] = (mac[0] & ~0x1) | 0x2; >> + >> + eth_setenv_enetaddr("ethaddr", mac); >> + eth_setenv_enetaddr("usbethaddr", mac); >> +} >> +#endif >> >> int board_usb_init(int index, enum usb_init_type init) >> { >> +#ifdef CONFIG_CMD_USB >> + struct pmic *p_pmic; >> + >> + /* Set Ref freq 0 => 24MHz, 1 => 26MHz*/ >> + /* Odroid Us have it at 24MHz, Odroid Xs at 26MHz */ >> + if (gd->board_type == ODROID_TYPE_U3) >> + gpio_direction_output(EXYNOS4X12_GPIO_X30, 0); >> + else >> + gpio_direction_output(EXYNOS4X12_GPIO_X30, 1); >> + >> + /* Disconnect, Reset, Connect */ >> + gpio_direction_output(EXYNOS4X12_GPIO_X34, 0); >> + gpio_direction_output(EXYNOS4X12_GPIO_X35, 0); >> + gpio_direction_output(EXYNOS4X12_GPIO_X35, 1); >> + gpio_direction_output(EXYNOS4X12_GPIO_X34, 1); >> + >> + /* Power off and on BUCK8 for LAN9730 */ >> + debug("LAN9730 - Turning power buck 8 OFF and ON.\n"); >> + >> + p_pmic = pmic_get("MAX77686_PMIC"); >> + if (p_pmic && !pmic_probe(p_pmic)) { >> + max77686_set_buck_mode(p_pmic, 8, OPMODE_OFF); >> + max77686_set_buck_voltage(p_pmic, 8, 750000); >> + max77686_set_buck_voltage(p_pmic, 8, 3300000); >> + max77686_set_buck_mode(p_pmic, 8, OPMODE_ON); >> + } >> + >> + set_usb_ethaddr(); >> +#endif >> + >> debug("USB_udc_probe\n"); >> return s3c_udc_probe(&s5pc210_otg_data); >> } >> diff --git a/drivers/usb/host/ehci-exynos.c b/drivers/usb/host/ehci-exynos.c >> index edd91a8..91f3c72 100644 >> --- a/drivers/usb/host/ehci-exynos.c >> +++ b/drivers/usb/host/ehci-exynos.c >> @@ -19,6 +19,7 @@ >> #include <asm/gpio.h> >> #include <asm-generic/errno.h> >> #include <linux/compat.h> >> +#include <power/max77686_pmic.h> >> #include "ehci.h" >> >> /* Declare global data pointer */ >> @@ -85,15 +86,10 @@ static int exynos_usb_parse_dt(const void *blob, struct >> exynos_ehci *exynos) >> } >> #endif >> >> -/* Setup the EHCI host controller. */ >> -static void setup_usb_phy(struct exynos_usb_phy *usb) >> +static void exynos5_setup_usb_phy(struct exynos_usb_phy *usb) >> { >> u32 hsic_ctrl; >> >> - set_usbhost_mode(USB20_PHY_CFG_HOST_LINK_EN); >> - >> - set_usbhost_phy_ctrl(POWER_USB_HOST_PHY_CTRL_EN); >> - >> clrbits_le32(&usb->usbphyctrl0, >> HOST_CTRL0_FSEL_MASK | >> HOST_CTRL0_COMMONON_N | >> @@ -150,8 +146,32 @@ static void setup_usb_phy(struct exynos_usb_phy *usb) >> EHCICTRL_ENAINCR16); >> } >> >> -/* Reset the EHCI host controller. */ >> -static void reset_usb_phy(struct exynos_usb_phy *usb) >> +static void exynos4412_setup_usb_phy(struct exynos4412_usb_phy *usb) >> +{ >> + writel(CLK_24MHZ, &usb->usbphyclk); >> + >> + clrbits_le32(&usb->usbphyctrl, (PHYPWR_NORMAL_MASK_HSIC0 | >> + PHYPWR_NORMAL_MASK_HSIC1 | PHYPWR_NORMAL_MASK_PHY1 | >> + PHYPWR_NORMAL_MASK_PHY0)); >> + >> + setbits_le32(&usb->usbphyrstcon, (RSTCON_HOSTPHY_SWRST | >> RSTCON_SWRST)); >> + udelay(10); >> + clrbits_le32(&usb->usbphyrstcon, (RSTCON_HOSTPHY_SWRST | >> RSTCON_SWRST)); >> +} >> + >> +static void setup_usb_phy(struct exynos_usb_phy *usb) >> +{ >> + set_usbhost_mode(USB20_PHY_CFG_HOST_LINK_EN); >> + >> + set_usbhost_phy_ctrl(POWER_USB_HOST_PHY_CTRL_EN); >> + >> + if (cpu_is_exynos5()) >> + exynos5_setup_usb_phy(usb); >> + else if (proid_is_exynos4412()) >> + exynos4412_setup_usb_phy((struct exynos4412_usb_phy *)usb); >> +} >> + >> +static void exynos5_reset_usb_phy(struct exynos_usb_phy *usb) >> { >> u32 hsic_ctrl; >> >> @@ -171,6 +191,22 @@ static void reset_usb_phy(struct exynos_usb_phy *usb) >> >> setbits_le32(&usb->hsicphyctrl1, hsic_ctrl); >> setbits_le32(&usb->hsicphyctrl2, hsic_ctrl); >> +} >> + >> +static void exynos4412_reset_usb_phy(struct exynos4412_usb_phy *usb) >> +{ >> + setbits_le32(&usb->usbphyctrl, (PHYPWR_NORMAL_MASK_HSIC0 | >> + PHYPWR_NORMAL_MASK_HSIC1 | PHYPWR_NORMAL_MASK_PHY1 | >> + PHYPWR_NORMAL_MASK_PHY0)); >> +} >> + >> +/* Reset the EHCI host controller. */ >> +static void reset_usb_phy(struct exynos_usb_phy *usb) >> +{ >> + if (cpu_is_exynos5()) >> + exynos5_reset_usb_phy(usb); >> + else if (proid_is_exynos4412()) >> + exynos4412_reset_usb_phy((struct exynos4412_usb_phy *)usb); >> >> set_usbhost_phy_ctrl(POWER_USB_HOST_PHY_CTRL_DISABLE); >> } >> diff --git a/include/configs/odroid.h b/include/configs/odroid.h >> index b616ac2..3fc0508 100644 >> --- a/include/configs/odroid.h >> +++ b/include/configs/odroid.h >> @@ -200,6 +200,19 @@ >> >> #define CONFIG_CMD_GPIO >> >> +/* USB */ >> +#define CONFIG_CMD_USB >> +#define CONFIG_USB_EHCI >> +#define CONFIG_USB_EHCI_EXYNOS >> +#define CONFIG_USB_STORAGE >> + >> +#define CONFIG_SYS_USB_EHCI_MAX_ROOT_PORTS 3 >> +#define CONFIG_CMD_NET >> +#define CONFIG_CMD_PING >> +#define CONFIG_CMD_DHCP >> +#define CONFIG_USB_HOST_ETHER >> +#define CONFIG_USB_ETHER_SMSC95XX >> + >> /* >> * Supported Odroid boards: X3, U3 >> * TODO: Add Odroid X support >> > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot