> > On further inspection not even that. :P > > The outcome of that switch is stored in the antenna variable, which is used with > > > > bcm43xx_mgmtframe_txantenna(dev, antenna); > > bcm43xx_set_rx_antenna(dev, antenna); > > > > So I guess that needs to be fixed to create a switch statement for > > the RX antenna and one for the TX antenna? > > Ah, yeah. I didn't have the complete code for this in my head. :) > > I'd rather like to have a helper function > int bcm43xx_antenna_from_ieee80211(u8 antenna) > which includes the switch statement and use it twice in the > config callback. Wanna do that? No problem, I will resend this patch with that helper function added. Ivo - To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
