Skip to contentSkip to conversation

staging: pi433: style fix - space after asterisk

PATCH In review Marcin Ciupak 2 participants 2 messages Aug 8, 2017 Staging

lore
Share & tools
lore

Marcin Ciupak

author
#0 lore raw

PATCH 1/1 staging: pi433: style fix - space after asterisk

This patch is intended to fix coding style issues in order to comply with kernel coding style guide as requested by TODO file.

It fixes the following checkpatch.pl error:

ERROR: "foo * bar" should be "foo *bar"

Note:
"WARNING: line over 80 characters" remains valid here and could be fixed by another patch.

Signed-off-by Marcin Ciupak <[email protected]>
---
 drivers/staging/pi433/rf69.c | 2 +-
 drivers/staging/pi433/rf69.h | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
drivers/staging/pi433/rf69.c +1 -1
index f83523e3395d..9f4e7173c688 100644
@@ -740,7 +740,7 @@ int rf69_set_sync_values(struct spi_device *spi, u8 syncValues[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 packetFormat packetFormat)
 {
 	#ifdef DEBUG
 		dev_dbg(&spi->dev, "set: packet format");
drivers/staging/pi433/rf69.h +1 -1
index b81e0762032e..fbccaae51c61 100644
@@ -61,7 +61,7 @@ int rf69_set_fifo_fill_condition(struct spi_device *spi, enum fifoFillCondition
 int rf69_set_sync_size(struct spi_device *spi, u8 sync_size);
 int rf69_set_sync_tolerance(struct spi_device *spi, u8 syncTolerance);
 int rf69_set_sync_values(struct spi_device *spi, u8 syncValues[8]);
-int rf69_set_packet_format(struct spi_device * spi, enum packetFormat packetFormat);
+int rf69_set_packet_format(struct spi_device *spi, enum packetFormat packetFormat);
 int rf69_set_crc_enable(struct spi_device *spi, enum optionOnOff optionOnOff);
 int rf69_set_adressFiltering(struct spi_device *spi, enum addressFiltering addressFiltering);
 int rf69_set_payload_length(struct spi_device *spi, u8 payloadLength);
Signature · 1 line
2.13.0
Reviewed-by on v1 1/1
#1 lore raw
Reviewed-by Marcus Wolf <[email protected]>

Thank you Marcin!

Quoted 58 lines

Marcin Ciupak <[email protected]> hat am 8. August 2017 um 15:54 geschrieben:

This patch is intended to fix coding style issues in order to comply with kernel coding style guide as requested by TODO file.

It fixes the following checkpatch.pl error:

ERROR: "foo * bar" should be "foo *bar"

Note:
"WARNING: line over 80 characters" remains valid here and could be fixed by another patch.

Signed-off-by Marcin Ciupak <[email protected]>

---
drivers/staging/pi433/rf69.c | 2 +-
drivers/staging/pi433/rf69.h | 2 +-

2 files changed, 2 insertions(+), 2 deletions(-)
drivers/staging/pi433/rf69.c +1 -1
index f83523e3395d..9f4e7173c688 100644
@@ -740,7 +740,7 @@ int rf69_set_sync_values(struct spi_device *spi, u8
 syncValues[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 packetFormat
 packetFormat)

{
#ifdef DEBUG
dev_dbg(&spi->dev, "set: packet format");

drivers/staging/pi433/rf69.h +0 -0
index b81e0762032e..fbccaae51c61 100644
@@ -61,7 +61,7 @@ int rf69_set_fifo_fill_condition(struct spi_device *spi,

enum fifoFillCondition
int rf69_set_sync_size(struct spi_device *spi, u8 sync_size);
int rf69_set_sync_tolerance(struct spi_device *spi, u8 syncTolerance); int rf69_set_sync_values(struct spi_device *spi, u8 syncValues[8]); -int rf69_set_packet_format(struct spi_device * spi, enum packetFormat packetFormat);
+int rf69_set_packet_format(struct spi_device *spi, enum packetFormat packetFormat);
int rf69_set_crc_enable(struct spi_device *spi, enum optionOnOff optionOnOff); int rf69_set_adressFiltering(struct spi_device *spi, enum addressFiltering addressFiltering);
int rf69_set_payload_length(struct spi_device *spi, u8 payloadLength); --
2.13.0