Commit e6dd870a authored by Valentin Vidic's avatar Valentin Vidic Committed by Greg Kroah-Hartman

staging: pi433: fix CamelCase for regValue

Local variable storing the new value for dio register
so replace with dio_value.  Update regaddr to dio_addr
to match.

Fixes checkpatch warnings:

  CHECK: Avoid CamelCase: <regValue>
Signed-off-by: default avatarValentin Vidic <Valentin.Vidic@CARNet.hr>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent f53c5d92
...@@ -505,27 +505,27 @@ int rf69_set_dio_mapping(struct spi_device *spi, u8 DIONumber, u8 value) ...@@ -505,27 +505,27 @@ int rf69_set_dio_mapping(struct spi_device *spi, u8 DIONumber, u8 value)
{ {
u8 mask; u8 mask;
u8 shift; u8 shift;
u8 regaddr; u8 dio_addr;
u8 regValue; u8 dio_value;
switch (DIONumber) { switch (DIONumber) {
case 0: case 0:
mask = MASK_DIO0; shift = SHIFT_DIO0; regaddr = REG_DIOMAPPING1; mask = MASK_DIO0; shift = SHIFT_DIO0; dio_addr = REG_DIOMAPPING1;
break; break;
case 1: case 1:
mask = MASK_DIO1; shift = SHIFT_DIO1; regaddr = REG_DIOMAPPING1; mask = MASK_DIO1; shift = SHIFT_DIO1; dio_addr = REG_DIOMAPPING1;
break; break;
case 2: case 2:
mask = MASK_DIO2; shift = SHIFT_DIO2; regaddr = REG_DIOMAPPING1; mask = MASK_DIO2; shift = SHIFT_DIO2; dio_addr = REG_DIOMAPPING1;
break; break;
case 3: case 3:
mask = MASK_DIO3; shift = SHIFT_DIO3; regaddr = REG_DIOMAPPING1; mask = MASK_DIO3; shift = SHIFT_DIO3; dio_addr = REG_DIOMAPPING1;
break; break;
case 4: case 4:
mask = MASK_DIO4; shift = SHIFT_DIO4; regaddr = REG_DIOMAPPING2; mask = MASK_DIO4; shift = SHIFT_DIO4; dio_addr = REG_DIOMAPPING2;
break; break;
case 5: case 5:
mask = MASK_DIO5; shift = SHIFT_DIO5; regaddr = REG_DIOMAPPING2; mask = MASK_DIO5; shift = SHIFT_DIO5; dio_addr = REG_DIOMAPPING2;
break; break;
default: default:
dev_dbg(&spi->dev, "set: illegal input param"); dev_dbg(&spi->dev, "set: illegal input param");
...@@ -533,13 +533,13 @@ int rf69_set_dio_mapping(struct spi_device *spi, u8 DIONumber, u8 value) ...@@ -533,13 +533,13 @@ int rf69_set_dio_mapping(struct spi_device *spi, u8 DIONumber, u8 value)
} }
// read reg // read reg
regValue = rf69_read_reg(spi, regaddr); dio_value = rf69_read_reg(spi, dio_addr);
// delete old value // delete old value
regValue = regValue & ~mask; dio_value = dio_value & ~mask;
// add new value // add new value
regValue = regValue | value << shift; dio_value = dio_value | value << shift;
// write back // write back
return rf69_write_reg(spi, regaddr, regValue); return rf69_write_reg(spi, dio_addr, dio_value);
} }
bool rf69_get_flag(struct spi_device *spi, enum flag flag) bool rf69_get_flag(struct spi_device *spi, enum flag flag)
......
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