[U-Boot] Testing u-boot-dm/next

[Stephen Warren <swarren@wwwdotorg.org>]

On 04/13/2015 10:27 AM, Stephen Warren wrote:
> On 04/08/2015 09:11 PM, Simon Glass wrote:
>> (Correcting address for Masahiro, sorry)
>>
>> On 8 April 2015 at 21:07, Simon Glass <sjg@chromium.org> wrote:
>>
>>> Hi,
>>>
>>> I have quite a few patches queued up in the next branch of u-boot-dm,
>>> ready for when the merge window options.
>>>
>>> If anyone has time and can give it a spin on their board, it would be
>>> much
>>> appreciated!
>
> On Jetson TK1, there's something up with USB.
...
> ... and here's u-boot-dm/next
>
>> Tegra124 (Jetson TK1) # usb start
>> starting USB...
>> USB-1:   USB EHCI 1.10
>> scanning bus 0 for devices... 1 USB Device(s) found
>> USB-1:   USB EHCI 1.10
>>
>> scanning bus 1 for devices... EHCI timed out on TD - token=0x80008d80
>>       USB device not accepting new address (error=2)
>> EHCI timed out on TD - token=0x80008d80
>>
>>       USB device not accepting new address (error=2)
>> 4 USB Device(s) found
>>        scanning usb for ethernet devices... 0 Ethernet Device(s) found

...
> Seaboard/Springbank appears to have the same issue. Additionally, the
> flashing process spews a ton of:
>
> ERROR: v7_dcache_inval_range - start address is not aligned - 0x3f77a428
> ERROR: v7_dcache_inval_range - stop address is not aligned - 0x3f77ac28

Both of those bisect to:

7bf0b2d00982 dm: usb: tegra: Move to driver model for USB

I wonder if the NAND issue is just a bug that's triggered by stack/data 
layout changes, and that commit tickles it?

For testing, it may be easier to load U-Boot into RAM that flash it 
every time. If so, since there's only 1 USB port, you'll need to use 
"usb reset" to re-scan that USB port. That will only work with the 
following patch, which I'll send in a minute:

diff --git a/include/configs/tegra-common-post.h 
b/include/configs/tegra-common-post.h
index c3ad8beb903d..9ab58555378c 100644
--- a/include/configs/tegra-common-post.h
+++ b/include/configs/tegra-common-post.h
@@ -26,10 +26,11 @@
  #define STDIN_KBD_KBC ""
  #endif

-#ifdef CONFIG_USB_KEYBOARD
+#if defined(CONFIG_USB_KEYBOARD) && !defined(CONFIG_SPL_BUILD)
  #define STDIN_KBD_USB ",usbkbd"
  #define CONFIG_SYS_USB_EVENT_POLL
  #define CONFIG_PREBOOT			"usb start"
+#define CONFIG_SYS_STDIO_DEREGISTER
  #else
  #define STDIN_KBD_USB ""
  #endif