mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Sascha Hauer <s.hauer@pengutronix.de>
To: Roberto Nibali <rnibali@gmail.com>
Cc: barebox@lists.infradead.org
Subject: Re: Booting mx25 based device from SD and NOR
Date: Wed, 30 May 2012 07:47:38 +0200	[thread overview]
Message-ID: <20120530054738.GF30400@pengutronix.de> (raw)
In-Reply-To: <CAONxwYO8Y9uYCr7VeGq9YbybRAGOZFZEQiKOTRfzm-=fH-0ihg@mail.gmail.com>

On Tue, May 29, 2012 at 12:14:52PM +0200, Roberto Nibali wrote:
> ADDENDUM:
> 
> 
> diff --git a/arch/arm/boards/fq-sid1-mx25-noah/Makefile
> b/arch/arm/boards/fq-sid1-mx25-noah/Makefile
> new file mode 100644
> index 0000000..1e9cd54
> --- /dev/null
> +++ b/arch/arm/boards/fq-sid1-mx25-noah/Makefile
> @@ -0,0 +1,26 @@
> +#
> +# (C) Copyright 2007 Juergen Beisert <jbe@pengutronix.de>

You should replace this with your own copyright. I think Jürgen can live
with not having a copyright on trivial files ;)

> diff --git a/arch/arm/boards/fq-sid1-mx25-noah/env/config

Have you considered using one of the environment templates? You
shouldn't duplicate them.

> b/arch/arm/boards/fq-sid1-mx25-noah/env/config
> new file mode 100644
> index 0000000..240056d
> --- /dev/null
> +++ b/arch/arm/boards/fq-sid1-mx25-noah/env/config
> @@ -0,0 +1,25 @@
> +#!/bin/sh
> +
> +autoboot_timeout=5
> +
> +# can be either 'nfs', 'tftp', 'nor', or 'mmc'
> +kernel=tftp
> +root=mmc
> +
> +version=/v2.1.8-rc
> +tftp_uimage=$version/uImage
> +tftp_jffs2=$version/console-image.jffs2
> +bootargs="console=ttymxc0,115200"
> +nfsroot="/home/develop/SID1/nfsroot"
> +nor_parts="512k(barebox)ro,512k(bareboxenv),3072k(kernel),-(root)"
> +nor_rootpart="/dev/mtdblock3"
> +mmc_kernel="/dev/mmcblk0p1"
> +mmc_rootpart="/dev/mmcblk0p2"
> +getip=dhcp
> +
> +# or set your networking parameters here
> +#eth0.ipaddr=192.168.1.80
> +#eth0.netmask=255.255.255.0
> +#eth0.gateway=a.b.c.d
> +eth0.serverip=192.168.1.23
> +eth0.ethaddr=00:50:c2:8c:e6:0e

*never* *ever* add MAC addresses to the default environment.

> +
> diff --git a/arch/arm/boards/fq-sid1-mx25-noah/flash_header.c
> b/arch/arm/boards/fq-sid1-mx25-noah/flash_header.c
> new file mode 100644
> index 0000000..982fc52
> --- /dev/null
> +++ b/arch/arm/boards/fq-sid1-mx25-noah/flash_header.c
> @@ -0,0 +1,88 @@
> +#include <common.h>
> +#include <mach/imx-flash-header.h>
> +#include <mach/imx-regs.h>
> +#include <asm/barebox-arm-head.h>
> +
> +extern void exception_vectors(void);
> +
> +void __naked __flash_header_start go(void)
> +{
> + barebox_arm_head();
> +}
> +
> + DEST_BASE = IMX_CS0_BASE
> +   #else
> + DEST_BASE = TEXT_BASE
> +   #endif
> +   FLASH_HEADER_BASE = (DEST_BASE + FLASH_HEADER_OFFSET)
> + */
> +
> +struct imx_dcd_entry __dcd_entry_section dcd_entry[] = {
> + /* NOR flash, CS0_CSCRU, CS0_CSCRL, CS0_CSCRA */
> + { .ptr_type = 4, .addr = 0xB8002000, .val = 0x0000D003, },
> + { .ptr_type = 4, .addr = 0xB8002004, .val = 0x00330D01, },
> + { .ptr_type = 4, .addr = 0xB8002008, .val = 0x00220800, },
> + /* DDR2 init */
> + { .ptr_type = 4, .addr = 0xb8001004, .val = 0x0076e83a, }, /* initial
> value for ESDCFG0 */

Your mailer wraps lines.

> +#endif
> +#ifdef CONFIG_DRIVER_VIDEO_IMX
> + //imx25_add_fb(&noah_fb_data);

Please remove dead code before posting this for inclusion

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

  reply	other threads:[~2012-05-30  5:47 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-22 12:11 Roberto Nibali
2012-05-22 18:42 ` Sascha Hauer
2012-05-23 10:43   ` Roberto Nibali
2012-05-23 11:47     ` Eric Bénard
2012-05-24 12:49       ` Roberto Nibali
2012-05-24 12:58         ` Eric Bénard
2012-05-24 13:18           ` Roberto Nibali
2012-05-24 13:31             ` Eric Bénard
2012-05-25  9:04               ` Roberto Nibali
2012-05-25 10:08                 ` Eric Bénard
2012-05-29  9:06                   ` Roberto Nibali
2012-05-29  9:29                     ` Sascha Hauer
2012-05-29  9:56                       ` Roberto Nibali
2012-05-29 10:14                         ` Roberto Nibali
2012-05-30  5:47                           ` Sascha Hauer [this message]
2012-05-30  6:39                             ` Juergen Beisert
2012-05-31 13:12                             ` Roberto Nibali
2012-05-31 17:06                               ` Sascha Hauer
2012-06-01 10:25                                 ` Roberto Nibali
2012-06-05  7:09                                   ` Sascha Hauer
2012-05-24 17:17             ` Sascha Hauer
2012-05-25  9:19               ` Roberto Nibali
2012-05-25 10:01                 ` Sascha Hauer
2012-05-29  9:26                   ` Roberto Nibali

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120530054738.GF30400@pengutronix.de \
    --to=s.hauer@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    --cc=rnibali@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox