RE: [PATCH] rtw89: fix error function parameter

From: Pkshih
Date: Wed Oct 20 2021 - 05:46:34 EST



> -----Original Message-----
> From: kvalo=codeaurora.org@xxxxxxxxxxxxxxxxx <kvalo=codeaurora.org@xxxxxxxxxxxxxxxxx> On Behalf Of Kalle
> Valo
> Sent: Wednesday, October 20, 2021 4:50 PM
> To: cgel.zte@xxxxxxxxx
> Cc: davem@xxxxxxxxxxxxx; kuba@xxxxxxxxxx; Pkshih <pkshih@xxxxxxxxxxx>; lv.ruyi@xxxxxxxxxx;
> linux-wireless@xxxxxxxxxxxxxxx; netdev@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Zeal Robot
> <zealci@xxxxxxxxxx>
> Subject: Re: [PATCH] rtw89: fix error function parameter
>
> cgel.zte@xxxxxxxxx wrote:
>
> > From: Lv Ruyi <lv.ruyi@xxxxxxxxxx>
> >
> > This patch fixes the following Coccinelle warning:
> > drivers/net/wireless/realtek/rtw89/rtw8852a.c:753:
> > WARNING possible condition with no effect (if == else)
> >
> > Reported-by: Zeal Robot <zealci@xxxxxxxxxx>
> > Signed-off-by: Lv Ruyi <lv.ruyi@xxxxxxxxxx>
> > Acked-by: Ping-Ke Shih <pkshih@xxxxxxxxxxx>
>
> Failed to apply, please rebase on top of wireless-drivers-next.
>
> error: patch failed: drivers/net/wireless/realtek/rtw89/rtw8852a.c:753
> error: drivers/net/wireless/realtek/rtw89/rtw8852a.c: patch does not apply
> error: Did you hand edit your patch?
> It does not apply to blobs recorded in its index.
> hint: Use 'git am --show-current-patch' to see the failed patch
> Applying: rtw89: fix error function parameter
> Using index info to reconstruct a base tree...
> Patch failed at 0001 rtw89: fix error function parameter
>
> Patch set to Changes Requested.
>

I think this is because the patch is translated into spaces instead of tabs,
in this and following statements.
" if (is_2g)"

--
Ping-Ke