On Mon, Sep 17, 2018 at 06:32:52PM +0200, Tomislav Požega wrote: > Enable LNAs only for the current operating band. Change power > amplifiers enabling style to the one in vco calibration routine > and drop redundant cap_bt_coexist enable of PA0. <snip> > - rt2x00_set_field32(&tx_pin, TX_PIN_CFG_PA_PE_A0_EN, > - rf->channel > 14); > - if (rt2x00_has_cap_bt_coexist(rt2x00dev)) > + if (rt2x00dev->curr_band == NL80211_BAND_5GHZ) { > + switch (rt2x00dev->default_ant.tx_chain_num) { > + case 3: > + rt2x00_set_field32(&tx_pin, TX_PIN_CFG_PA_PE_A2_EN, 1); > + case 2: > + rt2x00_set_field32(&tx_pin, TX_PIN_CFG_PA_PE_A1_EN, 1); > + case 1: > + rt2x00_set_field32(&tx_pin, TX_PIN_CFG_PA_PE_A0_EN, 1); > + break; > + } > + } else { > + switch (rt2x00dev->default_ant.tx_chain_num) { > + case 3: > + rt2x00_set_field32(&tx_pin, TX_PIN_CFG_PA_PE_G2_EN, 1); > + case 2: > + rt2x00_set_field32(&tx_pin, TX_PIN_CFG_PA_PE_G1_EN, 1); > + case 1: So we never disable PA_PE_Gx_EN bits for RT6352 since tx_pin variable is not nulifed, but read from register. I don't think this was intended. Thanks Stanislaw