[U-Boot] Best way of making some drivers common across kirkwood and orion5x SoCs?

[Prafulla Wadaskar]

Dear Albert


> -----Original Message-----
> From: u-boot-bounces at lists.denx.de 
> [mailto:u-boot-bounces at lists.denx.de] On Behalf Of Albert ARIBAUD
> Sent: Sunday, November 01, 2009 6:30 AM
> To: u-boot at lists.denx.de
> Subject: Re: [U-Boot] Best way of making some drivers common 
> across kirkwood and orion5x SoCs?
> 
> Wolfgang Denk a écrit :
> > Dear Albert ARIBAUD,
> > 
> > In message <4AEC53DB.6050509 at free.fr> you wrote:
> >> Would a name such as "marvell.h" be more acceptable? This would be
> > 
> > Hm... I expect there will be little chance to see MAarvell IP blocks
> > in non-Marvell SoCs, so this should work indeed. On the other hand,
> > why not chose a name that works for other CPUs / SoCs as 
> well. [So on
> > second thought maybe your "soc.h" was not that bad a name.]
> > 
> > Let's see what others suggest.

I propose the following strategy for this-
For orion- let's follow the standard development strategy used for ARM architectures including Kirkwood.
Since orion and kirkwood has lot of similarities but when both these will evolve for there entire supports there will be problems resulting code complexity.

So My suggestion is- let's develop orion as separate ARM SOC family as kirkwood.
Some of Kirkwood drivers can be directly used with Orion, that can be addressed through makefiles as we used NS16550 Serial support in Kirkwood. SOC specific configuration in the common drivers (for ex. Egiga) can always be addressed through individual <asm/arch/*.h>

> 
> While we wait for others' opinion, one quick question: do you 
> prefer me
> to submit this rework of the Marvell IP/drivers as individual patches,
> one for each IP/driver, or as a single patch for the whole rework?

It is always better to break down the entire work in the small pieces, it's easier to digest them faster :-)

Regards..
Prafulla . .

> 
> Amicalement,
> -- 
> Albert.
> 
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>