Commit 14c76150 authored by andrea merello's avatar andrea merello Committed by John W. Linville

rtl818x: add comments to explain few not obvious HW configs.

Certain HW options (TX packet retry count, CW configuration and
TX power configuration) can be specified in both the TX packet
descriptor and also into HW "global" registers.

The HW is thus configured to honour the global register or the
TX descriptor field depending by the case.

This patch adds few comments that hopefully clarify in which cases
the driver uses one method and in which cases it uses the other.
Signed-off-by: default avatarAndrea Merello <andrea.merello@gmail.com>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent 4da18bb4
...@@ -627,11 +627,23 @@ static int rtl8180_start(struct ieee80211_hw *dev) ...@@ -627,11 +627,23 @@ static int rtl8180_start(struct ieee80211_hw *dev)
if (priv->r8185) { if (priv->r8185) {
reg = rtl818x_ioread8(priv, &priv->map->CW_CONF); reg = rtl818x_ioread8(priv, &priv->map->CW_CONF);
/* CW is not on per-packet basis.
* in rtl8185 the CW_VALUE reg is used.
*/
reg &= ~RTL818X_CW_CONF_PERPACKET_CW; reg &= ~RTL818X_CW_CONF_PERPACKET_CW;
/* retry limit IS on per-packet basis.
* the short and long retry limit in TX_CONF
* reg are ignored
*/
reg |= RTL818X_CW_CONF_PERPACKET_RETRY; reg |= RTL818X_CW_CONF_PERPACKET_RETRY;
rtl818x_iowrite8(priv, &priv->map->CW_CONF, reg); rtl818x_iowrite8(priv, &priv->map->CW_CONF, reg);
reg = rtl818x_ioread8(priv, &priv->map->TX_AGC_CTL); reg = rtl818x_ioread8(priv, &priv->map->TX_AGC_CTL);
/* TX antenna and TX gain are not on per-packet basis.
* TX Antenna is selected by ANTSEL reg (RX in BB regs).
* TX gain is selected with CCK_TX_AGC and OFDM_TX_AGC regs
*/
reg &= ~RTL818X_TX_AGC_CTL_PERPACKET_GAIN; reg &= ~RTL818X_TX_AGC_CTL_PERPACKET_GAIN;
reg &= ~RTL818X_TX_AGC_CTL_PERPACKET_ANTSEL; reg &= ~RTL818X_TX_AGC_CTL_PERPACKET_ANTSEL;
reg |= RTL818X_TX_AGC_CTL_FEEDBACK_ANT; reg |= RTL818X_TX_AGC_CTL_FEEDBACK_ANT;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment