[PATCH V3 00/12] CCF support for Renesas r7s72100

Laurent Pinchart laurent.pinchart at ideasonboard.com
Fri Mar 7 11:36:25 EST 2014


Hi Wolfram,

On Friday 07 March 2014 17:29:47 Wolfram Sang wrote:
> Hi Laurent,
> 
> > 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.

> > 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>


More information about the linux-arm-kernel mailing list