On Thu, Jul 26, 2007 at 01:28:10AM -0500, Kumar Gala wrote:
>
>
> Begin forwarded message:
>
> >From: Zang Roy-r61911 <[EMAIL PROTECTED]>
> >Date: July 11, 2007 3:54:16 AM CDT
> >To: Paul Mackerras <[EMAIL PROTECTED]>
> >Cc: Kumar Gala <[EMAIL PROTECTED]>, linuxppc-dev list
> ><linuxppc-dev@ozlabs.org>
> >Subject: [PATCH] bootwrapper: adds cuboot for MPC7448HPC2 platform
> >
> >From: Roy Zang <[EMAIL PROTECTED]>
> >
> >This patch adds cuboot support for MPC7448HPC2 platform.
> >The cuImage can be used with legacy u-boot without FDT support.
> >
> >Signed-off-by: Roy Zang <[EMAIL PROTECTED]>
[snip]
> >--- /dev/null
> >+++ b/arch/powerpc/boot/cuboot-hpc2.c
> >@@ -0,0 +1,48 @@
> >+/*
> >+ * Copyright (C) 2007 Freescale Semiconductor, Inc. All rights
> >reserved.
> >+ *
> >+ * Author: Roy Zang <[EMAIL PROTECTED]>
> >+ *
> >+ * Description:
> >+ * Old U-boot compatibility for mpc7448hpc2 board
> >+ * Based on the code of Scott Wood <[EMAIL PROTECTED]>
> >+ * for 83xx and 85xx.
> >+ *
> >+ * This is free software; you can redistribute it and/or modify
> >+ * it under the terms of the GNU General Public License as
> >published by
> >+ * the Free Software Foundation; either version 2 of the
> >License, or
> >+ * (at your option) any later version.
> >+ *
> >+ */
> >+
> >+#include "ops.h"
> >+#include "stdio.h"
> >+#include "cuboot.h"
> >+
> >+#define TARGET_HAS_ENET1
> >+#include "ppcboot.h"
Including ppcboot.h without first defining TARGET_* defines which give
the type of system seems... fragile at best, broken at worst. Surely
this can only work by accident.
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev
