Re: [patch] of: check for IS_ERR()

From: David Miller
Date: Fri Feb 26 2010 - 04:54:10 EST


From: Dan Carpenter <error27@xxxxxxxxx>
Date: Fri, 26 Feb 2010 12:49:41 +0300

> get_phy_device() can return an ERR_PTR()
>
> Signed-off-by: Dan Carpenter <error27@xxxxxxxxx>
> ---
> I don't have a cross compile environment set up so I can't even compile
> test this. :/ But err.h is included so it should be OK.

It should return ERR_PTR() consistently. Checking for both
NULL and ERR_PTR() is undesirable.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/