On Tue, Aug 5, 2014 at 1:33 PM, Jindal, Sonika <sonika.jindal at intel.com> wrote: > > > On 8/5/2014 4:45 PM, Daniel Vetter wrote: >> >> On Tue, Aug 05, 2014 at 04:38:17PM +0530, sonika.jindal at intel.com wrote: >>> >>> From: Sonika Jindal <sonika.jindal at intel.com> >>> >>> Renaming defines to have levels instead of nominal values. >>> >>> Signed-off-by: Sonika Jindal <sonika.jindal at intel.com> >> >> >> You can't split up patches like this since this will break compilation. >> For larger stuff (and imo this is right above the cutoff) you first need >> to add the new functions/defines, then convert everyone over. And only >> when all the drivers are converted can we apply the patch to remove the >> old functions/defines. >> -Daniel >> > Got your concern. So, I will repost the first patch keeping both the defines > and an additional last patch for removing the extra defines.
Yeah. Btw for the actual replacement I highly recommend to do it with a semantic patch and cocci. There's unfortunately not a hole lot of good documentation around, but we recently started and it helps a lot with regenerating patches (e.g. for newly merged drm drivers) and avoiding mistakes for manual conversions. If we do a cocci patch we put it into the commit message, grep for @@. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch
