From: Mauro Carvalho Chehab <mchehab@s-opensource.com>
To: Linux Media Mailing List <linux-media@vger.kernel.org>
Cc: Mauro Carvalho Chehab <mchehab@s-opensource.com>,
Mauro Carvalho Chehab <mchehab@infradead.org>,
Geunyoung Kim <nenggun.kim@samsung.com>,
Junghak Sung <jh1009.sung@samsung.com>,
Patrick Boettcher <patrick.boettcher@posteo.de>,
Hans Verkuil <hans.verkuil@cisco.com>,
Matthias Schwarzott <zzam@gentoo.org>,
Heiner Kallweit <hkallweit1@gmail.com>,
Masahiro Yamada <yamada.masahiro@socionext.com>,
Michael Ira Krufky <mkrufky@linuxtv.org>,
Julia Lawall <Julia.Lawall@lip6.fr>,
Inki Dae <inki.dae@samsung.com>, Ole Ernst <olebowle@gmx.com>,
Wei Yongjun <weiyongjun1@huawei.com>,
Javier Martinez Canillas <javier@osg.samsung.com>,
Marcel Holtmann <marcel@holtmann.org>,
Arnd Bergmann <arnd@arndb.de>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Andrey Utkin <andrey.utkin@corp.bluecherry.net>,
Russell King <rmk+kernel@arm.linux.org.uk>,
Seung-Woo Kim <sw0312.kim@samsung.com>,
Sean Young <sean@mess.org>,
mjpeg-users@lists.sourceforge.net,
Andrew Morton <akpm@linux-foundation.org>,
Devin Heitmueller <dheitmueller@kernellabs.com>,
Dan Carpenter <dan.carpenter@oracle.com>,
Olli Salonen <olli.salonen@iki.fi>,
Samuel Ortiz <sameo@linux.intel.com>,
Kamil Debski <kamil@wypas.org>,
"Lad, Prabhakar" <prabhakar.csengg@gmail.com>,
Hans Verkuil <hverkuil@xs4all.nl>,
Hans Verkuil <hansverk@cisco.com>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Stephen Backway <stev391@gmail.com>,
Sakari Ailus <sakari.ailus@linux.intel.com>,
Mike Isely <isely@pobox.com>,
Geert Uytterhoeven <geert@linux-m68k.org>,
Max Kellermann <max@duempel.org>
Subject: [PATCH 00/35] Some printk fixups and improvements
Date: Wed, 16 Nov 2016 14:42:32 -0200 [thread overview]
Message-ID: <cover.1479314177.git.mchehab@s-opensource.com> (raw)
On Kernel 4.9, the KERN_CONT level is mandatory. So, we need to fix those
at the drivers. However, lots of media drivers don't follow the good practices
of using dev_foo() or pr_foo() macros.
This series convert several drivers the upstream debug macros, and fix
the issues related to KERN_CONT where needed.
I have another set of patches with dev_foo() conversions, but those require
manual testing, as a bad conversion could cause crashes or bad displayed
messages.
So, let's fix the easy cases. I'll keep working on the other patches when I
have spare time.
Please notice that today I merged a series of patches that may cause conflicts
with this series. I'll address it when merging it back at the media_tree.git, in a
couple of days.
Mauro Carvalho Chehab (35):
[media] stb0899_drv: get rid of continuation lines
[media] stv090x: get rid of continuation lines
[media] bt8xx/dst: use a more standard way to print messages
[media] bt8xx: use pr_foo() macros instead of printk()
[media] cx23885: use KERN_CONT where needed
[media] cx23885: convert it to use pr_foo() macros
[media] cx88: use KERN_CONT where needed
[media] cx88: convert it to use pr_foo() macros
[media] cx88: make checkpatch happier
[media] pluto2: use KERN_CONT where needed
[media] zoran: use KERN_CONT where needed
[media] wl128x: use KERNEL_CONT where needed
[media] pvrusb2: use KERNEL_CONT where needed
[media] ttusb_dec: use KERNEL_CONT where needed
[media] ttpci: cleanup debug macros and remove dead code
[media] dib0070: use pr_foo() instead of printk()
[media] dib0090: use pr_foo() instead of printk()
[media] dib3000mb: use pr_foo() instead of printk()
[media] dib3000mc: use pr_foo() instead of printk()
[media] dib7000m: use pr_foo() instead of printk()
[media] dib7000p: use pr_foo() instead of printk()
[media] dib8000: use pr_foo() instead of printk()
[media] dib9000: use pr_foo() instead of printk()
[media] dibx000_common: use pr_foo() instead of printk()
[media] af9005: remove a printk that would require a KERN_CONT
[media] tuner-core: use pr_foo, instead of internal printk macros
[media] v4l2-common: add a debug macro to be used with dev_foo()
[media] msp3400-driver: don't use KERN_CONT
[media] msp3400: convert it to use dev_foo() macros
[media] em28xx: convert it from pr_foo() to dev_foo()
[media] tvp5150: convert it to use dev_foo() macros
[media] tvp5150: Get rid of direct calls to printk()
[media] tvp5150: get rid of KERN_CONT
[media] rc-main: use pr_foo() macros
[media] tveeprom: print log messages using pr_foo()
drivers/media/common/tveeprom.c | 42 ++--
drivers/media/dvb-frontends/dib0070.c | 52 ++---
drivers/media/dvb-frontends/dib0090.c | 164 +++++++--------
drivers/media/dvb-frontends/dib3000mb.c | 137 ++++++-------
drivers/media/dvb-frontends/dib3000mb_priv.h | 16 +-
drivers/media/dvb-frontends/dib3000mc.c | 8 +-
drivers/media/dvb-frontends/dib7000m.c | 73 ++++---
drivers/media/dvb-frontends/dib7000p.c | 127 ++++++------
drivers/media/dvb-frontends/dib8000.c | 261 ++++++++++++------------
drivers/media/dvb-frontends/dib9000.c | 171 ++++++++--------
drivers/media/dvb-frontends/dibx000_common.c | 36 ++--
drivers/media/dvb-frontends/stb0899_drv.c | 21 +-
drivers/media/dvb-frontends/stv090x.c | 10 +-
drivers/media/i2c/msp3400-driver.c | 90 ++++----
drivers/media/i2c/msp3400-kthreads.c | 115 ++++++-----
drivers/media/i2c/tvp5150.c | 295 ++++++++++++++-------------
drivers/media/pci/bt8xx/btcx-risc.c | 46 +++--
drivers/media/pci/bt8xx/dst.c | 262 +++++++++++-------------
drivers/media/pci/bt8xx/dvb-bt8xx.c | 25 ++-
drivers/media/pci/cx23885/altera-ci.c | 13 +-
drivers/media/pci/cx23885/altera-ci.h | 14 +-
drivers/media/pci/cx23885/cimax2.c | 8 +-
drivers/media/pci/cx23885/cx23885-417.c | 57 +++---
drivers/media/pci/cx23885/cx23885-alsa.c | 21 +-
drivers/media/pci/cx23885/cx23885-cards.c | 49 +++--
drivers/media/pci/cx23885/cx23885-core.c | 139 ++++++-------
drivers/media/pci/cx23885/cx23885-dvb.c | 40 ++--
drivers/media/pci/cx23885/cx23885-f300.c | 2 +-
drivers/media/pci/cx23885/cx23885-i2c.c | 27 ++-
drivers/media/pci/cx23885/cx23885-input.c | 6 +-
drivers/media/pci/cx23885/cx23885-ir.c | 4 +-
drivers/media/pci/cx23885/cx23885-vbi.c | 7 +-
drivers/media/pci/cx23885/cx23885-video.c | 23 ++-
drivers/media/pci/cx23885/cx23885.h | 2 +
drivers/media/pci/cx23885/cx23888-ir.c | 6 +-
drivers/media/pci/cx23885/netup-eeprom.c | 4 +-
drivers/media/pci/cx23885/netup-init.c | 8 +-
drivers/media/pci/cx88/cx88-alsa.c | 104 +++++-----
drivers/media/pci/cx88/cx88-blackbird.c | 125 ++++++------
drivers/media/pci/cx88/cx88-cards.c | 188 ++++++++---------
drivers/media/pci/cx88/cx88-core.c | 245 +++++++++++-----------
drivers/media/pci/cx88/cx88-dsp.c | 37 ++--
drivers/media/pci/cx88/cx88-dvb.c | 173 ++++++++--------
drivers/media/pci/cx88/cx88-i2c.c | 108 +++++-----
drivers/media/pci/cx88/cx88-input.c | 21 +-
drivers/media/pci/cx88/cx88-mpeg.c | 216 +++++++++-----------
drivers/media/pci/cx88/cx88-reg.h | 5 +-
drivers/media/pci/cx88/cx88-tvaudio.c | 102 +++++----
drivers/media/pci/cx88/cx88-vbi.c | 32 +--
drivers/media/pci/cx88/cx88-video.c | 266 ++++++++++++------------
drivers/media/pci/cx88/cx88-vp3054-i2c.c | 50 ++---
drivers/media/pci/cx88/cx88.h | 40 ++--
drivers/media/pci/pluto2/pluto2.c | 4 +-
drivers/media/pci/ttpci/av7110.c | 15 --
drivers/media/pci/ttpci/av7110.h | 7 +-
drivers/media/pci/ttpci/budget.h | 8 +-
drivers/media/pci/zoran/zoran_device.c | 35 ++--
drivers/media/radio/wl128x/fmdrv_common.c | 16 +-
drivers/media/rc/rc-main.c | 8 +-
drivers/media/usb/dvb-usb/af9005.c | 1 -
drivers/media/usb/em28xx/em28xx-audio.c | 64 +++---
drivers/media/usb/em28xx/em28xx-camera.c | 60 ++++--
drivers/media/usb/em28xx/em28xx-cards.c | 129 +++++++-----
drivers/media/usb/em28xx/em28xx-core.c | 154 +++++++-------
drivers/media/usb/em28xx/em28xx-dvb.c | 89 ++++----
drivers/media/usb/em28xx/em28xx-i2c.c | 284 ++++++++++++++------------
drivers/media/usb/em28xx/em28xx-input.c | 42 ++--
drivers/media/usb/em28xx/em28xx-vbi.c | 6 +-
drivers/media/usb/em28xx/em28xx-video.c | 127 +++++++-----
drivers/media/usb/em28xx/em28xx.h | 3 -
drivers/media/usb/pvrusb2/pvrusb2-i2c-core.c | 14 +-
drivers/media/usb/ttusb-dec/ttusb_dec.c | 14 +-
drivers/media/v4l2-core/tuner-core.c | 106 +++++-----
include/media/v4l2-common.h | 7 +
74 files changed, 2638 insertions(+), 2648 deletions(-)
--
2.7.4
next reply other threads:[~2016-11-16 16:43 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-16 16:42 Mauro Carvalho Chehab [this message]
2016-11-16 16:42 ` [PATCH 01/35] [media] stb0899_drv: get rid of continuation lines Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 02/35] [media] stv090x: " Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 03/35] [media] bt8xx/dst: use a more standard way to print messages Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 04/35] [media] bt8xx: use pr_foo() macros instead of printk() Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 05/35] [media] cx23885: use KERN_CONT where needed Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 06/35] [media] cx23885: convert it to use pr_foo() macros Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 07/35] [media] cx88: use KERN_CONT where needed Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 08/35] [media] cx88: convert it to use pr_foo() macros Mauro Carvalho Chehab
2016-11-18 22:27 ` Andrey Utkin
2016-11-19 12:16 ` Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 09/35] [media] cx88: make checkpatch happier Mauro Carvalho Chehab
2016-11-18 22:25 ` Andrey Utkin
2016-11-19 12:14 ` Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 10/35] [media] pluto2: use KERN_CONT where needed Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 11/35] [media] zoran: " Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 12/35] [media] wl128x: use KERNEL_CONT " Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 13/35] [media] pvrusb2: " Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 14/35] [media] ttusb_dec: " Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 15/35] [media] ttpci: cleanup debug macros and remove dead code Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 16/35] [media] dib0070: use pr_foo() instead of printk() Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 17/35] [media] dib0090: " Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 18/35] [media] dib3000mb: " Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 19/35] [media] dib3000mc: " Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 20/35] [media] dib7000m: " Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 21/35] [media] dib7000p: " Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 22/35] [media] dib8000: " Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 23/35] [media] dib9000: " Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 24/35] [media] dibx000_common: " Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 25/35] [media] af9005: remove a printk that would require a KERN_CONT Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 26/35] [media] tuner-core: use pr_foo, instead of internal printk macros Mauro Carvalho Chehab
2016-11-16 16:42 ` [PATCH 27/35] [media] v4l2-common: add a debug macro to be used with dev_foo() Mauro Carvalho Chehab
2016-11-16 16:43 ` [PATCH 28/35] [media] msp3400-driver: don't use KERN_CONT Mauro Carvalho Chehab
2016-11-16 16:43 ` [PATCH 29/35] [media] msp3400: convert it to use dev_foo() macros Mauro Carvalho Chehab
2016-11-16 16:43 ` [PATCH 30/35] [media] em28xx: convert it from pr_foo() to dev_foo() Mauro Carvalho Chehab
2016-11-16 16:43 ` [PATCH 31/35] [media] tvp5150: convert it to use dev_foo() macros Mauro Carvalho Chehab
2016-11-16 16:43 ` [PATCH 32/35] [media] tvp5150: Get rid of direct calls to printk() Mauro Carvalho Chehab
2016-11-16 16:43 ` [PATCH 33/35] [media] tvp5150: get rid of KERN_CONT Mauro Carvalho Chehab
2016-11-16 16:43 ` [PATCH 34/35] [media] rc-main: use pr_foo() macros Mauro Carvalho Chehab
2016-11-16 16:43 ` [PATCH 35/35] [media] tveeprom: print log messages using pr_foo() Mauro Carvalho Chehab
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=cover.1479314177.git.mchehab@s-opensource.com \
--to=mchehab@s-opensource.com \
--cc=Julia.Lawall@lip6.fr \
--cc=akpm@linux-foundation.org \
--cc=andrey.utkin@corp.bluecherry.net \
--cc=arnd@arndb.de \
--cc=dan.carpenter@oracle.com \
--cc=dheitmueller@kernellabs.com \
--cc=geert@linux-m68k.org \
--cc=hans.verkuil@cisco.com \
--cc=hansverk@cisco.com \
--cc=hkallweit1@gmail.com \
--cc=hverkuil@xs4all.nl \
--cc=inki.dae@samsung.com \
--cc=isely@pobox.com \
--cc=javier@osg.samsung.com \
--cc=jh1009.sung@samsung.com \
--cc=kamil@wypas.org \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-media@vger.kernel.org \
--cc=marcel@holtmann.org \
--cc=max@duempel.org \
--cc=mchehab@infradead.org \
--cc=mchehab@kernel.org \
--cc=mjpeg-users@lists.sourceforge.net \
--cc=mkrufky@linuxtv.org \
--cc=nenggun.kim@samsung.com \
--cc=olebowle@gmx.com \
--cc=olli.salonen@iki.fi \
--cc=patrick.boettcher@posteo.de \
--cc=prabhakar.csengg@gmail.com \
--cc=rmk+kernel@arm.linux.org.uk \
--cc=sakari.ailus@linux.intel.com \
--cc=sameo@linux.intel.com \
--cc=sean@mess.org \
--cc=stev391@gmail.com \
--cc=sw0312.kim@samsung.com \
--cc=weiyongjun1@huawei.com \
--cc=yamada.masahiro@socionext.com \
--cc=zzam@gentoo.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;
as well as URLs for NNTP newsgroup(s).