Skip to contentSkip to conversation

arm: odroid: pmic77686: allow buck voltage settings

PATCH New Suriyan Ramasami 4 participants 17 messages Oct 29, 2014 – Nov 19, 2014

Subsystems (4)
Arm (32-bit) USB Arm Samsung Exynos USB ehci driver
lore
Share & tools
lore

Suriyan Ramasami

author
#0 lore raw

PATCH v3 1/3 arm: odroid: pmic77686: allow buck voltage settings

Allow to set the buck voltage for the max77686.
This will be used to reset the SMC LAN9730 ethernet on the odroids.

Signed-off-by Suriyan Ramasami <[email protected]>

---

Changes in v3:
* Correct ldo and buck validation logic
* Jaehoon/Przemyslaw - Use negative errno values for error condistions
* Albert - its buck not bucket

Changes in v2:
* Jaehoon - separate out this patch

Changes in v1:
- First try

 drivers/power/pmic/pmic_max77686.c | 52 +++++++++++++++++++++++++++++++++++---
 include/power/max77686_pmic.h      |  3 +++
 2 files changed, 52 insertions(+), 3 deletions(-)
drivers/power/pmic/pmic_max77686.c +49 -3
index df1fd91..95b1a57 100644
@@ -42,11 +42,30 @@ static unsigned int max77686_ldo_volt2hex(int ldo, ulong uV)
 	return 0;
 }
 
+static int max77686_buck_volt2hex(int buck, ulong uV)
+{
+	int hex = 0;
+
+	if (buck < 5 || buck > 9) {
+		debug("%s: buck %d is not supported\n", __func__, buck);
+		return -EINVAL;
+	}
+
+	hex = (uV - 750000) / 50000;
+
+	if (hex >= 0 && hex <= MAX77686_BUCK_VOLT_MAX_HEX)
+		return hex;
+
+	debug("%s: %ld is wrong voltage value for BUCK%d\n",
+	      __func__, uV, buck);
+	return -EINVAL;
+}
+
 int max77686_set_ldo_voltage(struct pmic *p, int ldo, ulong uV)
 {
 	unsigned int val, ret, hex, adr;
 
-	if (ldo < 1 && ldo > 26) {
+	if (ldo < 1 || ldo > 26) {
 		printf("%s: %d is wrong ldo number\n", __func__, ldo);
 		return -1;
 	}

@@ -68,11 +87,38 @@ int max77686_set_ldo_voltage(struct pmic *p, int ldo, ulong uV)
 	return ret;
 }
 
+int max77686_set_buck_voltage(struct pmic *p, int buck, ulong uV)
+{
+	unsigned int val, adr;
+	int hex, ret;
+
+	if (buck < 5 || buck > 9) {
+		printf("%s: %d is an unsupported bucket number\n",
+		       __func__, buck);
+		return -EINVAL;
+	}
+
+	adr = max77686_buck_addr[buck] + 1;
+	hex = max77686_buck_volt2hex(buck, uV);
+
+	if (hex < 0)
+		return hex;
+
+	ret = pmic_reg_read(p, adr, &val);
+	if (ret)
+		return ret;
+
+	val &= ~MAX77686_BUCK_VOLT_MASK;
+	ret |= pmic_reg_write(p, adr, val | hex);
+
+	return ret;
+}
+
 int max77686_set_ldo_mode(struct pmic *p, int ldo, char opmode)
 {
 	unsigned int val, ret, adr, mode;
 
-	if (ldo < 1 && 26 < ldo) {
+	if (ldo < 1 || 26 < ldo) {
 		printf("%s: %d is wrong ldo number\n", __func__, ldo);
 		return -1;
 	}

@@ -157,7 +203,7 @@ int max77686_set_buck_mode(struct pmic *p, int buck, char opmode)
 	/* mode */
 	switch (opmode) {
 	case OPMODE_OFF:
-		mode = MAX77686_BUCK_MODE_OFF;
+		mode = MAX77686_BUCK_MODE_OFF << mode_shift;
 		break;
 	case OPMODE_STANDBY:
 		switch (buck) {
include/power/max77686_pmic.h +3 -0
index c2a772a..b0e4255 100644
@@ -150,6 +150,7 @@ enum {
 
 int max77686_set_ldo_voltage(struct pmic *p, int ldo, ulong uV);
 int max77686_set_ldo_mode(struct pmic *p, int ldo, char opmode);
+int max77686_set_buck_voltage(struct pmic *p, int buck, ulong uV);
 int max77686_set_buck_mode(struct pmic *p, int buck, char opmode);
 
 #define MAX77686_LDO_VOLT_MAX_HEX	0x3f

@@ -159,6 +160,8 @@ int max77686_set_buck_mode(struct pmic *p, int buck, char opmode);
 #define MAX77686_LDO_MODE_STANDBY	(0x01 << 0x06)
 #define MAX77686_LDO_MODE_LPM		(0x02 << 0x06)
 #define MAX77686_LDO_MODE_ON		(0x03 << 0x06)
+#define MAX77686_BUCK_VOLT_MAX_HEX	0x3f
+#define MAX77686_BUCK_VOLT_MASK		0x3f
 #define MAX77686_BUCK_MODE_MASK		0x03
 #define MAX77686_BUCK_MODE_SHIFT_1	0x00
 #define MAX77686_BUCK_MODE_SHIFT_2	0x04
Signature · 1 line
1.9.1
author
#1 lore raw

PATCH v3 2/3 arm: odroid: enable/disable usb host phy for exynos4412

Enable/disable the usb host phy on the odroid U/X2 boards which are based on the Exynos4412 SOC.

Signed-off-by Suriyan Ramasami <[email protected]>

---

Changes in v3:
* Minkyu - do not mix cpu_is... and proid_is...

Changes in v2:
* Jaehoon - separate this patch out

Changes in v1:
* First try

 arch/arm/cpu/armv7/exynos/power.c        | 27 +++++++++++++++++++++++++++
 arch/arm/include/asm/arch-exynos/power.h |  7 +++++++
 2 files changed, 34 insertions(+)
arch/arm/cpu/armv7/exynos/power.c +27 -0
index e1ab3d6..1520d64 100644
@@ -53,10 +53,37 @@ 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 (cpu_is_exynos4())
+		if (proid_is_exynos4412())
+			exynos4412_set_usbhost_phy_ctrl(enable);
 }
 
 static void exynos5_set_usbdrd_phy_ctrl(unsigned int enable)
arch/arm/include/asm/arch-exynos/power.h +7 -0
index e8a98a5..3f97b31 100644
@@ -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];
Signature · 1 line
1.9.1
#2 lore raw

PATCH v3 2/3 arm: odroid: enable/disable usb host phy for exynos4412

Quoted 13 lines · Suriyan Ramasami

Enable/disable the usb host phy on the odroid U/X2 boards which are based on the Exynos4412 SOC.

Signed-off-by Suriyan Ramasami <[email protected]>

---

Changes in v3:
* Minkyu - do not mix cpu_is... and proid_is...

Changes in v2:
* Jaehoon - separate this patch out

Quoted 7 lines

Changes in v1:
* First try

 arch/arm/cpu/armv7/exynos/power.c        | 27 +++++++++++++++++++++++++++
 arch/arm/include/asm/arch-exynos/power.h |  7 +++++++
 2 files changed, 34 insertions(+)

applied to u-boot-samsung.

Thanks,
Minkyu Kang.

author
#3 lore raw

PATCH v3 3/3 arm: odroid: usb: add support for usb host including ethernet

In reply to Suriyan Ramasami

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.

Signed-off-by Suriyan Ramasami <[email protected]>

---

Changes in v3:
* removed set_usb_ethaddr() and related code as the GUID registers do not seem to be documented anywhere. This is sad, as this mechanism allows for each Odroid to boot up with the same MAC address every time, but no two odroids shall have the same MAC address on boot. This ensures multiple odroids in the same LAN to come up without conflicting MAC addresses.
* Minkyu - Do not mix cpu_is... and proid_is...

Changes in v2:
* Jaehoon - Split power.[ch] as a separate patch
* Removed an unneeded header file from ehci-exynos.c
* Jaehoon - Fix indentation in the dts file

Changes in v1:
* First try

 arch/arm/dts/exynos4412-odroid.dts      | 11 +++++++
 arch/arm/include/asm/arch-exynos/ehci.h | 13 ++++++++
 board/samsung/odroid/odroid.c           | 32 +++++++++++++++++++
 drivers/usb/host/ehci-exynos.c          | 55 ++++++++++++++++++++++++++++-----
 include/configs/odroid.h                | 13 ++++++++
 5 files changed, 116 insertions(+), 8 deletions(-)
arch/arm/dts/exynos4412-odroid.dts +11 -0
index 4c5e2b3..2df1801 100644
@@ -67,4 +67,15 @@
 		div = <0x3>;
 		index = <4>;
 	};
+
+	ehci at 12580000 {
+		compatible = "samsung,exynos-ehci";
+		reg = <0x12580000 0x100>;
+		#address-cells = <1>;
+		#size-cells = <1>;
+		phy {
+			compatible = "samsung,exynos-usb-phy";
+			reg = <0x125B0000 0x100>;
+		};
+	};
 };
arch/arm/include/asm/arch-exynos/ehci.h +13 -0
index d2d70bd..3800fa9 100644
@@ -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);
board/samsung/odroid/odroid.c +32 -0
index 5edb250..d671f13 100644
@@ -453,9 +453,41 @@ 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)
 
 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);
