All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH 0/4 v2] AM35X: Add musb support
@ 2010-07-09  6:13 Ajay Kumar Gupta
  2010-07-09  6:13 ` [U-Boot] [PATCH 1/4 v2] AM35x: Adding SCM general register definitions Ajay Kumar Gupta
                   ` (2 more replies)
  0 siblings, 3 replies; 14+ messages in thread
From: Ajay Kumar Gupta @ 2010-07-09  6:13 UTC (permalink / raw)
  To: u-boot

Remy,

These patches add support for musb in AM35x.
Changes from v1:
	- Moved SCM definitions to am35x specific file am35x_def.h

Patches have been created against uboot-ti/master tree (as on 9th July)
but they also apply cleanly to both denx/master and uboot-usb/master
as there is no interdependent changes.

Regards,
Ajay

Ajay Kumar Gupta (4):
  AM35x: Adding SCM general register definitions
  musb: MSC host support for AM35x
  musb: am35x: Workaround for fifo read issue
  AM3517EVM: musb: add usb config

 arch/arm/include/asm/arch-omap3/am35x_def.h |   52 +++++++++
 drivers/usb/musb/Makefile                   |    1 +
 drivers/usb/musb/am35x.c                    |  150 +++++++++++++++++++++++++++
 drivers/usb/musb/am35x.h                    |   94 +++++++++++++++++
 drivers/usb/musb/musb_core.c                |    6 +
 drivers/usb/musb/musb_udc.c                 |    2 +
 include/configs/am3517_evm.h                |   38 +++++++
 include/usb.h                               |    2 +-
 8 files changed, 344 insertions(+), 1 deletions(-)
 create mode 100644 arch/arm/include/asm/arch-omap3/am35x_def.h
 create mode 100644 drivers/usb/musb/am35x.c
 create mode 100644 drivers/usb/musb/am35x.h

^ permalink raw reply	[flat|nested] 14+ messages in thread

* [U-Boot] [PATCH 1/4 v2] AM35x: Adding SCM general register definitions
  2010-07-09  6:13 [U-Boot] [PATCH 0/4 v2] AM35X: Add musb support Ajay Kumar Gupta
@ 2010-07-09  6:13 ` Ajay Kumar Gupta
  2010-07-09  6:13   ` [U-Boot] [PATCH 2/4 v2] musb: MSC host support for AM35x Ajay Kumar Gupta
  2010-07-16  4:44 ` [U-Boot] [PATCH 0/4 v2] AM35X: Add musb support Gupta, Ajay Kumar
  2010-08-02  8:18 ` Remy Bohmer
  2 siblings, 1 reply; 14+ messages in thread
From: Ajay Kumar Gupta @ 2010-07-09  6:13 UTC (permalink / raw)
  To: u-boot

Adding general register structure of system control module (SCM)
of AM35x. This would be required to access devconf2 and ip_sw_reset
register in musb module.

Signed-off-by: Ajay Kumar Gupta <ajay.gupta@ti.com>
---
 arch/arm/include/asm/arch-omap3/am35x_def.h |   52 +++++++++++++++++++++++++++
 1 files changed, 52 insertions(+), 0 deletions(-)
 create mode 100644 arch/arm/include/asm/arch-omap3/am35x_def.h

diff --git a/arch/arm/include/asm/arch-omap3/am35x_def.h b/arch/arm/include/asm/arch-omap3/am35x_def.h
new file mode 100644
index 0000000..81942a8
--- /dev/null
+++ b/arch/arm/include/asm/arch-omap3/am35x_def.h
@@ -0,0 +1,52 @@
+/*
+ * am35x_def.h - TI's AM35x specific definitions.
+ *
+ * Based on arch/arm/include/asm/arch-omap3/cpu.h
+ *
+ * Author: Ajay Kumar Gupta <ajay.gupta@ti.com>
+ *
+ * Copyright (c) 2010 Texas Instruments Incorporated
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+ */
+
+#ifndef _AM35X_DEF_H_
+#define _AM35X_DEF_H_
+
+#if !(defined(__KERNEL_STRICT_NAMES) || defined(__ASSEMBLY__))
+#include <asm/types.h>
+#endif /* !(__KERNEL_STRICT_NAMES || __ASSEMBLY__) */
+
+#ifndef __KERNEL_STRICT_NAMES
+#ifndef __ASSEMBLY__
+
+/* General register mappings of system control module */
+#define AM35X_SCM_GEN_BASE	0x48002270
+struct am35x_scm_general {
+	u32 res1[0xC4];		/* 0x000 - 0x30C */
+	u32 devconf2;		/* 0x310 */
+	u32 devconf3;		/* 0x314 */
+	u32 res2[0x2];		/* 0x318 - 0x31C */
+	u32 cba_priority;	/* 0x320 */
+	u32 lvl_intr_clr;	/* 0x324 */
+	u32 ip_sw_reset;	/* 0x328 */
+	u32 ipss_clk_ctrl;	/* 0x32C */
+};
+#define am35x_scm_general_regs ((struct am35x_scm_general *)AM35X_SCM_GEN_BASE)
+
+#endif /*__ASSEMBLY__ */
+#endif /* __KERNEL_STRICT_NAMES */
+
+#endif /* _AM35X_DEF_H_ */
-- 
1.6.2.4

^ permalink raw reply related	[flat|nested] 14+ messages in thread

* [U-Boot] [PATCH 2/4 v2] musb: MSC host support for AM35x
  2010-07-09  6:13 ` [U-Boot] [PATCH 1/4 v2] AM35x: Adding SCM general register definitions Ajay Kumar Gupta
@ 2010-07-09  6:13   ` Ajay Kumar Gupta
  2010-07-09  6:13     ` [U-Boot] [PATCH 3/4 v2] musb: am35x: Workaround for fifo read issue Ajay Kumar Gupta
  2010-08-01 11:16     ` [U-Boot] [PATCH 2/4 v2] musb: MSC host support for AM35x Remy Bohmer
  0 siblings, 2 replies; 14+ messages in thread
From: Ajay Kumar Gupta @ 2010-07-09  6:13 UTC (permalink / raw)
  To: u-boot

Tested MSC Host on AM3517EVM.

Signed-off-by: Ajay Kumar Gupta <ajay.gupta@ti.com>
---
 drivers/usb/musb/Makefile   |    1 +
 drivers/usb/musb/am35x.c    |  118 +++++++++++++++++++++++++++++++++++++++++++
 drivers/usb/musb/am35x.h    |   94 ++++++++++++++++++++++++++++++++++
 drivers/usb/musb/musb_udc.c |    2 +
 include/usb.h               |    2 +-
 5 files changed, 216 insertions(+), 1 deletions(-)
 create mode 100644 drivers/usb/musb/am35x.c
 create mode 100644 drivers/usb/musb/am35x.h

diff --git a/drivers/usb/musb/Makefile b/drivers/usb/musb/Makefile
index 397f5fe..7d23e06 100644
--- a/drivers/usb/musb/Makefile
+++ b/drivers/usb/musb/Makefile
@@ -31,6 +31,7 @@ COBJS-$(CONFIG_USB_BLACKFIN) += blackfin_usb.o
 COBJS-$(CONFIG_USB_DAVINCI) += davinci.o
 COBJS-$(CONFIG_USB_OMAP3) += omap3.o
 COBJS-$(CONFIG_USB_DA8XX) += da8xx.o
+COBJS-$(CONFIG_USB_AM35X) += am35x.o
 
 COBJS	:= $(COBJS-y)
 SRCS	:= $(COBJS:.o=.c)
