Re: [PATCH 1/2] I-pipe: Core implementation

[Karim Yaghmour]

Philippe Gerum wrote:
> There's a fourth one (ipipe/x86.c) added by the arch-dependent patch, 
> but yes, I agree that this could sound rather overkill to have this 
> support in its own dir, especially a top-level one. The files under 
> ipipe/ can be built as a loadable module, hence the current layout.
> Would you see this belonging to, e.g., the driver tree instead?

How about this instead:

Arch-indepedent parts:
----------------------
include/linux/ipipe.h

kernel/ipipe/Kconfig      (formerly ipipe/Kconfig)
kernel/ipipe/Makefile     (formerly ipipe/Makefile)
kernel/ipipe/core.c       (formerly kernel/ipipe.c)
kernel/ipipe/generic.c    (formerly ipipe/generi.c)

Arch-dependent parts:
---------------------
include/asm-i386/ipipe.h

arch/i386/kernel/ipipe-core.c  (formerly arch/i386/kernel/ipipe.c)
arch/i386/kernel/ipipe-root.c  (formerly ipipe/x86.c)

Seems to me that the above makes more sense. Albeit you would have
parts of the module in kernel/ipipe/* and the rest in
arch/*/kernel/ipipe*.

Karim
-- 
Author, Speaker, Developer, Consultant
Pushing Embedded and Real-Time Linux Systems Beyond the Limits
http://www.opersys.com || karim@xxxxxxxxxxx || 1-866-677-4546
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/