Colin King <colin.king@xxxxxxxxxxxxx> wrote: > From: Colin Ian King <colin.king@xxxxxxxxxxxxx> > > Function _rtl8821ae_mrate_idx_to_arfr_id is functionally identical to > the generic version rtl_mrate_idx_to_arfr_id, so remove > _rtl8821ae_mrate_idx_to_arfr_id and use the generic one instead. > > This also fixes a missing break statement found by CoverityScan in > _rtl8821ae_mrate_idx_to_arfr_id, namely: CID#1167237 ("Missing break > in switch") > > Thanks to Joe Perches for spotting this when I submitted an earlier patch. > > Fixes: 3c05bedb5fef ("Staging: rtl8812ae: Add Realtek 8821 PCI WIFI driver") > Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx> > ACKed-by: Larry Finger <Larry.Finger@xxxxxxxxxxxx> Patch applied to wireless-drivers-next.git, thanks. c894696188d5 rtlwifi: rtl8821ae: replace _rtl8821ae_mrate_idx_to_arfr_id with generic version -- https://patchwork.kernel.org/patch/10630249/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches