On 07/07/2023 10:23, Martin Kaistra wrote: > Hi Georg, > > Am 05.07.23 um 11:00 schrieb Georg Müller: >> Hi Martin, >> [...] >> Please give me a hint if and how I may help tracking down the issue to >> enable AP mode for 8188EU as well. > > I would suggest the following: > - in rtl8xxxu_fill_txdesc_v3, set macid, HW_SEQ like in the v2 function > - 8188e does decide on the tx rate in software, so rtl8188e_handle_ra_tx_report2() will probably need some adjustments to handle multiple macids (maybe ra_info in rtl8xxxu_priv should be an array of size RTL8XXXU_MAX_MAC_ID_NUM?) ra_info should be an array(?), except the dynamic_tx_rpt_timing_counter member should be moved to rtl8xxxu_priv. In the vendor driver it was a static variable in the odm_rate_decision_8188e() function. An array or some dynamic data structure? The array would take up at least 3072 bytes, including when the driver is handling some other chip which doesn't need the array. Additionally, rtl8xxxu_sta_add() and rtl8xxxu_sta_remove() need to increase/decrease REG_TX_REPORT_CTRL + 1, otherwise you will only get correct rate control for the first station which connects to the AP.
