Commit c436ef3b authored by Valentin Vidic's avatar Valentin Vidic Committed by Greg Kroah-Hartman
Browse files

staging: pi433: fix CamelCase for packetFormat enum 



Fixes checkpatch warnings:

  CHECK: Avoid CamelCase: <packetFormat>
  CHECK: Avoid CamelCase: <packetLengthFix>
  CHECK: Avoid CamelCase: <packetLengthVar>

Signed-off-by: default avatarValentin Vidic <Valentin.Vidic@CARNet.hr>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent e0c12b6f

drivers/staging/pi433/pi433_if.c

+4 −4
Original line number Diff line number Diff line
@@ -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;

	}

drivers/staging/pi433/rf69.c

+4 −4
Original line number Diff line number Diff line
@@ -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");

drivers/staging/pi433/rf69.h

+1 −1
Original line number Diff line number Diff line
@@ -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);

drivers/staging/pi433/rf69_enum.h

+3 −3
Original line number Diff line number Diff line
@@ -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 {

CRA Git | Maintained and supported by SUSTech CRA and CCSE