Timed out waiting for HCHalted. USBSTS: 000000

Author: lupyuen

README.md

@@ -76,15 +76,30 @@ https://github.com/lupyuen/pinephone-nuttx-usb/blob/b80499b3b8ec837fe2110e9476e8
 TODO
 
 ```text
-DEBUGASSERT(sizeof(struct ehci_overlay_s) == SIZEOF_EHCI_OVERLAY);
+a64_ehci_initialize: sizeof(struct a64_qh_s)=72
+a64_ehci_initialize: sizeof(struct a64_qtd_s)=32
+a64_ehci_initialize: sizeof(struct ehci_itd_s)=64
+a64_ehci_initialize: sizeof(struct ehci_sitd_s)=28
+a64_ehci_initialize: sizeof(struct ehci_qtd_s)=32
+a64_ehci_initialize: sizeof(struct ehci_overlay_s)=32
+a64_ehci_initialize: sizeof(struct ehci_qh_s)=48
+a64_ehci_initialize: sizeof(struct ehci_fstn_s)=8
 ```
 
-TODO
+a64_qh_s Now: 48 + 8 + 4 + 8 = 68
 
-```text
-sizeof(struct ehci_overlay_s)=32
+Align to 32-bit = 72
+
+Need to align to 0x20 = 32
+
+Pad to 96
+
+```c
+  uint8_t pad2[96 - 72]; // TODO: Pad from 72 to 96 bytes for 64-bit platform
 ```
 
+a64_qh_s Previously: 48 + 4 + 4 + 8 = 64
+
 # Output Log
 
 ```text
@@ -114,8 +129,8 @@ Found U-Boot script /boot.scr
 653 bytes read in 3 ms (211.9 KiB/s)
 ## Executing script at 4fc00000
 gpio: pin 114 (gpio 114) value is 1
-348854 bytes read in 21 ms (15.8 MiB/s)
-Uncompressed size: 10514432 = 0xA07000
+356655 bytes read in 20 ms (17 MiB/s)
+Uncompressed size: 10534912 = 0xA0C000
 36162 bytes read in 5 ms (6.9 MiB/s)
 1078500 bytes read in 50 ms (20.6 MiB/s)
 ## Flattened Device Tree blob at 4fa00000
@@ -125,35 +140,24 @@ Uncompressed size: 10514432 = 0xA07000
 
 Starting kernel ...
 
-usbhost_registerclass: Registering class:0x40120658 nids:2
-a64_ehci_initialize: sizeof(struct a64_qh_s)=72
+usbhost_registerclass: Registering class:0x40124838 nids:2
+a64_ehci_initialize: sizeof(struct a64_qh_s)=96
 a64_ehci_initialize: sizeof(struct a64_qtd_s)=32
-_assert: Current Version: NuttX  12.0.3 4d922be-dirty Mar  7 2023 15:54:47 arm64
-_assert: Assertion failed : at file: chip/a64_ehci.c:4996 task: nsh_main 0x4008b0d0
-up_dump_register: stack = 0x40129660
-up_dump_register: x0:   0x40129660          x1:   0xa
-up_dump_register: x2:   0x20                x3:   0x400efb22
-up_dump_register: x4:   0x4a10              x5:   0x0
-up_dump_register: x6:   0x4                 x7:   0x88
-up_dump_register: x8:   0x40a88268          x9:   0x0
-up_dump_register: x10:  0x1105000           x11:  0x5
-up_dump_register: x12:  0x0                 x13:  0x1
-up_dump_register: x14:  0x0                 x15:  0x1c28000
-up_dump_register: x16:  0x0                 x17:  0x1
-up_dump_register: x18:  0x0                 x19:  0x0
-up_dump_register: x20:  0x40a8d010          x21:  0x400ef92d
-up_dump_register: x22:  0x0                 x23:  0x1384
-up_dump_register: x24:  0x4011e6fe          x25:  0x40120000
-up_dump_register: x26:  0x0                 x27:  0x0
-up_dump_register: x28:  0x0                 x29:  0x0
-up_dump_register: x30:  0x4008b078        
-up_dump_register: 
-up_dump_register: STATUS Registers:
-up_dump_register: SPSR:      0x40000005        
-up_dump_register: ELR:       0x40081000        
-up_dump_register: SP_EL0:    0x40a8f300        
-up_dump_register: SP_ELX:    0x40a8f250        
-up_dump_register: TPIDR_EL0: 0x40a8d010        
-up_dump_register: TPIDR_EL1: 0x40a8d010        
-up_dump_register: EXE_DEPTH: 0xffffffffffffffff
+a64_ehci_initialize: sizeof(struct ehci_itd_s)=64
+a64_ehci_initialize: sizeof(struct ehci_sitd_s)=28
+a64_ehci_initialize: sizeof(struct ehci_qtd_s)=32
+a64_ehci_initialize: sizeof(struct ehci_overlay_s)=32
+a64_ehci_initialize: sizeof(struct ehci_qh_s)=48
+a64_ehci_initialize: sizeof(struct ehci_fstn_s)=8
+EHCI Initializing EHCI Stack
+a64_printreg: 01c1b010<-00000000
+a64_printreg: 01c1b014->00000000
+EHCI ERROR: Timed out waiting for HCHalted. USBSTS: 000000
+EHCI ERROR: a64_reset failed: 110
+a64_usbhost_initialize: ERROR: a64_ehci_initialize failed
+ERROR: Couldn't start usb -19
+nsh: mkfatfs: command not found
+
+NuttShell (NSH) NuttX-12.0.3
+nsh> 
 ```

a64_ehci.c

@@ -52,6 +52,7 @@
 ////#define CONFIG_USBHOST_INT_DISABLE // TODO CONFIG_USBHOST_INT_DISABLE
 #define CONFIG_A64_EHCI_REGDEBUG // TODO CONFIG_A64_EHCI_REGDEBUG
 #define ARMV7M_DCACHE_LINESIZE 32 // TODO ARMV7M_DCACHE_LINESIZE
+#define up_udelay(x) up_mdelay(x) // TODO up_udelay
 
 #if defined(CONFIG_A64_USBOTG) && defined(CONFIG_USBHOST)
 
@@ -197,6 +198,7 @@ struct a64_qh_s
   struct a64_epinfo_s *epinfo; /* Endpoint used for the transfer */
   uint32_t fqp;                  /* First qTD in the list (physical address) */
   uint8_t pad[8];                /* Padding to assure 32-byte alignment */
+  uint8_t pad2[96 - 72]; // TODO: Pad from 72 to 96 bytes for 64-bit platform
 };
 
 /* Internal representation of the EHCI Queue Element Transfer Descriptor