mwifiex: Reduce endian conversion for REG Host Commands
authorPrasun Maiti <prasunmaiti87@gmail.com>
Mon, 27 Jun 2016 10:13:22 +0000 (15:43 +0530)
committerKalle Valo <kvalo@codeaurora.org>
Mon, 18 Jul 2016 19:32:35 +0000 (22:32 +0300)
For multiple REG Host Commands (e.g HostCmd_CMD_802_11_EEPROM_ACCESS,
HostCmd_CMD_MAC_REG_ACCESS etc.) "cpu_to_leX"-converted values are
saved to driver. So, "leX_to_cpu" conversion is required too many
times afterwards in driver.

This patch reduces the endian: conversion without saving "cpu_to_leX"
converted values in driver. This will convert endianness in prepare
command and command response path.

Signed-off-by: Prasun Maiti <prasunmaiti87@gmail.com>
Acked-by: Amitkumar Karwar <akarwar@marvell.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/marvell/mwifiex/ioctl.h
drivers/net/wireless/marvell/mwifiex/sta_cmd.c
drivers/net/wireless/marvell/mwifiex/sta_cmdresp.c
drivers/net/wireless/marvell/mwifiex/sta_ioctl.c

index f5b8fd1fad1b3117b6142d05e179fed59779605a..70429815ff53e96a8c713c6c4844f61c03637a3e 100644 (file)
@@ -343,16 +343,16 @@ enum {
 };
 
 struct mwifiex_ds_reg_rw {
-       __le32 type;
-       __le32 offset;
-       __le32 value;
+       u32 type;
+       u32 offset;
+       u32 value;
 };
 
 #define MAX_EEPROM_DATA 256
 
 struct mwifiex_ds_read_eeprom {
-       __le16 offset;
-       __le16 byte_count;
+       u16 offset;
+       u16 byte_count;
        u8 value[MAX_EEPROM_DATA];
 };
 
index 8c658495bf6608b64c95d6fc49777659d5d7f31d..7897037b0992bc34e7212962c7e98c7d936ca50d 100644 (file)
@@ -1148,9 +1148,8 @@ static int mwifiex_cmd_reg_access(struct host_cmd_ds_command *cmd,
                cmd->size = cpu_to_le16(sizeof(*mac_reg) + S_DS_GEN);
                mac_reg = &cmd->params.mac_reg;
                mac_reg->action = cpu_to_le16(cmd_action);
-               mac_reg->offset =
-                       cpu_to_le16((u16) le32_to_cpu(reg_rw->offset));
-               mac_reg->value = reg_rw->value;
+               mac_reg->offset = cpu_to_le16((u16) reg_rw->offset);
+               mac_reg->value = cpu_to_le32(reg_rw->value);
                break;
        }
        case HostCmd_CMD_BBP_REG_ACCESS:
@@ -1160,9 +1159,8 @@ static int mwifiex_cmd_reg_access(struct host_cmd_ds_command *cmd,
                cmd->size = cpu_to_le16(sizeof(*bbp_reg) + S_DS_GEN);
                bbp_reg = &cmd->params.bbp_reg;
                bbp_reg->action = cpu_to_le16(cmd_action);
-               bbp_reg->offset =
-                       cpu_to_le16((u16) le32_to_cpu(reg_rw->offset));
-               bbp_reg->value = (u8) le32_to_cpu(reg_rw->value);
+               bbp_reg->offset = cpu_to_le16((u16) reg_rw->offset);
+               bbp_reg->value = (u8) reg_rw->value;
                break;
        }
        case HostCmd_CMD_RF_REG_ACCESS:
@@ -1172,8 +1170,8 @@ static int mwifiex_cmd_reg_access(struct host_cmd_ds_command *cmd,
                cmd->size = cpu_to_le16(sizeof(*rf_reg) + S_DS_GEN);
                rf_reg = &cmd->params.rf_reg;
                rf_reg->action = cpu_to_le16(cmd_action);
-               rf_reg->offset = cpu_to_le16((u16) le32_to_cpu(reg_rw->offset));
-               rf_reg->value = (u8) le32_to_cpu(reg_rw->value);
+               rf_reg->offset = cpu_to_le16((u16) reg_rw->offset);
+               rf_reg->value = (u8) reg_rw->value;
                break;
        }
        case HostCmd_CMD_PMIC_REG_ACCESS:
