From: Valentin Vidic Date: Mon, 5 Mar 2018 07:02:14 +0000 (+0100) Subject: staging: pi433: fix CamelCase for packetFormat enum X-Git-Tag: v4.19~1302^2~224 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=c436ef3b9abfcdb3b6db0c87f3d11d628cd0d426;p=platform%2Fkernel%2Flinux-rpi.git staging: pi433: fix CamelCase for packetFormat enum Fixes checkpatch warnings: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: CHECK: Avoid CamelCase: Signed-off-by: Valentin Vidic Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c index aad1debd..b1cbec8 100644 --- a/drivers/staging/pi433/pi433_if.c +++ b/drivers/staging/pi433/pi433_if.c @@ -222,11 +222,11 @@ rf69_set_rx_cfg(struct pi433_device *dev, struct pi433_rx_cfg *rx_cfg) return ret; } if (rx_cfg->enable_length_byte == OPTION_ON) { - ret = rf69_set_packet_format(dev->spi, packetLengthVar); + ret = rf69_set_packet_format(dev->spi, packet_length_var); if (ret < 0) return ret; } else { - ret = rf69_set_packet_format(dev->spi, packetLengthFix); + ret = rf69_set_packet_format(dev->spi, packet_length_fix); if (ret < 0) return ret; } @@ -337,11 +337,11 @@ rf69_set_tx_cfg(struct pi433_device *dev, struct pi433_tx_cfg *tx_cfg) } if (tx_cfg->enable_length_byte == OPTION_ON) { - ret = rf69_set_packet_format(dev->spi, packetLengthVar); + ret = rf69_set_packet_format(dev->spi, packet_length_var); if (ret < 0) return ret; } else { - ret = rf69_set_packet_format(dev->spi, packetLengthFix); + ret = rf69_set_packet_format(dev->spi, packet_length_fix); if (ret < 0) return ret; } diff --git a/drivers/staging/pi433/rf69.c b/drivers/staging/pi433/rf69.c index b2cea5f..54d8105 100644 --- a/drivers/staging/pi433/rf69.c +++ b/drivers/staging/pi433/rf69.c @@ -661,12 +661,12 @@ int rf69_set_sync_values(struct spi_device *spi, u8 sync_values[8]) return retval; } -int rf69_set_packet_format(struct spi_device *spi, enum packetFormat packetFormat) +int rf69_set_packet_format(struct spi_device *spi, enum packet_format packet_format) { - switch (packetFormat) { - case packetLengthVar: + switch (packet_format) { + case packet_length_var: return rf69_set_bit(spi, REG_PACKETCONFIG1, MASK_PACKETCONFIG1_PAKET_FORMAT_VARIABLE); - case packetLengthFix: + case packet_length_fix: return rf69_clear_bit(spi, REG_PACKETCONFIG1, MASK_PACKETCONFIG1_PAKET_FORMAT_VARIABLE); default: dev_dbg(&spi->dev, "set: illegal input param"); diff --git a/drivers/staging/pi433/rf69.h b/drivers/staging/pi433/rf69.h index cf89b55..7b7932a 100644 --- a/drivers/staging/pi433/rf69.h +++ b/drivers/staging/pi433/rf69.h @@ -50,7 +50,7 @@ int rf69_disable_sync(struct spi_device *spi); int rf69_set_fifo_fill_condition(struct spi_device *spi, enum fifo_fill_condition fifo_fill_condition); int rf69_set_sync_size(struct spi_device *spi, u8 sync_size); int rf69_set_sync_values(struct spi_device *spi, u8 sync_values[8]); -int rf69_set_packet_format(struct spi_device *spi, enum packetFormat packetFormat); +int rf69_set_packet_format(struct spi_device *spi, enum packet_format packet_format); int rf69_enable_crc(struct spi_device *spi); int rf69_disable_crc(struct spi_device *spi); int rf69_set_address_filtering(struct spi_device *spi, enum address_filtering address_filtering); diff --git a/drivers/staging/pi433/rf69_enum.h b/drivers/staging/pi433/rf69_enum.h index cba3c5b..682b667 100644 --- a/drivers/staging/pi433/rf69_enum.h +++ b/drivers/staging/pi433/rf69_enum.h @@ -118,9 +118,9 @@ enum fifo_fill_condition { always }; -enum packetFormat { - packetLengthFix, - packetLengthVar +enum packet_format { + packet_length_fix, + packet_length_var }; enum tx_start_condition {