Re: [PATCH 05/17] drm/sun4i: Reorder some code in DE2

From: Maxime Ripard
Date: Tue Nov 28 2017 - 15:26:52 EST


Hi,

On Mon, Nov 27, 2017 at 09:57:38PM +0100, Jernej Skrabec wrote:
> While DE2 driver works, parts of the code are not in optimal place. Reorder
> it so it will be easier to support multiple planes.
>
> This commit doesn't do any functional change besides removing two not
> very useful debug messages.
>
> Signed-off-by: Jernej Skrabec <jernej.skrabec@xxxxxxxx>

So I have the same feeling here that it could be split into several
patches, but you should at least explain why and how that code is more
optimal than it used to.

Thanks!
Maxime

--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

Attachment: signature.asc
Description: PGP signature