staging: pi433: Comply with 80 character column limit

Lines have been split where it makes sense to shorten them in order
to comply with the coding standards and fix checkpatch.pl warnings.
There are still lines left that are too long, however breaking
those would impair readability.

Changes in v2:
 - now working on staging-next branch of the staging tree
 - the changes to the defines previously made are deleted
   due to readability

Signed-off-by: Sophie Matter <sophie.matter@web.de>
Signed-off-by: Rico Schrage <rico.schrage@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c
index 293602d87c0fdd5d3a469f2ba870ee61153f5e19..cced28afe45481fad01f5dd74f96eafce1ad11e4 100644 (file)
--- a/drivers/staging/pi433/pi433_if.c
+++ b/drivers/staging/pi433/pi433_if.c
@@ -826,11 +826,15 @@ pi433_write(struct file *filp, const char __user *buf,
        instance = filp->private_data;
        device = instance->device;
 
-       /* check, whether internal buffer (tx thread) is big enough for requested size */
+       /*
+        * check, whether internal buffer (tx thread) is big enough
+        * for requested size
+        */
        if (count > MAX_MSG_SIZE)
                return -EMSGSIZE;
 
-       /* write the following sequence into fifo:
+       /*
+        * write the following sequence into fifo:
         * - tx_cfg
         * - size of message
         * - message
@@ -1116,7 +1120,10 @@ static int pi433_probe(struct spi_device *spi)
        /* setup spi parameters */
        spi->mode = 0x00;
        spi->bits_per_word = 8;
-       /* spi->max_speed_hz = 10000000;  1MHz already set by device tree overlay */
+       /*
+        * spi->max_speed_hz = 10000000;
+        * 1MHz already set by device tree overlay
+        */
 
        retval = spi_setup(spi);
        if (retval) {

diff --git a/drivers/staging/pi433/pi433_if.h b/drivers/staging/pi433/pi433_if.h
index 0e0c1b0ab1a6013134056e23525a4bc94bdf15b6..2d4fa77c793e6a4e9a68de3b132330119b885a25 100644 (file)
--- a/drivers/staging/pi433/pi433_if.h
+++ b/drivers/staging/pi433/pi433_if.h
@@ -43,7 +43,8 @@ enum option_on_off {
 /* IOCTL structs and commands */
 
 /**
- * struct pi433_tx_config - describes the configuration of the radio module for sending
+ * struct pi433_tx_config
+ * describes the configuration of the radio module for sending
  * @frequency:
  * @bit_rate:
  * @modulation:
@@ -90,7 +91,8 @@ struct pi433_tx_cfg {
 };
 
 /**
- * struct pi433_rx_config - describes the configuration of the radio module for sending
+ * struct pi433_rx_config
+ * describes the configuration of the radio module for sending
  * @frequency:
  * @bit_rate:
  * @modulation:

diff --git a/drivers/staging/pi433/rf69.c b/drivers/staging/pi433/rf69.c
index 724c24ce142802cd88e44985ea34af2646c16272..77e0a677ea96441107e3537036ad55a38d3638e3 100644 (file)
--- a/drivers/staging/pi433/rf69.c
+++ b/drivers/staging/pi433/rf69.c
@@ -128,9 +128,13 @@ int rf69_set_mode(struct spi_device *spi, enum mode mode)
        return rf69_read_mod_write(spi, REG_OPMODE, MASK_OPMODE_MODE,
                                   mode_map[mode]);
 
-       // we are using packet mode, so this check is not really needed
-       // but waiting for mode ready is necessary when going from sleep because the FIFO may not be immediately available from previous mode
-       //while (_mode == RF69_MODE_SLEEP && (READ_REG(REG_IRQFLAGS1) & RF_IRQFLAGS1_MODEREADY) == 0x00); // Wait for ModeReady
+       /*
+        * we are using packet mode, so this check is not really needed
+        * but waiting for mode ready is necessary when going from sleep
+        * because the FIFO may not be immediately available from previous mode
+        * while (_mode == RF69_MODE_SLEEP && (READ_REG(REG_IRQFLAGS1) &
+                 RF_IRQFLAGS1_MODEREADY) == 0x00); // Wait for ModeReady
+        */
 }
 
 int rf69_set_data_mode(struct spi_device *spi, u8 data_mode)