Hi Christoph, On Tuesday, 9. May 2006 17:15, Christoph Hellwig wrote: > > Signed-off-by: Christian Limpach <Christian.Limpach@xxxxxxxxxxxx> > > Signed-off-by: Chris Wright <chrisw@xxxxxxxxxxxx> > > --- > > include/xen/interface/arch-x86_32.h | 197 +++++++++++++++ > > that kind of stuff needs to go to asm/ > > > include/xen/interface/event_channel.h | 205 +++++++++++++++ > > instead of interface please use something shorter, we'll see this > all over the includes statements. intf for example. I like them and think they are quite clear. Documentation/CodingStyle Chapter 4: Naming seem to apply here. And since you type the include only ONCE per file, this looks like a good trade, doesn't it? Regards Ingo Oeser
