[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <54AA61D2.1040002@freescale.com>
Date: Mon, 5 Jan 2015 18:05:06 +0800
From: Liu Ying <Ying.Liu@...escale.com>
To: <dri-devel@...ts.freedesktop.org>, <stefan.wahren@...e.com>,
<devicetree@...r.kernel.org>, <linux@....linux.org.uk>,
<kernel@...gutronix.de>, <linux-kernel@...r.kernel.org>,
<a.hajda@...sung.com>, <mturquette@...aro.org>,
<linux-arm-kernel@...ts.infradead.org>, <andyshrk@...il.com>
Subject: Re: [PATCH RFC v7 16/21] drm: panel: Add support for Himax HX8369A
MIPI DSI panel
Hi Daniel,
On 01/05/2015 04:54 PM, Daniel Vetter wrote:
> On Tue, Dec 30, 2014 at 11:34:06AM +0800, Liu Ying wrote:
>> This patch adds support for Himax HX8369A MIPI DSI panel.
>>
>> Reviewed-by: Andrzej Hajda <a.hajda@...sung.com>
>> Signed-off-by: Liu Ying <Ying.Liu@...escale.com>
>> ---
>> v6->v7:
>> * Address Andrzej Hajda's following comments.
>> * Simplify the return logic in hx8369a_dcs_write().
>> * Replace the macro hx8369a_dsi_init_helper() with a function array to improve
>> the code quality.
>> * Handle error cases during getting gpios in probe().
>> * Add 'Reviewed-by: Andrzej Hajda <a.hajda@...sung.com>'.
>
> If you only update one patch in a big series it's imo better to just
> resend that one to avoid spamming the mailing list with noise. Imo it also
> helps to keep the discussion all nicely grouped together.
Thanks for your suggestion. I would try to make people happy about
the way the series is submitted for review.
>
> In any case please give a short overview of what changed in the cover
> letter when resending the entire series so that people don't have to go
> through all the invidual patches.
Yes, absolutely. I gave short overviews of what changed in the very
beginning of the cover letters from v2 to v8 and I will do this in
every necessary follow-up version.
Regards,
Liu Ying
> -Daniel
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists