[coreboot] patch: mcp55 usb for v3

Marc Jones marc.jones at amd.com
Tue Aug 12 01:06:41 CEST 2008


ron minnich wrote:
> On Mon, Aug 11, 2008 at 3:37 AM, Carl-Daniel Hailfinger
>   
>>> +#warning Figure out if we want usb debug direct
>>>
>>>       
>> Hm?
>>
>>     
>>> +#define CONFIG_USBDEBUG_DIRECT 0
>>>
>>>       
>> Move that one to a Kconfig variable.
>>     
>
> we might as well. I just don't know if we ever used it. Anyone?
>
>
>   

Seems like it should be a kconfig. I don't know if USBDEBUG works but 
there is a lot of stuff in v2 for it.

>>> +#if CONFIG_USBDEBUG_DIRECT
>>> +#include <usbdebug_direct.h>
>>> +#endif
>>> +
>>> +// yeah right. not yet. extern struct ehci_debug_info dbg_info;
>>>
>>>       
>> More context, please.
>>     
>
> I don't know what it is.
>
>   

>> Separate function please.
>>     
>
> Again, I left the warning there because we don't want to do this the
> way we did in v2. So it is yet to be resolved.
>
>   
I am OK with the warning for now.

Acked-by: Marc Jones <marc.jones at amd.com>

-- 
Marc Jones
Senior Firmware Engineer
(970) 226-9684 Office
mailto:Marc.Jones at amd.com
http://www.amd.com/embeddedprocessors






More information about the coreboot mailing list