+	}
+
+#endif
+
 	debug("USB_udc_probe\n");
 	return s3c_udc_probe(&s5pc210_otg_data);
 }
drivers/usb/host/ehci-exynos.c +47 -8

76 lines not on this page · raw message

include/configs/odroid.h +13 -0
index b928af8..807e96b 100644
@@ -198,6 +198,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
Signature · 1 line
1.9.1
#4 lore raw

PATCH v3 3/3 arm: odroid: usb: add support for usb host including ethernet

Quoted 160 lines · Suriyan Ramasami

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.

Signed-off-by Suriyan Ramasami <[email protected]>

---

Changes in v3:
* removed set_usb_ethaddr() and related code as the GUID registers do not seem to be documented anywhere. This is sad, as this mechanism allows for each Odroid to boot up with the same MAC address every time, but no two odroids shall have the same MAC address on boot. This ensures multiple odroids in the same LAN to come up without conflicting MAC addresses.
* Minkyu - Do not mix cpu_is... and proid_is...

Changes in v2:
* Jaehoon - Split power.[ch] as a separate patch
* Removed an unneeded header file from ehci-exynos.c
* Jaehoon - Fix indentation in the dts file

Changes in v1:
* First try

 arch/arm/dts/exynos4412-odroid.dts      | 11 +++++++
 arch/arm/include/asm/arch-exynos/ehci.h | 13 ++++++++
 board/samsung/odroid/odroid.c           | 32 +++++++++++++++++++
 drivers/usb/host/ehci-exynos.c          | 55 ++++++++++++++++++++++++++++-----
 include/configs/odroid.h                | 13 ++++++++
 5 files changed, 116 insertions(+), 8 deletions(-)
arch/arm/dts/exynos4412-odroid.dts +11 -0
index 4c5e2b3..2df1801 100644
@@ -67,4 +67,15 @@
 		div = <0x3>;
 		index = <4>;
 	};
