Commit 2dfc2ded authored by Markus Elfring's avatar Markus Elfring Committed by Peter Huewe

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: default avatarMarkus Elfring <elfring@users.sourceforge.net>
Signed-off-by: default avatarPeter Huewe <peterhuewe@gmx.de>
parent 9fd8e5a2
...@@ -206,8 +206,7 @@ static int i2c_atmel_remove(struct i2c_client *client) ...@@ -206,8 +206,7 @@ static int i2c_atmel_remove(struct i2c_client *client)
struct device *dev = &(client->dev); struct device *dev = &(client->dev);
struct tpm_chip *chip = dev_get_drvdata(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); tpm_remove_hardware(dev);
kfree(chip); kfree(chip);
return 0; return 0;
......
...@@ -630,8 +630,7 @@ static int i2c_nuvoton_remove(struct i2c_client *client) ...@@ -630,8 +630,7 @@ static int i2c_nuvoton_remove(struct i2c_client *client)
struct device *dev = &(client->dev); struct device *dev = &(client->dev);
struct tpm_chip *chip = dev_get_drvdata(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); tpm_remove_hardware(dev);
kfree(chip); kfree(chip);
return 0; return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment