2014-10-31 21:07:09

by Mariusz Gorski

[permalink] [raw]
Subject: [PATCH] Remove outdated TODO tasks

Remove Lindent and checkpatch.pl tasks from TODO file as the first one
is obsolete and the other one is already done.

Signed-off-by: Mariusz Gorski <[email protected]>
---
drivers/staging/panel/TODO | 2 --
1 file changed, 2 deletions(-)

diff --git a/drivers/staging/panel/TODO b/drivers/staging/panel/TODO
index a4be749..c8040c1 100644
--- a/drivers/staging/panel/TODO
+++ b/drivers/staging/panel/TODO
@@ -1,6 +1,4 @@
TODO:
- - checkpatch.pl cleanups
- - Lindent
- review major/minor usages
- review userspace api
- see if all of this could be easier done in userspace instead.
--
2.1.3


2014-10-31 21:12:29

by Konrad Zapalowicz

[permalink] [raw]
Subject: Re: [PATCH] Remove outdated TODO tasks

On 10/31, Mariusz Gorski wrote:
> Remove Lindent and checkpatch.pl tasks from TODO file as the first one
> is obsolete and the other one is already done.

Great however your topic is ill formated. It should contain the "staging:
panel:" prefix.

thanks,
konrad

>
> Signed-off-by: Mariusz Gorski <[email protected]>
> ---
> drivers/staging/panel/TODO | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/staging/panel/TODO b/drivers/staging/panel/TODO
> index a4be749..c8040c1 100644
> --- a/drivers/staging/panel/TODO
> +++ b/drivers/staging/panel/TODO
> @@ -1,6 +1,4 @@
> TODO:
> - - checkpatch.pl cleanups
> - - Lindent
> - review major/minor usages
> - review userspace api
> - see if all of this could be easier done in userspace instead.
> --
> 2.1.3
>
> _______________________________________________
> devel mailing list
> [email protected]
> http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

2014-11-01 04:42:57

by Sudip Mukherjee

[permalink] [raw]
Subject: Re: [PATCH] Remove outdated TODO tasks

On Fri, Oct 31, 2014 at 10:06:46PM +0100, Mariusz Gorski wrote:
> Remove Lindent and checkpatch.pl tasks from TODO file as the first one
> is obsolete and the other one is already done.

when i checked panel.c with checkpatch and with --strict option, i got 20 checks.

thanks
sudip

>
> Signed-off-by: Mariusz Gorski <[email protected]>
> ---
> drivers/staging/panel/TODO | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/staging/panel/TODO b/drivers/staging/panel/TODO
> index a4be749..c8040c1 100644
> --- a/drivers/staging/panel/TODO
> +++ b/drivers/staging/panel/TODO
> @@ -1,6 +1,4 @@
> TODO:
> - - checkpatch.pl cleanups
> - - Lindent
> - review major/minor usages
> - review userspace api
> - see if all of this could be easier done in userspace instead.
> --
> 2.1.3
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/