Messages in this thread | | | From | Prasun Maiti <> | Date | Fri, 17 Jun 2016 18:41:39 +0530 | Subject | [Query] mwifiex: few observations to reduce number of endian conversions |
| |
Hi Amitkumar,
I have two observations:
1. I have found that in the command response path for host command "HostCmd_CMD_802_11_EEPROM_ACCESS", a "0" value has been endian converted. It can only be a safe futuristic approach for any non-zero value there however! Otherwise, the endian conversion can be removed.
2. For multiple Host Commands (e.g HostCmd_CMD_802_11_EEPROM_ACCESS etc.) "cpu_to_leX"-converted values are saved to driver. So "leX_to_cpu" conversion is required too many times afterwards in driver. On the contrary, we can save the values to driver without any conversion, and only command buffer(s) are prepared with endian converted values. In this way we can gain some efficiency [code size / time] by reducing the number of endian conversion considerably.
Please let me know your opinion on the above.
-- Thanks, Prasun
| |