From f53c5d92b596b6f0a7dd1a636d1018e51230b271 Mon Sep 17 00:00:00 2001 From: Valentin Vidic Date: Sat, 27 Jan 2018 10:42:15 +0100 Subject: [PATCH] staging: pi433: fix CamelCase for newValue Local variable storing the new value for bandwidth register so replace with bandwidth. Fixes checkpatch warnings: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic Signed-off-by: Greg Kroah-Hartman --- drivers/staging/pi433/rf69.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/pi433/rf69.c b/drivers/staging/pi433/rf69.c index 261ba1643f5a..ef56deccd0af 100644 --- a/drivers/staging/pi433/rf69.c +++ b/drivers/staging/pi433/rf69.c @@ -425,7 +425,7 @@ int rf69_set_lna_gain(struct spi_device *spi, enum lnaGain lnaGain) static int rf69_set_bandwidth_intern(struct spi_device *spi, u8 reg, enum mantisse mantisse, u8 exponent) { - u8 newValue; + u8 bandwidth; // check value for mantisse and exponent if (exponent > 7) { @@ -441,29 +441,29 @@ static int rf69_set_bandwidth_intern(struct spi_device *spi, u8 reg, } // read old value - newValue = rf69_read_reg(spi, reg); + bandwidth = rf69_read_reg(spi, reg); // "delete" mantisse and exponent = just keep the DCC setting - newValue = newValue & MASK_BW_DCC_FREQ; + bandwidth = bandwidth & MASK_BW_DCC_FREQ; // add new mantisse switch (mantisse) { case mantisse16: - newValue = newValue | BW_MANT_16; + bandwidth = bandwidth | BW_MANT_16; break; case mantisse20: - newValue = newValue | BW_MANT_20; + bandwidth = bandwidth | BW_MANT_20; break; case mantisse24: - newValue = newValue | BW_MANT_24; + bandwidth = bandwidth | BW_MANT_24; break; } // add new exponent - newValue = newValue | exponent; + bandwidth = bandwidth | exponent; // write back - return rf69_write_reg(spi, reg, newValue); + return rf69_write_reg(spi, reg, bandwidth); } int rf69_set_bandwidth(struct spi_device *spi, enum mantisse mantisse, u8 exponent) -- 2.30.2