diff --git a/drivers/usb/musb/am35x.c b/drivers/usb/musb/am35x.c
new file mode 100644
index 0000000..2024940
--- /dev/null
+++ b/drivers/usb/musb/am35x.c
@@ -0,0 +1,118 @@
+/*
+ * am35x.c - TI's AM35x platform specific usb wrapper functions.
+ *
+ * Author: Ajay Kumar Gupta <ajay.gupta@ti.com>
+ *
+ * Based on drivers/usb/musb/da8xx.c
+ *
+ * Copyright (c) 2010 Texas Instruments Incorporated
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+ */
+#include <common.h>
+
+#include "am35x.h"
+
+/* MUSB platform configuration */
+struct musb_config musb_cfg = {
+	.regs		= (struct musb_regs *)AM35X_USB_OTG_CORE_BASE,
+	.timeout	= AM35X_USB_OTG_TIMEOUT,
+	.musb_speed	= 0,
+};
+
+/*
+ * Enable the USB phy
+ */
+static u8 phy_on(void)
+{
+	u32 devconf2;
+	u32 timeout;
+
+	devconf2 = readl(&am35x_scm_general_regs->devconf2);
+
+	devconf2 &= ~(DEVCONF2_RESET | DEVCONF2_PHYPWRDN | DEVCONF2_OTGPWRDN |
+		      DEVCONF2_OTGMODE | DEVCONF2_REFFREQ |
+		      DEVCONF2_PHY_GPIOMODE);
+	devconf2 |= DEVCONF2_SESENDEN | DEVCONF2_VBDTCTEN | DEVCONF2_PHY_PLLON |
+		    DEVCONF2_REFFREQ_13MHZ | DEVCONF2_DATPOL;
+
+	writel(devconf2, &am35x_scm_general_regs->devconf2);
+
+	/* wait until the USB phy is turned on */
+	timeout = musb_cfg.timeout;
+	while (timeout--)
+		if (readl(&am35x_scm_general_regs->devconf2) & DEVCONF2_PHYCKGD)
+			return 1;
+
+	/* USB phy was not turned on */
+	return 0;
+}
+
+/*
+ * Disable the USB phy
+ */
+static void phy_off(void)
+{
+	u32 devconf2;
+
+	/*
+	 * Power down the on-chip PHY.
+	 */
+	devconf2 = readl(&am35x_scm_general_regs->devconf2);
+
+	devconf2 &= ~DEVCONF2_PHY_PLLON;
+	devconf2 |= DEVCONF2_PHYPWRDN | DEVCONF2_OTGPWRDN;
+	writel(devconf2, &am35x_scm_general_regs->devconf2);
+}
+
+/*
+ * This function performs platform specific initialization for usb0.
+ */
+int musb_platform_init(void)
+{
+	u32 revision;
+	u32 sw_reset;
+
+	/* global usb reset */
+	sw_reset = readl(&am35x_scm_general_regs->ip_sw_reset);
+	sw_reset |= (1 << 0);
+	writel(sw_reset, &am35x_scm_general_regs->ip_sw_reset);
+	sw_reset &= ~(1 << 0);
+	writel(sw_reset, &am35x_scm_general_regs->ip_sw_reset);
+
+	/* reset the controller */
+	writel(0x1, &am35x_usb_regs->control);
+	udelay(5000);
+
+	/* start the on-chip usb phy and its pll */
+	if (phy_on() == 0)
+		return -1;
+
+	/* Returns zero if e.g. not clocked */
+	revision = readl(&am35x_usb_regs->revision);
+	if (revision == 0)
+		return -1;
+
+	return 0;
+}
+
+/*
+ * This function performs platform specific deinitialization for usb0.
+ */
+void musb_platform_deinit(void)
+{
+	/* Turn off the phy */
+	phy_off();
+}
diff --git a/drivers/usb/musb/am35x.h b/drivers/usb/musb/am35x.h
new file mode 100644
index 0000000..756c3ae
--- /dev/null
+++ b/drivers/usb/musb/am35x.h
@@ -0,0 +1,94 @@
+/*
+ * am35x.h - TI's AM35x platform specific usb wrapper definitions.
+ *
+ * Author: Ajay Kumar Gupta <ajay.gupta@ti.com>
+ *
+ * Based on drivers/usb/musb/da8xx.h
+ *
+ * Copyright (c) 2010 Texas Instruments Incorporated
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+ */
+
+#ifndef __AM35X_USB_H__
+#define __AM35X_USB_H__
+
+#include <asm/arch/am35x_def.h>
+#include "musb_core.h"
+
+/* Base address of musb wrapper */
+#define AM35X_USB_OTG_BASE	0x5C040000
+
+/* Base address of musb core */
+#define AM35X_USB_OTG_CORE_BASE	(AM35X_USB_OTG_BASE + 0x400)
+
+/* Timeout for AM35x usb module */
+#define AM35X_USB_OTG_TIMEOUT	0x3FFFFFF
+
+/*
+ * AM35x platform USB wrapper register overlay.
+ */
+struct am35x_usb_regs {
+	u32	revision;
+	u32	control;
+	u32	status;
+	u32	emulation;
+	u32	reserved0[1];
+	u32	autoreq;
+	u32	srpfixtime;
+	u32	ep_intsrc;
+	u32	ep_intsrcset;
+	u32	ep_intsrcclr;
+	u32	ep_intmsk;
+	u32	ep_intmskset;
+	u32	ep_intmskclr;
+	u32	ep_intsrcmsked;
+	u32	reserved1[1];
+	u32	core_intsrc;
+	u32	core_intsrcset;
+	u32	core_intsrcclr;
+	u32	core_intmsk;
+	u32	core_intmskset;
+	u32	core_intmskclr;
+	u32	core_intsrcmsked;
+	u32	reserved2[1];
+	u32	eoi;
+	u32	mop_sop_en;
+	u32	reserved3[2];
+	u32	txmode;
+	u32	rxmode;
+	u32	epcount_mode;
+};
+
+#define am35x_usb_regs ((struct am35x_usb_regs *)AM35X_USB_OTG_BASE)
+
+/* USB 2.0 PHY Control */
+#define DEVCONF2_PHY_GPIOMODE	(1 << 23)
+#define DEVCONF2_OTGMODE	(3 << 14)
+#define DEVCONF2_SESENDEN	(1 << 13)       /* Vsess_end comparator */
+#define DEVCONF2_VBDTCTEN	(1 << 12)       /* Vbus comparator */
+#define DEVCONF2_REFFREQ_24MHZ	(2 << 8)
+#define DEVCONF2_REFFREQ_26MHZ	(7 << 8)
+#define DEVCONF2_REFFREQ_13MHZ	(6 << 8)
+#define DEVCONF2_REFFREQ	(0xf << 8)
+#define DEVCONF2_PHYCKGD	(1 << 7)
+#define DEVCONF2_VBUSSENSE	(1 << 6)
+#define DEVCONF2_PHY_PLLON	(1 << 5)        /* override PLL suspend */
+#define DEVCONF2_RESET		(1 << 4)
+#define DEVCONF2_PHYPWRDN	(1 << 3)
+#define DEVCONF2_OTGPWRDN	(1 << 2)
+#define DEVCONF2_DATPOL		(1 << 1)
+
+#endif	/* __AM35X_USB_H__ */
diff --git a/drivers/usb/musb/musb_udc.c b/drivers/usb/musb/musb_udc.c
index fc43cf4..6f6ed61 100644
--- a/drivers/usb/musb/musb_udc.c
+++ b/drivers/usb/musb/musb_udc.c
@@ -57,6 +57,8 @@
 #include "musb_core.h"
 #if defined(CONFIG_USB_OMAP3)
 #include "omap3.h"
