2024-06-12 14:08:46

by Teddy Engel

Subject: [PATCH] staging: vt6656: Remove line from TODO

Remove checkpatch.pl line from TODO as no valid checkpatch
recommendations still present.

Signed-off-by: Teddy Engel <[email protected]>
---
drivers/staging/vt6656/TODO | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/staging/vt6656/TODO b/drivers/staging/vt6656/TODO
index e154b2f3b247..876cdccb6948 100644
--- a/drivers/staging/vt6656/TODO
+++ b/drivers/staging/vt6656/TODO
@@ -11,7 +11,6 @@ TODO:
- switch to use LIB80211
- switch to use MAC80211
- use kernel coding style
-- checkpatch.pl fixes
- sparse fixes
- integrate with drivers/net/wireless

--
2.39.2

2024-06-12 20:58:00

by Philipp Hortmann

Subject: Re: [PATCH] staging: vt6656: Remove line from TODO

On 6/12/24 16:07, Teddy Engel wrote:
> Remove checkpatch.pl line from TODO as no valid checkpatch
> recommendations still present.
>
> Signed-off-by: Teddy Engel <[email protected]>
> ---
> drivers/staging/vt6656/TODO | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/staging/vt6656/TODO b/drivers/staging/vt6656/TODO
> index e154b2f3b247..876cdccb6948 100644
> --- a/drivers/staging/vt6656/TODO
> +++ b/drivers/staging/vt6656/TODO
> @@ -11,7 +11,6 @@ TODO:
> - switch to use LIB80211
> - switch to use MAC80211
> - use kernel coding style
> -- checkpatch.pl fixes
> - sparse fixes
> - integrate with drivers/net/wireless
>

Hi Teddy,

to me this is a second version of a previous patch. Please issue a new
version. Your next Patch with this content is V3 as this was a V2

If you send in a second version of this patch please use a change
history. Description from Dan under:
https://staticthinking.wordpress.com/2022/07/27/how-to-send-a-v2-patch/

Thanks for your support.

Bye Philipp