Re: [PATCH 1/6] bootwrapper: arch/powerpc/boot code reorg

On Wed, Aug 02, 2006 at 02:53:47PM +1000, Paul Mackerras wrote:

Mark A. Greer writes:

quoted

Abstract the operations used in the bootwrapper.  The operations
have been divided up into platform ops (platform_ops), firware ops
(fw_ops), device tree ops (dt_ops), and console ops (console_ops).

Overall the approach looks good.

The ops structure seems like a reasonable concept, but I question
whether we need to have platform_ops separate from fw_ops, since the
firmware is essentially part of the implementation of the platform.

Sometimes the people run different fw's on the same platform.  An
example is the sandpoint which comes with DINK but others run uboot on
it.

Also I don't see why we need to do a double indirection to get to each
ops function.

Okay.

Also, we will probably want to create some directory structure under
arch/powerpc/boot in the longer term.

Agreed.

quoted

+#ifdef __powerpc64__
+typedef unsigned long		u64;
+#else
+typedef unsigned long long	u64;
+#endif

This is potentially confusing, because the __powerpc64__ relates to
the boot wrapper not the kernel, and we don't build a 64-bit boot
wrapper on any platform (not yet anyway).  Maybe this needs a comment.

Okay.

Mark