Do some printk cleanups to avoid troubles with continuation lines
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest25 patches 4 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/25 | tuner-xc2028: mark printk continuation lines as suchdrivers/ | +48 | 0 replies | |
| 2/25 | tuner-xc2028: don't break long linesdrivers/ | +9 | 0 replies | |
| 3/25 | em28xx: don't break long linesdrivers/ drivers/ drivers/ +3 more | +33 | 0 replies | |
| 4/25 | em28xx: mark printk continuation lines as suchdrivers/ | +6 | 0 replies | |
| 5/25 | em28xx: use pr_foo instead of em28xx-specific printk macrosdrivers/ drivers/ drivers/ +6 more | +215 | 0 replies | |
| 6/25 | em28xx: convert the remaining printks to pr_foodrivers/ drivers/ drivers/ +6 more | +146 | 0 replies | |
| 7/25 | dvb-core: don't break long linesdrivers/ | +1 | 0 replies | |
| 8/25 | tuner-core: don't break long linesdrivers/ | +3 | 0 replies | |
| 9/25 | tuner-core: use %&ph for small buffer dumpsdrivers/ | +3 | 0 replies | |
| 10/25 | radio-bcm2048: don't ignore errorsdrivers/ | +2 | 0 replies | |
| 11/25 | dvb-core: use pr_foo() instead of printk()drivers/ drivers/ drivers/ +3 more | +146 | 3 replies | |
| 12/25 | dvb_demux: convert an internal ifdef into a Kconfig optiondrivers/ drivers/ | +18 | 0 replies | |
| 13/25 | dvb_demux: uncomment a packet loss check codedrivers/ | +6 | 0 replies | |
| 14/25 | dvb-core: get rid of demux optional circular bufferdrivers/ drivers/ drivers/ +2 more | +6 | 0 replies | |
| 15/25 | dvb-core: move dvb_filter out of the DVB coredrivers/ drivers/ drivers/ +3 more | +59 | 0 replies | |
| 16/25 | dvb_filter: get rid of dead codedrivers/ | +0 | 0 replies | |
| 17/25 | dvb_filter: use KERN_CONT where neededdrivers/ | +6 | 0 replies | |
| 18/25 | uvc_driver: use KERN_CONT where neededdrivers/ | +15 | 1 reply | |
| 19/25 | imon: use %*ph to do small hexa dumpsdrivers/ | +2 | 0 replies | |
| 20/25 | mt20xx: use %*ph to do small hexa dumpsdrivers/ | +5 | 0 replies | |
| 21/25 | tvaudio: mark printk continuation lines as suchdrivers/ | +3 | 0 replies | |
| 22/25 | flexcop-i2c: mark printk continuation lines as suchdrivers/ | +5 | 0 replies | |
| 23/25 | cx2341x: mark printk continuation lines as suchdrivers/ | +6 | 0 replies | |
| 24/25 | dvb-pll: use pr_foo() macros instead of printk()drivers/ | +12 | 0 replies | |
| 25/25 | nxt6000: use pr_foo() macros instead of printk()drivers/ | +76 | 0 replies |
Reviews trailers per patch
1 of 25 patches has a Reviewed-by or Acked-by.
| Patch | Laurent Pinchart |
|---|---|
| 1/25 | |
| 2/25 | |
| 3/25 | |
| 4/25 | |
| 5/25 | |
| 6/25 | |
| 7/25 | |
| 8/25 | |
| 9/25 | |
| 10/25 | |
| 11/25 | |
| 12/25 | |
| 13/25 | |
| 14/25 | |
| 15/25 | |
| 16/25 | |
| 17/25 | |
| 18/25 | R-b |
| 19/25 | |
| 20/25 | |
| 21/25 | |
| 22/25 | |
| 23/25 | |
| 24/25 | |
| 25/25 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH 00/25] Do some printk cleanups to avoid troubles with continuation lines
This commit:
commit 563873318d328d9bbab4b00dfd835ac7c7e28697 Merge: 24532f768121 bfd8d3f23b51 Author: Linus Torvalds <[email protected]> Date:���Mon Oct 10 09:29:50 2016 -0700 Merge branch 'printk-cleanups'
Affects dmesg prints like:
printk(KERN_INFO "foo");
printk(" bar\n);Won't be doing the same output, as the Kernel won't understand the second printk as a continuation.
That's said, some media drivers use some very old driver-specific way of outputting messages, that came before pr_foo() and dev_foo() started being used at the subsystem.
This patch series start fixing those issues. I have another series of ~30-40 patches already, but they require more work, as they use dev_foo(), with could potentially cause some regression, and touch on a lot of drivers.
They also depend on a big patch that merges big printk lines, that I also want to split into smaller ones.
So, let me flush a series of cases that handle simpler cases.
Mauro Carvalho Chehab (25):
[media] tuner-xc2028: mark printk continuation lines as such [media] tuner-xc2028: don't break long lines
[media] em28xx: don't break long lines
[media] em28xx: mark printk continuation lines as such [media] em28xx: use pr_foo instead of em28xx-specific printk macros [media] em28xx: convert the remaining printks to pr_foo [media] dvb-core: don't break long lines
[media] tuner-core: don't break long lines
[media] tuner-core: use %&ph for small buffer dumps [media] radio-bcm2048: don't ignore errors
[media] dvb-core: use pr_foo() instead of printk() [media] dvb_demux: convert an internal ifdef into a Kconfig option [media] dvb_demux: uncomment a packet loss check code [media] dvb-core: get rid of demux optional circular buffer [media] dvb-core: move dvb_filter out of the DVB core [media] dvb_filter: get rid of dead code
[media] dvb_filter: use KERN_CONT where needed
[media] uvc_driver: use KERN_CONT where needed
[media] imon: use %*ph to do small hexa dumps
[media] mt20xx: use %*ph to do small hexa dumps
[media] tvaudio: mark printk continuation lines as such [media] flexcop-i2c: mark printk continuation lines as such [media] cx2341x: mark printk continuation lines as such [media] dvb-pll: use pr_foo() macros instead of printk() [media] nxt6000: use pr_foo() macros instead of printk()
drivers/media/common/b2c2/flexcop-common.h | 1 -
drivers/media/common/b2c2/flexcop-i2c.c | 10 +-
drivers/media/common/cx2341x.c | 12 +-
drivers/media/dvb-core/Kconfig | 13 +
drivers/media/dvb-core/Makefile | 2 +-
drivers/media/dvb-core/demux.h | 5 +-
drivers/media/dvb-core/dmxdev.c | 28 +-
drivers/media/dvb-core/dvb_ca_en50221.c | 57 +-
drivers/media/dvb-core/dvb_demux.c | 115 ++--
drivers/media/dvb-core/dvb_demux.h | 2 -
drivers/media/dvb-core/dvb_filter.c | 603 ---------------------
drivers/media/dvb-core/dvb_frontend.c | 12 +-
drivers/media/dvb-core/dvb_net.c | 85 +--
drivers/media/dvb-core/dvbdev.c | 25 +-
drivers/media/dvb-frontends/dvb-pll.c | 19 +-
drivers/media/dvb-frontends/nxt6000.c | 136 +++--
drivers/media/i2c/tvaudio.c | 5 +-
drivers/media/pci/ttpci/Makefile | 2 +-
drivers/media/pci/ttpci/dvb_filter.c | 114 ++++
drivers/media/{dvb-core => pci/ttpci}/dvb_filter.h | 0
drivers/media/rc/imon.c | 7 +-
drivers/media/tuners/mt20xx.c | 21 +-
drivers/media/tuners/tuner-xc2028.c | 116 ++--
drivers/media/usb/em28xx/em28xx-audio.c | 45 +-
drivers/media/usb/em28xx/em28xx-camera.c | 53 +-
drivers/media/usb/em28xx/em28xx-cards.c | 153 +++---
drivers/media/usb/em28xx/em28xx-core.c | 105 ++--
drivers/media/usb/em28xx/em28xx-dvb.c | 73 ++-
drivers/media/usb/em28xx/em28xx-i2c.c | 183 +++----
drivers/media/usb/em28xx/em28xx-input.c | 29 +-
drivers/media/usb/em28xx/em28xx-vbi.c | 7 +-
drivers/media/usb/em28xx/em28xx-video.c | 105 ++--
drivers/media/usb/em28xx/em28xx.h | 18 +-
drivers/media/usb/ttusb-dec/ttusb_dec.c | 58 +-
drivers/media/usb/uvc/uvc_driver.c | 30 +-
drivers/media/v4l2-core/tuner-core.c | 15 +-
drivers/staging/media/bcm2048/radio-bcm2048.c | 2 +
37 files changed, 896 insertions(+), 1370 deletions(-)
delete mode 100644 drivers/media/dvb-core/dvb_filter.c
create mode 100644 drivers/media/pci/ttpci/dvb_filter.c
rename drivers/media/{dvb-core => pci/ttpci}/dvb_filter.h (100%)Signature · 1 line
2.7.4