Re: [PATCH] [v3] staging: rtl8723bs: Clean up dead code

From: Dan Carpenter
Date: Tue Jan 08 2019 - 04:59:15 EST


Your subject does work for `git am` but the more normal way to format
it is:

[PATCH v4] staging: rtl8723bs: Clean up dead code

On Mon, Jan 07, 2019 at 05:20:52PM -0600, Aditya Pakki wrote:
> rtw_wps_start() is part of dead code due to CONFIG_INTEL_WIDI.
> The fix removes the deadcode and replaces the function with NULL
> in rtw_private_handler. Identified as part of copy_from_user bug.
>
> Changes from v1 & 2:
> The return value of copy_from_user() is not checked appropriately.
> However, maintainers identified the patch fixing the issue is not
> required as function rts_wps_start() is dead code and can be removed.
>
> Signed-off-by: Aditya Pakki <pakki001@xxxxxxx>
> ---

The "Changed from" comments need to go here under the --- cut off line.
It gets removed from the final kernel git log.

That information is just for maintainers to remember what was discussed
earlier. It's not needed for the permanent record.

> .../staging/rtl8723bs/os_dep/ioctl_linux.c | 34 +------------------
> 1 file changed, 1 insertion(+), 33 deletions(-)

regards,
dan carpenter