Am 09.08.2011 um 11:12 schrieb Mark Cave-Ayland:
On 08/08/11 16:07, Alexander von Gluck wrote:
Below is a quick patch to introduce the bus-frequency field for each CPU.
This exists on every Apple OpenFirmware i've seen and is expected on Haiku OS.
The change to config/scripts/switch-arch isn't required but lets you compile openbios using Haiku's PowerPC cross-compiler.
Also included is a extra tracing line for troubleshooting MMU issues.
This all looks sensible to me, however I'd like to leave this to either Alex or Andreas to do the final commit as they the current PPC maintainers.
Am a bit busy ATM and out-of-sync with qemu-devel... The [PATCH] as such is inacceptable to me (should've been [RFC] or a series of [PATCH]es) and is missing an SoB.
The switch-arch part looked okay, and if you and Blue agree, I'd sign off and apply that part - Haiku uses ELF just like Linux.
In fw_cfg.h there appeared to be some unrelated tab-to-whitespace changes and the init.c part surely needs matching QEMU support. Like I said earlier, I think fw_cfg is the wrong way to go long-term for such cosmetic stuff (we'd get a huge influx) but I wouldn't object to it as an interim solution if QEMU / Alex G. feel it's okay. The value is pretty meaningless on QEMU, so we should critically question Haiku's reliance upon it rather than blindly supplying it.
Andreas