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

Reply via email to