[U-Boot] [PATCH 1/3] arm: add CONFIG_MACH_TYPE option and documentation

2011-07-04 Thread Igor Grinberg
CONFIG_MACH_TYPE can be used to set the machine type number in the common arm code instead of setting it in the board code. Signed-off-by: Igor Grinberg --- README | 12 arch/arm/lib/board.c |5 + 2 files changed, 17 insertions(+), 0 deletions(-) diff --git

Re: [U-Boot] [PATCH 1/3] arm: add CONFIG_MACH_TYPE option and documentation

2011-07-04 Thread Christopher Harvey
On Mon, Jul 04, 2011 at 12:00:19PM +0300, Igor Grinberg wrote: > CONFIG_MACH_TYPE can be used to set the machine type number in the > common arm code instead of setting it in the board code. > > Signed-off-by: Igor Grinberg > --- > README | 12 > arch/arm/lib/board.c

Re: [U-Boot] [PATCH 1/3] arm: add CONFIG_MACH_TYPE option and documentation

2011-07-04 Thread Albert ARIBAUD
Hi Harvey, Le 04/07/2011 23:06, Christopher Harvey a écrit : > On Mon, Jul 04, 2011 at 12:00:19PM +0300, Igor Grinberg wrote: >> CONFIG_MACH_TYPE can be used to set the machine type number in the >> common arm code instead of setting it in the board code. >> >> Signed-off-by: Igor Grinberg >> ---

Re: [U-Boot] [PATCH 1/3] arm: add CONFIG_MACH_TYPE option and documentation

2011-07-04 Thread Albert ARIBAUD
Le 05/07/2011 00:03, Albert ARIBAUD a écrit : > Hi Harvey, Sorry Christopher, I mixed up. > Le 04/07/2011 23:06, Christopher Harvey a écrit : >> On Mon, Jul 04, 2011 at 12:00:19PM +0300, Igor Grinberg wrote: >>> CONFIG_MACH_TYPE can be used to set the machine type number in the >>> common arm cod

Re: [U-Boot] [PATCH 1/3] arm: add CONFIG_MACH_TYPE option and documentation

2011-07-04 Thread Wolfgang Denk
Dear Christopher Harvey, In message <20110704210619.ga3...@harvey-pc.matrox.com> you wrote: > > I'm curious, is it a feature that bd->bi_arch_number can be set at > runtime? Do any boards actually make a decision about what value to Yes, this is a feature. It comes in handy in a number of cases.

Re: [U-Boot] [PATCH 1/3] arm: add CONFIG_MACH_TYPE option and documentation

2011-07-05 Thread Igor Grinberg
On 07/05/11 00:06, Christopher Harvey wrote: > On Mon, Jul 04, 2011 at 12:00:19PM +0300, Igor Grinberg wrote: >> CONFIG_MACH_TYPE can be used to set the machine type number in the >> common arm code instead of setting it in the board code. >> >> Signed-off-by: Igor Grinberg >> --- >> README

Re: [U-Boot] [PATCH 1/3] arm: add CONFIG_MACH_TYPE option and documentation

2011-07-05 Thread charvey
On Tue, Jul 05, 2011 at 12:16:12AM +0200, Wolfgang Denk wrote: > Dear Christopher Harvey, > > In message <20110704210619.ga3...@harvey-pc.matrox.com> you wrote: > > > > I'm curious, is it a feature that bd->bi_arch_number can be set at > > runtime? Do any boards actually make a decision about what

Re: [U-Boot] [PATCH 1/3] arm: add CONFIG_MACH_TYPE option and documentation

2011-07-05 Thread Igor Grinberg
On 07/05/11 17:08, char...@matrox.com wrote: > On Tue, Jul 05, 2011 at 12:16:12AM +0200, Wolfgang Denk wrote: >> Dear Christopher Harvey, >> >> In message <20110704210619.ga3...@harvey-pc.matrox.com> you wrote: >>> I'm curious, is it a feature that bd->bi_arch_number can be set at >>> runtime? Do

Re: [U-Boot] [PATCH 1/3] arm: add CONFIG_MACH_TYPE option and documentation

2011-07-06 Thread Albert ARIBAUD
Hi Igor, Le 04/07/2011 11:00, Igor Grinberg a écrit : > CONFIG_MACH_TYPE can be used to set the machine type number in the > common arm code instead of setting it in the board code. > > Signed-off-by: Igor Grinberg > --- > README | 12 > arch/arm/lib/board.c |5

Re: [U-Boot] [PATCH 1/3] arm: add CONFIG_MACH_TYPE option and documentation

2011-07-06 Thread Igor Grinberg
On 07/06/11 21:53, Albert ARIBAUD wrote: > Hi Igor, > > Le 04/07/2011 11:00, Igor Grinberg a écrit : >> CONFIG_MACH_TYPE can be used to set the machine type number in the >> common arm code instead of setting it in the board code. >> >> Signed-off-by: Igor Grinberg >> --- >> README

Re: [U-Boot] [PATCH 1/3] arm: add CONFIG_MACH_TYPE option and documentation

2011-07-07 Thread Albert ARIBAUD
Hi Igor, Le 06/07/2011 22:05, Igor Grinberg a écrit : > On 07/06/11 21:53, Albert ARIBAUD wrote: > >> Hi Igor, >> >> Le 04/07/2011 11:00, Igor Grinberg a écrit : >>> CONFIG_MACH_TYPE can be used to set the machine type number in the >>> common arm code instead of setting it in the board code. >>>

Re: [U-Boot] [PATCH 1/3] arm: add CONFIG_MACH_TYPE option and documentation

2011-07-07 Thread Igor Grinberg
On 07/07/11 19:07, Albert ARIBAUD wrote: > Hi Igor, > > Le 06/07/2011 22:05, Igor Grinberg a écrit : >> On 07/06/11 21:53, Albert ARIBAUD wrote: >> >>> Hi Igor, >>> >>> Le 04/07/2011 11:00, Igor Grinberg a écrit : CONFIG_MACH_TYPE can be used to set the machine type number in the common

Re: [U-Boot] [PATCH 1/3] arm: add CONFIG_MACH_TYPE option and documentation

2011-07-07 Thread Albert ARIBAUD
Le 07/07/2011 18:51, Igor Grinberg a écrit : >>> If we have this option and it is documented, then any new board can use it >>> instead of thinking (although it is simple) where and how to dereference >>> the bi_arch_number. >> >> Not sure I get you there. Can you elaborate on a more precise examp

Re: [U-Boot] [PATCH 1/3] arm: add CONFIG_MACH_TYPE option and documentation

2011-07-07 Thread Igor Grinberg
On 07/07/11 20:46, Albert ARIBAUD wrote: > Le 07/07/2011 18:51, Igor Grinberg a écrit : > If we have this option and it is documented, then any new board can use it instead of thinking (although it is simple) where and how to dereference the bi_arch_number. >>> >>> Not sure I get you

Re: [U-Boot] [PATCH 1/3] arm: add CONFIG_MACH_TYPE option and documentation

2011-07-12 Thread Igor Grinberg
Hi Albert, On 07/08/11 00:06, Igor Grinberg wrote: > On 07/07/11 20:46, Albert ARIBAUD wrote: >> Le 07/07/2011 18:51, Igor Grinberg a écrit : >> > If we have this option and it is documented, then any new board can use it > instead of thinking (although it is simple) where and how to deref

Re: [U-Boot] [PATCH 1/3] arm: add CONFIG_MACH_TYPE option and documentation

2011-07-14 Thread Albert ARIBAUD
Hi Igor, Le 13/07/2011 07:52, Igor Grinberg a écrit : > Hi Albert, > > On 07/08/11 00:06, Igor Grinberg wrote: >> On 07/07/11 20:46, Albert ARIBAUD wrote: >>> Le 07/07/2011 18:51, Igor Grinberg a écrit : >>> >> If we have this option and it is documented, then any new board can use >> it

Re: [U-Boot] [PATCH 1/3] arm: add CONFIG_MACH_TYPE option and documentation

2011-07-14 Thread Albert ARIBAUD
Hi again Igor, Le 14/07/2011 16:10, Albert ARIBAUD a écrit : > Pragmatism against perfection: let's go for 2, then. Change the boards > you intended to change, and from now on reviewers for any change to a > board should point out the move to CONFIG_MACH_TYPE as mandatory. Forgot to mention: Th

Re: [U-Boot] [PATCH 1/3] arm: add CONFIG_MACH_TYPE option and documentation

2011-07-14 Thread Igor Grinberg
On 07/14/11 17:20, Albert ARIBAUD wrote: > Hi again Igor, > > Le 14/07/2011 16:10, Albert ARIBAUD a écrit : > >> Pragmatism against perfection: let's go for 2, then. Change the boards >> you intended to change, and from now on reviewers for any change to a >> board should point out the move to CON