From: Greg KH <greg@kroah.com>
To: Ravishankar Karkala Mallikarjunayya <ravishankar.km@greenturtles.in>
Cc: gregkh@suse.de, wfp5p@virginia.edu, devel@driverdev.osuosl.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 03/24] Staging: comedi: fix warning issue in pcl818.c
Date: Sun, 23 Oct 2011 10:20:00 +0200 [thread overview]
Message-ID: <20111023082000.GC13409@kroah.com> (raw)
In-Reply-To: <1319091550-2558-3-git-send-email-ravishankar.km@greenturtles.in>
On Thu, Oct 20, 2011 at 11:48:49AM +0530, Ravishankar Karkala Mallikarjunayya wrote:
> This is a patch to the pcl818.c file that fixes up a warning
> found by the checkpatch.pl tool.
> Fix brace coding style and NULL initilization issue.
> Converted printks to dev_printk.
> Removed unnecessary printk statements.
That's a lot of different things all at once here. Please only do one
"change" per patch. So this should be split up into multiple patches.
Same goes for your other 24 patches, care to break them up as well, as
it's easier to review and verify that you got it correct.
thanks,
greg k-h
next prev parent reply other threads:[~2011-10-23 8:20 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-10-20 6:18 [PATCH 01/24] Staging: comedi: fix printk issue in das6402.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:18 ` [PATCH 02/24] Staging: comedi: fix printk coding style issue in Ravishankar Karkala Mallikarjunayya
2011-10-23 8:19 ` Greg KH
2011-10-20 6:18 ` [PATCH 03/24] Staging: comedi: fix warning issue in pcl818.c Ravishankar Karkala Mallikarjunayya
2011-10-23 8:20 ` Greg KH [this message]
2011-10-20 6:18 ` [PATCH 04/24] Staging: comedi: fix code cleanup issue in pcmuio.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:18 ` [PATCH 05/24] Staging: comedi: fix printk issue in c6xdigio.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:18 ` [PATCH 06/24] Staging: comedi: fix printk issue in ni_pcimio.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:18 ` [PATCH 7/24] Staging: comedi: fix printk issue in ni_labpc_cs.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:18 ` [PATCH 08/24] Staging: comedi: fix printk issue in das800.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:18 ` [PATCH 9/24] Staging: comedi: fix warning issue in dt3000.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:18 ` [PATCH 10/24] Staging: comedi: fix printk issue in cb_pcidas64.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:18 ` [PATCH 11/24] Staging: comedi: fix printk issue in cb_pcidda.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:18 ` [PATCH 12/24] Staging: comedi: fix printk issue in cb_pcidio.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:18 ` [PATCH 13/24] Staging: comedi: fix printk issue in cb_pcimdas.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:19 ` [PATCH 14/24] Staging: comedi: fix printk issue in cb_pcimdda.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:19 ` [PATCH 15/24] Staging: comedi: fix printk issue in contec_pci_dio.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:19 ` [PATCH 16/24] Staging: comedi: fix printk issue in amplc_pci230.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:19 ` [PATCH 17/24] Staging: comedi: fix printk issue in adv_pci_dio.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:19 ` [PATCH 18/24] Staging: comedi: fix printk issue in ni_at_a2150.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:19 ` [PATCH 19/24] Staging: comedi: fix warning issue in contec_pci_dio.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:19 ` [PATCH 20/24] Staging: comedi: fix printk issue in cb_pcidas.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:19 ` [PATCH 21/24] Staging: comedi: fix warning issue in cb_das16_cs.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:19 ` [PATCH 22/24] Staging: comedi: fix printk issue in das1800.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:19 ` [PATCH 23/24] Staging: comedi: fix printk issue in adv_pci1710.c Ravishankar Karkala Mallikarjunayya
2011-10-20 6:19 ` [PATCH 24/24] Staging: comedi: fix code cleanup in daqboard2000.c Ravishankar Karkala Mallikarjunayya
2011-10-23 8:18 ` [PATCH 01/24] Staging: comedi: fix printk issue in das6402.c Greg KH
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=20111023082000.GC13409@kroah.com \
--to=greg@kroah.com \
--cc=devel@driverdev.osuosl.org \
--cc=gregkh@suse.de \
--cc=linux-kernel@vger.kernel.org \
--cc=ravishankar.km@greenturtles.in \
--cc=wfp5p@virginia.edu \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.