[OpenBIOS] [PATCH] More complete PowerPC OpenFirmware emulation -- adding bus-frequency

[Natalia Portillo]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Hi all,

El 09/08/2011, a las 22:13, Alexander von Gluck escribió:

> On Tue, 9 Aug 2011 22:18:47 +0200, Andreas Färber wrote:
>> 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.
> 
> The whitespace changes just fix the white spaces of those variables directly
> above that don't match anything else. feel free to leave them out.
> 
> I can see your argument on blindly supporting it, Haiku checks for it and
> cpu frequency and stops booting if it is missing. I could just change haiku's
> code to not require it.. but:
> 
> * I'd rather have the information then not (even if it is just 0)
> * the parameter exists on *every* Apple openfirmware implementation
>   (i've verified this on a G3 bronze powerbook, G3 iBook, G4 QuickSilver, G5 iMac)
> * as the parameter exists on all new-world Apple systems, who knows what other
>   operating systems this breaks compatibility with?

Mac OS X and probably also Mac OS.


> Thanks!
> -- Alex
> 
> -- 
> OpenBIOS                 http://openbios.org/
> Mailinglist:  http://lists.openbios.org/mailman/listinfo
> Free your System - May the Forth be with you

-----BEGIN PGP SIGNATURE-----
Version: GnuPG/MacGPG2 v2.0.17 (Darwin)
Comment: GPGTools - http://gpgtools.org

iF4EAREIAAYFAk5Bo1cACgkQv/wfOsykIRTHXgD/eGnFozmCyG3TX8h2fCSCIKLE
5IsYxUcVaOn1/9Elis8BAMIuumJzXtI8u5wUJH/uiWLVaFJFEkHofX4pu8sFhuoo
=oov2
-----END PGP SIGNATURE-----