From: Peter Huewe Date: Tue, 29 Jan 2013 21:02:02 +0000 (+0100) Subject: char/tpm/tpm_i2c_stm_st33: drop temporary variable for return value X-Git-Tag: firefly_0821_release~3680^2~627^2~15 X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=7333549bf4f345d338825dcd5b85400600e4cae7;p=firefly-linux-kernel-4.4.55.git char/tpm/tpm_i2c_stm_st33: drop temporary variable for return value We don't need a temporary variable just to store the return value which gets return in the next statement. Signed-off-by: Peter Huewe Signed-off-by: Kent Yoder --- diff --git a/drivers/char/tpm/tpm_i2c_stm_st33.c b/drivers/char/tpm/tpm_i2c_stm_st33.c index b14c4b59958d..1f5f71e14abe 100644 --- a/drivers/char/tpm/tpm_i2c_stm_st33.c +++ b/drivers/char/tpm/tpm_i2c_stm_st33.c @@ -96,16 +96,14 @@ enum tis_defaults { static int write8_reg(struct i2c_client *client, u8 tpm_register, u8 *tpm_data, u16 tpm_size) { - int value = 0; struct st33zp24_platform_data *pin_infos; pin_infos = client->dev.platform_data; pin_infos->tpm_i2c_buffer[0][0] = tpm_register; memcpy(&pin_infos->tpm_i2c_buffer[0][1], tpm_data, tpm_size); - value = i2c_master_send(client, pin_infos->tpm_i2c_buffer[0], + return i2c_master_send(client, pin_infos->tpm_i2c_buffer[0], tpm_size + 1); - return value; } /* write8_reg() */ /*