From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Mauro Carvalho Chehab <mchehab@s-opensource.com>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>,
Mauro Carvalho Chehab <mchehab@infradead.org>,
Mauro Carvalho Chehab <mchehab@kernel.org>
Subject: Re: [PATCH 18/25] [media] uvc_driver: use KERN_CONT where needed
Date: Mon, 17 Oct 2016 16:50:56 +0300 [thread overview]
Message-ID: <2921419.5PiyCZEmd7@avalon> (raw)
In-Reply-To: <cb3853709bfff02c2a967112531e87d49c74f4fd.1476466574.git.mchehab@s-opensource.com>
Hi Mauro,
Thank you for the patch.
On Friday 14 Oct 2016 14:45:56 Mauro Carvalho Chehab wrote:
> Some continuation messages are not using KERN_CONT.
>
> Since commit 563873318d32 ("Merge branch 'printk-cleanups"),
> this won't work as expected anymore. So, let's add KERN_CONT
> to those lines.
>
> Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
> drivers/media/usb/uvc/uvc_driver.c | 30 +++++++++++++++---------------
> 1 file changed, 15 insertions(+), 15 deletions(-)
>
> diff --git a/drivers/media/usb/uvc/uvc_driver.c
> b/drivers/media/usb/uvc/uvc_driver.c index 302e284a95eb..9c4b56b4a9c6
> 100644
> --- a/drivers/media/usb/uvc/uvc_driver.c
> +++ b/drivers/media/usb/uvc/uvc_driver.c
> @@ -1309,7 +1309,7 @@ static int uvc_scan_chain_entity(struct
> uvc_video_chain *chain, switch (UVC_ENTITY_TYPE(entity)) {
> case UVC_VC_EXTENSION_UNIT:
> if (uvc_trace_param & UVC_TRACE_PROBE)
> - printk(" <- XU %d", entity->id);
> + printk(KERN_CONT " <- XU %d", entity->id);
>
> if (entity->bNrInPins != 1) {
> uvc_trace(UVC_TRACE_DESCR, "Extension unit %d has more
"
> @@ -1321,7 +1321,7 @@ static int uvc_scan_chain_entity(struct
> uvc_video_chain *chain,
>
> case UVC_VC_PROCESSING_UNIT:
> if (uvc_trace_param & UVC_TRACE_PROBE)
> - printk(" <- PU %d", entity->id);
> + printk(KERN_CONT " <- PU %d", entity->id);
>
> if (chain->processing != NULL) {
> uvc_trace(UVC_TRACE_DESCR, "Found multiple "
> @@ -1334,7 +1334,7 @@ static int uvc_scan_chain_entity(struct
> uvc_video_chain *chain,
>
> case UVC_VC_SELECTOR_UNIT:
> if (uvc_trace_param & UVC_TRACE_PROBE)
> - printk(" <- SU %d", entity->id);
> + printk(KERN_CONT " <- SU %d", entity->id);
>
> /* Single-input selector units are ignored. */
> if (entity->bNrInPins == 1)
> @@ -1353,7 +1353,7 @@ static int uvc_scan_chain_entity(struct
> uvc_video_chain *chain, case UVC_ITT_CAMERA:
> case UVC_ITT_MEDIA_TRANSPORT_INPUT:
> if (uvc_trace_param & UVC_TRACE_PROBE)
> - printk(" <- IT %d\n", entity->id);
> + printk(KERN_CONT " <- IT %d\n", entity->id);
>
> break;
>
> @@ -1361,17 +1361,17 @@ static int uvc_scan_chain_entity(struct
> uvc_video_chain *chain, case UVC_OTT_DISPLAY:
> case UVC_OTT_MEDIA_TRANSPORT_OUTPUT:
> if (uvc_trace_param & UVC_TRACE_PROBE)
> - printk(" OT %d", entity->id);
> + printk(KERN_CONT " OT %d", entity->id);
>
> break;
>
> case UVC_TT_STREAMING:
> if (UVC_ENTITY_IS_ITERM(entity)) {
> if (uvc_trace_param & UVC_TRACE_PROBE)
> - printk(" <- IT %d\n", entity->id);
> + printk(KERN_CONT " <- IT %d\n", entity->id);
> } else {
> if (uvc_trace_param & UVC_TRACE_PROBE)
> - printk(" OT %d", entity->id);
> + printk(KERN_CONT " OT %d", entity->id);
> }
>
> break;
> @@ -1416,9 +1416,9 @@ static int uvc_scan_chain_forward(struct
> uvc_video_chain *chain, list_add_tail(&forward->chain, &chain->entities);
> if (uvc_trace_param & UVC_TRACE_PROBE) {
> if (!found)
> - printk(" (->");
> + printk(KERN_CONT " (->");
>
> - printk(" XU %d", forward->id);
> + printk(KERN_CONT " XU %d", forward->id);
> found = 1;
> }
> break;
> @@ -1436,16 +1436,16 @@ static int uvc_scan_chain_forward(struct
> uvc_video_chain *chain, list_add_tail(&forward->chain, &chain->entities);
> if (uvc_trace_param & UVC_TRACE_PROBE) {
> if (!found)
> - printk(" (->");
> + printk(KERN_CONT " (->");
>
> - printk(" OT %d", forward->id);
> + printk(KERN_CONT " OT %d", forward->id);
> found = 1;
> }
> break;
> }
> }
> if (found)
> - printk(")");
> + printk(KERN_CONT ")");
>
> return 0;
> }
> @@ -1471,7 +1471,7 @@ static int uvc_scan_chain_backward(struct
> uvc_video_chain *chain, }
>
> if (uvc_trace_param & UVC_TRACE_PROBE)
> - printk(" <- IT");
> + printk(KERN_CONT " <- IT");
>
> chain->selector = entity;
> for (i = 0; i < entity->bNrInPins; ++i) {
> @@ -1485,14 +1485,14 @@ static int uvc_scan_chain_backward(struct
> uvc_video_chain *chain, }
>
> if (uvc_trace_param & UVC_TRACE_PROBE)
> - printk(" %d", term->id);
> + printk(KERN_CONT " %d", term->id);
>
> list_add_tail(&term->chain, &chain->entities);
> uvc_scan_chain_forward(chain, term, entity);
> }
>
> if (uvc_trace_param & UVC_TRACE_PROBE)
> - printk("\n");
> + printk(KERN_CONT "\n");
>
> id = 0;
> break;
--
Regards,
Laurent Pinchart
next prev parent reply other threads:[~2016-10-17 13:50 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-10-14 17:45 [PATCH 00/25] Do some printk cleanups to avoid troubles with continuation lines Mauro Carvalho Chehab
2016-10-14 17:45 ` [PATCH 01/25] [media] tuner-xc2028: mark printk continuation lines as such Mauro Carvalho Chehab
2016-10-14 17:45 ` [PATCH 02/25] [media] tuner-xc2028: don't break long lines Mauro Carvalho Chehab
2016-10-14 17:45 ` [PATCH 03/25] [media] em28xx: " Mauro Carvalho Chehab
2016-10-14 17:45 ` [PATCH 04/25] [media] em28xx: mark printk continuation lines as such Mauro Carvalho Chehab
2016-10-14 17:45 ` [PATCH 05/25] [media] em28xx: use pr_foo instead of em28xx-specific printk macros Mauro Carvalho Chehab
2016-10-14 17:45 ` [PATCH 06/25] [media] em28xx: convert the remaining printks to pr_foo Mauro Carvalho Chehab
2016-10-14 17:45 ` [PATCH 07/25] [media] dvb-core: don't break long lines Mauro Carvalho Chehab
2016-10-14 17:45 ` [PATCH 08/25] [media] tuner-core: " Mauro Carvalho Chehab
2016-10-14 17:45 ` [PATCH 09/25] [media] tuner-core: use %&ph for small buffer dumps Mauro Carvalho Chehab
2016-10-14 17:45 ` [PATCH 10/25] [media] radio-bcm2048: don't ignore errors Mauro Carvalho Chehab
2016-10-14 17:45 ` [PATCH 11/25] [media] dvb-core: use pr_foo() instead of printk() Mauro Carvalho Chehab
2016-10-14 18:22 ` SF Markus Elfring
2016-10-21 10:34 ` Mauro Carvalho Chehab
2016-10-14 20:15 ` Arnd Bergmann
2016-10-14 17:45 ` [PATCH 12/25] [media] dvb_demux: convert an internal ifdef into a Kconfig option Mauro Carvalho Chehab
2016-10-14 17:45 ` [PATCH 13/25] [media] dvb_demux: uncomment a packet loss check code Mauro Carvalho Chehab
2016-10-14 17:45 ` [PATCH 14/25] [media] dvb-core: get rid of demux optional circular buffer Mauro Carvalho Chehab
2016-10-14 17:45 ` [PATCH 15/25] [media] dvb-core: move dvb_filter out of the DVB core Mauro Carvalho Chehab
2016-10-14 17:45 ` [PATCH 16/25] [media] dvb_filter: get rid of dead code Mauro Carvalho Chehab
2016-10-14 17:45 ` [PATCH 17/25] [media] dvb_filter: use KERN_CONT where needed Mauro Carvalho Chehab
2016-10-14 17:45 ` [PATCH 18/25] [media] uvc_driver: " Mauro Carvalho Chehab
2016-10-17 13:50 ` Laurent Pinchart [this message]
2016-10-14 17:45 ` [PATCH 19/25] [media] imon: use %*ph to do small hexa dumps Mauro Carvalho Chehab
2016-10-14 17:45 ` [PATCH 20/25] [media] mt20xx: " Mauro Carvalho Chehab
2016-10-14 17:45 ` [PATCH 21/25] [media] tvaudio: mark printk continuation lines as such Mauro Carvalho Chehab
2016-10-14 17:46 ` [PATCH 22/25] [media] flexcop-i2c: " Mauro Carvalho Chehab
2016-10-14 17:46 ` [PATCH 23/25] [media] cx2341x: " Mauro Carvalho Chehab
2016-10-14 17:46 ` [PATCH 24/25] [media] dvb-pll: use pr_foo() macros instead of printk() Mauro Carvalho Chehab
2016-10-14 17:46 ` [PATCH 25/25] [media] nxt6000: " 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=2921419.5PiyCZEmd7@avalon \
--to=laurent.pinchart@ideasonboard.com \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@infradead.org \
--cc=mchehab@kernel.org \
--cc=mchehab@s-opensource.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