Removing device driver code in mainboard.c for Tyan boards?
Eric W. Biederman
ebiederman at lnxi.com
Fri Nov 5 18:49:01 CET 2004
YhLu <YhLu at tyan.com> writes:
> I changed e7501 to use root_complex too.
Could we please reserve this construct for chipsets
with multiple northbridges?
I think it is just going to be extra noise on a
simple chipset like the e7501.
And since the e7501 is an example to a certain extent having it
doing extra work be confusing and a pain.
> If the chip_ops has no .name, then should use
> if CONFIG_CHIP_NAME
> config chip.h
> end
>
> also .c include that chip_ops
> #if CONFIG_CHIP_NAME
> ...chip_ops defininition
> #endif
>
> Regards
>
> Such as norththbridge, socket, mainboard(root).
They can. But I am about to suggest simply removing the config
directive, and assuming the ``config chip.h'' is always given. The
overhead is almost zero for a zero filled structure. And always
requiring a chip.h would tend to simply the process of writing code
for a new chip.
Eric
More information about the coreboot
mailing list