Hi Sean,

On Sun, 27 Apr 2008 15:25:46 -0400 Sean MacLennan <[EMAIL PROTECTED]> wrote:
>
> +++ b/arch/powerpc/platforms/44x/warp-nand.c
> @@ -11,8 +11,10 @@
>  #include <linux/mtd/partitions.h>
>  #include <linux/mtd/nand.h>
>  #include <linux/mtd/ndfc.h>
> +#include <linux/of_platform.h>

You really want linux.of.h (unless there was some preexisting reason to
require of_platform.h).

> +static void pika_setup_critical_temp(struct i2c_client *client)
> +{
> +     struct device_node *np;
> +     int irq, rc;
> +
> +     /* Setup power LEDS for possible critical temp */
> +     np = of_find_compatible_node(NULL, NULL, "ibm,gpio-440EP");
> +     if (np)
> +             np = of_find_compatible_node(np, NULL, "ibm,gpio-440EP");

Did you really mean to test if (!np) above instead?

-- 
Cheers,
Stephen Rothwell                    [EMAIL PROTECTED]
http://www.canb.auug.org.au/~sfr/

Attachment: pgpmvWFstEC2p.pgp
Description: PGP signature

_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Reply via email to