[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