Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d01a4e04 authored by Colin Ian King's avatar Colin Ian King Committed by Kalle Valo
Browse files

airo: remove unused variables len and dev and clean up formatting 



Variables len and dev assigned values but are never used hence they
are redundant and can be removed.  Also add in white space in for-loop
and remove some { } braces on if statement.

Cleans up clang warnings:
warning: variable 'len' set but not used [-Wunused-but-set-variable]
warning: variable 'dev' set but not used [-Wunused-but-set-variable]

Signed-off-by: default avatarColin Ian King <colin.king@canonical.com>
Signed-off-by: default avatarKalle Valo <kvalo@codeaurora.org>
parent 2ddd82ee

drivers/net/wireless/cisco/airo.c

+3 −5
Original line number Diff line number Diff line
@@ -3419,7 +3419,7 @@ static void airo_handle_rx(struct airo_info *ai) 


static void airo_handle_tx(struct airo_info *ai, u16 status)

{

	int i, len = 0, index = -1;

	int i, index = -1;

	u16 fid;



	if (test_bit(FLAG_MPI, &ai->flags)) {
@@ -3444,11 +3444,9 @@ static void airo_handle_tx(struct airo_info *ai, u16 status) 
	fid = IN4500(ai, TXCOMPLFID);



	for (i = 0; i < MAX_FIDS; i++) {

		if ((ai->fids[i] & 0xffff) == fid) {

			len = ai->fids[i] >> 16;

		if ((ai->fids[i] & 0xffff) == fid)

			index = i;

	}

	}



	if (index != -1) {

		if (status & EV_TXEXC)

drivers/net/wireless/cisco/airo_cs.c

+0 −3
Original line number Diff line number Diff line
@@ -102,11 +102,8 @@ static int airo_cs_config_check(struct pcmcia_device *p_dev, void *priv_data) 


static int airo_config(struct pcmcia_device *link)

{

	struct local_info *dev;

	int ret;



	dev = link->priv;



	dev_dbg(&link->dev, "airo_config\n");



	link->config_flags |= CONF_ENABLE_IRQ | CONF_AUTO_SET_VPP |