"zhengbin (A)" <zhengbi...@huawei.com> writes: > On 2019/10/9 20:58, Kalle Valo wrote: >> zhengbin <zhengbi...@huawei.com> writes: >> >>> Fixes gcc '-Wunused-but-set-variable' warning: >>> >>> drivers/net/wireless/realtek/rtlwifi/rtl8192ee/fw.c: In function >>> rtl92ee_download_fw: >>> drivers/net/wireless/realtek/rtlwifi/rtl8192ee/fw.c:111:6: warning: >>> variable err set but not used [-Wunused-but-set-variable] >>> >>> Reported-by: Hulk Robot <hul...@huawei.com> >>> Signed-off-by: zhengbin <zhengbi...@huawei.com> >>> Acked-by: Ping-Ke Shih <pks...@realtek.com> >> There's no changelog, why did you resend? Document clearly the changes >> so that maintainers don't need to guess what has changed: > > Failed to apply: > > fatal: corrupt patch at line 13 > error: could not build fake ancestor > Applying: rtlwifi: rtl8192ee: Remove set but not used variable 'err' > Patch failed at 0001 rtlwifi: rtl8192ee: Remove set but not used variable > 'err' > The copy of the patch that failed is found in: .git/rebase-apply/patch > > So I resend this.
Ok, thanks. But next time include the changelog automatically and mark the patch as v2. And read the documentation: https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches -- Kalle Valo
