[coreboot] r950 - coreboot-v3/northbridge/amd/k8

Myles Watson mylesgw at gmail.com
Thu Oct 23 21:20:37 CEST 2008


> -	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







More information about the coreboot mailing list