Re: [PATCH v2] b43: LP-PHY: Initialize TX power control

[Larry Finger <Larry.Finger@lwfinger.net>]

Gábor Stefanik wrote:
> The HW TX power control init still needs work.
> The SW init is complete according to the specs.
> 
> Signed-off-by: Gábor Stefanik <netrolller.3d@gmail.com>
> ---
> v2: Address Michael's comments, and update for the dummy transmission
> changes.
> 
> This depends on both the rev0/1 RC calibration patch and the
> updated V4 dummy transmission patch I submitted earlier.
> 
> Larry, please review the SPEC TODO/SPEC FIXME comments!
> 
> drivers/net/wireless/b43/phy_lp.c |  105
> ++++++++++++++++++++++++++++++++++---
> drivers/net/wireless/b43/phy_lp.h |    5 ++
> 2 files changed, 103 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/net/wireless/b43/phy_lp.c
> b/drivers/net/wireless/b43/phy_lp.c
> index 2441a8d..b4e51f0 100644
> --- a/drivers/net/wireless/b43/phy_lp.c
> +++ b/drivers/net/wireless/b43/phy_lp.c
> @@ -1257,18 +1257,109 @@ static void lpphy_calibration(struct b43_wldev
> *dev)
>     b43_mac_enable(dev);
> }
> 
> +static void lpphy_set_tssi_mux(struct b43_wldev *dev, enum
> tssi_mux_mode mode)
> +{
> +    if (mode != TSSI_MUX_EXT) {
> +        b43_radio_set(dev, B2063_PA_SP1, 0x2);
> +        b43_phy_set(dev, B43_PHY_OFDM(0xF3), 0x1000);
> +        b43_radio_write(dev, B2063_PA_CTL10, 0x51);
> +        if (mode == TSSI_MUX_POSTPA) {
> +            b43_radio_mask(dev, B2063_PA_SP1, 0xFFFE);
> +            b43_phy_mask(dev, B43_LPPHY_AFE_CTL_OVRVAL, 0xFFC7);
> +        } else {
> +            b43_radio_maskset(dev, B2063_PA_SP1, 0xFFFE, 0x1);
> +            b43_phy_maskset(dev, B43_LPPHY_AFE_CTL_OVRVAL,
> +                    0xFFC7, 0x20);
> +        }
> +    } else {
> +        //SPEC TODO

Put a WARN_ON(1) here. This branch would be an error. No, I don't
understand it either!

Larry