On Thu, Aug 23, 2012 at 12:26:53AM +0200, Marek Vasut wrote: > Dear dingu...@altera.com, > > > From: Dinh Nguyen <dingu...@altera.com> > > > > Add minimal support for Altera's SOCFPGA Cyclone 5 hardware. > > > > Signed-off-by: Dinh Nguyen <dingu...@altera.com> > > Signed-off-by: Pavel Machek <pa...@denx.de> > > [...] > > Please CC albert with new arches. > > > diff --git a/Makefile b/Makefile > > index 5ce5cc3..12aa372 100644 > > --- a/Makefile > > +++ b/Makefile > > @@ -369,7 +369,7 @@ BOARD_SIZE_CHECK = > > endif > > > > # Always append ALL so that arch config.mk's can add custom ones > > -ALL-y += $(obj)u-boot.srec $(obj)u-boot.bin $(obj)System.map > > +ALL-y += $(obj)u-boot.srec $(obj)u-boot.bin $(obj)u-boot.img > > $(obj)System.map > > This won't slide
You can place this into arch/arm/cpu/armv7/socfpga/config.mk (see am33xx/config.mk or any of the others) for examples. [snip] > > diff --git a/arch/arm/cpu/armv7/socfpga/spl.c > > b/arch/arm/cpu/armv7/socfpga/spl.c new file mode 100644 > > index 0000000..7bd1e2f > > --- /dev/null > > +++ b/arch/arm/cpu/armv7/socfpga/spl.c > > @@ -0,0 +1,182 @@ > > + /* > > + * Copyright (C) 2012 Altera Corporation <www.altera.com> > > + * > > + * This program 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. > > + * > > + * This program is distributed in the hope that it will be useful, > > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > > + * GNU General Public License for more details. > > + * > > + * You should have received a copy of the GNU General Public License > > + * along with this program. If not, see <http://www.gnu.org/licenses/>. > > + */ > > + > > +#include <common.h> > > +#include <asm/io.h> > > +#include <asm/u-boot.h> > > +#include <asm/utils.h> > > +#include <asm/arch/socfpga_spl.h> > > +#include <version.h> > > +#include <image.h> > > +#include <malloc.h> > > +#include <asm/arch/reset_manager.h> > > + > > +DECLARE_GLOBAL_DATA_PTR; > > + > > +static u32 *boot_params_ptr; > > +static struct spl_image_info spl_image; > > + > > +extern void __malloc_start, __malloc_end, __stack_start; > > + > > +/* Define global data structure pointer to it */ > > +static gd_t gdata __attribute__ ((section(".data"))); > > + > > +/* > > + * Error action > > + */ > > +inline void hang(void) > > +{ > > + puts("### ERROR ### Please RESET the board ###\n"); > > + for (;;) > > + ; > > +} > > + > > +/* > > + * Read the mkimage signature to extract info such as entry point. > > + * The header is generated by tools/mkimage > > + */ > > +static unsigned long spl_parse_image_header(const struct image_header > > *header) +{ > > + /* checking for mkimage signature */ > > + if (__be32_to_cpu(header->ih_magic) == IH_MAGIC) { > > + /* Valid image. Extract information out of header */ > > + spl_image.size = __be32_to_cpu(header->ih_size); > > + spl_image.entry_point = __be32_to_cpu(header->ih_load); > > + spl_image.load_addr = __be32_to_cpu(header->ih_load); > > + spl_image.os = header->ih_os; > > + spl_image.name = (const char *)&header->ih_name; > > + spl_image.crc = __be32_to_cpu(header->ih_dcrc); > > + debug("Subsequent boot image info\n"); > > + debug(" Image Name: %.*s\n", IH_NMLEN, spl_image.name); > > + debug(" Data Size: %d\n", spl_image.size); > > + debug(" Load Address: 0x%x\n", spl_image.load_addr); > > + debug(" Entry Point: 0x%X\n", spl_image.entry_point); > > + } else { > > + /* Not a valid image as mkimage signature not found */ > > + printf("Error : mkimage signature not found - ih_magic = %x\n", > > + header->ih_magic); > > + return 1; > > + } > > + > > + /* Checking image type. Do any customize stuff per image type here */ > > + switch (spl_image.os) { > > + case IH_OS_U_BOOT: > > + debug(" Image Type: U-Boot\n"); > > + break; > > + default: > > + printf(" Image Type: Unknown (%d)\n", spl_image.os); > > + ; > > + } > > + return 0; > > +} > > I think I don't even wanna know what's the purpose here :-) It's the omap-common/spl.c code with the bits for IH_OS_LINUX taken out :) Can you please re-do your series on top of the SPL series I just posted that provides a common SPL framework? Thanks! -- Tom _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot