On Fri, Sep 26, 2008 at 03:25:48PM -0500, Jon Loeliger wrote: > From: Jon Loeliger <[EMAIL PROTECTED]> > > This code is derived from code posted by David Gibson.
[snip] > + for (i = 0; i < ARRAY_SIZE(expr_table); i++) > + if (fdt32_to_cpu(res[i]) != expr_table[i].result) Hrm.. when you fixed the endian bug here (I found it myself the other day), you might have thought to fix it... > + FAIL("Incorrect result for expression \"%s\"," > + " 0x%x instead of 0x%x\n", > + expr_table[i].expr, res[i], ^^^^^^ ...here, too. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson _______________________________________________ devicetree-discuss mailing list devicetree-discuss@ozlabs.org https://ozlabs.org/mailman/listinfo/devicetree-discuss