Jakub Sitnicki <[email protected]> writes:
> As opposed to the vendor driver (rtl8188eu), extract from efuse only
> these TX power values that are going to be of use later on.
>
> Signed-off-by: Jakub Sitnicki <[email protected]>
> ---
>  drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 35 
> +++++++++++++++++++++++-
>  drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.h | 24 ++++++++++++++++
>  2 files changed, 58 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c 
> b/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c
> index 344161b..90da707 100644
> --- a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c
> +++ b/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c
> @@ -1900,7 +1900,40 @@ static int rtl8192cu_parse_efuse(struct rtl8xxxu_priv 
> *priv)
>  
>  static int rtl8188eu_parse_efuse(struct rtl8xxxu_priv *priv)
>  {
> -     return -EOPNOTSUPP; /* Not implemented */
> +     struct rtl8188eu_efuse *efuse = &priv->efuse_wifi.efuse8188;
> +     struct rtl8188eu_tx_power *tx_power = &priv->tx_power.tx_power8188;
> +
> +     if (efuse->rtl_id != cpu_to_le16(0x8129))
> +             return -EINVAL;
> +
> +     ether_addr_copy(priv->mac_addr, efuse->mac_addr);
> +
> +     memcpy(tx_power->cck_index_A,
> +            efuse->tx_power.cck_index_A,
> +            sizeof(tx_power->cck_index_A));
> +
> +     memcpy(tx_power->ht40_1s_index_A,
> +            efuse->tx_power.ht40_1s_index_A,
> +            sizeof(tx_power->ht40_1s_index_A));
> +
> +     memcpy(tx_power->ht20_ofdm_index_diff,
> +            efuse->tx_power.ht20_ofdm_index_diff,
> +            sizeof(tx_power->ht20_ofdm_index_diff));
> +
> +     dev_info(&priv->udev->dev, "Vendor: %.7s\n",
> +              efuse->vendor_name);
> +     dev_info(&priv->udev->dev, "Product: %.11s\n",
> +              efuse->device_name);
> +
> +     /* XXX: Do we need to read the rf_regulatory flag? */
> +
> +     if (rtl8xxxu_debug & RTL8XXXU_DEBUG_EFUSE) {
> +             print_hex_dump_bytes("rtl8xxxu: ", DUMP_PREFIX_OFFSET,
> +                                  priv->efuse_wifi.raw,
> +                                  sizeof(priv->efuse_wifi.raw));
> +     }
> +
> +     return 0;
>  }
>  
>  static int
> diff --git a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.h 
> b/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.h
> index 79f47e3..ec49504 100644
> --- a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.h
> +++ b/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.h
> @@ -507,6 +507,22 @@ struct rtl8192cu_efuse {
>       u8 customer_id;
>  };
>  
> +struct rtl8188eu_efuse {
> +     __le16 rtl_id;
> +     u8 res0[0x0e];
> +     struct {                                                /* 0x10 */
> +             u8 cck_index_A[6];
> +             u8 ht40_1s_index_A[5];
> +             struct nibble_pair ht20_ofdm_index_diff[1];
> +     } tx_power;
> +     u8 res1[0xbb];
> +     u8 mac_addr[ETH_ALEN];                                  /* 0xd7 */
> +     u8 res2[0x02];
> +     u8 vendor_name[7];                                      /* 0xdf */
> +     u8 res3[0x02];
> +     u8 device_name[11];                                     /* 0xe8 */
> +};

This is incomplete - I think it should look more like this:

struct rtl8188eu_efuse {
        __le16 rtl_id;
        u8 res0[0x0e];
        u8 cck_tx_power_index_A[3];     /* 0x10 */
        u8 cck_tx_power_index_B[3];
        u8 ht40_1s_tx_power_index_A[3]; /* 0x16 */
        u8 ht40_1s_tx_power_index_B[3];
        u8 res1[0x9c];
        u8 channel_plan;                /* 0xb8 */
        u8 xtal_k;
        u8 thermal_meter;
        u8 iqk_lck;
        u8 res2[5];
        u8 rf_board_option;
        u8 rf_feature_option;
        u8 rf_bt_setting;
        u8 eeprom_version;
        u8 eeprom_customer_id;
        u8 res3[3];
        u8 rf_antenna_option;           /* 0xc9 */
        u8 res4[6];
        u8 vid;                         /* 0xd0 */
        u8 res5[1];
        u8 pid;                         /* 0xd2 */
        u8 res6[1];
        u8 usb_optional_function;
        u8 res7[2];
        u8 mac_addr[ETH_ALEN];          /* 0xd7 */
        u8 res8[2];
        u8 vendor_name[7];
        u8 res9[2];
        u8 device_name[0x0b];           /* 0xe8 */
        u8 res10[2];
        u8 serial[0x0b];                /* 0xf5 */
        u8 res11[0x30];
        u8 unknown[0x0d];               /* 0x130 */
        u8 res12[0xc3];
};

> +
>  struct rtl8xxxu_reg8val {
>       u16 reg;
>       u8 val;
> @@ -587,6 +603,12 @@ struct rtl8723au_tx_power {
>       struct nibble_pair ht20_max_offset[3];
>  };
>  
> +struct rtl8188eu_tx_power {
> +     u8 cck_index_A[6];
> +     u8 ht40_1s_index_A[5];
> +     struct nibble_pair ht20_ofdm_index_diff[1];
> +};
> +
>  struct rtl8xxxu_priv {
>       struct ieee80211_hw *hw;
>       struct usb_device *udev;
> @@ -607,6 +629,7 @@ struct rtl8xxxu_priv {
>       char chip_name[8];
>       union {
>               struct rtl8723au_tx_power tx_power8723;
> +             struct rtl8188eu_tx_power tx_power8188;
>       } tx_power;
>       u32 chip_cut:4;
>       u32 rom_rev:4;
> @@ -659,6 +682,7 @@ struct rtl8xxxu_priv {
>               u8 raw[EFUSE_MAP_LEN];
>               struct rtl8723au_efuse efuse8723;
>               struct rtl8192cu_efuse efuse8192;
> +             struct rtl8188eu_efuse efuse8188;
>       } efuse_wifi;
>       u32 adda_backup[RTL8XXXU_ADDA_REGS];
>       u32 mac_backup[RTL8XXXU_MAC_REGS];

There is a lot of conflict with the last couple of patches - please
rebase them against current.

Thanks,
Jes
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to