Re: [PATCH -mm 0/4] MIPS BCM947xx CPUs support

[Aurelien Jarno <aurelien@aurel32.net>]

Ralf Baechle a écrit :
> On Mon, Aug 06, 2007 at 05:07:01PM +0200, Aurelien Jarno wrote:
> 
>> The following series of patches add basic support for the BCM947xx 
>> CPUs. CFE support still needs work and thus is not included in those 
>> patches, so the command line has to be included in the kernel. 
>> Everything else is fully functional and the resulting kernel works
>> fine on a Netgear WGT634U.
> 
> Any reason why not to simply share the CFE related code with what the
> Broadcom Sibyte family of chips are using?

That's actually the plan, but there is some work to do before. BCM947xx
CFE code is too BCM947xx centric, and the one in the sibyte/cfe
directory is too SiByte centric. For example the SiByte code assume that
the environment variables are in a serial EEPROM, whereas the BCM947xx
code assumes that it is in the FLASH.

Merging both will take some time, and if we accept to not get the kernel
parameters from CFE, it is possible to run BCM947xx board with these
patches.

>> I am submitting those patches for inclusion into -mm as they depend
>> on features that are not present in the linux-mips git tree, but are
>> present in the -mm series, namely Sonic Silicon Backplane bus support.
> 
> That won't fly as akpm is pulling from the MIPS tree also, so there
> will be conflicts really soon.
> 

I don't see a patch corresponding to the MIPS tree in the broken-out
directory. Anyway what other solution do you propose? I can see:
- Integrate the patches that have the most risk of conflicts (I think
patch #1) into the MIPS tree.
- Integrate all BCM947xx patches into the MIPS tree accepting that it
can compile without additional patches.
- Integrate SSB patches into the MIPS tree.

Aurelien

-- 
  .''`.  Aurelien Jarno	            | GPG: 1024D/F1BCDB73
 : :' :  Debian developer           | Electrical Engineer
 `. `'   aurel32@debian.org         | aurelien@aurel32.net
   `-    people.debian.org/~aurel32 | www.aurel32.net