[U-Boot] [PATCH 08/30] powerpc/km82xx: rename mgcoge2ne to mgcoge3ne board support

Holger Brunck holger.brunck at keymile.com
Mon May 2 11:22:09 CEST 2011


On 05/02/2011 10:42 AM, Wolfgang Denk wrote:
> Dear Holger Brunck,
> 
> In message <4DBE6962.1010609 at keymile.com> you wrote:
>>
>> the SDRAM configuration is different to mgcoge. And the CFI flash on mgcoge has
>> a dual die in a single chip and therefore e.g.CONFIG_SYS_MAX_FLASH_BANKS is
>> different to mgcoge3ne.
> 
> OK, but these can trivially be handled within one set of board files.
> 

with a lot of #ifdef MGCOGE3NE in one config file or do I misunderstand something?

>> Additionaly some functional differences are part of the board support, this is
>> added later on. E.g. mgcoge3ne is part of mgcoge3 which has two processors
>> deployed and there are some connections between the processors via GPIOs which
>> are not present on mgcoge. So in my opinion an additional board suport is
>> reasonable in this case.
> 
> Well, it's your board and you have to maintain it in the long run, but
> I'm not convinced that this is a wise decision.
> 

So your proposal would be to do something like
mgcoge  powerpc   mpc8260     km82xx  keymile - mgcoge3ne:MGCOGE3NE,...
in boards.cfg and only one config file for both boards?

Best regards
Holger Brunck





More information about the U-Boot mailing list