From: Mariusz Gorski <marius.gorski@gmail.com>
To: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
Cc: Konrad Zapalowicz <bergo.torino@gmail.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Willy Tarreau <willy@meta-x.org>,
devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3] staging: panel: Remove outdated TODO tasks
Date: Sat, 1 Nov 2014 11:50:41 +0100 [thread overview]
Message-ID: <20141101105040.GA21226@firebird> (raw)
In-Reply-To: <20141101045916.GB3642@sudip-PC>
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/
prev parent reply other threads:[~2014-11-01 10:50 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-31 22:01 [PATCH v3] staging: panel: Remove outdated TODO tasks Mariusz Gorski
2014-10-31 22:08 ` Konrad Zapalowicz
2014-11-01 4:59 ` Sudip Mukherjee
2014-11-01 10:50 ` Mariusz Gorski [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20141101105040.GA21226@firebird \
--to=marius.gorski@gmail.com \
--cc=bergo.torino@gmail.com \
--cc=devel@driverdev.osuosl.org \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=sudipm.mukherjee@gmail.com \
--cc=willy@meta-x.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox