From: Tom <Tom.Rix@windriver.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] v2 Linuxstamp II 9260 support
Date: Wed, 04 Nov 2009 18:39:41 -0600 [thread overview]
Message-ID: <4AF21ECD.8000308@windriver.com> (raw)
In-Reply-To: <c785bba30911041619o5a52febdx55960ae994775d1d@mail.gmail.com>
Paul Thomas wrote:
>> Another partition.c...
>> Looking for
>>
>> ugrep Ulf
>> ./board/ronetix/pm9263/partition.c
>> * Ulf Samuelsson <ulf@atmel.com>
>> ./board/ronetix/pm9261/partition.c
>> * Ulf Samuelsson <ulf@atmel.com>
>> ./board/atmel/at91sam9261ek/partition.c
>> * Ulf Samuelsson <ulf@atmel.com>
>> ./board/atmel/at91sam9260ek/partition.c
>> * Ulf Samuelsson <ulf@atmel.com>
>> ./board/atmel/at91sam9263ek/partition.c
>> * Ulf Samuelsson <ulf@atmel.com>
>> ./board/atmel/at91sam9rlek/partition.c
>> * Ulf Samuelsson <ulf@atmel.com>
>> ./board/atmel/at91cap9adk/partition.c
>> * Ulf Samuelsson <ulf@atmel.com>
>> ./board/atmel/at91rm9200dk/partition.c
>> * Ulf Samuelsson <ulf@atmel.com>
>> ./board/atmel/at91rm9200ek/led.c
>> * Ulf Samuelsson <ulf@atmel.com>
>> ./board/esd/meesc/partition.c
>> * Ulf Samuelsson <ulf@atmel.com>
>> ./include/configs/at91rm9200ek.h
>> * Ulf Samuelsson <ulf@atmel.com>
>>
>> Why do we need another copy?
>> Please move this to arm926ejs/at91/
>> If you are feeling adventurous, move the others, else I will take
>> care of it once this change is committed
>>
>
> I think the partition.c file should stay with the board and not move
> to arm926ejs/at91/. It's probably the case that most of the boards are
> using the 8MB dataflash with the same partition table, but it is still
> a board specific configuration.
>
I have looked at about half of them.
It looks like partition tables are identical.
The difference is more entries in the CS table.
I am interested in a way to generalize this file
and move it to the cpu layer so all the common
or nearly common files can be reduced to 1 file
Tom
> thanks,
> Paul
next prev parent reply other threads:[~2009-11-05 0:39 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-11-04 5:35 [U-Boot] [PATCH] v2 Linuxstamp II 9260 support Paul Thomas
2009-11-04 23:47 ` Tom
2009-11-05 0:19 ` Paul Thomas
2009-11-05 0:39 ` Tom [this message]
2009-11-05 4:10 ` Paul Thomas
2009-11-05 20:16 ` Wolfgang Denk
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=4AF21ECD.8000308@windriver.com \
--to=tom.rix@windriver.com \
--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.