char: tpm: Deletion of unnecessary checks before the function call "tpm_dev_vendor_release"

The tpm_dev_vendor_release() function tests whether its argument is NULL
and then returns immediately. Thus the test around the call is not needed.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Peter Huewe <peterhuewe@gmx.de>

2 files changed, 2 insertions, 4 deletions

diff --git a/drivers/char/tpm/tpm_i2c_atmel.c b/drivers/char/tpm/tpm_i2c_atmel.c
index 503a85ae176c..5f448886417f 100644
--- a/drivers/char/tpm/tpm_i2c_atmel.c
+++ b/drivers/char/tpm/tpm_i2c_atmel.c
@@ -206,8 +206,7 @@ static int i2c_atmel_remove(struct i2c_client *client)
         struct device *dev = &(client->dev);
         struct tpm_chip *chip = dev_get_drvdata(dev);
 
-        if (chip)
-                tpm_dev_vendor_release(chip);
+        tpm_dev_vendor_release(chip);
         tpm_remove_hardware(dev);
         kfree(chip);
         return 0;
diff --git a/drivers/char/tpm/tpm_i2c_nuvoton.c b/drivers/char/tpm/tpm_i2c_nuvoton.c
index 23c7b137a7fd..bbb4997438c3 100644
--- a/drivers/char/tpm/tpm_i2c_nuvoton.c
+++ b/drivers/char/tpm/tpm_i2c_nuvoton.c
@@ -630,8 +630,7 @@ static int i2c_nuvoton_remove(struct i2c_client *client)
         struct device *dev = &(client->dev);
         struct tpm_chip *chip = dev_get_drvdata(dev);
 
-        if (chip)
-                tpm_dev_vendor_release(chip);
+        tpm_dev_vendor_release(chip);
         tpm_remove_hardware(dev);
         kfree(chip);
         return 0;