+
+	ehci at 12580000 {
+		compatible = "samsung,exynos-ehci";
+		reg = <0x12580000 0x100>;
+		#address-cells = <1>;
+		#size-cells = <1>;
+		phy {
+			compatible = "samsung,exynos-usb-phy";
+			reg = <0x125B0000 0x100>;
+		};
+	};
 };
arch/arm/include/asm/arch-exynos/ehci.h +13 -0
index d2d70bd..3800fa9 100644
@@ -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);
board/samsung/odroid/odroid.c +32 -0
index 5edb250..d671f13 100644
@@ -453,9 +453,41 @@ 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)
 
 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);
+	}
+
+#endif
+
 	debug("USB_udc_probe\n");
 	return s3c_udc_probe(&s5pc210_otg_data);
 }
drivers/usb/host/ehci-exynos.c +6 -8
index edd91a8..6fdbf57 100644
@@ -85,15 +85,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 +145,34 @@ 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 |

drivers/usb/host/ehci-exynos.c static void setup_usb_phy(struct exynos_usb_phy *usb) · line 162

Quoted 10 lines
drivers/usb/host/ehci-exynos.c +10 -0
+		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)
+{

Well, I didn't know exactly. but i think this is something strange. is it always bypassed the "struct exynos_usb_phy"?

And don't need to modify the below code into exynos_usb_parse_dt()?

	/*
	 * Get the base address for usbphy from the device node
	 */
	exynos->usb = (struct exynos_usb_phy *)fdtdec_get_addr(blob, node,
								"reg");
	if (exynos->usb == NULL) {
		debug("Can't get the usbphy register address\n");
		return -ENXIO;
	}

Best Regards,
Jaehoon Chung

Quoted 66 lines
drivers/usb/host/ehci-exynos.c +31 -0
+	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 (cpu_is_exynos4())
+		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 +192,24 @@ 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 (cpu_is_exynos4())
+		if (proid_is_exynos4412())
+			exynos4412_reset_usb_phy((struct exynos4412_usb_phy *)
+						 usb);
 
 	set_usbhost_phy_ctrl(POWER_USB_HOST_PHY_CTRL_DISABLE);
 }
include/configs/odroid.h +13 -0
index b928af8..807e96b 100644
@@ -198,6 +198,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
author
#5 lore raw

PATCH v3 3/3 arm: odroid: usb: add support for usb host including ethernet

Hello Jaehoon Chung,

Quoted 177 lines · Jaehoon Chung
Quoted 160 lines · Suriyan Ramasami

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.

Signed-off-by Suriyan Ramasami <[email protected]>

---

Changes in v3:
* removed set_usb_ethaddr() and related code as the GUID registers do not seem to be documented anywhere. This is sad, as this mechanism allows for each Odroid to boot up with the same MAC address every time, but no two odroids shall have the same MAC address on boot. This ensures multiple odroids in the same LAN to come up without conflicting MAC addresses.
* Minkyu - Do not mix cpu_is... and proid_is...

Changes in v2:
* Jaehoon - Split power.[ch] as a separate patch
* Removed an unneeded header file from ehci-exynos.c
* Jaehoon - Fix indentation in the dts file

Changes in v1:
* First try

 arch/arm/dts/exynos4412-odroid.dts      | 11 +++++++
 arch/arm/include/asm/arch-exynos/ehci.h | 13 ++++++++
 board/samsung/odroid/odroid.c           | 32 +++++++++++++++++++
 drivers/usb/host/ehci-exynos.c          | 55 ++++++++++++++++++++++++++++-----
 include/configs/odroid.h                | 13 ++++++++
 5 files changed, 116 insertions(+), 8 deletions(-)
arch/arm/dts/exynos4412-odroid.dts +11 -0
index 4c5e2b3..2df1801 100644
@@ -67,4 +67,15 @@
              div = <0x3>;
              index = <4>;
      };
+
+     ehci at 12580000 {
+             compatible = "samsung,exynos-ehci";
+             reg = <0x12580000 0x100>;
+             #address-cells = <1>;
+             #size-cells = <1>;
+             phy {
+                     compatible = "samsung,exynos-usb-phy";
+                     reg = <0x125B0000 0x100>;
+             };
+     };
 };
arch/arm/include/asm/arch-exynos/ehci.h +13 -0
index d2d70bd..3800fa9 100644
@@ -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);
board/samsung/odroid/odroid.c +32 -0
index 5edb250..d671f13 100644
@@ -453,9 +453,41 @@ 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)
 
 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);
+     }
+
+#endif
+
      debug("USB_udc_probe\n");
      return s3c_udc_probe(&s5pc210_otg_data);
 }
drivers/usb/host/ehci-exynos.c +6 -8
index edd91a8..6fdbf57 100644
@@ -85,15 +85,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 +145,34 @@ 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 |

drivers/usb/host/ehci-exynos.c static void setup_usb_phy(struct exynos_usb_phy *usb) · line 162

