Larry Finger <Larry.Finger@xxxxxxxxxxxx> writes: > On 8/20/19 9:08 AM, Kalle Valo wrote: >> Larry Finger <Larry.Finger@xxxxxxxxxxxx> wrote: >> >>> In the process of converting the bit manipulation macros were converted >>> to use GENMASK(), the compiler reported a value too big for the field. >>> The offending statement was trying to write 0x100 into a 5-bit field. >>> An accompaning comment says to set bit 3, thus the code is changed >>> appropriately. >>> >>> This error has been in the driver since its initial submission. >>> >>> Fixes: 29d00a3e46bb ("rtlwifi: rtl8192cu: Add routine trx") >>> Signed-off-by: Larry Finger <Larry.Finger@xxxxxxxxxxxx> >> >> I only see patch 5 on patchwork, what happened to patches 1-4? > > I pulled them at the last minute, but forgot to rename the subject. > Please apply that fix alone. The others will follow when I get time; > however, they are code improvements. Only this one was a code fix. Ok, I was just worried if patchwork had lost of them somehow. -- https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches