On Mon, 5 May 2014 08:46:24 -0700
Tim Harvey <thar...@gateworks.com> wrote:
> +cc Edward Lin
>
> On Mon, May 5, 2014 at 2:14 AM, Stefano Babic <sba...@denx.de> wrote:
> > Hi Tim,
> >
...
> >> --- /dev/null
> >> +++ b/arch/arm/imx-common/spl.c
> >> @@ -0,0 +1,79 @@
> >> +/*
> >> + * Copyright (C) 2014 Gateworks Corporation
> >> + * Author: Tim Harvey <thar...@gateworks.com>
> >> + *
> >> + * SPDX-License-Identifier: GPL-2.0+
> >> + */
> >> +
> >
> > Just a little remark. Do you have written the function on your own or
> > have you ported it from Freescale's 2009.08 ? Apart names, it looks similar.
> >
> > If this is the case, you should also add Freescale's Copyright to the
> > header.
>
> Hi Stefano,
>
> I've never looked at Freescales 2009.08 code myself (I have an
> aversion to vendor specific forks of u-boot almost 5 years old heh),
> but its likely bits of the function came from there originally but my
> inspiration likely came from Edward Lin's patch a while back. A
> switch/case seems the logical approach given the bitfield from the
> IMX6 reference manual with several don't-cares spread around and I
> felt that was much easier to read than trying to convert it all the
> shifts and masks.
>
> I can add additional copyright's if anyone feels its necessary - I've
> added Edward to the cc to see if he feels its necessary.
>
> Tim
>
> >
> >> +#include <common.h>
> >> +#include <asm/io.h>
> >> +#include <asm/arch/imx-regs.h>
> >> +#include <asm/spl.h>
> >> +#include <spl.h>
> >> +
> >> +#if defined(CONFIG_MX6)
> >> +/* determine boot device from SRC_SBMR1 register (BOOT_CFG[4:1]) */
> >> +u32 spl_boot_device(void)
> >> +{
> >> + struct src *psrc = (struct src *)SRC_BASE_ADDR;
> >> + unsigned reg = readl(&psrc->sbmr1);
> >> +
> >> + /* BOOT_CFG1[7:4] - see IMX6DQRM Table 8-8 */
> >> + switch ((reg & 0x000000FF) >> 4) {
> >> + /* EIM: See 8.5.1, Table 8-9 */
> >> + case 0x0:
> >> + /* BOOT_CFG1[3]: NOR/OneNAND Selection */
> >> + if ((reg & 0x00000008) >> 3)
> >> + return BOOT_DEVICE_ONENAND;
> >> + else
> >> + return BOOT_DEVICE_NOR;
> >> + break;
> >> + /* SATA: See 8.5.4, Table 8-20 */
> >> + case 0x2:
> >> + return BOOT_DEVICE_SATA;
> >> + /* Serial ROM: See 8.5.5.1, Table 8-22 */
> >> + case 0x3:
> >> + /* BOOT_CFG4[2:0] */
> >> + switch ((reg & 0x07000000) >> 24) {
> >> + case 0x0 ... 0x4:
> >> + return BOOT_DEVICE_SPI;
> >> + case 0x5 ... 0x7:
> >> + return BOOT_DEVICE_I2C;
> >> + }
> >> + break;
> >> + /* SD/eSD: 8.5.3, Table 8-15 */
> >> + case 0x4:
> >> + case 0x5:
> >> + return BOOT_DEVICE_MMC1;
> >> + /* MMC/eMMC: 8.5.3 */
> >> + case 0x6:
> >> + case 0x7:
> >> + return BOOT_DEVICE_MMC1;
> >> + /* NAND Flash: 8.5.2 */
> >> + case 0x8 ... 0xf:
> >> + return BOOT_DEVICE_NAND;
> >> + }
> >> + return BOOT_DEVICE_NONE;
> >> +}
> >> +#endif
> >> +
> >> +#if defined(CONFIG_SPL_MMC_SUPPORT)
> >> +/* called from spl_mmc to see type of boot mode for storage (RAW or FAT)
> >> */
> >> +u32 spl_boot_mode(void)
> >> +{
> >> + switch (spl_boot_device()) {
> >> + /* for MMC return either RAW or FAT mode */
> >> + case BOOT_DEVICE_MMC1:
> >> + case BOOT_DEVICE_MMC2:
> >> +#ifdef CONFIG_SPL_FAT_SUPPORT
> >> + return MMCSD_MODE_FAT;
> >> +#else
> >> + return MMCSD_MODE_RAW;
> >> +#endif
> >> + break;
> >> + default:
> >> + puts("spl: ERROR: unsupported device\n");
> >> + hang();
> >> + }
> >> +}
> >> +#endif
> >>
Tim, Stefano,
that piece of code indeed originates from Freescale's 2009.08 u-boot.
(Edward has changed jobs, so the CC:d address is of no use.)
regards,
//Tapani
ps: Tim, thanks for getting the SPL framework mainlined!
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