+#elif defined(CONFIG_USB_AM35X)
+#include "am35x.h"
 #elif defined(CONFIG_USB_DAVINCI)
 #include "davinci.h"
 #endif
diff --git a/include/usb.h b/include/usb.h
index bc4ccfe..afd65e3 100644
--- a/include/usb.h
+++ b/include/usb.h
@@ -133,7 +133,7 @@ struct usb_device {
 	defined(CONFIG_USB_SL811HS) || defined(CONFIG_USB_ISP116X_HCD) || \
 	defined(CONFIG_USB_R8A66597_HCD) || defined(CONFIG_USB_DAVINCI) || \
 	defined(CONFIG_USB_OMAP3) || defined(CONFIG_USB_DA8XX) || \
-	defined(CONFIG_USB_BLACKFIN)
+	defined(CONFIG_USB_BLACKFIN) || defined(CONFIG_USB_AM35X)
 
 int usb_lowlevel_init(void);
 int usb_lowlevel_stop(void);
-- 
1.6.2.4

^ permalink raw reply related	[flat|nested] 14+ messages in thread

* [U-Boot] [PATCH 3/4 v2] musb: am35x: Workaround for fifo read issue
  2010-07-09  6:13   ` [U-Boot] [PATCH 2/4 v2] musb: MSC host support for AM35x Ajay Kumar Gupta
@ 2010-07-09  6:13     ` Ajay Kumar Gupta
  2010-07-09  6:13       ` [U-Boot] [PATCH 4/4] AM3517EVM: musb: add usb config Ajay Kumar Gupta
  2010-08-01 11:16       ` [U-Boot] [PATCH 3/4 v2] musb: am35x: Workaround for fifo read issue Remy Bohmer
  2010-08-01 11:16     ` [U-Boot] [PATCH 2/4 v2] musb: MSC host support for AM35x Remy Bohmer
  1 sibling, 2 replies; 14+ messages in thread
From: Ajay Kumar Gupta @ 2010-07-09  6:13 UTC (permalink / raw)
  To: u-boot

AM35x supports only 32bit read operations so we need to have
workaround for 8bit and 16bit read operations.

Signed-off-by: Ajay Kumar Gupta <ajay.gupta@ti.com>
---
 drivers/usb/musb/am35x.c     |   32 ++++++++++++++++++++++++++++++++
 drivers/usb/musb/musb_core.c |    6 ++++++
 2 files changed, 38 insertions(+), 0 deletions(-)

diff --git a/drivers/usb/musb/am35x.c b/drivers/usb/musb/am35x.c
index 2024940..1706c13 100644
--- a/drivers/usb/musb/am35x.c
+++ b/drivers/usb/musb/am35x.c
@@ -116,3 +116,35 @@ void musb_platform_deinit(void)
 	/* Turn off the phy */
 	phy_off();
 }
+
+/*
+ * This function reads data from endpoint fifo for AM35x
+ * which supports only 32bit read operation.
+ *
+ * ep           - endpoint number
+ * length       - number of bytes to read from FIFO
+ * fifo_data    - pointer to data buffer into which data is read
+ */
+__attribute__((weak))
+void read_fifo(u8 ep, u32 length, void *fifo_data)
+{
+	u8  *data = (u8 *)fifo_data;
+	u32 val;
+	int i;
+
+	/* select the endpoint index */
+	writeb(ep, &musbr->index);
+
+	if (length > 4) {
+		for (i = 0; i < (length >> 2); i++) {
+			val = readl(&musbr->fifox[ep]);
+			memcpy(data, &val, 4);
+			data += 4;
+		}
+		length %= 4;
+	}
+	if (length > 0) {
+		val = readl(&musbr->fifox[ep]);
+		memcpy(data, &val, length);
+	}
+}
diff --git a/drivers/usb/musb/musb_core.c b/drivers/usb/musb/musb_core.c
index dc740cf..6fe2c39 100644
--- a/drivers/usb/musb/musb_core.c
+++ b/drivers/usb/musb/musb_core.c
@@ -142,6 +142,11 @@ void write_fifo(u8 ep, u32 length, void *fifo_data)
 }
 
 /*
+ * AM35x supports only 32bit read operations so
+ * use seperate read_fifo() function for it.
+ */
+#ifndef CONFIG_USB_AM35X
+/*
  * This function reads data from endpoint fifo
  *
  * ep           - endpoint number
@@ -160,3 +165,4 @@ void read_fifo(u8 ep, u32 length, void *fifo_data)
 	while (length--)
 		*data++ = readb(&musbr->fifox[ep]);
 }
+#endif /* CONFIG_USB_AM35X */
-- 
1.6.2.4

^ permalink raw reply related	[flat|nested] 14+ messages in thread

* [U-Boot] [PATCH 4/4] AM3517EVM: musb: add usb config
  2010-07-09  6:13     ` [U-Boot] [PATCH 3/4 v2] musb: am35x: Workaround for fifo read issue Ajay Kumar Gupta
@ 2010-07-09  6:13       ` Ajay Kumar Gupta
  2010-08-01 11:16       ` [U-Boot] [PATCH 3/4 v2] musb: am35x: Workaround for fifo read issue Remy Bohmer
  1 sibling, 0 replies; 14+ messages in thread
From: Ajay Kumar Gupta @ 2010-07-09  6:13 UTC (permalink / raw)
  To: u-boot

Enabling USB HOST in defconfig.

Signed-off-by: Ajay Kumar Gupta <ajay.gupta@ti.com>
---
 include/configs/am3517_evm.h |   38 ++++++++++++++++++++++++++++++++++++++
 1 files changed, 38 insertions(+), 0 deletions(-)

diff --git a/include/configs/am3517_evm.h b/include/configs/am3517_evm.h
index 10af21b..a9045d8 100644
--- a/include/configs/am3517_evm.h
+++ b/include/configs/am3517_evm.h
@@ -99,6 +99,44 @@
 #define CONFIG_OMAP3_MMC		1
 #define CONFIG_DOS_PARTITION		1
 
+/*
+ * USB configuration
+ * Enable CONFIG_MUSB_HCD for Host functionalities MSC, keyboard
+ * Enable CONFIG_MUSB_UDC for Device functionalities.
+ */
+#define CONFIG_USB_AM35X		1
+#define CONFIG_MUSB_HCD			1
+
+#ifdef CONFIG_USB_AM35X
+
+#ifdef CONFIG_MUSB_HCD
+#define CONFIG_CMD_USB
+
+#define CONFIG_USB_STORAGE
+#define CONGIG_CMD_STORAGE
+#define CONFIG_CMD_FAT
+
+#ifdef CONFIG_USB_KEYBOARD
+#define CONFIG_SYS_USB_EVENT_POLL
+#define CONFIG_PREBOOT "usb start"
+#endif /* CONFIG_USB_KEYBOARD */
+
+#endif /* CONFIG_MUSB_HCD */
+
+#ifdef CONFIG_MUSB_UDC
+/* USB device configuration */
+#define CONFIG_USB_DEVICE		1
+#define CONFIG_USB_TTY			1
+#define CONFIG_SYS_CONSOLE_IS_IN_ENV	1
+/* Change these to suit your needs */
+#define CONFIG_USBD_VENDORID		0x0451
+#define CONFIG_USBD_PRODUCTID		0x5678
+#define CONFIG_USBD_MANUFACTURER	"Texas Instruments"
+#define CONFIG_USBD_PRODUCT_NAME	"AM3517EVM"
+#endif /* CONFIG_MUSB_UDC */
+
+#endif /* CONFIG_USB_AM35X */
+
 /* commands to include */
 #include <config_cmd_default.h>
 
-- 
1.6.2.4

^ permalink raw reply related	[flat|nested] 14+ messages in thread

* [U-Boot] [PATCH 0/4 v2] AM35X: Add musb support
  2010-07-09  6:13 [U-Boot] [PATCH 0/4 v2] AM35X: Add musb support Ajay Kumar Gupta
  2010-07-09  6:13 ` [U-Boot] [PATCH 1/4 v2] AM35x: Adding SCM general register definitions Ajay Kumar Gupta
@ 2010-07-16  4:44 ` Gupta, Ajay Kumar
  2010-07-16 19:08   ` Paulraj, Sandeep
  2010-08-02  8:18 ` Remy Bohmer
  2 siblings, 1 reply; 14+ messages in thread
From: Gupta, Ajay Kumar @ 2010-07-16  4:44 UTC (permalink / raw)
  To: u-boot

> These patches add support for musb in AM35x.
> Changes from v1:
> 	- Moved SCM definitions to am35x specific file am35x_def.h
> 
> Patches have been created against uboot-ti/master tree (as on 9th July)
> but they also apply cleanly to both denx/master and uboot-usb/master
> as there is no interdependent changes.

Remy,
	Any comment on this patch set ?

Regards,
Ajay
> 
> Regards,
> Ajay
> 
> Ajay Kumar Gupta (4):
>   AM35x: Adding SCM general register definitions
>   musb: MSC host support for AM35x
>   musb: am35x: Workaround for fifo read issue
>   AM3517EVM: musb: add usb config
> 
>  arch/arm/include/asm/arch-omap3/am35x_def.h |   52 +++++++++
>  drivers/usb/musb/Makefile                   |    1 +
>  drivers/usb/musb/am35x.c                    |  150
> +++++++++++++++++++++++++++
>  drivers/usb/musb/am35x.h                    |   94 +++++++++++++++++
>  drivers/usb/musb/musb_core.c                |    6 +
>  drivers/usb/musb/musb_udc.c                 |    2 +
>  include/configs/am3517_evm.h                |   38 +++++++
>  include/usb.h                               |    2 +-
>  8 files changed, 344 insertions(+), 1 deletions(-)
>  create mode 100644 arch/arm/include/asm/arch-omap3/am35x_def.h
>  create mode 100644 drivers/usb/musb/am35x.c
>  create mode 100644 drivers/usb/musb/am35x.h

^ permalink raw reply	[flat|nested] 14+ messages in thread

* [U-Boot] [PATCH 0/4 v2] AM35X: Add musb support
  2010-07-16  4:44 ` [U-Boot] [PATCH 0/4 v2] AM35X: Add musb support Gupta, Ajay Kumar
@ 2010-07-16 19:08   ` Paulraj, Sandeep
  2010-07-19 11:59     ` Gupta, Ajay Kumar
  0 siblings, 1 reply; 14+ messages in thread
From: Paulraj, Sandeep @ 2010-07-16 19:08 UTC (permalink / raw)
  To: u-boot


> 
> > These patches add support for musb in AM35x.
> > Changes from v1:
> > 	- Moved SCM definitions to am35x specific file am35x_def.h
> >
> > Patches have been created against uboot-ti/master tree (as on 9th July)
> > but they also apply cleanly to both denx/master and uboot-usb/master
> > as there is no interdependent changes.
> 
> Remy,
> 	Any comment on this patch set ?
> 
> Regards,
> Ajay


He sent me an e-mail offline.
He is on vacation without Wifi and laptop.

Sandeep

^ permalink raw reply	[flat|nested] 14+ messages in thread

* [U-Boot] [PATCH 0/4 v2] AM35X: Add musb support
  2010-07-16 19:08   ` Paulraj, Sandeep
@ 2010-07-19 11:59     ` Gupta, Ajay Kumar
  0 siblings, 0 replies; 14+ messages in thread
From: Gupta, Ajay Kumar @ 2010-07-19 11:59 UTC (permalink / raw)
  To: u-boot

> > Remy,
> > 	Any comment on this patch set ?
> >
> > Regards,
> > Ajay
> 
> 
> He sent me an e-mail offline.
> He is on vacation without Wifi and laptop.

Thanks for updating on this.

Regards,
Ajay
> 
> Sandeep

^ permalink raw reply	[flat|nested] 14+ messages in thread

* [U-Boot] [PATCH 3/4 v2] musb: am35x: Workaround for fifo read issue
  2010-07-09  6:13     ` [U-Boot] [PATCH 3/4 v2] musb: am35x: Workaround for fifo read issue Ajay Kumar Gupta
  2010-07-09  6:13       ` [U-Boot] [PATCH 4/4] AM3517EVM: musb: add usb config Ajay Kumar Gupta
@ 2010-08-01 11:16       ` Remy Bohmer
  1 sibling, 0 replies; 14+ messages in thread
From: Remy Bohmer @ 2010-08-01 11:16 UTC (permalink / raw)
  To: u-boot

Hi,

2010/7/9 Ajay Kumar Gupta <ajay.gupta@ti.com>:
> AM35x supports only 32bit read operations so we need to have
> workaround for 8bit and 16bit read operations.
>
> Signed-off-by: Ajay Kumar Gupta <ajay.gupta@ti.com>
> ---
> ?drivers/usb/musb/am35x.c ? ? | ? 32 ++++++++++++++++++++++++++++++++
> ?drivers/usb/musb/musb_core.c | ? ?6 ++++++
> ?2 files changed, 38 insertions(+), 0 deletions(-)

Acked-by: Remy Bohmer <linux@bohmer.net>

^ permalink raw reply	[flat|nested] 14+ messages in thread

* [U-Boot] [PATCH 2/4 v2] musb: MSC host support for AM35x
  2010-07-09  6:13   ` [U-Boot] [PATCH 2/4 v2] musb: MSC host support for AM35x Ajay Kumar Gupta
  2010-07-09  6:13     ` [U-Boot] [PATCH 3/4 v2] musb: am35x: Workaround for fifo read issue Ajay Kumar Gupta
@ 2010-08-01 11:16     ` Remy Bohmer
  2010-08-02  4:19       ` Gupta, Ajay Kumar
  2010-08-08 21:06       ` Wolfgang Denk
  1 sibling, 2 replies; 14+ messages in thread
From: Remy Bohmer @ 2010-08-01 11:16 UTC (permalink / raw)
  To: u-boot

Hi,

2010/7/9 Ajay Kumar Gupta <ajay.gupta@ti.com>:
> Tested MSC Host on AM3517EVM.
>
> Signed-off-by: Ajay Kumar Gupta <ajay.gupta@ti.com>
> ---
> ?drivers/usb/musb/Makefile ? | ? ?1 +
> ?drivers/usb/musb/am35x.c ? ?| ?118 +++++++++++++++++++++++++++++++++++++++++++
> ?drivers/usb/musb/am35x.h ? ?| ? 94 ++++++++++++++++++++++++++++++++++
> ?drivers/usb/musb/musb_udc.c | ? ?2 +
> ?include/usb.h ? ? ? ? ? ? ? | ? ?2 +-
> ?5 files changed, 216 insertions(+), 1 deletions(-)
> ?create mode 100644 drivers/usb/musb/am35x.c
> ?create mode 100644 drivers/usb/musb/am35x.h

Acked-by: Remy Bohmer <linux@bohmer.net>

^ permalink raw reply	[flat|nested] 14+ messages in thread

* [U-Boot] [PATCH 2/4 v2] musb: MSC host support for AM35x
  2010-08-01 11:16     ` [U-Boot] [PATCH 2/4 v2] musb: MSC host support for AM35x Remy Bohmer
@ 2010-08-02  4:19       ` Gupta, Ajay Kumar
  2010-08-08 21:06       ` Wolfgang Denk
  1 sibling, 0 replies; 14+ messages in thread
From: Gupta, Ajay Kumar @ 2010-08-02  4:19 UTC (permalink / raw)
  To: u-boot

Hi,
> 2010/7/9 Ajay Kumar Gupta <ajay.gupta@ti.com>:
> > Tested MSC Host on AM3517EVM.
> >
> > Signed-off-by: Ajay Kumar Gupta <ajay.gupta@ti.com>
> > ---
> > ?drivers/usb/musb/Makefile ? | ? ?1 +
> > ?drivers/usb/musb/am35x.c ? ?| ?118
> +++++++++++++++++++++++++++++++++++++++++++
> > ?drivers/usb/musb/am35x.h ? ?| ? 94 ++++++++++++++++++++++++++++++++++
> > ?drivers/usb/musb/musb_udc.c | ? ?2 +
> > ?include/usb.h ? ? ? ? ? ? ? | ? ?2 +-
> > ?5 files changed, 216 insertions(+), 1 deletions(-)
> > ?create mode 100644 drivers/usb/musb/am35x.c
> > ?create mode 100644 drivers/usb/musb/am35x.h
> 
> Acked-by: Remy Bohmer <linux@bohmer.net>

Remy,
	Please merge the patch set supporting AM3517 musb in your
uboot-usb tree.

Regards,
Ajay

^ permalink raw reply	[flat|nested] 14+ messages in thread

* [U-Boot] [PATCH 0/4 v2] AM35X: Add musb support
  2010-07-09  6:13 [U-Boot] [PATCH 0/4 v2] AM35X: Add musb support Ajay Kumar Gupta
  2010-07-09  6:13 ` [U-Boot] [PATCH 1/4 v2] AM35x: Adding SCM general register definitions Ajay Kumar Gupta
  2010-07-16  4:44 ` [U-Boot] [PATCH 0/4 v2] AM35X: Add musb support Gupta, Ajay Kumar
@ 2010-08-02  8:18 ` Remy Bohmer
  2 siblings, 0 replies; 14+ messages in thread
From: Remy Bohmer @ 2010-08-02  8:18 UTC (permalink / raw)
  To: u-boot

Hi,

2010/7/9 Ajay Kumar Gupta <ajay.gupta@ti.com>:
> Remy,
>
> These patches add support for musb in AM35x.
> Changes from v1:
> ? ? ? ?- Moved SCM definitions to am35x specific file am35x_def.h
>
> Patches have been created against uboot-ti/master tree (as on 9th July)
> but they also apply cleanly to both denx/master and uboot-usb/master
> as there is no interdependent changes.
>
> Regards,
> Ajay
>
> Ajay Kumar Gupta (4):
> ?AM35x: Adding SCM general register definitions
> ?musb: MSC host support for AM35x
> ?musb: am35x: Workaround for fifo read issue
> ?AM3517EVM: musb: add usb config
>
> ?arch/arm/include/asm/arch-omap3/am35x_def.h | ? 52 +++++++++
> ?drivers/usb/musb/Makefile ? ? ? ? ? ? ? ? ? | ? ?1 +
> ?drivers/usb/musb/am35x.c ? ? ? ? ? ? ? ? ? ?| ?150 +++++++++++++++++++++++++++
> ?drivers/usb/musb/am35x.h ? ? ? ? ? ? ? ? ? ?| ? 94 +++++++++++++++++
> ?drivers/usb/musb/musb_core.c ? ? ? ? ? ? ? ?| ? ?6 +
> ?drivers/usb/musb/musb_udc.c ? ? ? ? ? ? ? ? | ? ?2 +
> ?include/configs/am3517_evm.h ? ? ? ? ? ? ? ?| ? 38 +++++++
> ?include/usb.h ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? | ? ?2 +-
> ?8 files changed, 344 insertions(+), 1 deletions(-)
> ?create mode 100644 arch/arm/include/asm/arch-omap3/am35x_def.h
> ?create mode 100644 drivers/usb/musb/am35x.c
> ?create mode 100644 drivers/usb/musb/am35x.h

Applied to u-boot-usb master.

Thanks.

Remy

^ permalink raw reply	[flat|nested] 14+ messages in thread

* [U-Boot] [PATCH 2/4 v2] musb: MSC host support for AM35x
  2010-08-01 11:16     ` [U-Boot] [PATCH 2/4 v2] musb: MSC host support for AM35x Remy Bohmer
  2010-08-02  4:19       ` Gupta, Ajay Kumar
@ 2010-08-08 21:06       ` Wolfgang Denk
  2010-08-09 13:48         ` Remy Bohmer
  1 sibling, 1 reply; 14+ messages in thread
From: Wolfgang Denk @ 2010-08-08 21:06 UTC (permalink / raw)
  To: u-boot

Dear Remy Bohmer,

In message <AANLkTi=GCQgny4myVVVUAUvksemu67zWSNzkpyDM79dY@mail.gmail.com> you wrote:
> 
> 2010/7/9 Ajay Kumar Gupta <ajay.gupta@ti.com>:
> > Tested MSC Host on AM3517EVM.
> >
> > Signed-off-by: Ajay Kumar Gupta <ajay.gupta@ti.com>
> > ---
> >  drivers/usb/musb/Makefile   |    1 +
> >  drivers/usb/musb/am35x.c    |  118 +++++++++++++++++++++++++++++++++++++++++++
> >  drivers/usb/musb/am35x.h    |   94 ++++++++++++++++++++++++++++++++++
> >  drivers/usb/musb/musb_udc.c |    2 +
> >  include/usb.h               |    2 +-
> >  5 files changed, 216 insertions(+), 1 deletions(-)
> >  create mode 100644 drivers/usb/musb/am35x.c
> >  create mode 100644 drivers/usb/musb/am35x.h
>
> Acked-by: Remy Bohmer <linux@bohmer.net>

You ACK these? I would expect they go in through the USB repository.
What is your expectation - who should apply these patches? 

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
When all is said and done, more is said than done.

^ permalink raw reply	[flat|nested] 14+ messages in thread

* [U-Boot] [PATCH 2/4 v2] musb: MSC host support for AM35x
  2010-08-08 21:06       ` Wolfgang Denk
@ 2010-08-09 13:48         ` Remy Bohmer
  0 siblings, 0 replies; 14+ messages in thread
From: Remy Bohmer @ 2010-08-09 13:48 UTC (permalink / raw)
  To: u-boot

Hi Wolfgang,

2010/8/8 Wolfgang Denk <wd@denx.de>:
> Dear Remy Bohmer,
>
> In message <AANLkTi=GCQgny4myVVVUAUvksemu67zWSNzkpyDM79dY@mail.gmail.com> you wrote:
>>
>> 2010/7/9 Ajay Kumar Gupta <ajay.gupta@ti.com>:
>> > Tested MSC Host on AM3517EVM.
>> >
>> > Signed-off-by: Ajay Kumar Gupta <ajay.gupta@ti.com>
>> > ---
>> > ?drivers/usb/musb/Makefile ? | ? ?1 +
>> > ?drivers/usb/musb/am35x.c ? ?| ?118 +++++++++++++++++++++++++++++++++++++++++++
>> > ?drivers/usb/musb/am35x.h ? ?| ? 94 ++++++++++++++++++++++++++++++++++
>> > ?drivers/usb/musb/musb_udc.c | ? ?2 +
>> > ?include/usb.h ? ? ? ? ? ? ? | ? ?2 +-
>> > ?5 files changed, 216 insertions(+), 1 deletions(-)
>> > ?create mode 100644 drivers/usb/musb/am35x.c
>> > ?create mode 100644 drivers/usb/musb/am35x.h
>>
>> Acked-by: Remy Bohmer <linux@bohmer.net>
>
> You ACK these? I would expect they go in through the USB repository.
> What is your expectation - who should apply these patches?

I guess I was not fully awake when I sent that, I thought it was part
of a bigger series that should go in the u-boot-ti tree...
Anyway, I applied it to the u-boot-usb tree already a week ago...

Kind regards,

Remy

^ permalink raw reply	[flat|nested] 14+ messages in thread

end of thread, other threads:[~2010-08-09 13:48 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-07-09  6:13 [U-Boot] [PATCH 0/4 v2] AM35X: Add musb support Ajay Kumar Gupta
2010-07-09  6:13 ` [U-Boot] [PATCH 1/4 v2] AM35x: Adding SCM general register definitions Ajay Kumar Gupta
2010-07-09  6:13   ` [U-Boot] [PATCH 2/4 v2] musb: MSC host support for AM35x Ajay Kumar Gupta
2010-07-09  6:13     ` [U-Boot] [PATCH 3/4 v2] musb: am35x: Workaround for fifo read issue Ajay Kumar Gupta
2010-07-09  6:13       ` [U-Boot] [PATCH 4/4] AM3517EVM: musb: add usb config Ajay Kumar Gupta
2010-08-01 11:16       ` [U-Boot] [PATCH 3/4 v2] musb: am35x: Workaround for fifo read issue Remy Bohmer
2010-08-01 11:16     ` [U-Boot] [PATCH 2/4 v2] musb: MSC host support for AM35x Remy Bohmer
2010-08-02  4:19       ` Gupta, Ajay Kumar
2010-08-08 21:06       ` Wolfgang Denk
2010-08-09 13:48         ` Remy Bohmer
2010-07-16  4:44 ` [U-Boot] [PATCH 0/4 v2] AM35X: Add musb support Gupta, Ajay Kumar
2010-07-16 19:08   ` Paulraj, Sandeep
2010-07-19 11:59     ` Gupta, Ajay Kumar
2010-08-02  8:18 ` Remy Bohmer

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.