aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGustavo A. R. Silva <gustavo@embeddedor.com>2018-08-01 19:31:01 -0500
committerDavid S. Miller <davem@davemloft.net>2018-08-02 14:43:04 -0700
commitd3e2a25bcd0b24f0905554ef0762eb77cbca7ad5 (patch)
tree757dfebfe14ef2cb406165a22a1813cdb8eed08b
parentnet: hns: remove redundant variables 'max_frm' and 'tmp_mac_key' (diff)
downloadlinux-dev-d3e2a25bcd0b24f0905554ef0762eb77cbca7ad5.tar.xz
linux-dev-d3e2a25bcd0b24f0905554ef0762eb77cbca7ad5.zip
xen-netback: use true and false for boolean values
Return statements in functions returning bool should use true or false instead of an integer value. This issue was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com> Acked-by: Wei Liu <wei.liu2@citrix.com> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--drivers/net/xen-netback/netback.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/xen-netback/netback.c b/drivers/net/xen-netback/netback.c
index a27daa23c9dc..3621e05a7494 100644
--- a/drivers/net/xen-netback/netback.c
+++ b/drivers/net/xen-netback/netback.c
@@ -1603,9 +1603,9 @@ static void xenvif_ctrl_action(struct xenvif *vif)
static bool xenvif_ctrl_work_todo(struct xenvif *vif)
{
if (likely(RING_HAS_UNCONSUMED_REQUESTS(&vif->ctrl)))
- return 1;
+ return true;
- return 0;
+ return false;
}
irqreturn_t xenvif_ctrl_irq_fn(int irq, void *data)