Re: [U-Boot] [PATCH 3/3] Armada100: gplugD: Add USB command support

2012-01-26 Thread Prafulla Wadaskar


 -Original Message-
 From: Ajay Bhargav [mailto:ajay.bhar...@einfochips.com]
 Sent: 25 January 2012 17:50
 To: li...@bohmer.net
 Cc: Prafulla Wadaskar; marek.va...@gmail.com; u-boot@lists.denx.de;
 Ajay Bhargav
 Subject: [PATCH 3/3] Armada100: gplugD: Add USB command support
 
 This patch adds support for USB commands and USB storage device for
 Marvell gplugD
 
 Signed-off-by: Ajay Bhargav ajay.bhar...@einfochips.com
 ---
  include/configs/gplugd.h |   17 +
  1 files changed, 17 insertions(+), 0 deletions(-)
 
 diff --git a/include/configs/gplugd.h b/include/configs/gplugd.h
 index 9813309..f8dc74f 100644
 --- a/include/configs/gplugd.h
 +++ b/include/configs/gplugd.h
 @@ -134,4 +134,21 @@
  #define CONFIG_CMD_EDITENV
  #define CONFIG_CMD_SAVEENV
 
 +#define CONFIG_CMD_USB

This line should be moved up where all the commands are defined (below 
config_cmd_default.h)

 +
 +#ifdef CONFIG_CMD_USB
 +#define CONFIG_USB_EHCI
 +#define CONFIG_USB_EHCI_ARMADA100
 +#define CONFIG_EHCI_IS_TDI
 +#define CONFIG_USB_STORAGE
 +
 +#ifndef CONFIG_DOS_PARTITION

Does this undef needed? remove it you can.

Regards..
Prafulla . . .
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 3/3] Armada100: gplugD: Add USB command support

2012-01-26 Thread Ajay Bhargav

- Prafulla Wadaskar prafu...@marvell.com wrote:

  -Original Message-
  From: Ajay Bhargav [mailto:ajay.bhar...@einfochips.com]
  Sent: 25 January 2012 17:50
  To: li...@bohmer.net
  Cc: Prafulla Wadaskar; marek.va...@gmail.com; u-boot@lists.denx.de;
  Ajay Bhargav
  Subject: [PATCH 3/3] Armada100: gplugD: Add USB command support
  
  This patch adds support for USB commands and USB storage device for
  Marvell gplugD
[...snip...]
  
  +#ifdef CONFIG_CMD_USB
  +#define CONFIG_USB_EHCI
  +#define CONFIG_USB_EHCI_ARMADA100
  +#define CONFIG_EHCI_IS_TDI
  +#define CONFIG_USB_STORAGE
  +
  +#ifndef CONFIG_DOS_PARTITION
 
 Does this undef needed? remove it you can.
 
 Regards..
 Prafulla . . .
 

I will review it again and remove it if not needed.

Regards,
Ajay Bhargav
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 3/3] Armada100: gplugD: Add USB command support

2012-01-25 Thread Ajay Bhargav
This patch adds support for USB commands and USB storage device for
Marvell gplugD

Signed-off-by: Ajay Bhargav ajay.bhar...@einfochips.com
---
 include/configs/gplugd.h |   17 +
 1 files changed, 17 insertions(+), 0 deletions(-)

diff --git a/include/configs/gplugd.h b/include/configs/gplugd.h
index 9813309..f8dc74f 100644
--- a/include/configs/gplugd.h
+++ b/include/configs/gplugd.h
@@ -134,4 +134,21 @@
 #define CONFIG_CMD_EDITENV
 #define CONFIG_CMD_SAVEENV
 
+#define CONFIG_CMD_USB
+
+#ifdef CONFIG_CMD_USB
+#define CONFIG_USB_EHCI
+#define CONFIG_USB_EHCI_ARMADA100
+#define CONFIG_EHCI_IS_TDI
+#define CONFIG_USB_STORAGE
+
+#ifndef CONFIG_DOS_PARTITION
+#define CONFIG_DOS_PARTITION
+#endif /* CONFIG_DOS_PARTITION */
+
+#define CONFIG_ISO_PARTITION
+#define CONFIG_SUPPORT_VFAT
+#define CONFIG_CMD_EXT2
+#endif /* CONFIG_CMD_USB */
+
 #endif /* __CONFIG_GPLUGD_H */
-- 
1.7.7.2

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot