Re: [PATCH] [media] V4L/videobuf2-memops: use pr_debug for debug messages

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Acked-by: Kyungmin Park <kyunginn.,park@xxxxxxxxxxx>

---

I think it's better to add the videobuf2 maintainer entry for proper
person to know the changes.
In this case, Marek is missing.

If any objection, I will make a patch.

Thank you,
Kyungmin Park

2011/6/2 Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx>:
> Otherwise they clutter the dmesg buffer even on a production kernel.
>
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx>
> ---
>  drivers/media/video/videobuf2-memops.c |    6 +++---
>  1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/media/video/videobuf2-memops.c b/drivers/media/video/videobuf2-memops.c
> index 5370a3a..1987e1b1 100644
> --- a/drivers/media/video/videobuf2-memops.c
> +++ b/drivers/media/video/videobuf2-memops.c
> @@ -177,7 +177,7 @@ int vb2_mmap_pfn_range(struct vm_area_struct *vma, unsigned long paddr,
>
>        vma->vm_ops->open(vma);
>
> -       printk(KERN_DEBUG "%s: mapped paddr 0x%08lx at 0x%08lx, size %ld\n",
> +       pr_debug("%s: mapped paddr 0x%08lx at 0x%08lx, size %ld\n",
>                        __func__, paddr, vma->vm_start, size);
>
>        return 0;
> @@ -195,7 +195,7 @@ static void vb2_common_vm_open(struct vm_area_struct *vma)
>  {
>        struct vb2_vmarea_handler *h = vma->vm_private_data;
>
> -       printk(KERN_DEBUG "%s: %p, refcount: %d, vma: %08lx-%08lx\n",
> +       pr_debug("%s: %p, refcount: %d, vma: %08lx-%08lx\n",
>               __func__, h, atomic_read(h->refcount), vma->vm_start,
>               vma->vm_end);
>
> @@ -213,7 +213,7 @@ static void vb2_common_vm_close(struct vm_area_struct *vma)
>  {
>        struct vb2_vmarea_handler *h = vma->vm_private_data;
>
> -       printk(KERN_DEBUG "%s: %p, refcount: %d, vma: %08lx-%08lx\n",
> +       pr_debug("%s: %p, refcount: %d, vma: %08lx-%08lx\n",
>               __func__, h, atomic_read(h->refcount), vma->vm_start,
>               vma->vm_end);
>
> --
> 1.7.5.3
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux