Laurent Vivier wrote:
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).
Right. The reason for this is that all of the C memory management routines are in Forth order, and so if they are being called via CIF then the argument order needs to be reversed.
At the moment, the memory management routines are registered in 2 places: within the MMU/memory package (called from Forth) and within the CIF package. So what I did was add claim/release words back into forth/system/ciface.fs to swap the argument order and then pass onto the correct function.
However: the PPC and Sparc architectures use different C functions when called from CIF; hence I registered copies of whichever functions were used for that architecture as "cif-claim" and "cif-release" and then call these fixed names from "claim" and "release" in forth/system/ciface.fs. Does that help?
ATB,
Mark.