On Thu, 2 Jun 2022 13:59:38 -0700, quic_jjohnson@xxxxxxxxxxx wrote: > why an embedded block? seems to serve no purpose > >> + bw = rate_ctrl_info.field.bw; >> + bf = agg_report->bf; >> + >> + cntrs = &stats->tx.he[bw][nss][mcs][gi][bf]; > > bad indentation > >> + } >> + break; >> + case WRS_MODE_VHT: >> + bw = rate_ctrl_info.field.bw; >> + nss = (rate_ctrl_info.field.mcs_index >> 4); >> + mcs = (rate_ctrl_info.field.mcs_index & 0xF); >> + gi = rate_ctrl_info.field.gi; >> + bf = agg_report->bf; >> + >> + cntrs = &stats->tx.vht[bw][nss][mcs][gi][bf]; > > bad indentation Correct – that is an artifact from old version of code. Will fix that! Best regards, Viktor Barna