leds: tlc591xx: fix return value check in tlc591xx_probe()

After device_get_match_data(), tlc591xx is not checked, add
check for it and also check np after dev_of_node.

Reported-by: Hulk Robot <[e-mail]>
Signed-off-by: Yang Yingliang <[e-mail]>
Signed-off-by: Pavel Machek <[e-mail]>

drivers/leds/leds-tlc591xx.c

···
 tlc591xx_probe(struct i2c_client *client,
 	       const struct i2c_device_id *id)
 {
-	struct device_node *np = dev_of_node(&client->dev), *child;
+	struct device_node *np, *child;
 	struct device *dev = &client->dev;
 	const struct tlc591xx *tlc591xx;
 	struct tlc591xx_priv *priv;
 	int err, count, reg;
 
-	tlc591xx = device_get_match_data(dev);
+	np = dev_of_node(dev);
 	if (!np)
+		return -ENODEV;
+
+	tlc591xx = device_get_match_data(dev);
+	if (!tlc591xx)
 		return -ENODEV;
 
 	count = of_get_available_child_count(np);