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

Commit 8afceb1e authored by Michael Buesch's avatar Michael Buesch Committed by John W. Linville
Browse files

[PATCH] bcm43xx: fix the remaining sparse warnings.

parent b5e868ed
Loading
Loading
Loading
Loading
+2 −2
Original line number Diff line number Diff line
@@ -2509,7 +2509,7 @@ static int bcm43xx_validate_chip(struct bcm43xx_private *bcm)
	return -ENODEV;
}

void bcm43xx_init_struct_phyinfo(struct bcm43xx_phyinfo *phy)
static void bcm43xx_init_struct_phyinfo(struct bcm43xx_phyinfo *phy)
{
	/* Initialize a "phyinfo" structure. The structure is already
	 * zeroed out.
@@ -2521,7 +2521,7 @@ void bcm43xx_init_struct_phyinfo(struct bcm43xx_phyinfo *phy)
	spin_lock_init(&phy->lock);
}

void bcm43xx_init_struct_radioinfo(struct bcm43xx_radioinfo *radio)
static void bcm43xx_init_struct_radioinfo(struct bcm43xx_radioinfo *radio)
{
	/* Initialize a "radioinfo" structure. The structure is already
	 * zeroed out.
+8 −3
Original line number Diff line number Diff line
@@ -1057,9 +1057,14 @@ static void bcm43xx_phy_initg(struct bcm43xx_private *bcm)
			bcm43xx_phy_write(bcm, 0x002F, 0x0202);
	}

	if ((bcm->sprom.boardflags & BCM43xx_BFL_RSSI) == 0) {
		FIXME();//FIXME: 0x7FFFFFFF should be 16-bit !
		bcm43xx_nrssi_hw_update(bcm, (u16)0x7FFFFFFF);
	if (!(bcm->sprom.boardflags & BCM43xx_BFL_RSSI)) {
		/* The specs state to update the NRSSI LT with
		 * the value 0x7FFFFFFF here. I think that is some weird
		 * compiler optimization in the original driver.
		 * Essentially, what we do here is resetting all NRSSI LT
		 * entries to -32 (see the limit_value() in nrssi_hw_update())
		 */
		bcm43xx_nrssi_hw_update(bcm, 0xFFFF);
		bcm43xx_calc_nrssi_threshold(bcm);
	} else if (phy->connected) {
		if (radio->nrssi[0] == -1000) {