[coreboot] [PATCH] YABEL debug flags
Myles Watson
mylesgw at gmail.com
Fri Feb 19 21:28:00 CET 2010
> Yes, so do I... But making it depending on YABEL_DEBUG_FLAGS may not be the
>> right thing, unless the code is also modified to check for the bits in
>> yabel debug flags explicitly.
>>
> I'm not sure why a non-zero check isn't good enough. I haven't used the
> YABEL debug flags very much.
>
>
> Because you'd suddenly get IO debug outputs whether you specify
> YABEL_DEBUG_CPU_PREFETCH or YABEL_DEBUG_IO :-)
>
Yep. That's bad.
> Well, no need to worry for now... I think it's fine. I'll prepare my patch
> because that takes care of this in a place or two....
>
Sounds good.
Thanks,
Myles
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20100219/99084a0e/attachment.html>
More information about the coreboot
mailing list