diff options
| author | Simon Glass <sjg@chromium.org> | 2017-09-17 16:54:54 -0600 |
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2017-10-08 20:41:09 -0600 |
| commit | ae06a1b9960aa97bd876840ebdb644c2952cd482 (patch) | |
| tree | 5125f2dde49762c757fb42084178832e3114548c | |
| parent | 7c84319af9c76084f50f3f2b97545bfa05f3971d (diff) | |
dm: gpio: pca953x: Drop pointless data structure checks
These checks cannot fail since driver model will not call a driver's
method if it cannot fully create the driver data structures.
It is confusing to have these checks and others might copy them. Drop this
code.
Signed-off-by: Simon Glass <sjg@chromium.org>
| -rw-r--r-- | drivers/gpio/pca953x_gpio.c | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/drivers/gpio/pca953x_gpio.c b/drivers/gpio/pca953x_gpio.c index 4962f25230..791d1d1516 100644 --- a/drivers/gpio/pca953x_gpio.c +++ b/drivers/gpio/pca953x_gpio.c @@ -249,22 +249,11 @@ static int pca953x_probe(struct udevice *dev) { struct pca953x_info *info = dev_get_platdata(dev); struct gpio_dev_priv *uc_priv = dev_get_uclass_priv(dev); - struct dm_i2c_chip *chip = dev_get_parent_platdata(dev); char name[32], *str; int addr; ulong driver_data; int ret; - if (!info) { - dev_err(dev, "platdata not ready\n"); - return -ENOMEM; - } - - if (!chip) { - dev_err(dev, "i2c not ready\n"); - return -ENODEV; - } - addr = fdtdec_get_int(gd->fdt_blob, dev_of_offset(dev), "reg", 0); if (addr == 0) return -ENODEV; |