From: Pan Bian Date: Sun, 12 Nov 2017 15:38:09 +0000 (+0800) Subject: net: dsa: lan9303: correctly check return value of devm_gpiod_get_optional X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=97438abcfb80656d4aedaca21b8ff8db4fcc93a1;p=linux.git net: dsa: lan9303: correctly check return value of devm_gpiod_get_optional Function devm_gpiod_get_optional() returns an ERR_PTR on failure. Its return value should not be validated by a NULL check. Instead, use IS_ERR. Signed-off-by: Pan Bian Reviewed-by: Andrew Lunn Signed-off-by: David S. Miller --- diff --git a/drivers/net/dsa/lan9303-core.c b/drivers/net/dsa/lan9303-core.c index fdfdb0edfe624..b24566bb74d2b 100644 --- a/drivers/net/dsa/lan9303-core.c +++ b/drivers/net/dsa/lan9303-core.c @@ -1301,7 +1301,7 @@ static void lan9303_probe_reset_gpio(struct lan9303 *chip, chip->reset_gpio = devm_gpiod_get_optional(chip->dev, "reset", GPIOD_OUT_LOW); - if (!chip->reset_gpio) { + if (IS_ERR(chip->reset_gpio)) { dev_dbg(chip->dev, "No reset GPIO defined\n"); return; }