[PATCH] powerpc: consolidate mpc83xx platform files
Scott Wood
scottwood at freescale.com
Wed Dec 13 09:41:19 EST 2006
Kumar Gala wrote:
> On Dec 12, 2006, at 3:30 PM, Scott Wood wrote:
>> They *may* not want to (and they certainly shouldn't be forced to), but
>> some may not want to define a new ppc_md (or modify a probe function)
>> for every new board if all of the differences are encapsulated in the
>> device tree. I thought one of the main goals of having a device tree is
>> that if it's done right, the kernel need not know about every single
>> model of board, just the different components that a device tree can
>> specify.
>
> That's true, and if that's the case you'd just set your "model" to
> match an existing supported ppc_md.
Having an 831x explicitly claim to be an 834x is just a tad icky...
>> If a board has truly board-specific logic that needs custom code in the
>> kernel itself (rather than the bootloader), then it can go in as a
>> driver with a device tree node (this should be done with the BCSR stuff
>> where needed).
>
> This is not always the case, there are times when you have board
> specific modifications you make in the early kernel code.
Sure -- I'm not proposing doing away with board-specific machine
descriptions entirely, just reducing the circumstances where they're
required.
> True, but I dont see what the desire is to create a 'generic' 83xx
> support. Who gets to define what is considered 'generic'?
"Generic" is any board that has needs that aren't expressed in the
device tree.
> I'm all for refactoring code so my board code
> is simpler, but at the end of the day I know there are people that are
> going to need board specific code for their environments.
And an 83xx-generic machine description does not stop them from doing
so. "Generic" does not mean "universal". It means "there's nothing
special about this board". If you need board-specific code in the
kernel, then don't label it generic.
-Scott
More information about the Linuxppc-dev
mailing list