From: Gregory CLEMENT <gregory.clement@free-electrons.com>
To: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Lior Amsalem <alior@marvell.com>,
barebox@lists.infradead.org, Willy Tarreau <w@1wt.eu>,
Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
Subject: Re: [PATCH 0/7] Basic support for Marvell Armada 370/XP SoC
Date: Wed, 08 May 2013 01:46:05 +0200 [thread overview]
Message-ID: <5189923D.5090901@free-electrons.com> (raw)
In-Reply-To: <51898E2F.3010404@free-electrons.com>
On 05/08/2013 01:28 AM, Gregory CLEMENT wrote:
> On 05/03/2013 06:51 PM, Thomas Petazzoni wrote:
>> Hello,
> Hi Thomas,
>
> it was a great work, I am currently trying to use but unfortunately,
> I didn't managed to run barebox on the Armada XP OpenBlocks AX3 from
> Plathome :/
But I managed to run barebox on the Mirabox!
>
> I get something like that:
>
> kwboot -t -b barebix.bin.kw -B 115200 /dev/ttyUSB0
> Sending boot message. Please reboot the target...-
>
> [here I power up the AX3]
>
> Sending boot image...
> 0 % [......................................................................]
> 4 % [......................................................................]
> 9 % [......................................................................]
> 14 % [......................................................................]
> 18 % [......................................................................]
> 23 % [......................................................................]
> 28 % [......................................................................]
> 33 % [......................................................................]
> 37 % [......................................................................]
> 42 % [......................................................................]
> 47 % [......................................................................]
> 52 % [.............................................................BootROM: Invalid header ID
> Booting from NOR flash
> DDR3 Training Sequence - Ver 3.0.0
>
> I also found typo in the command line example you gave to use kwbimage.
>
> I will try the other platform soon.
>
> [...]
>>
>> An image is then later created with:
>>
>> ./scripts/kwbimage -x -i <path/to/kwbimage.cfg> -o <image>
>
> I think it should be:
> ./scripts/kwbimage -c -i <path/to/kwbimage.cfg> -o <image>
>
> Thanks!
>>
>> For each board, the kwbimage.cfg file is typically located in
>> arch/arm/boards/<board-name>/. The DDR3 training code must however
>> be extracted from an existing bootloader image of your board,
>> usually the one provided by the board manufacturer.
>>
>> * A kwboot tool to push a bootloader through UART. It is directly
>> taken from U-Boot source code, to which I've added some fixes:
>>
>> - Extend the timeouts, to actually make it work on Armada
>> 370/XP. This has originally been found by Willy Tarreau.
>>
>> - Ignore non-Xmodem characters, so that the original DDR3 training
>> code can be used without modifications (Willy had to change it to
>> make it output its messages on a different serial port, otherwise
>> it was confusing the Xmodem implementation)
>>
>> - Output to stdout all the non-Xmodem characters so that if
>> something goes wrong during the transfer, we have some
>> informations. It also shows the messages output by the DDR3
>> training code.
>>
>> - Remove the 'patch' feature that patches an image to have the UART
>> type. This requires a knowledge of the header format, which is
>> different between version 0 (kirkwood) and version 1 (armada
>> 370/xp). It is not really needed anyway since kwbimage can
>> extract and create images.
>>
>> * The SoC-level code, which for now only consists in a minimal
>> clocksource driver, a function to register an UART, a fixed-rate
>> clock, and a function that determines the amount of RAM by looking
>> at the SDRAM windows registers.
>>
>> * The board-level code for the Armada 370 Mirabox from Globalscale,
>> the Armada XP OpenBlocks AX3 from Plathome and the Armada XP GP
>> from Marvell.
>>
>> Reviews and comments welcome!
>>
>> Best regards,
>>
>> Thomas
>>
>> Thomas Petazzoni (7):
>> scripts: allow lines longer than 80 cols with printf() in checkpatch
>> scripts: new kwbimage manipulation tool for Marvell SoC boot images
>> scripts: add kwboot tool
>> arm: initial support for Marvell Armada 370/XP SoCs
>> arm: add basic support for Armada XP OpenBlocks AX3 platform
>> arm: add basic support for the Armada 370 Mirabox platform
>> arm: add basic support for the Armada XP GP platform
>>
>> arch/arm/Kconfig | 8 +
>> arch/arm/Makefile | 4 +
>> arch/arm/boards/globalscale-mirabox/Makefile | 2 +
>> arch/arm/boards/globalscale-mirabox/config.h | 4 +
>> .../globalscale-mirabox/globalscale-mirabox.c | 26 +
>> arch/arm/boards/globalscale-mirabox/kwbimage.cfg | 8 +
>> arch/arm/boards/globalscale-mirabox/lowlevel.c | 26 +
>> arch/arm/boards/marvell-armada-xp-gp/Makefile | 2 +
>> arch/arm/boards/marvell-armada-xp-gp/config.h | 4 +
>> arch/arm/boards/marvell-armada-xp-gp/kwbimage.cfg | 8 +
>> arch/arm/boards/marvell-armada-xp-gp/lowlevel.c | 25 +
>> .../marvell-armada-xp-gp/marvell-armada-xp-gp.c | 25 +
>> arch/arm/boards/plathome-openblocks-ax3/Makefile | 2 +
>> arch/arm/boards/plathome-openblocks-ax3/config.h | 4 +
>> .../boards/plathome-openblocks-ax3/kwbimage.cfg | 8 +
>> arch/arm/boards/plathome-openblocks-ax3/lowlevel.c | 25 +
>> .../plathome-openblocks-ax3.c | 25 +
>> arch/arm/configs/globalscale_mirabox_defconfig | 8 +
>> arch/arm/configs/marvell_armada_xp_gp_defconfig | 10 +
>> arch/arm/configs/plathome_openblocks_ax3_defconfig | 9 +
>> arch/arm/mach-mvebu/Kconfig | 54 +
>> arch/arm/mach-mvebu/Makefile | 1 +
>> arch/arm/mach-mvebu/core.c | 142 +++
>> arch/arm/mach-mvebu/include/mach/clkdev.h | 7 +
>> arch/arm/mach-mvebu/include/mach/debug_ll.h | 40 +
>> arch/arm/mach-mvebu/include/mach/mvebu.h | 22 +
>> drivers/clocksource/Kconfig | 4 +
>> drivers/clocksource/Makefile | 1 +
>> drivers/clocksource/mvebu.c | 90 ++
>> scripts/.gitignore | 2 +
>> scripts/Makefile | 3 +-
>> scripts/checkpatch.pl | 2 +-
>> scripts/kwbimage.c | 1224 ++++++++++++++++++++
>> scripts/kwboot.c | 717 ++++++++++++
>> 34 files changed, 2540 insertions(+), 2 deletions(-)
>> create mode 100644 arch/arm/boards/globalscale-mirabox/Makefile
>> create mode 100644 arch/arm/boards/globalscale-mirabox/config.h
>> create mode 100644 arch/arm/boards/globalscale-mirabox/globalscale-mirabox.c
>> create mode 100644 arch/arm/boards/globalscale-mirabox/kwbimage.cfg
>> create mode 100644 arch/arm/boards/globalscale-mirabox/lowlevel.c
>> create mode 100644 arch/arm/boards/marvell-armada-xp-gp/Makefile
>> create mode 100644 arch/arm/boards/marvell-armada-xp-gp/config.h
>> create mode 100644 arch/arm/boards/marvell-armada-xp-gp/kwbimage.cfg
>> create mode 100644 arch/arm/boards/marvell-armada-xp-gp/lowlevel.c
>> create mode 100644 arch/arm/boards/marvell-armada-xp-gp/marvell-armada-xp-gp.c
>> create mode 100644 arch/arm/boards/plathome-openblocks-ax3/Makefile
>> create mode 100644 arch/arm/boards/plathome-openblocks-ax3/config.h
>> create mode 100644 arch/arm/boards/plathome-openblocks-ax3/kwbimage.cfg
>> create mode 100644 arch/arm/boards/plathome-openblocks-ax3/lowlevel.c
>> create mode 100644 arch/arm/boards/plathome-openblocks-ax3/plathome-openblocks-ax3.c
>> create mode 100644 arch/arm/configs/globalscale_mirabox_defconfig
>> create mode 100644 arch/arm/configs/marvell_armada_xp_gp_defconfig
>> create mode 100644 arch/arm/configs/plathome_openblocks_ax3_defconfig
>> create mode 100644 arch/arm/mach-mvebu/Kconfig
>> create mode 100644 arch/arm/mach-mvebu/Makefile
>> create mode 100644 arch/arm/mach-mvebu/core.c
>> create mode 100644 arch/arm/mach-mvebu/include/mach/clkdev.h
>> create mode 100644 arch/arm/mach-mvebu/include/mach/debug_ll.h
>> create mode 100644 arch/arm/mach-mvebu/include/mach/mvebu.h
>> create mode 100644 drivers/clocksource/mvebu.c
>> create mode 100644 scripts/kwbimage.c
>> create mode 100644 scripts/kwboot.c
>>
>
>
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next prev parent reply other threads:[~2013-05-07 23:46 UTC|newest]
Thread overview: 63+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-03 16:51 Thomas Petazzoni
2013-05-03 16:51 ` [PATCH 1/7] scripts: allow lines longer than 80 cols with printf() in checkpatch Thomas Petazzoni
2013-05-06 7:23 ` Sascha Hauer
2013-05-06 7:27 ` Thomas Petazzoni
2013-05-03 16:51 ` [PATCH 2/7] scripts: new kwbimage manipulation tool for Marvell SoC boot images Thomas Petazzoni
2013-05-04 19:51 ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-04 20:32 ` Jason Cooper
2013-05-05 6:40 ` Sascha Hauer
2013-05-05 11:19 ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-05 11:40 ` Willy Tarreau
2013-05-06 13:53 ` Thomas Petazzoni
2013-05-06 13:54 ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-06 14:06 ` Thomas Petazzoni
2013-05-06 14:04 ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-06 14:13 ` Thomas Petazzoni
2013-05-06 14:14 ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-06 14:31 ` Willy Tarreau
2013-05-06 19:34 ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-06 19:53 ` Willy Tarreau
2013-05-06 20:21 ` Thomas Petazzoni
2013-05-06 20:35 ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-06 20:44 ` Thomas Petazzoni
2013-05-06 20:56 ` Sascha Hauer
2013-05-06 21:03 ` Thomas Petazzoni
2013-05-07 6:31 ` Sascha Hauer
2013-05-07 9:33 ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-07 9:46 ` Willy Tarreau
2013-05-07 9:47 ` Sascha Hauer
2013-05-07 9:57 ` Thomas Petazzoni
2013-05-06 20:48 ` Sascha Hauer
2013-05-06 14:21 ` Jason Cooper
2013-05-06 19:39 ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-07 6:44 ` Sascha Hauer
2013-05-06 19:41 ` Sascha Hauer
2013-05-06 19:16 ` Sascha Hauer
2013-05-06 20:22 ` Thomas Petazzoni
2013-05-07 23:30 ` Gregory CLEMENT
2013-05-03 16:51 ` [PATCH 3/7] scripts: add kwboot tool Thomas Petazzoni
2013-05-03 16:51 ` [PATCH 4/7] arm: initial support for Marvell Armada 370/XP SoCs Thomas Petazzoni
2013-05-06 14:09 ` Thomas Petazzoni
2013-05-06 14:07 ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-06 14:30 ` Sascha Hauer
2013-05-06 14:34 ` Thomas Petazzoni
2013-05-06 14:46 ` Sascha Hauer
2013-05-06 14:48 ` Thomas Petazzoni
2013-05-03 16:51 ` [PATCH 5/7] arm: add basic support for Armada XP OpenBlocks AX3 platform Thomas Petazzoni
2013-05-05 6:59 ` Sascha Hauer
2013-05-03 16:51 ` [PATCH 6/7] arm: add basic support for the Armada 370 Mirabox platform Thomas Petazzoni
2013-05-03 16:51 ` [PATCH 7/7] arm: add basic support for the Armada XP GP platform Thomas Petazzoni
2013-05-04 17:15 ` [PATCH 0/7] Basic support for Marvell Armada 370/XP SoC Jason Cooper
2013-05-04 17:23 ` Re[2]: " Alexander Shiyan
2013-05-04 17:49 ` Thomas Petazzoni
2013-05-04 17:52 ` Re[2]: " Alexander Shiyan
2013-05-04 17:52 ` Thomas Petazzoni
2013-05-04 18:34 ` Jason Cooper
2013-05-05 6:33 ` Sascha Hauer
2013-05-06 13:54 ` Thomas Petazzoni
2013-05-06 19:28 ` Sascha Hauer
2013-05-06 20:32 ` Thomas Petazzoni
2013-05-07 23:28 ` Gregory CLEMENT
2013-05-07 23:46 ` Gregory CLEMENT [this message]
2013-05-08 10:46 ` Thomas Petazzoni
2013-05-08 21:02 ` Gregory CLEMENT
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=5189923D.5090901@free-electrons.com \
--to=gregory.clement@free-electrons.com \
--cc=alior@marvell.com \
--cc=barebox@lists.infradead.org \
--cc=ezequiel.garcia@free-electrons.com \
--cc=thomas.petazzoni@free-electrons.com \
--cc=w@1wt.eu \
/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