YhLu YhLu@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