All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sumit Semwal <sumit.semwal@linaro.org>
To: Tomi Valkeinen <tomi.valkeinen@ti.com>
Cc: Chandrabhanu Mahapatra <cmahapatra@ti.com>,
	linux-omap@vger.kernel.org, linux-fbdev@vger.kernel.org
Subject: Re: [PATCH V4 0/5] OMAPDSS: Enable dynamic debug printing
Date: Wed, 10 Oct 2012 10:38:38 +0000	[thread overview]
Message-ID: <50754D5E.8030206@linaro.org> (raw)
In-Reply-To: <1349441191.5836.8.camel@deskari>

Tomi, Chandrabhanu,

On Friday 05 October 2012 06:16 PM, Tomi Valkeinen wrote:
> On Sat, 2012-09-29 at 16:19 +0530, Chandrabhanu Mahapatra wrote:
>> Hi everyone,
>> this patch series aims at cleaning up of DSS of printk()'s enabled with
>> dss_debug and replace them with generic dynamic debug printing.
>
> Except for the missing debug print conversions in dsi.c this looks good.
> Do you want me to apply the current series and you can send the dsi.c
> patch later, or do you want to fix the dsi.c also before I apply?

With the change for DSI, please feel free to add

Reviewed-by: Sumit Semwal <sumit.semwal@ti.com>

BR,
~Sumit.
>
>   Tomi
>


WARNING: multiple messages have this Message-ID (diff)
From: Sumit Semwal <sumit.semwal@linaro.org>
To: Tomi Valkeinen <tomi.valkeinen@ti.com>
Cc: Chandrabhanu Mahapatra <cmahapatra@ti.com>,
	linux-omap@vger.kernel.org, linux-fbdev@vger.kernel.org
Subject: Re: [PATCH V4 0/5] OMAPDSS: Enable dynamic debug printing
Date: Wed, 10 Oct 2012 15:56:38 +0530	[thread overview]
Message-ID: <50754D5E.8030206@linaro.org> (raw)
In-Reply-To: <1349441191.5836.8.camel@deskari>

Tomi, Chandrabhanu,

On Friday 05 October 2012 06:16 PM, Tomi Valkeinen wrote:
> On Sat, 2012-09-29 at 16:19 +0530, Chandrabhanu Mahapatra wrote:
>> Hi everyone,
>> this patch series aims at cleaning up of DSS of printk()'s enabled with
>> dss_debug and replace them with generic dynamic debug printing.
>
> Except for the missing debug print conversions in dsi.c this looks good.
> Do you want me to apply the current series and you can send the dsi.c
> patch later, or do you want to fix the dsi.c also before I apply?

With the change for DSI, please feel free to add

Reviewed-by: Sumit Semwal <sumit.semwal@ti.com>

BR,
~Sumit.
>
>   Tomi
>


  reply	other threads:[~2012-10-10 10:38 UTC|newest]

Thread overview: 62+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-25  6:03 [PATCH] OMAPDSS: Cleanup DSSDBG with dynamic pr_debug function Chandrabhanu Mahapatra
2012-09-25  6:15 ` Chandrabhanu Mahapatra
2012-09-25  6:24 ` Tomi Valkeinen
2012-09-25  6:24   ` Tomi Valkeinen
2012-09-25  9:30   ` Mahapatra, Chandrabhanu
2012-09-25  9:42     ` Mahapatra, Chandrabhanu
2012-09-25  9:57     ` Tomi Valkeinen
2012-09-25  9:57       ` Tomi Valkeinen
2012-09-26  5:15 ` [PATCH V2 0/2] OMAPDSS: Enable dynamic debug printing Chandrabhanu Mahapatra
2012-09-26  5:27   ` Chandrabhanu Mahapatra
2012-09-26  5:15   ` [PATCH V2 1/2] OMAPDSS: Cleanup DSSDBG with dynamic pr_debug function Chandrabhanu Mahapatra
2012-09-26  5:27     ` Chandrabhanu Mahapatra
2012-09-26  5:16   ` [PATCH V2 2/2] OMAPDSS: Remove dss_debug variable Chandrabhanu Mahapatra
2012-09-26  5:28     ` Chandrabhanu Mahapatra
2012-09-26 14:29   ` [PATCH V2 0/2] OMAPDSS: Enable dynamic debug printing Tomi Valkeinen
2012-09-26 14:29     ` Tomi Valkeinen
2012-09-27 10:50     ` Mahapatra, Chandrabhanu
2012-09-27 10:50       ` Mahapatra, Chandrabhanu
2012-09-27 11:01       ` Tomi Valkeinen
2012-09-27 11:01         ` Tomi Valkeinen
2012-09-28 10:23   ` [PATCH V3 0/3] " Chandrabhanu Mahapatra
2012-09-28 10:35     ` Chandrabhanu Mahapatra
2012-09-28 10:23     ` [PATCH V3 1/3] OMAPDSS: Move definition of DEBUG flag to Makefile Chandrabhanu Mahapatra
2012-09-28 10:35       ` Chandrabhanu Mahapatra
2012-09-28 10:23     ` [PATCH V3 2/3] OMAPDSS: Cleanup DSSDBG with dynamic pr_debug function Chandrabhanu Mahapatra
2012-09-28 10:35       ` Chandrabhanu Mahapatra
2012-09-28 11:22       ` Tomi Valkeinen
2012-09-28 11:22         ` Tomi Valkeinen
2012-09-28 11:30         ` Mahapatra, Chandrabhanu
2012-09-28 11:42           ` Mahapatra, Chandrabhanu
2012-09-28 11:37           ` Tomi Valkeinen
2012-09-28 11:37             ` Tomi Valkeinen
2012-09-28 10:23     ` [PATCH V3 3/3] OMAPDSS: Remove dss_debug variable Chandrabhanu Mahapatra
2012-09-28 10:35       ` Chandrabhanu Mahapatra
2012-09-28 11:34     ` [PATCH V3 0/3] OMAPDSS: Enable dynamic debug printing Tomi Valkeinen
2012-09-28 11:34       ` Tomi Valkeinen
2012-09-28 12:11       ` Mahapatra, Chandrabhanu
2012-09-28 12:23         ` Mahapatra, Chandrabhanu
2012-09-29 10:49     ` [PATCH V4 0/5] " Chandrabhanu Mahapatra
2012-09-29 10:51       ` Chandrabhanu Mahapatra
2012-09-29 10:49       ` [PATCH V4 1/5] OMAPDSS: Move definition of DEBUG flag to Makefile Chandrabhanu Mahapatra
2012-09-29 10:51         ` Chandrabhanu Mahapatra
2012-09-29 10:49       ` [PATCH V4 2/5] OMAPDSS: Create new debug config options Chandrabhanu Mahapatra
2012-09-29 10:52         ` Chandrabhanu Mahapatra
2012-09-29 15:15         ` Sumit Semwal
2012-09-29 15:27           ` Sumit Semwal
2012-10-01  7:48         ` [PATCH V5 " Chandrabhanu Mahapatra
2012-10-01  7:51           ` Chandrabhanu Mahapatra
2012-09-29 10:49       ` [PATCH V4 3/5] OMAPDSS: Cleanup DSSDBG with dynamic pr_debug function Chandrabhanu Mahapatra
2012-09-29 10:52         ` Chandrabhanu Mahapatra
2012-09-29 10:49       ` [PATCH V4 4/5] OMAPDSS: Replace multi part debug prints with pr_debug Chandrabhanu Mahapatra
2012-09-29 10:52         ` Chandrabhanu Mahapatra
2012-10-05 12:33         ` Tomi Valkeinen
2012-10-05 12:33           ` Tomi Valkeinen
2012-10-10  9:22         ` [PATCH V5 " Chandrabhanu Mahapatra
2012-10-10  9:34           ` Chandrabhanu Mahapatra
2012-09-29 10:49       ` [PATCH V4 5/5] OMAPDSS: Remove dss_debug variable Chandrabhanu Mahapatra
2012-09-29 10:52         ` Chandrabhanu Mahapatra
2012-10-05 12:46       ` [PATCH V4 0/5] OMAPDSS: Enable dynamic debug printing Tomi Valkeinen
2012-10-05 12:46         ` Tomi Valkeinen
2012-10-10 10:26         ` Sumit Semwal [this message]
2012-10-10 10:38           ` Sumit Semwal

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=50754D5E.8030206@linaro.org \
    --to=sumit.semwal@linaro.org \
    --cc=cmahapatra@ti.com \
    --cc=linux-fbdev@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=tomi.valkeinen@ti.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.