@@ -1183,9 +1181,8 @@ static int mwifiex_cmd_reg_access(struct host_cmd_ds_command *cmd,
                cmd->size = cpu_to_le16(sizeof(*pmic_reg) + S_DS_GEN);
                pmic_reg = &cmd->params.pmic_reg;
                pmic_reg->action = cpu_to_le16(cmd_action);
-               pmic_reg->offset =
-                               cpu_to_le16((u16) le32_to_cpu(reg_rw->offset));
-               pmic_reg->value = (u8) le32_to_cpu(reg_rw->value);
+               pmic_reg->offset = cpu_to_le16((u16) reg_rw->offset);
+               pmic_reg->value = (u8) reg_rw->value;
                break;
        }
        case HostCmd_CMD_CAU_REG_ACCESS:
@@ -1195,9 +1192,8 @@ static int mwifiex_cmd_reg_access(struct host_cmd_ds_command *cmd,
                cmd->size = cpu_to_le16(sizeof(*cau_reg) + S_DS_GEN);
                cau_reg = &cmd->params.rf_reg;
                cau_reg->action = cpu_to_le16(cmd_action);
-               cau_reg->offset =
-                               cpu_to_le16((u16) le32_to_cpu(reg_rw->offset));
-               cau_reg->value = (u8) le32_to_cpu(reg_rw->value);
+               cau_reg->offset = cpu_to_le16((u16) reg_rw->offset);
+               cau_reg->value = (u8) reg_rw->value;
                break;
        }
        case HostCmd_CMD_802_11_EEPROM_ACCESS:
@@ -1208,8 +1204,8 @@ static int mwifiex_cmd_reg_access(struct host_cmd_ds_command *cmd,
 
                cmd->size = cpu_to_le16(sizeof(*cmd_eeprom) + S_DS_GEN);
                cmd_eeprom->action = cpu_to_le16(cmd_action);
-               cmd_eeprom->offset = rd_eeprom->offset;
-               cmd_eeprom->byte_count = rd_eeprom->byte_count;
+               cmd_eeprom->offset = cpu_to_le16(rd_eeprom->offset);
+               cmd_eeprom->byte_count = cpu_to_le16(rd_eeprom->byte_count);
                cmd_eeprom->value = 0;
                break;
        }
index 9050d06998e63421d85ab9fd6a794c8638cd45d5..ccf54932e32185ec5508306c954b1a7fc8fe3416 100644 (file)
@@ -786,45 +786,44 @@ static int mwifiex_ret_reg_access(u16 type, struct host_cmd_ds_command *resp,
        switch (type) {
        case HostCmd_CMD_MAC_REG_ACCESS:
                r.mac = &resp->params.mac_reg;
-               reg_rw->offset = cpu_to_le32((u32) le16_to_cpu(r.mac->offset));
-               reg_rw->value = r.mac->value;
+               reg_rw->offset = (u32) le16_to_cpu(r.mac->offset);
+               reg_rw->value = le32_to_cpu(r.mac->value);
                break;
        case HostCmd_CMD_BBP_REG_ACCESS:
                r.bbp = &resp->params.bbp_reg;
-               reg_rw->offset = cpu_to_le32((u32) le16_to_cpu(r.bbp->offset));
-               reg_rw->value = cpu_to_le32((u32) r.bbp->value);
+               reg_rw->offset = (u32) le16_to_cpu(r.bbp->offset);
+               reg_rw->value = (u32) r.bbp->value;
                break;
 
        case HostCmd_CMD_RF_REG_ACCESS:
                r.rf = &resp->params.rf_reg;
-               reg_rw->offset = cpu_to_le32((u32) le16_to_cpu(r.rf->offset));
-               reg_rw->value = cpu_to_le32((u32) r.bbp->value);
+               reg_rw->offset = (u32) le16_to_cpu(r.rf->offset);
+               reg_rw->value = (u32) r.bbp->value;
                break;
        case HostCmd_CMD_PMIC_REG_ACCESS:
                r.pmic = &resp->params.pmic_reg;
-               reg_rw->offset = cpu_to_le32((u32) le16_to_cpu(r.pmic->offset));
-               reg_rw->value = cpu_to_le32((u32) r.pmic->value);
+               reg_rw->offset = (u32) le16_to_cpu(r.pmic->offset);
+               reg_rw->value = (u32) r.pmic->value;
                break;
        case HostCmd_CMD_CAU_REG_ACCESS:
                r.rf = &resp->params.rf_reg;
-               reg_rw->offset = cpu_to_le32((u32) le16_to_cpu(r.rf->offset));
-               reg_rw->value = cpu_to_le32((u32) r.rf->value);
+               reg_rw->offset = (u32) le16_to_cpu(r.rf->offset);
+               reg_rw->value = (u32) r.rf->value;
                break;
        case HostCmd_CMD_802_11_EEPROM_ACCESS:
                r.eeprom = &resp->params.eeprom;
-               pr_debug("info: EEPROM read len=%x\n", r.eeprom->byte_count);
-               if (le16_to_cpu(eeprom->byte_count) <
-                   le16_to_cpu(r.eeprom->byte_count)) {
-                       eeprom->byte_count = cpu_to_le16(0);
+               pr_debug("info: EEPROM read len=%x\n",
+                               le16_to_cpu(r.eeprom->byte_count));
+               if (eeprom->byte_count < le16_to_cpu(r.eeprom->byte_count)) {
+                       eeprom->byte_count = 0;
                        pr_debug("info: EEPROM read length is too big\n");
                        return -1;
                }
-               eeprom->offset = r.eeprom->offset;
-               eeprom->byte_count = r.eeprom->byte_count;
-               if (le16_to_cpu(eeprom->byte_count) > 0)
+               eeprom->offset = le16_to_cpu(r.eeprom->offset);
+               eeprom->byte_count = le16_to_cpu(r.eeprom->byte_count);
+               if (eeprom->byte_count > 0)
                        memcpy(&eeprom->value, &r.eeprom->value,
-                              le16_to_cpu(r.eeprom->byte_count));
-
+                              min((u16)MAX_EEPROM_DATA, eeprom->byte_count));
                break;
        default:
                return -1;
index 2ba5397272e2148c731291d20c0d56373f225449..e06647a327b68b24f710afe22c2066d65894270c 100644 (file)
@@ -1251,7 +1251,7 @@ static int mwifiex_reg_mem_ioctl_reg_rw(struct mwifiex_private *priv,
 {
        u16 cmd_no;
 
-       switch (le32_to_cpu(reg_rw->type)) {
+       switch (reg_rw->type) {
        case MWIFIEX_REG_MAC:
                cmd_no = HostCmd_CMD_MAC_REG_ACCESS;
                break;
@@ -1286,9 +1286,9 @@ mwifiex_reg_write(struct mwifiex_private *priv, u32 reg_type,
 {
        struct mwifiex_ds_reg_rw reg_rw;
 
-       reg_rw.type = cpu_to_le32(reg_type);
-       reg_rw.offset = cpu_to_le32(reg_offset);
-       reg_rw.value = cpu_to_le32(reg_value);
+       reg_rw.type = reg_type;
+       reg_rw.offset = reg_offset;
+       reg_rw.value = reg_value;
 
        return mwifiex_reg_mem_ioctl_reg_rw(priv, &reg_rw, HostCmd_ACT_GEN_SET);
 }
@@ -1306,14 +1306,14 @@ mwifiex_reg_read(struct mwifiex_private *priv, u32 reg_type,
        int ret;
        struct mwifiex_ds_reg_rw reg_rw;
 
-       reg_rw.type = cpu_to_le32(reg_type);
-       reg_rw.offset = cpu_to_le32(reg_offset);
+       reg_rw.type = reg_type;
+       reg_rw.offset = reg_offset;
        ret = mwifiex_reg_mem_ioctl_reg_rw(priv, &reg_rw, HostCmd_ACT_GEN_GET);
 
        if (ret)
                goto done;
 
-       *value = le32_to_cpu(reg_rw.value);
+       *value = reg_rw.value;
 
 done:
        return ret;
@@ -1332,15 +1332,16 @@ mwifiex_eeprom_read(struct mwifiex_private *priv, u16 offset, u16 bytes,
        int ret;
        struct mwifiex_ds_read_eeprom rd_eeprom;
 
-       rd_eeprom.offset = cpu_to_le16((u16) offset);
-       rd_eeprom.byte_count = cpu_to_le16((u16) bytes);
+       rd_eeprom.offset =  offset;
+       rd_eeprom.byte_count = bytes;
 
        /* Send request to firmware */
        ret = mwifiex_send_cmd(priv, HostCmd_CMD_802_11_EEPROM_ACCESS,
                               HostCmd_ACT_GEN_GET, 0, &rd_eeprom, true);
 
        if (!ret)
-               memcpy(value, rd_eeprom.value, MAX_EEPROM_DATA);
+               memcpy(value, rd_eeprom.value, min((u16)MAX_EEPROM_DATA,
+                      rd_eeprom.byte_count));
        return ret;
 }