From mboxrd@z Thu Jan 1 00:00:00 1970 From: laurent.pinchart@ideasonboard.com (Laurent Pinchart) Date: Fri, 07 Mar 2014 17:36:25 +0100 Subject: [PATCH V3 00/12] CCF support for Renesas r7s72100 In-Reply-To: <20140307162946.GG28943@katana> References: <1394208048-32495-1-git-send-email-wsa@the-dreams.de> <2368991.bjBJjms8qn@avalon> <20140307162946.GG28943@katana> Message-ID: <1552759.qGZNIXqrb1@avalon> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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: