[OpenBIOS] PATCH v2: Rework CIF interface to handle parameters in reverse order
Laurent Vivier
Laurent at vivier.eu
Sun Nov 29 23:15:20 CET 2009
Le dimanche 29 novembre 2009 à 13:30 +0000, Mark Cave-Ayland a écrit :
> Hi everyone,
>
> Here is a second version of the patch which attempts to reverse the
> argument order of the CIF Forth words so that they can be called
> correctly from both Forth and C.
>
> The patch is fairly straightforward; probably the only unexpected part
> is the need to rename the existing /openprom/client-services "claim" and
> "release" words to "cif-claim" and "cif-release" respectively. This is
> because we need to use the "claim" and "release" words in
> forth/system/ciface.fs to reverse the argument order before calling the
> real underlying words.
>
> I've tested this on a FC12 PPC CD as well as my SPARC64 Milax CD and
> with this version of the patch, the CIF words are called correctly in
> both cases. I'd like to apply this reasonably soon, so please can people
> test on their setups to make sure it doesn't break anything?
>
> Signed-off-by: Mark Cave-Ayland <mark.cave-ayland at siriusit.co.uk>
Just one question about the patch: Why do you rename "claim" and
"release" to "cif-claim" and "cif-release" ?
Moreover, I think it is not compliant with OpenFirmware specification
(IEEE 1295).
Regards,
Laurent
--
--------------------- laurent at vivier.eu ----------------------
"Tout ce qui est impossible reste à accomplir" Jules Verne
"Things are only impossible until they're not" Jean-Luc Picard
More information about the OpenBIOS
mailing list