[U-Boot] [PATCH 1/6] usb : musb : Add high speed field in usb_device structure

Abraham, Thomas t-abraham at ti.com
Wed Dec 17 22:12:21 CET 2008


Hi Remy, 

> -----Original Message-----
> From: l.pinguin at gmail.com [mailto:l.pinguin at gmail.com] On 
> Behalf Of Remy Bohmer
> Sent: Wednesday, December 17, 2008 1:48 AM
> To: Abraham, Thomas
> 

<snip>

> 
> Same, Michael combined slow+high into 1 single element called 
> 'speed', what I would prefer here.
> 
> Can you please rebase your work on the U-boot-usb tree, 'next' branch?
> And streamline these changes with the work Michael already has done?

Ok. I will rebase with u-boot-usb next branch. 

For mentor usb controller, it would still be necessary to record the speed of the device when it is found connected to a hub port. So the update of usb->speed would have to be done as below in 'usb_hub_port_connect_change' function. Would this be acceptable?

 /* Allocate a new device struct for it */
 usb = usb_alloc_new_device();
 usb->speed = (portstatus & USB_PORT_STAT_LOW_SPEED) ? 1 : 0;
+usb->speed = (portstatus & USB_PORT_STAT_HIGH_SPEED) ? 1 : 0;

> 
> Kind Regards,
> 
> Remy

Thanks,
Thomas.



More information about the U-Boot mailing list