Quoted 10 lines
drivers/usb/host/ehci-exynos.c +10 -0
+             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)
+{

Well, I didn't know exactly. but i think this is something strange. is it always bypassed the "struct exynos_usb_phy"?

And don't need to modify the below code into exynos_usb_parse_dt()?

Quoted 9 lines
        /*
         * Get the base address for usbphy from the device node
         */
        exynos->usb = (struct exynos_usb_phy *)fdtdec_get_addr(blob, node,
                                                                "reg");
        if (exynos->usb == NULL) {
                debug("Can't get the usbphy register address\n");
                return -ENXIO;
        }

Well, its a pointer and its cast to (exynos4412_usb_phy *) later on (in function setup_usb_phy()).

If this cast looks funky, we could change exynos->usb to be of type void * and cast it to appropriate (exynos4_usb_phy *) or (exynos4412_us_phy *) as needed.

I do not know which is preferable. As far as I see it, its a pointer, and depending on the SoC we cast it differently.

Please let me know if you have any ideas of how you want to change it for it to be less confusing (if that is indeed the case).

Thanks
- Suriyan

Quoted 71 lines

Best Regards,
Jaehoon Chung

Quoted 66 lines
drivers/usb/host/ehci-exynos.c +31 -0
+     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 (cpu_is_exynos4())
+             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 +192,24 @@ 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 (cpu_is_exynos4())
+             if (proid_is_exynos4412())
+                     exynos4412_reset_usb_phy((struct exynos4412_usb_phy *)
+                                              usb);
 
      set_usbhost_phy_ctrl(POWER_USB_HOST_PHY_CTRL_DISABLE);
 }
include/configs/odroid.h +13 -0
index b928af8..807e96b 100644
@@ -198,6 +198,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
author
#6 lore raw

PATCH v3 3/3 arm: odroid: usb: add support for usb host including ethernet

Quoted 197 lines · Suriyan Ramasami

Hello Jaehoon Chung,

Quoted 177 lines · Jaehoon Chung
Quoted 160 lines · Suriyan Ramasami

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.

Signed-off-by Suriyan Ramasami <[email protected]>

---

Changes in v3:
* removed set_usb_ethaddr() and related code as the GUID registers do not seem to be documented anywhere. This is sad, as this mechanism allows for each Odroid to boot up with the same MAC address every time, but no two odroids shall have the same MAC address on boot. This ensures multiple odroids in the same LAN to come up without conflicting MAC addresses.
* Minkyu - Do not mix cpu_is... and proid_is...

Changes in v2:
* Jaehoon - Split power.[ch] as a separate patch
* Removed an unneeded header file from ehci-exynos.c
* Jaehoon - Fix indentation in the dts file

Changes in v1:
* First try

 arch/arm/dts/exynos4412-odroid.dts      | 11 +++++++
 arch/arm/include/asm/arch-exynos/ehci.h | 13 ++++++++
 board/samsung/odroid/odroid.c           | 32 +++++++++++++++++++
 drivers/usb/host/ehci-exynos.c          | 55 ++++++++++++++++++++++++++++-----
 include/configs/odroid.h                | 13 ++++++++
 5 files changed, 116 insertions(+), 8 deletions(-)
arch/arm/dts/exynos4412-odroid.dts +11 -0
index 4c5e2b3..2df1801 100644
@@ -67,4 +67,15 @@
              div = <0x3>;
              index = <4>;
      };
+
+     ehci at 12580000 {
+             compatible = "samsung,exynos-ehci";
+             reg = <0x12580000 0x100>;
+             #address-cells = <1>;
+             #size-cells = <1>;
+             phy {
+                     compatible = "samsung,exynos-usb-phy";
+                     reg = <0x125B0000 0x100>;
+             };
+     };
 };
arch/arm/include/asm/arch-exynos/ehci.h +13 -0
index d2d70bd..3800fa9 100644
@@ -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);
board/samsung/odroid/odroid.c +32 -0
index 5edb250..d671f13 100644
@@ -453,9 +453,41 @@ 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)
 
 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);
+     }
+
+#endif
+
      debug("USB_udc_probe\n");
      return s3c_udc_probe(&s5pc210_otg_data);
 }
drivers/usb/host/ehci-exynos.c +6 -8
index edd91a8..6fdbf57 100644
@@ -85,15 +85,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 +145,34 @@ 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 |

drivers/usb/host/ehci-exynos.c static void setup_usb_phy(struct exynos_usb_phy *usb) · line 162

Quoted 10 lines
drivers/usb/host/ehci-exynos.c +10 -0
+             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)
+{

Well, I didn't know exactly. but i think this is something strange. is it always bypassed the "struct exynos_usb_phy"?

And don't need to modify the below code into exynos_usb_parse_dt()?

Quoted 9 lines
        /*
         * Get the base address for usbphy from the device node
         */
        exynos->usb = (struct exynos_usb_phy *)fdtdec_get_addr(blob, node,
                                                                "reg");
        if (exynos->usb == NULL) {
                debug("Can't get the usbphy register address\n");
                return -ENXIO;
        }

Well, its a pointer and its cast to (exynos4412_usb_phy *) later on (in function setup_usb_phy()).

If this cast looks funky, we could change exynos->usb to be of type void * and cast it to appropriate (exynos4_usb_phy *) or (exynos4412_us_phy *) as needed.

Quoted 6 lines

I do not know which is preferable. As far as I see it, its a pointer, and depending on the SoC we cast it differently.

Please let me know if you have any ideas of how you want to change it for it to be less confusing (if that is indeed the case).

Another thought is to change struct exynos_usb_phy to be as follows using a union which encompasses the exynos4412 usb register mappings:

/* Register map for PHY control */

struct exynos_usb_phy {
        union {
               unsigned int usbphyctrl0;
               unsigned int usbphyctrl;
        };
        union {
               unsigned int usbphytune0;
               unsigned int usbphyclk;
        };
        union {
               unsigned int reserved1[2];
               unsigned int usbphyrstcon;
        };
        unsigned int hsicphyctrl1;
        unsigned int hsicphytune1;
        unsigned int reserved2[2];
        unsigned int hsicphyctrl2;
        unsigned int hsicphytune2;
        unsigned int reserved3[2];
        unsigned int ehcictrl;
        unsigned int ohcictrl;
        unsigned int usbotgsys;
        unsigned int reserved4;
        unsigned int usbotgtune;
};

This to me, looks more confusing. Another approach would be to rename struct exynos_usb_phy to exynos5_usb_phy as the register mapping is more pertinent to exynos5 SoCs, and change more code to make it more clean.

Your thoughts on this are welcome.
Regards
- Suriyan

Quoted 74 lines

Thanks
- Suriyan

Quoted 71 lines

Best Regards,
Jaehoon Chung

Quoted 66 lines
drivers/usb/host/ehci-exynos.c +31 -0
+     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 (cpu_is_exynos4())
+             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 +192,24 @@ 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 (cpu_is_exynos4())
+             if (proid_is_exynos4412())
+                     exynos4412_reset_usb_phy((struct exynos4412_usb_phy *)
+                                              usb);
 
      set_usbhost_phy_ctrl(POWER_USB_HOST_PHY_CTRL_DISABLE);
 }
