Re: [U-Boot] [PATCH 2/3] arm: socfpga: config: Enable USB support
On Sun, 21 Sep 2014, Marek Vasut wrote: Enable support for the DWC2 USB controller. Signed-off-by: Marek Vasut ma...@denx.de Cc: Chin Liang See cl...@altera.com Cc: Dinh Nguyen dingu...@altera.com Cc: Albert Aribaud albert.u.b...@aribaud.net Cc: Tom Rini tr...@ti.com Cc: Wolfgang Denk w...@denx.de Cc: Pavel Machek pa...@denx.de --- include/configs/socfpga_cyclone5_common.h | 8 1 file changed, 8 insertions(+) diff --git a/include/configs/socfpga_cyclone5_common.h b/include/configs/socfpga_cyclone5_common.h index 49504dc..3f8f91f 100644 --- a/include/configs/socfpga_cyclone5_common.h +++ b/include/configs/socfpga_cyclone5_common.h @@ -157,6 +157,14 @@ #define CONFIG_BAUDRATE 115200 /* + * USB + */ +#ifdef CONFIG_CMD_USB +#define CONFIG_USB_DWC2 +#define CONFIG_USB_STORAGE Don't you also need: #define CONFIG_USB_DWC2_REG_ADDR 0xffb4 for the base address for the USB1 on SOCFPGA? BR, Dinh ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 2/3] arm: socfpga: config: Enable USB support
On Tue, Sep 23, 2014 at 2:55 PM, Dinh Nguyen dingu...@opensource.altera.com wrote: On Sun, 21 Sep 2014, Marek Vasut wrote: Enable support for the DWC2 USB controller. Signed-off-by: Marek Vasut ma...@denx.de Cc: Chin Liang See cl...@altera.com Cc: Dinh Nguyen dingu...@altera.com Cc: Albert Aribaud albert.u.b...@aribaud.net Cc: Tom Rini tr...@ti.com Cc: Wolfgang Denk w...@denx.de Cc: Pavel Machek pa...@denx.de --- include/configs/socfpga_cyclone5_common.h | 8 1 file changed, 8 insertions(+) diff --git a/include/configs/socfpga_cyclone5_common.h b/include/configs/socfpga_cyclone5_common.h index 49504dc..3f8f91f 100644 --- a/include/configs/socfpga_cyclone5_common.h +++ b/include/configs/socfpga_cyclone5_common.h @@ -157,6 +157,14 @@ #define CONFIG_BAUDRATE 115200 /* + * USB + */ +#ifdef CONFIG_CMD_USB +#define CONFIG_USB_DWC2 +#define CONFIG_USB_STORAGE Don't you also need: #define CONFIG_USB_DWC2_REG_ADDR 0xffb4 for the base address for the USB1 on SOCFPGA? Now that I saw patch 50/51 arm: socfpga: Split SoCFPGA configuration, I think these defines should go into socfpga_cyclone5.h Dinh ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 2/3] arm: socfpga: config: Enable USB support
On Tuesday, September 23, 2014 at 09:55:59 PM, Dinh Nguyen wrote: On Sun, 21 Sep 2014, Marek Vasut wrote: Enable support for the DWC2 USB controller. Signed-off-by: Marek Vasut ma...@denx.de Cc: Chin Liang See cl...@altera.com Cc: Dinh Nguyen dingu...@altera.com Cc: Albert Aribaud albert.u.b...@aribaud.net Cc: Tom Rini tr...@ti.com Cc: Wolfgang Denk w...@denx.de Cc: Pavel Machek pa...@denx.de --- include/configs/socfpga_cyclone5_common.h | 8 1 file changed, 8 insertions(+) diff --git a/include/configs/socfpga_cyclone5_common.h b/include/configs/socfpga_cyclone5_common.h index 49504dc..3f8f91f 100644 --- a/include/configs/socfpga_cyclone5_common.h +++ b/include/configs/socfpga_cyclone5_common.h @@ -157,6 +157,14 @@ #define CONFIG_BAUDRATE115200 /* + * USB + */ +#ifdef CONFIG_CMD_USB +#define CONFIG_USB_DWC2 +#define CONFIG_USB_STORAGE Don't you also need: #define CONFIG_USB_DWC2_REG_ADDR 0xffb4 for the base address for the USB1 on SOCFPGA? Yes, you need to do that, but there are actually two USB controllers in the SoCFPGA CV, so this is up to the user to pick one and define it in his separate board header file (socfpga_cyclone5.h or similar). So far, the DWC2 driver supports only single controller (this needs to be fixed). Best regards, Marek Vasut ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 2/3] arm: socfpga: config: Enable USB support
On Wednesday, September 24, 2014 at 12:21:04 AM, Dinh Nguyen wrote: On Tue, Sep 23, 2014 at 2:55 PM, Dinh Nguyen dingu...@opensource.altera.com wrote: On Sun, 21 Sep 2014, Marek Vasut wrote: Enable support for the DWC2 USB controller. Signed-off-by: Marek Vasut ma...@denx.de Cc: Chin Liang See cl...@altera.com Cc: Dinh Nguyen dingu...@altera.com Cc: Albert Aribaud albert.u.b...@aribaud.net Cc: Tom Rini tr...@ti.com Cc: Wolfgang Denk w...@denx.de Cc: Pavel Machek pa...@denx.de --- include/configs/socfpga_cyclone5_common.h | 8 1 file changed, 8 insertions(+) diff --git a/include/configs/socfpga_cyclone5_common.h b/include/configs/socfpga_cyclone5_common.h index 49504dc..3f8f91f 100644 --- a/include/configs/socfpga_cyclone5_common.h +++ b/include/configs/socfpga_cyclone5_common.h @@ -157,6 +157,14 @@ #define CONFIG_BAUDRATE 115200 /* + * USB + */ +#ifdef CONFIG_CMD_USB +#define CONFIG_USB_DWC2 +#define CONFIG_USB_STORAGE Don't you also need: #define CONFIG_USB_DWC2_REG_ADDR 0xffb4 for the base address for the USB1 on SOCFPGA? Now that I saw patch 50/51 arm: socfpga: Split SoCFPGA configuration, I think these defines should go into socfpga_cyclone5.h Yeah, the board specific goes into socfpga_cyclone5.h , since that's de-facto support for the SoCDK board. The generic part goes into socfpga_cyclone5_common.h . Best regards, Marek Vasut ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH 2/3] arm: socfpga: config: Enable USB support
Enable support for the DWC2 USB controller. Signed-off-by: Marek Vasut ma...@denx.de Cc: Chin Liang See cl...@altera.com Cc: Dinh Nguyen dingu...@altera.com Cc: Albert Aribaud albert.u.b...@aribaud.net Cc: Tom Rini tr...@ti.com Cc: Wolfgang Denk w...@denx.de Cc: Pavel Machek pa...@denx.de --- include/configs/socfpga_cyclone5_common.h | 8 1 file changed, 8 insertions(+) diff --git a/include/configs/socfpga_cyclone5_common.h b/include/configs/socfpga_cyclone5_common.h index 49504dc..3f8f91f 100644 --- a/include/configs/socfpga_cyclone5_common.h +++ b/include/configs/socfpga_cyclone5_common.h @@ -157,6 +157,14 @@ #define CONFIG_BAUDRATE115200 /* + * USB + */ +#ifdef CONFIG_CMD_USB +#define CONFIG_USB_DWC2 +#define CONFIG_USB_STORAGE +#endif + +/* * U-Boot environment */ #define CONFIG_SYS_CONSOLE_IS_IN_ENV -- 2.0.0 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
