On Thu, Nov 5, 2009 at 7:53 AM, Guennadi Liakhovetski <g.liakhovetski@xxxxxx> wrote: > On Wed, 4 Nov 2009, Antonio Ospite wrote: > >> Signed-off-by: Antonio Ospite <ospite@xxxxxxxxxxxxxxxxx> >> Signed-off-by: Bart Visscher <bartv@xxxxxxxxxx> > > Is this patch going via Bart? Or should this be an Acked-by? Sometimes there are multiple people working on the same patch, and I don't see anyway for the SOB process to address this correctly since the author could be _only_ one and the SOB list respresents a patch passing route. So basically I don't care too much about SOB by multiple people yet not strictly conform to the patch forwarding route. > ... > > A general question for the ezx.c: wouldn't it be better to convert that > full-of-ifdef's file to a mach-pxa/ezx/ directory? > I'm only concerned that mach-pxa/* is already very crowded, I'm even thinkin of merging all those eseries things in a single file. The real difficulty with ezx is that they share many things, and the sharing sometimes is not just common, it can be shared by some of the platforms but not all - even if one eventually separates them into different files, there still needs to be a lot #ifdef .. #endif. Since these #ifdef .. #endif are actually inclusive, and can even be removed if no one cares about the size compiling all the platforms together, I don't see them as big problems. > Thanks > Guennadi > --- > Guennadi Liakhovetski, Ph.D. > Freelance Open-Source Software Developer > http://www.open-technology.de/ > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
