> -     switch((base >> 8) & 7) {
> -             case 0: return "No interleave";
> -             case 1: return "2 nodes";
> -             case 3: return "4 nodes";
> -             case 7: return "8 nodes";
> -             default: return "Reserved";
> +     /* TODO: Are these fallthroughs intentional? */

I don't think they'll fall through because of the return statements.

Thanks,
Myles




--
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to