All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 7/7] add support for palm treo 680 board
Date: Fri, 12 Apr 2013 19:26:27 +0200	[thread overview]
Message-ID: <201304121926.27431.marex@denx.de> (raw)
In-Reply-To: <51680344.80800@newsguy.com>

Dear Mike Dunn,

> On 04/11/2013 12:20 PM, Marek Vasut wrote:
> > Dear Mike Dunn,
> > 
> >> A quick overview of u-boot implementation on the treo 680...
> >> 
> >> The treo 680 has a Diskonchip G4 nand flash chip.  This device has a 2k
> >> region that maps to the system bus at the reset vector in a NOR-like
> >> fashion so that it can be used as the boot device.  The phone is shipped
> >> with this 2k region configured as write-protected (can't be modified)
> >> and programmed with an initial program loader (IPL).  At power-up, this
> >> IPL loads the contents of two flash blocks to SDRAM and jumps to it. 
> >> The capacity of the two blocks is not large enough to hold all of
> >> u-boot, so a u-boot SPL is used.  To conserve flash space, these two
> >> blocks and the necessary number of subsequent blocks are programmed
> >> with a concatenated spl + u-boot image.  That way, the IPL will also
> >> load a portion of u-boot proper, and when the spl runs, it relocates
> >> the portion of u-boot that the IPL has already loaded, and then resumes
> >> loading the remaining part of u-boot before jumping to it.
> >> 
> >> The default_environment is used (CONFIG_ENV_IS_NOWHERE) because I didn't
> >> think that having a writable environment was worth the cost of a flash
> >> block, although adding it would be straightforward.  I abuse the
> >> CONFIG_EXTRA_ENV_SETTINGS option to specify the usbtty for the console
> >> (CONFIG_SYS_CONSOLE_IS_IN_ENV).
> >> 
> >> Support for the LCD is included, but currently it is only useful for
> >> displaying the u-boot splash screen.  But if u-boot is built without the
> >> usbtty console, it does display the auto-boot progress nicely.
> >> 
> >> Signed-off-by: Mike Dunn <mikedunn@newsguy.com>
> > 
> > I think the tool shall really go as a separate patch. Besides, can the
> > tool not be implemented as a part of u-boot's mkimage infrastructure?
> 
> OK, I can make the flash_u-boot utility (which writes u-boot to the flash
> boot blocks) a separate patch.
> 
> As for making it part of mkimage...  I didn't really consider that because
> based on my (limited) knowledge, I figured they are unrelated.  As I
> understand it, mkimage creates an OS image file that u-boot can parse and
> load.  flash_u-boot is a utility that performs the task of writing u-boot
> itself to flash.

It can create an bootloader image that can be written to flash using standard 
mtd utilities. Does this not cut it for you? Why do you need a separate flasher, 
because the G4 is special ?

> I figured that mine was a special case, since u-boot must be written in a
> special format (redundant pages) and in a special manner (alternate 4k
> regions skipped), with the flash device in a special mode, and so it can
> not be done in the normal manner; e.g., 'nandwrite' from mtd-utils, or its
> u-boot 'nand write' equivalent, even if you first ran the u-boot image
> through a separate utility that simply converted the format of the image.
> 
> Hope that makes sense.  Any insight appreciated.  I'll take a look at
> what's in the tools directory.

I see ... so the G4 is such a horrible beast :( OK

Best regards,
Marek Vasut

  reply	other threads:[~2013-04-12 17:26 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-11  0:45 [U-Boot] [PATCH v2 0/7] palm treo 680 smartphone board support Mike Dunn
2013-04-11  0:45 ` [U-Boot] [PATCH v2 1/7] pxa_lcd: add the ACX544AKN lcd device Mike Dunn
2013-04-11 19:14   ` Marek Vasut
2013-04-11  0:45 ` [U-Boot] [PATCH v2 2/7] pxa_lcd: make lcd_enable() a weak pointer Mike Dunn
2013-04-11 19:15   ` Marek Vasut
2013-04-11  0:45 ` [U-Boot] [PATCH v2 3/7] pxa27x_udc: remove call to unimplemented set_GPIO_mode() Mike Dunn
2013-04-11 19:16   ` Marek Vasut
2013-04-11  0:45 ` [U-Boot] [PATCH v2 4/7] arm: bootm: call udc_disable()before booting linux Mike Dunn
2013-04-11 19:18   ` Marek Vasut
2013-04-11  0:45 ` [U-Boot] [PATCH v2 5/7] lib: import bitrev library from the linux kernel Mike Dunn
2013-04-11 19:19   ` Marek Vasut
2013-04-12 12:01     ` Mike Dunn
2013-04-12 17:28       ` Marek Vasut
2013-04-13 19:27       ` Wolfgang Denk
2013-04-11  0:45 ` [U-Boot] [PATCH v2 6/7] mtd: nand: add driver for diskonchip g4 nand flash Mike Dunn
2013-04-11 19:19   ` Marek Vasut
2013-04-12 16:29     ` Scott Wood
2013-04-11 21:00   ` Scott Wood
2013-04-12 13:36     ` Mike Dunn
2013-04-12 17:27     ` Marek Vasut
2013-04-12 17:34       ` Scott Wood
2013-04-11  0:45 ` [U-Boot] [PATCH v2 7/7] add support for palm treo 680 board Mike Dunn
2013-04-11 19:20   ` Marek Vasut
2013-04-12 12:51     ` Mike Dunn
2013-04-12 17:26       ` Marek Vasut [this message]
2013-04-14 17:44 ` [U-Boot] [PATCH v2 0/7] palm treo 680 smartphone board support Marek Vasut
2013-04-14 20:10   ` Mike Dunn
2013-04-14 20:13     ` Marek Vasut

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=201304121926.27431.marex@denx.de \
    --to=marex@denx.de \
    --cc=u-boot@lists.denx.de \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.