[PATCH v3] staging: panel: Remove outdated TODO tasks

[PATCH v3] staging: panel: Remove outdated TODO tasks

[Mariusz Gorski]

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 <marius.gorski@gmail.com>
---
v3: Fixed commit summary

 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

Re: [PATCH v3] staging: panel: Remove outdated TODO tasks

[Konrad Zapalowicz]

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.
> 
> Signed-off-by: Mariusz Gorski <marius.gorski@gmail.com>
> ---
> v3: Fixed commit summary

Great. Thanks.

regards,
konrad
 
>  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
> devel@linuxdriverproject.org
> http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH v3] staging: panel: Remove outdated TODO tasks

[Sudip Mukherjee]

On Fri, Oct 31, 2014 at 11:08:34PM +0100, Konrad Zapalowicz wrote:
> 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.
> > 

when i checked panel.c with checkpatch and with --strict option, i got 20 checks.
(sent the same reply to v1 patch also).

thanks
sudip

> > Signed-off-by: Mariusz Gorski <marius.gorski@gmail.com>
> > ---
> > v3: Fixed commit summary
> 
> Great. Thanks.
> 
> regards,
> konrad
>  
> >  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
> > devel@linuxdriverproject.org
> > http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH v3] staging: panel: Remove outdated TODO tasks

[Mariusz Gorski]

On Sat, Nov 01, 2014 at 10:29:16AM +0530, Sudip Mukherjee wrote:
> On Fri, Oct 31, 2014 at 11:08:34PM +0100, Konrad Zapalowicz wrote:
> > 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.
> > > 
> 
> when i checked panel.c with checkpatch and with --strict option, i got 20 checks.
> (sent the same reply to v1 patch also).

Good point, supid. Thanks! I guess the checkpatch.pl issues deserve a
separate patch, so I'll just reduce the scope of this one.

Cheers,
Mariusz

> thanks
> sudip
> 
> > > Signed-off-by: Mariusz Gorski <marius.gorski@gmail.com>
> > > ---
> > > v3: Fixed commit summary
> > 
> > Great. Thanks.
> > 
> > regards,
> > konrad
> >  
> > >  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
> > > devel@linuxdriverproject.org
> > > http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > Please read the FAQ at  http://www.tux.org/lkml/