Re: [U-Boot] [PATCH 1/2] efikamx: refine USB support

2012-09-04 Thread Stefano Babic
On 23/08/2012 16:52, Matt Sealey wrote:
 Because of the way USB pad settings are handled it doesn't make sense to
 be able to build the Efika MX board support without CONFIG_CMD_USB turned
 on. So, we change the build to always compile in USB support.
 
 We do not need to check for CONFIG_CMD_USB like we do with CONFIG_MXC_SPI
 since the USB subsystem will error out of the compile for us.
 
 Additionally, the following behaviors have changed;
 
 * Smartbook preboot should not set input and output to USB keyboard as
   there is no display support
 * board_eth_init is implemented such that it does not cause U-Boot to
   report an explicit failure (CPU Net Initialization Failed).
 
 Since Ethernet is implemented via USB (fixed on Smarttop, pluggable on
 Smartbook, and handled by usb start) - the warning that is left
 (No ethernet found) is perfectly reasonable at the point it is printed
 since the USB system hasn't been started and nothing has been probed yet.
 
 Signed-off-by: Matt Sealey m...@genesi-usa.com
 Cc: Stefano Babic sba...@denx.de
 Cc: Marek Vasut ma...@denx.de
 ---

Applied (after rebasing) on u-boot-imx, thanks.

Best regards,
Stefano Babic


-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 1/2] efikamx: refine USB support

2012-08-23 Thread Matt Sealey
Because of the way USB pad settings are handled it doesn't make sense to
be able to build the Efika MX board support without CONFIG_CMD_USB turned
on. So, we change the build to always compile in USB support.

We do not need to check for CONFIG_CMD_USB like we do with CONFIG_MXC_SPI
since the USB subsystem will error out of the compile for us.

Additionally, the following behaviors have changed;

* Smartbook preboot should not set input and output to USB keyboard as
  there is no display support
* board_eth_init is implemented such that it does not cause U-Boot to
  report an explicit failure (CPU Net Initialization Failed).

Since Ethernet is implemented via USB (fixed on Smarttop, pluggable on
Smartbook, and handled by usb start) - the warning that is left
(No ethernet found) is perfectly reasonable at the point it is printed
since the USB system hasn't been started and nothing has been probed yet.

Signed-off-by: Matt Sealey m...@genesi-usa.com
Cc: Stefano Babic sba...@denx.de
Cc: Marek Vasut ma...@denx.de
---
 board/genesi/mx51_efikamx/Makefile  |6 +-
 board/genesi/mx51_efikamx/efikamx-usb.c |   12 
 board/genesi/mx51_efikamx/efikamx.c |3 ---
 3 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/board/genesi/mx51_efikamx/Makefile 
b/board/genesi/mx51_efikamx/Makefile
index bd2174f..f95356f 100644
--- a/board/genesi/mx51_efikamx/Makefile
+++ b/board/genesi/mx51_efikamx/Makefile
@@ -27,11 +27,7 @@ include $(TOPDIR)/config.mk
 
 LIB= $(obj)lib$(BOARD).o
 
-COBJS  := efikamx.o
-
-ifdef  CONFIG_CMD_USB
-COBJS  += efikamx-usb.o
-endif
+COBJS  := efikamx.o efikamx-usb.o
 
 SRCS   := $(COBJS:.o=.c)
 OBJS   := $(addprefix $(obj),$(COBJS))
diff --git a/board/genesi/mx51_efikamx/efikamx-usb.c 
b/board/genesi/mx51_efikamx/efikamx-usb.c
index abd358a..ae6d1be 100644
--- a/board/genesi/mx51_efikamx/efikamx-usb.c
+++ b/board/genesi/mx51_efikamx/efikamx-usb.c
@@ -201,3 +201,15 @@ void board_ehci_hcd_postinit(struct usb_ehci *ehci, int 
port)
   IMX_GPIO_NR(2, 20));
}
 }
+
+/*
+ * Ethernet on the Smarttop is on the USB bus. Rather than give an error about
+ * CPU Net Initialization Failed, just pass this test since no other settings
+ * are required. Smartbook doesn't have built-in Ethernet but we will let it
+ * pass anyway considering someone may have plugged in a USB stick and all
+ * they need to do is run usb start.
+ */
+int board_eth_init(bd_t *bis)
+{
+   return 0;
+}
diff --git a/board/genesi/mx51_efikamx/efikamx.c 
b/board/genesi/mx51_efikamx/efikamx.c
index b9064fb..d9c499d 100644
--- a/board/genesi/mx51_efikamx/efikamx.c
+++ b/board/genesi/mx51_efikamx/efikamx.c
@@ -484,9 +484,6 @@ int board_late_init(void)
imx_iomux_v3_setup_multiple_pads(efikamx_pata_pads, 
ARRAY_SIZE(efikamx_pata_pads));
efikamx_usb_setup_pads();
 
-   if (machine_is_efikasb())
-   setenv(preboot, usb reset ; setenv stdin usbkbd\0);
-
return 0;
 }
 
-- 
1.7.9.5

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