include/configs/odroid.h +13 -0
index b928af8..807e96b 100644
@@ -198,6 +198,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
#7 lore raw

PATCH v3 3/3 arm: odroid: usb: add support for usb host including ethernet

In reply to Suriyan Ramasami
Quoted 24 lines · Suriyan Ramasami

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.

Signed-off-by Suriyan Ramasami <[email protected]>

---

Changes in v3:
* removed set_usb_ethaddr() and related code as the GUID registers do not seem to be documented anywhere. This is sad, as this mechanism allows for each Odroid to boot up with the same MAC address every time, but no two odroids shall have the same MAC address on boot. This ensures multiple odroids in the same LAN to come up without conflicting MAC addresses.
* Minkyu - Do not mix cpu_is... and proid_is...

Changes in v2:
* Jaehoon - Split power.[ch] as a separate patch
* Removed an unneeded header file from ehci-exynos.c
* Jaehoon - Fix indentation in the dts file

Changes in v1:
* First try

Quoted 7 lines
 arch/arm/dts/exynos4412-odroid.dts      | 11 +++++++
 arch/arm/include/asm/arch-exynos/ehci.h | 13 ++++++++
 board/samsung/odroid/odroid.c           | 32 +++++++++++++++++++
 drivers/usb/host/ehci-exynos.c          | 55 ++++++++++++++++++++++++++++-----
 include/configs/odroid.h                | 13 ++++++++
 5 files changed, 116 insertions(+), 8 deletions(-)

applied to u-boot-samsung.

Thanks,
Minkyu Kang.

author
#8 lore raw

PATCH v3 3/3 arm: odroid: usb: add support for usb host including ethernet

Hello Minkyu Kang/Przemyslaw and of course anyone else who can direct me,

Thanks for the commit.

I just realized that the code in this particular patch is missing
the gpio_request() call before calling the gpio_direction_output() call, and hence fails to initialize the USB3503A hub. This was pointed out by Przemyslaw in another patch of mine (odroid: blue LED)

So, my question is, this patch is in u-boot-samsung. To correct the
above mentioned error, do I submit a patch directed to u-boot-samsung? If so how do I do it? I mean, do I elaborate this in the subject /.body of the mail?

Thanks and regards,
- Suriyan

Quoted 38 lines · Minkyu Kang
Quoted 24 lines · Suriyan Ramasami

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.

Signed-off-by Suriyan Ramasami <[email protected]>

---

Changes in v3:
* removed set_usb_ethaddr() and related code as the GUID registers do not seem to be documented anywhere. This is sad, as this mechanism allows for each Odroid to boot up with the same MAC address every time, but no two odroids shall have the same MAC address on boot. This ensures multiple odroids in the same LAN to come up without conflicting MAC addresses.
* Minkyu - Do not mix cpu_is... and proid_is...

Changes in v2:
* Jaehoon - Split power.[ch] as a separate patch
* Removed an unneeded header file from ehci-exynos.c
* Jaehoon - Fix indentation in the dts file

Changes in v1:
* First try

Quoted 7 lines
 arch/arm/dts/exynos4412-odroid.dts      | 11 +++++++
 arch/arm/include/asm/arch-exynos/ehci.h | 13 ++++++++
 board/samsung/odroid/odroid.c           | 32 +++++++++++++++++++
 drivers/usb/host/ehci-exynos.c          | 55 ++++++++++++++++++++++++++++-----
 include/configs/odroid.h                | 13 ++++++++
 5 files changed, 116 insertions(+), 8 deletions(-)

applied to u-boot-samsung.

Thanks,
Minkyu Kang.

#9 lore raw

PATCH v3 3/3 arm: odroid: usb: add support for usb host including ethernet

Hi,

Quoted 11 lines · Suriyan Ramasami

Hello Minkyu Kang/Przemyslaw and of course anyone else who can direct me,

Thanks for the commit.

I just realized that the code in this particular patch is missing
the gpio_request() call before calling the gpio_direction_output() call, and hence fails to initialize the USB3503A hub. This was pointed out by Przemyslaw in another patch of mine (odroid: blue LED)

So, my question is, this patch is in u-boot-samsung. To correct the
above mentioned error, do I submit a patch directed to u-boot-samsung?

Yes. Please send new patch.

Quoted 2 lines

If so how do I do it? I mean, do I elaborate this in the subject /.body of the mail?

sorry, I can't catch the point of your question.

Quoted 3 lines

Thanks and regards,
- Suriyan

Thanks,
Minkyu Kang.

#10 lore raw

PATCH v3 3/3 arm: odroid: usb: add support for usb host including ethernet

Dear Minkyu,

Quoted 20 lines · Minkyu Kang

Hi,

Quoted 11 lines · Suriyan Ramasami

Hello Minkyu Kang/Przemyslaw and of course anyone else who can direct me,

Thanks for the commit.

I just realized that the code in this particular patch is missing
the gpio_request() call before calling the gpio_direction_output() call, and hence fails to initialize the USB3503A hub. This was pointed out by Przemyslaw in another patch of mine (odroid: blue LED)

So, my question is, this patch is in u-boot-samsung. To correct the
above mentioned error, do I submit a patch directed to u-boot-samsung?

Yes. Please send new patch.

Quoted 2 lines

If so how do I do it? I mean, do I elaborate this in the subject /.body of the mail?

Quoted 10 lines

sorry, I can't catch the point of your question.

Quoted 3 lines

Thanks and regards,
- Suriyan

Thanks,
Minkyu Kang.

Could you please wait for the test/ack next time before you merge the patches to the tree? It will prevent before making an additional work on testing or fixing some code.

Best regards,

Signature · 4 lines
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com
#11 lore raw

PATCH v3 3/3 arm: odroid: usb: add support for usb host including ethernet

Dear Przemyslaw Marczak,

Quoted 29 lines · Przemyslaw Marczak

Dear Minkyu,

Quoted 26 lines · Minkyu Kang

Hi,

Quoted 11 lines · Suriyan Ramasami

Hello Minkyu Kang/Przemyslaw and of course anyone else who can direct me,

Thanks for the commit.

I just realized that the code in this particular patch is missing
the gpio_request() call before calling the gpio_direction_output() call, and hence fails to initialize the USB3503A hub. This was pointed out by Przemyslaw in another patch of mine (odroid: blue LED)

So, my question is, this patch is in u-boot-samsung. To correct the
above mentioned error, do I submit a patch directed to u-boot-samsung?

Yes. Please send new patch.

Quoted 2 lines

If so how do I do it? I mean, do I elaborate this in the subject /.body of the mail?

sorry, I can't catch the point of your question.

Quoted 3 lines

Thanks and regards,
- Suriyan

Quoted 8 lines
Quoted 4 lines

Thanks,
Minkyu Kang.

Could you please wait for the test/ack next time before you merge the patches to the tree? It will prevent before making an additional work on testing or fixing some code.

Best regards,

It's my bad that I didn't check it carefully. I usually wait somebody's response.
But this patch is waited 18 days on the queue. How long will I wait?

Thanks,
Minkyu Kang.

#12 lore raw

PATCH v3 3/3 arm: odroid: usb: add support for usb host including ethernet

Hello Minkyu,

Quoted 41 lines · Minkyu Kang

Dear Przemyslaw Marczak,

Quoted 29 lines · Przemyslaw Marczak

Dear Minkyu,

Quoted 26 lines · Minkyu Kang

Hi,

Quoted 11 lines · Suriyan Ramasami

Hello Minkyu Kang/Przemyslaw and of course anyone else who can direct me,

Thanks for the commit.

I just realized that the code in this particular patch is missing
the gpio_request() call before calling the gpio_direction_output() call, and hence fails to initialize the USB3503A hub. This was pointed out by Przemyslaw in another patch of mine (odroid: blue LED)

So, my question is, this patch is in u-boot-samsung. To correct the
above mentioned error, do I submit a patch directed to u-boot-samsung?

Yes. Please send new patch.

Quoted 2 lines

If so how do I do it? I mean, do I elaborate this in the subject /.body of the mail?

sorry, I can't catch the point of your question.

Quoted 3 lines

Thanks and regards,
- Suriyan

Quoted 8 lines
Quoted 4 lines

Thanks,
Minkyu Kang.

Could you please wait for the test/ack next time before you merge the patches to the tree? It will prevent before making an additional work on testing or fixing some code.

Best regards,

Quoted 8 lines

It's my bad that I didn't check it carefully. I usually wait somebody's response.
But this patch is waited 18 days on the queue. How long will I wait?

Thanks,
Minkyu Kang.

It's easy to miss something without test on a hardware and this is the reason why the code should be tested before the merge.

The 18 days is quite long, I agree with you - but is this a reason for those patches to be merged, without the test?

You are a busy person at work, the same as I am, but I hope that "ping" somebody to check the code is a good manner.

And then I believe, we can provide stable and tested master tree as a maintainers of our boards.

Best regards,

Signature · 4 lines
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com
#13 lore raw

PATCH v3 3/3 arm: odroid: usb: add support for usb host including ethernet

In reply to Suriyan Ramasami

Hello Suriyan,

Quoted 9 lines · Suriyan Ramasami

Hello Minkyu Kang/Przemyslaw and of course anyone else who can direct me,

Thanks for the commit.

I just realized that the code in this particular patch is missing
the gpio_request() call before calling the gpio_direction_output() call, and hence fails to initialize the USB3503A hub. This was pointed out by Przemyslaw in another patch of mine (odroid: blue LED)

Quoted 9 lines

So, my question is, this patch is in u-boot-samsung. To correct the
above mentioned error, do I submit a patch directed to u-boot-samsung? If so how do I do it? I mean, do I elaborate this in the subject /.body of the mail?

Thanks and regards,
- Suriyan

Yes, the gpio_request() was missed in your code - it prints errors about it but the HUB works fine. So when you send the fixed version this should be okay. Please check ums command before send.

Please also check setting buck value at function board_usb_init(), the buck is disabled and before enable - the value changes two times - probably by mistake.

And the gpio_request(...) you can put into board_gpio_init - request gpio only one.

The odroid is available on the market, so I would prefer keep some documentation in the doc/README.odroid.
Could you please add short section about the USB usage in U-Boot? This would be useful.

Quoted 34 lines · Minkyu Kang
Quoted 33 lines · Suriyan Ramasami
Quoted 31 lines

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.

Signed-off-by Suriyan Ramasami <[email protected]>

---

Changes in v3:
* removed set_usb_ethaddr() and related code as the GUID registers do not seem to be documented anywhere. This is sad, as this mechanism allows for each Odroid to boot up with the same MAC address every time, but no two odroids shall have the same MAC address on boot. This ensures multiple odroids in the same LAN to come up without conflicting MAC addresses.
* Minkyu - Do not mix cpu_is... and proid_is...

Changes in v2:
* Jaehoon - Split power.[ch] as a separate patch
* Removed an unneeded header file from ehci-exynos.c
* Jaehoon - Fix indentation in the dts file

Changes in v1:
* First try

  arch/arm/dts/exynos4412-odroid.dts      | 11 +++++++
  arch/arm/include/asm/arch-exynos/ehci.h | 13 ++++++++
  board/samsung/odroid/odroid.c           | 32 +++++++++++++++++++
  drivers/usb/host/ehci-exynos.c          | 55 ++++++++++++++++++++++++++++-----
  include/configs/odroid.h                | 13 ++++++++
  5 files changed, 116 insertions(+), 8 deletions(-)
Quoted 6 lines
Quoted 5 lines

applied to u-boot-samsung.

Thanks,
Minkyu Kang.

Best regards,

Signature · 4 lines
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com
author
#14 lore raw

PATCH v3 3/3 arm: odroid: usb: add support for usb host including ethernet

Hello Przemyslaw,

Quoted 18 lines · Przemyslaw Marczak

Hello Suriyan,

Quoted 15 lines · Suriyan Ramasami

Hello Minkyu Kang/Przemyslaw and of course anyone else who can direct me,

Thanks for the commit.

I just realized that the code in this particular patch is missing
the gpio_request() call before calling the gpio_direction_output() call, and hence fails to initialize the USB3503A hub. This was pointed out by Przemyslaw in another patch of mine (odroid: blue LED)

So, my question is, this patch is in u-boot-samsung. To correct the
above mentioned error, do I submit a patch directed to u-boot-samsung? If so how do I do it? I mean, do I elaborate this in the subject /.body of the mail?

Quoted 9 lines
Quoted 4 lines

Thanks and regards,
- Suriyan

Yes, the gpio_request() was missed in your code - it prints errors about it but the HUB works fine. So when you send the fixed version this should be okay. Please check ums command before send.

Sure, will do!

Quoted 4 lines

Please also check setting buck value at function board_usb_init(), the buck is disabled and before enable - the value changes two times - probably by mistake.

I think there is an error in the implementation of max77686_set_buck_mode(..., buck #, OPMODE_OFF), or I have some gap in my understanding of it. I believe that when you set OPMODE_OFF it should turn it OFF right? as in no voltage? This does not seem to be the case.
I have to turn the power off and then on, on BUCK8 (LAN9730 feeds off of it), to reset the LAN when a usb reset is issued, else it will not be detected again when usb start is re-issued.
Initially, I had max77686_set_buck_mode(p_pmic, 9, OPMODE_OFF); delay; max77686(set_buck_mode(p_pmic, 9, OPMODE_ON); but it did not power recycle the LAN9730 chip.
I do not have access to the MAX77686 data sheets to debug this. Hence, in my current code I set the voltage to the lowest possible value, and then bring it back up to 3.3V to simulate a reset. In fact, I can remove the OPMODE_OFF and OPMODE_ON code, and it works. Please let me know if there is any change needed in the OPMODE_OFF code for it to work the way it should, or correct my understanding of what it is supposed to do.

Currently, I am thinking of just having:

                max77686_set_buck_voltage(p_pmic, 8, 750000);
                max77686_set_buck_voltage(p_pmic, 8, 3300000);

and leave the OPMODE_OFF and OPMODE_ON out of it.

Quoted 3 lines

And the gpio_request(...) you can put into board_gpio_init - request gpio only one.

Yes.

Quoted 4 lines

The odroid is available on the market, so I would prefer keep some documentation in the doc/README.odroid.
Could you please add short section about the USB usage in U-Boot? This would be useful.

OK!

Quoted 56 lines
Quoted 43 lines · Minkyu Kang
Quoted 40 lines · Suriyan Ramasami
Quoted 37 lines

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.

Signed-off-by Suriyan Ramasami <[email protected]>

---

Changes in v3:
* removed set_usb_ethaddr() and related code as the GUID registers do not
seem to be documented anywhere. This is sad, as this mechanism allows for each Odroid to boot up with the same MAC address every time, but
no
two odroids shall have the same MAC address on boot. This ensures
multiple
odroids in the same LAN to come up without conflicting MAC addresses.
* Minkyu - Do not mix cpu_is... and proid_is...

Changes in v2:
* Jaehoon - Split power.[ch] as a separate patch
* Removed an unneeded header file from ehci-exynos.c
* Jaehoon - Fix indentation in the dts file

Changes in v1:
* First try

  arch/arm/dts/exynos4412-odroid.dts      | 11 +++++++
  arch/arm/include/asm/arch-exynos/ehci.h | 13 ++++++++
  board/samsung/odroid/odroid.c           | 32 +++++++++++++++++++
  drivers/usb/host/ehci-exynos.c          | 55
++++++++++++++++++++++++++++-----
  include/configs/odroid.h                | 13 ++++++++
  5 files changed, 116 insertions(+), 8 deletions(-)
Quoted 6 lines
Quoted 5 lines

applied to u-boot-samsung.

Thanks,
Minkyu Kang.

Best regards,
--
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com

#15 lore raw

PATCH v3 3/3 arm: odroid: usb: add support for usb host including ethernet

Hello Suriyan,

Quoted 58 lines · Suriyan Ramasami

Hello Przemyslaw,

Quoted 18 lines · Przemyslaw Marczak

Hello Suriyan,

Quoted 15 lines · Suriyan Ramasami

Hello Minkyu Kang/Przemyslaw and of course anyone else who can direct me,

Thanks for the commit.

I just realized that the code in this particular patch is missing
the gpio_request() call before calling the gpio_direction_output() call, and hence fails to initialize the USB3503A hub. This was pointed out by Przemyslaw in another patch of mine (odroid: blue LED)

So, my question is, this patch is in u-boot-samsung. To correct the
above mentioned error, do I submit a patch directed to u-boot-samsung? If so how do I do it? I mean, do I elaborate this in the subject /.body of the mail?

Quoted 9 lines
Quoted 4 lines

Thanks and regards,
- Suriyan

Yes, the gpio_request() was missed in your code - it prints errors about it but the HUB works fine. So when you send the fixed version this should be okay. Please check ums command before send.

Sure, will do!

Quoted 4 lines

Please also check setting buck value at function board_usb_init(), the buck is disabled and before enable - the value changes two times - probably by mistake.

I think there is an error in the implementation of max77686_set_buck_mode(..., buck #, OPMODE_OFF), or I have some gap in my understanding of it. I believe that when you set OPMODE_OFF it should turn it OFF right? as in no voltage? This does not seem to be the case.
I have to turn the power off and then on, on BUCK8 (LAN9730 feeds off of it), to reset the LAN when a usb reset is issued, else it will not be detected again when usb start is re-issued.
Initially, I had max77686_set_buck_mode(p_pmic, 9, OPMODE_OFF); delay; max77686(set_buck_mode(p_pmic, 9, OPMODE_ON); but it did not power recycle the LAN9730 chip.
I do not have access to the MAX77686 data sheets to debug this. Hence, in my current code I set the voltage to the lowest possible value, and then bring it back up to 3.3V to simulate a reset. In fact, I can remove the OPMODE_OFF and OPMODE_ON code, and it works. Please let me know if there is any change needed in the OPMODE_OFF code for it to work the way it should, or correct my understanding of what it is supposed to do.

Quoted 5 lines

Currently, I am thinking of just having:

                 max77686_set_buck_voltage(p_pmic, 8, 750000);
                 max77686_set_buck_voltage(p_pmic, 8, 3300000);

and leave the OPMODE_OFF and OPMODE_ON out of it.

Ok, now I see the issue. So I looked into the documentation and there is a "ENB8" pin in PMIC package - this pin allows steering BUCK8 ON/OFF by the hardware. If ENB8 is set to "low", then you can do on/off - when "high" then you can't change it's state by I2C write. This PMIC has a lot of such gpio for setting the default power-on state of ldo/buck regulators.

The signal of ENB8 in OdroidU3 schematic is missed, I can suppose that it is connected to one of SOC RST signal as it is on Odroid X2.

So doing the reset by setting low/high voltage value - seems to be good in this case.

Quoted 4 lines
Quoted 3 lines

And the gpio_request(...) you can put into board_gpio_init - request gpio only one.

Quoted 65 lines

Yes.

Quoted 4 lines

The odroid is available on the market, so I would prefer keep some documentation in the doc/README.odroid.
Could you please add short section about the USB usage in U-Boot? This would be useful.

OK!

Quoted 56 lines
Quoted 43 lines · Minkyu Kang
Quoted 40 lines · Suriyan Ramasami
Quoted 37 lines

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.

Signed-off-by Suriyan Ramasami <[email protected]>

---

Changes in v3:
* removed set_usb_ethaddr() and related code as the GUID registers do not
seem to be documented anywhere. This is sad, as this mechanism allows for each Odroid to boot up with the same MAC address every time, but
no
two odroids shall have the same MAC address on boot. This ensures
multiple
odroids in the same LAN to come up without conflicting MAC addresses.
* Minkyu - Do not mix cpu_is... and proid_is...

Changes in v2:
* Jaehoon - Split power.[ch] as a separate patch
* Removed an unneeded header file from ehci-exynos.c
* Jaehoon - Fix indentation in the dts file

Changes in v1:
* First try

   arch/arm/dts/exynos4412-odroid.dts      | 11 +++++++
   arch/arm/include/asm/arch-exynos/ehci.h | 13 ++++++++
   board/samsung/odroid/odroid.c           | 32 +++++++++++++++++++
   drivers/usb/host/ehci-exynos.c          | 55
++++++++++++++++++++++++++++-----
   include/configs/odroid.h                | 13 ++++++++
   5 files changed, 116 insertions(+), 8 deletions(-)
Quoted 6 lines
Quoted 5 lines

applied to u-boot-samsung.

Thanks,
Minkyu Kang.

Best regards,
--
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com

Best regards,

Signature · 4 lines
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com
#16 lore raw

PATCH v3 1/3 arm: odroid: pmic77686: allow buck voltage settings

In reply to Suriyan Ramasami
Quoted 15 lines · Suriyan Ramasami

Allow to set the buck voltage for the max77686.
This will be used to reset the SMC LAN9730 ethernet on the odroids.

Signed-off-by Suriyan Ramasami <[email protected]>

---

Changes in v3:
* Correct ldo and buck validation logic
* Jaehoon/Przemyslaw - Use negative errno values for error condistions
* Albert - its buck not bucket

Changes in v2:
* Jaehoon - separate out this patch

Quoted 7 lines

Changes in v1:
- First try

 drivers/power/pmic/pmic_max77686.c | 52 +++++++++++++++++++++++++++++++++++---
 include/power/max77686_pmic.h      |  3 +++
 2 files changed, 52 insertions(+), 3 deletions(-)

applied to u-boot-samsung.

Thanks,
Minkyu Kang.