From: Sven Leykauf <sven_leykauf1@web.de>
To: devel@driverdev.osuosl.org
Cc: Daniel Bauer <daniel.j.bauer@fau.de>,
Sven Leykauf <sven_leykauf1@web.de>,
linux-kernel@i4.cs.fau.de
Subject: [PATCH 2/2] pi433: Fix indentation according to coding style
Date: Tue, 3 Dec 2019 18:54:49 +0100 [thread overview]
Message-ID: <20191203175447.28765-2-sven_leykauf1@web.de> (raw)
In-Reply-To: <20191203175447.28765-1-sven_leykauf1@web.de>
Fix indentation so that no line exceeds the 80 character border.
Put the return command one line under the default case, so it looks
better.
Co-developed-by: Daniel Bauer <daniel.j.bauer@fau.de>
Signed-off-by: Daniel Bauer <daniel.j.bauer@fau.de>
Signed-off-by: Sven Leykauf <sven_leykauf1@web.de>
---
drivers/staging/pi433/rf69.c | 56 ++++++++++++++++++++++++------------
1 file changed, 37 insertions(+), 19 deletions(-)
diff --git a/drivers/staging/pi433/rf69.c b/drivers/staging/pi433/rf69.c
index 6b13f92028c7..6cdd46682aa9 100644
--- a/drivers/staging/pi433/rf69.c
+++ b/drivers/staging/pi433/rf69.c
@@ -596,42 +596,60 @@ bool rf69_get_flag(struct spi_device *spi, enum flag flag)
{
switch (flag) {
case mode_switch_completed:
- return (rf69_read_reg(spi, REG_IRQFLAGS1) & MASK_IRQFLAGS1_MODE_READY);
+ return (rf69_read_reg(spi, REG_IRQFLAGS1) &
+ MASK_IRQFLAGS1_MODE_READY);
case ready_to_receive:
- return (rf69_read_reg(spi, REG_IRQFLAGS1) & MASK_IRQFLAGS1_RX_READY);
+ return (rf69_read_reg(spi, REG_IRQFLAGS1) &
+ MASK_IRQFLAGS1_RX_READY);
case ready_to_send:
- return (rf69_read_reg(spi, REG_IRQFLAGS1) & MASK_IRQFLAGS1_TX_READY);
+ return (rf69_read_reg(spi, REG_IRQFLAGS1) &
+ MASK_IRQFLAGS1_TX_READY);
case pll_locked:
- return (rf69_read_reg(spi, REG_IRQFLAGS1) & MASK_IRQFLAGS1_PLL_LOCK);
+ return (rf69_read_reg(spi, REG_IRQFLAGS1) &
+ MASK_IRQFLAGS1_PLL_LOCK);
case rssi_exceeded_threshold:
- return (rf69_read_reg(spi, REG_IRQFLAGS1) & MASK_IRQFLAGS1_RSSI);
+ return (rf69_read_reg(spi, REG_IRQFLAGS1) &
+ MASK_IRQFLAGS1_RSSI);
case timeout:
- return (rf69_read_reg(spi, REG_IRQFLAGS1) & MASK_IRQFLAGS1_TIMEOUT);
+ return (rf69_read_reg(spi, REG_IRQFLAGS1) &
+ MASK_IRQFLAGS1_TIMEOUT);
case automode:
- return (rf69_read_reg(spi, REG_IRQFLAGS1) & MASK_IRQFLAGS1_AUTOMODE);
+ return (rf69_read_reg(spi, REG_IRQFLAGS1) &
+ MASK_IRQFLAGS1_AUTOMODE);
case sync_address_match:
- return (rf69_read_reg(spi, REG_IRQFLAGS1) & MASK_IRQFLAGS1_SYNC_ADDRESS_MATCH);
+ return (rf69_read_reg(spi, REG_IRQFLAGS1) &
+ MASK_IRQFLAGS1_SYNC_ADDRESS_MATCH);
case fifo_full:
- return (rf69_read_reg(spi, REG_IRQFLAGS2) & MASK_IRQFLAGS2_FIFO_FULL);
+ return (rf69_read_reg(spi, REG_IRQFLAGS2) &
+ MASK_IRQFLAGS2_FIFO_FULL);
/*
* case fifo_not_empty:
- * return (rf69_read_reg(spi, REG_IRQFLAGS2) & MASK_IRQFLAGS2_FIFO_NOT_EMPTY);
+ * return (rf69_read_reg(spi, REG_IRQFLAGS2) &
+ * MASK_IRQFLAGS2_FIFO_NOT_EMPTY);
*/
case fifo_empty:
- return !(rf69_read_reg(spi, REG_IRQFLAGS2) & MASK_IRQFLAGS2_FIFO_NOT_EMPTY);
+ return !(rf69_read_reg(spi, REG_IRQFLAGS2) &
+ MASK_IRQFLAGS2_FIFO_NOT_EMPTY);
case fifo_level_below_threshold:
- return (rf69_read_reg(spi, REG_IRQFLAGS2) & MASK_IRQFLAGS2_FIFO_LEVEL);
+ return (rf69_read_reg(spi, REG_IRQFLAGS2) &
+ MASK_IRQFLAGS2_FIFO_LEVEL);
case fifo_overrun:
- return (rf69_read_reg(spi, REG_IRQFLAGS2) & MASK_IRQFLAGS2_FIFO_OVERRUN);
+ return (rf69_read_reg(spi, REG_IRQFLAGS2) &
+ MASK_IRQFLAGS2_FIFO_OVERRUN);
case packet_sent:
- return (rf69_read_reg(spi, REG_IRQFLAGS2) & MASK_IRQFLAGS2_PACKET_SENT);
+ return (rf69_read_reg(spi, REG_IRQFLAGS2) &
+ MASK_IRQFLAGS2_PACKET_SENT);
case payload_ready:
- return (rf69_read_reg(spi, REG_IRQFLAGS2) & MASK_IRQFLAGS2_PAYLOAD_READY);
+ return (rf69_read_reg(spi, REG_IRQFLAGS2) &
+ MASK_IRQFLAGS2_PAYLOAD_READY);
case crc_ok:
- return (rf69_read_reg(spi, REG_IRQFLAGS2) & MASK_IRQFLAGS2_CRC_OK);
+ return (rf69_read_reg(spi, REG_IRQFLAGS2) &
+ MASK_IRQFLAGS2_CRC_OK);
case battery_low:
- return (rf69_read_reg(spi, REG_IRQFLAGS2) & MASK_IRQFLAGS2_LOW_BAT);
- default: return false;
+ return (rf69_read_reg(spi, REG_IRQFLAGS2) &
+ MASK_IRQFLAGS2_LOW_BAT);
+ default:
+ return false;
}
}
@@ -725,7 +743,7 @@ int rf69_set_packet_format(struct spi_device *spi,
MASK_PACKETCONFIG1_PACKET_FORMAT_VARIABLE);
case packet_length_fix:
return rf69_clear_bit(spi, REG_PACKETCONFIG1,
- MASK_PACKETCONFIG1_PACKET_FORMAT_VARIABLE);
+ MASK_PACKETCONFIG1_PACKET_FORMAT_VARIABLE);
default:
dev_dbg(&spi->dev, "set: illegal input param");
return -EINVAL;
--
2.20.1
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
next prev parent reply other threads:[~2019-12-03 18:00 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-12-03 17:54 [PATCH 1/2] pi433: Fix indentation according to coding style Sven Leykauf
2019-12-03 17:54 ` Sven Leykauf [this message]
2019-12-03 18:14 ` [PATCH 2/2] " Dan Carpenter
2019-12-03 18:30 ` Dan Carpenter
2019-12-03 18:06 ` [PATCH 1/2] " Dan Carpenter
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20191203175447.28765-2-sven_leykauf1@web.de \
--to=sven_leykauf1@web.de \
--cc=daniel.j.bauer@fau.de \
--cc=devel@driverdev.osuosl.org \
--cc=linux-kernel@i4.cs.fau.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).