From: "Leonid V. Fedorenchik" <leonidsbox@gmail.com>
To: "Palash Bandyopadhyay" <Palash.Bandyopadhyay@conexant.com>
Cc: "Greg Kroah-Hartman" <gregkh@suse.de>,
"Mauro Carvalho Chehab" <mchehab@redhat.com>,
"Ruslan Pisarev" <ruslan@rpisarev.org.ua>,
"Ilia Mirkin" <imirkin@alum.mit.edu>,
"Joe Perches" <joe@perches.com>,
"Hans Verkuil" <hverkuil@xs4all.nl>,
"Youquan Song" <youquan.song@intel.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"devel@linuxdriverproject.org" <devel@linuxdriverproject.org>
Subject: Re: [PATCH v4 00/19] Staging: cx25821: Fix style issues
Date: Sat, 3 Sep 2011 11:50:37 +0800 [thread overview]
Message-ID: <20110903115037.184dd503@inspire> (raw)
In-Reply-To: <34B38BE41EDBA046A4AFBB591FA311320502550FA8@NBMBX01.bbnet.ad>
On Fri, 2 Sep 2011 09:05:41 -0700
"Palash Bandyopadhyay" <Palash.Bandyopadhyay@conexant.com> wrote:
> Thanks a lot Leonid. Changes look good to me.
My pleasure. Thank you for the review.
>
> Regards,
> Palash
>
> -----Original Message-----
> From: Leonid V. Fedorenchik [mailto:leonidsbox@gmail.com]
> Sent: Thursday, September 01, 2011 8:55 PM
> To: Greg Kroah-Hartman
> Cc: Mauro Carvalho Chehab; Ruslan Pisarev; Ilia Mirkin; Joe Perches; Palash Bandyopadhyay; Hans Verkuil; Arnd Bergmann; Youquan Song; linux-kernel@vger.kernel.org; devel@linuxdriverproject.org; Leonid V. Fedorenchik
> Subject: [PATCH v4 00/19] Staging: cx25821: Fix style issues
>
> This patch series fixes some style issues in drivers/staging/cx25821
>
> I split my previous patches per logical change and combined them into
> one patch series.
>
> Changes in v4 based on reviews of v3.
>
> Leonid V. Fedorenchik (19):
> Staging: cx25821: Get rid of typedef in cx25821.h
> Staging: cx25821: Rename member name in struct cx25821_dev
> Staging: cx25821: Delete file cx25821-gpio.h
> Staging: cx25821: Move EXPORT_SYMBOL() to the right place
> Staging: cx25821: Use DEFINE_PCI_DEVICE_TABLE for cx25821_pci_tbl
> Staging: cx25821: Fix wrong statement indent in cx25821-cards.c
> Staging: cx25821: Fix wrong statement indent in cx25821-core.c
> Staging: cx25821: Fix wrong statement indent in cx25821-i2c.c
> Staging: cx25821: Fix too long lines in cx25821-audio.h
> Staging: cx25821: Fix obscure line endings in cx25821-audio-upstream.c
> Staging: cx25821: Fix long lines in cx25821-audio-upstream.c
> Staging: cx25821: Fix long lines in cx25821-audio-upstream.h
> Staging: cx25821: Replace :? by if-else in cx25821-video-upstream-ch2.c
> Staging: cx25821: Fix labels' placement in cx25821-video-upstream-ch2.c
> Staging: cx25821: Fix wrong line endings in cx25821-video-upstream-ch2.c
> Staging: cx25821: Fix long lines in cx25821-video-upstream-ch2.c
> Staging: cx25821: Fix long lines in cx25821-video-upstream-ch2.h
> Staging: cx25821: Fix long lines in cx25821-video-upstream.c
> Staging: cx25821: Fix long lines in cx25821-video-upstream.h
>
> drivers/staging/cx25821/cx25821-audio-upstream.c | 41 +-
> drivers/staging/cx25821/cx25821-audio-upstream.h | 9 +-
> drivers/staging/cx25821/cx25821-audio.h | 6 +-
> drivers/staging/cx25821/cx25821-cards.c | 26 +-
> drivers/staging/cx25821/cx25821-core.c | 503 ++++++++++----------
> drivers/staging/cx25821/cx25821-gpio.c | 1 +
> drivers/staging/cx25821/cx25821-gpio.h | 2 -
> drivers/staging/cx25821/cx25821-i2c.c | 30 +-
> .../staging/cx25821/cx25821-video-upstream-ch2.c | 79 ++--
> .../staging/cx25821/cx25821-video-upstream-ch2.h | 83 +++-
> drivers/staging/cx25821/cx25821-video-upstream.c | 16 +-
> drivers/staging/cx25821/cx25821-video-upstream.h | 78 +++-
> drivers/staging/cx25821/cx25821.h | 10 +-
> 13 files changed, 465 insertions(+), 419 deletions(-)
> delete mode 100644 drivers/staging/cx25821/cx25821-gpio.h
>
> Conexant E-mail Firewall (Conexant.Com) made the following annotations
> ---------------------------------------------------------------------
> ********************** Legal Disclaimer ****************************
>
> "This email may contain confidential and privileged material for the sole use of the intended recipient. Any unauthorized review, use or distribution by others is strictly prohibited. If you have received the message in error, please advise the sender by reply email and delete the message. Thank you."
>
> **********************************************************************
>
> ---------------------------------------------------------------------
>
Leonid V. Fedorenchik
prev parent reply other threads:[~2011-09-03 3:51 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-09-02 3:55 [PATCH v4 00/19] Staging: cx25821: Fix style issues Leonid V. Fedorenchik
2011-09-02 3:55 ` [PATCH v4 01/19] Staging: cx25821: Get rid of typedef in cx25821.h Leonid V. Fedorenchik
2011-09-02 3:55 ` [PATCH v4 02/19] Staging: cx25821: Rename member name in struct cx25821_dev Leonid V. Fedorenchik
2011-09-02 3:55 ` [PATCH v4 03/19] Staging: cx25821: Delete file cx25821-gpio.h Leonid V. Fedorenchik
2011-09-02 3:55 ` [PATCH v4 04/19] Staging: cx25821: Move EXPORT_SYMBOL() to the right place Leonid V. Fedorenchik
2011-09-02 3:55 ` [PATCH v4 05/19] Staging: cx25821: Use DEFINE_PCI_DEVICE_TABLE for cx25821_pci_tbl Leonid V. Fedorenchik
2011-09-02 3:55 ` [PATCH v4 06/19] Staging: cx25821: Fix wrong statement indent in cx25821-cards.c Leonid V. Fedorenchik
2011-09-02 3:55 ` [PATCH v4 07/19] Staging: cx25821: Fix wrong statement indent in cx25821-core.c Leonid V. Fedorenchik
2011-09-02 3:55 ` [PATCH v4 08/19] Staging: cx25821: Fix wrong statement indent in cx25821-i2c.c Leonid V. Fedorenchik
2011-09-02 3:55 ` [PATCH v4 09/19] Staging: cx25821: Fix too long lines in cx25821-audio.h Leonid V. Fedorenchik
2011-09-02 3:55 ` [PATCH v4 10/19] Staging: cx25821: Fix obscure line endings in cx25821-audio-upstream.c Leonid V. Fedorenchik
2011-09-02 3:55 ` [PATCH v4 11/19] Staging: cx25821: Fix long lines " Leonid V. Fedorenchik
2011-09-02 3:55 ` [PATCH v4 12/19] Staging: cx25821: Fix long lines in cx25821-audio-upstream.h Leonid V. Fedorenchik
2011-09-02 3:55 ` [PATCH v4 13/19] Staging: cx25821: Replace :? by if-else in cx25821-video-upstream-ch2.c Leonid V. Fedorenchik
2011-09-02 3:55 ` [PATCH v4 14/19] Staging: cx25821: Fix labels' placement " Leonid V. Fedorenchik
2011-09-02 3:55 ` [PATCH v4 15/19] Staging: cx25821: Fix wrong line endings " Leonid V. Fedorenchik
2011-09-02 3:55 ` [PATCH v4 16/19] Staging: cx25821: Fix long lines " Leonid V. Fedorenchik
2011-09-02 3:55 ` [PATCH v4 17/19] Staging: cx25821: Fix long lines in cx25821-video-upstream-ch2.h Leonid V. Fedorenchik
2011-09-02 3:55 ` [PATCH v4 18/19] Staging: cx25821: Fix long lines in cx25821-video-upstream.c Leonid V. Fedorenchik
2011-09-02 3:55 ` [PATCH v4 19/19] Staging: cx25821: Fix long lines in cx25821-video-upstream.h Leonid V. Fedorenchik
2011-09-02 9:16 ` [PATCH v4 00/19] Staging: cx25821: Fix style issues Dan Carpenter
2011-09-02 11:10 ` Leonid V. Fedorenchik
2011-09-02 10:34 ` Valdis.Kletnieks
2011-09-02 11:23 ` Leonid V. Fedorenchik
2011-09-02 16:05 ` Palash Bandyopadhyay
2011-09-03 3:50 ` Leonid V. Fedorenchik [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=20110903115037.184dd503@inspire \
--to=leonidsbox@gmail.com \
--cc=Palash.Bandyopadhyay@conexant.com \
--cc=devel@linuxdriverproject.org \
--cc=gregkh@suse.de \
--cc=hverkuil@xs4all.nl \
--cc=imirkin@alum.mit.edu \
--cc=joe@perches.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mchehab@redhat.com \
--cc=ruslan@rpisarev.org.ua \
--cc=youquan.song@intel.com \
/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