On Fri, 2 Sep 2011 09:05:41 -0700
"Palash Bandyopadhyay" <[email protected]> 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:[email protected]] 
> 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; 
> [email protected]; [email protected]; 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
_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to