[OpenBIOS] [PATCH] Updated some device tree properties to match names used by Apple

[Mark Cave-Ayland]

On 12/04/14 13:18, BALATON Zoltan wrote:

> Signed-off-by: BALATON Zoltan <balaton at eik.bme.hu>
>
> Index: openbios-devel/arch/ppc/qemu/init.c
> ===================================================================
> --- openbios-devel/arch/ppc/qemu/init.c	(revision 1286)
> +++ openbios-devel/arch/ppc/qemu/init.c	(working copy)
> @@ -702,6 +732,12 @@
>       push_str("/");
>       fword("find-device");
>
> +    /* Apple calls the root node device-tree */
> +    if (is_apple()) {
> +        push_str("device-tree");
> +        fword("device-name");
> +    }
> +

I'd like to see the fact that this is a workaround for a 
MorphOS-specific bug recorded in both the comment and the commit message 
(I seem to remember it was mentioned in your original patch description?).

The reason for this is that OpenBIOS strives to be an IEEE1275-compliant 
firmware compatible, and so where we deviate from behaviour documented 
in the specification (particularly for compatibility purposes), I'd like 
a clear record of why the change is needed in order to prevent someone 
coming along later and inadvertently changing it.


ATB,

Mark.