Hi Wolfram,

On Friday 07 March 2014 17:29:47 Wolfram Sang wrote:

Hi Laurent,

quoted

Beside the comment to patch 06/12, I would have moved patches 06/12 to
08/12 to the beginning of the series, in order to avoid adding a clock
workaround for the serial clock only to remove it a couple of patches
later.

That would have saved only 2 lines since we need the workaround anyhow
(MTU2) and would break the incremental approach of this series somewhat.
I'd like to keep it.

It would have saved one patch. If Simon needs to apply CCF support separately 
I'm fine with that approach, but if both changes are applied at the same time 
the SCIF patches could go in first.

quoted

Oh, and one might ask you to fix the 80 columns checkpatch warnings for
patch 01/12 :-)

Broken intentionally for the sake of readability :)

-- 
Regards,

Laurent Pinchart
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 490 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140307/4351711a/attachment.sig>