Roopni Devanathan <quic_rdevanat@xxxxxxxxxxx> wrote: > From: Pradeep Kumar Chitrapu <quic_pradeepc@xxxxxxxxxxx> > > Before firmware assert, if there is a station interface in the device > which is not associated with an AP, the basic rates are set to zero. > Following this, during firmware recovery, when basic rates are zero, > ath12k driver is sending invalid rate codes, which are negative values, > to firmware. This results in firmware assert. > > Fix this by checking if rate codes are valid, before sending them > to the firmware. > > Tested-on: QCN9274 hw2.0 PCI WLAN.WBE.1.4.1-00199-QCAHKSWPL_SILICONZ-1 > > Signed-off-by: Pradeep Kumar Chitrapu <quic_pradeepc@xxxxxxxxxxx> > Signed-off-by: Roopni Devanathan <quic_rdevanat@xxxxxxxxxxx> For the __ffs() part, Reviewed-by: Ping-Ke Shih <pkshih@xxxxxxxxxxx>