OpenBIOS February 2016

openbios@openbios.org

11 participants
39 discussions

advice/help on OF device nodes?
by Alyssa Milburn 07 Feb '16

07 Feb '16

Hi, I wanted to do the below (based on the nvram code in the same file), but it doesn't work (I always get zero). Can anyone explain what I'm doing wrong? (This isn't a suggested fix/patch, I'm just wondering what I'm missing.) - Alyssa @@ -162,6 +162,7 @@ macio_nvram_get(char *buf) static void openpic_init(const char *path, phys_addr_t addr) { + phandle_t chosen; phandle_t dnode; int props[2]; char buf[128]; @@ -186,6 +187,11 @@ openpic_init(const char *path, phys_addr_t addr) set_int_property(dnode, "clock-frequency", 4166666); fword("finish-device"); + + chosen = find_dev("/chosen"); + push_str(buf); + fword("open-dev"); + set_int_property(chosen, "interrupt-controller", POP()); } DECLARE_NODE(ob_macio, INSTALL_OPEN, sizeof(int), "Tmac-io");

3 3

patch to allow Mac OS 9 to boot
by Programmingkid 06 Feb '16

06 Feb '16

http://www.openfirmware.info/pipermail/openbios/2015-June/008722.html diff --git a/openbios-devel/forth/bootstrap/interpreter.fs b/openbios-devel/forth/bootstrap/interpreter.fs index 5187058..74a109f 100644 --- a/openbios-devel/forth/bootstrap/interpreter.fs +++ b/openbios-devel/forth/bootstrap/interpreter.fs @@ -163,7 +163,7 @@ defer outer-interpreter : evaluate ( str len -- ?? ) 2dup + -rot over + over do - i c@ 0a = if + i c@ dup 0a = swap 0d = or if i over - This is most of the patch that is needed to make OpenBIOS successfully boot Mac OS 9. From the bootstrap.fs file, the 0a means linefeed. The file also says the 0d means carret '^'? If this patch is to look for linefeeds and carriage returns, why use the carret character here? This is a solution I came up with to fix incompatibility with other operating systems and still be able to boot Mac OS 9. The adler32 word is only used on Mac OS 9, so what if it sets a flag variable to indicate Mac OS 9 is being used. Then OpenBIOS uses that flag to determine which line of code should be used. Here is the pseudo-code: In adler32.... set using_mac_os_9 = true In the evaluate word: if (using_mac_os_9 == true) { i c@ dup 0a = swap 0d = or if } else { i c@ 0a = if } Does this sound good?

4 15

[PATCH] add copyright message to root node at startup
by Programmingkid 06 Feb '16

06 Feb '16

Mac OS 9 and earlier require that OpenBIOS has Apple's copyright message in the root node, under the copyright property. This patch adds that message. This patch also removes the code from the adler32 word that did the same thing. Operating systems might need to read the copyright message before the adler32 word is called. So this patch takes care of that problem by supplying the copyright message at startup. This makes the copyright adding code in the adler32 word obsolete. Only the Macintosh emulator will have this message added to the root node. Other machines such as PREP don't run Apple's software so the copyright message will not be added to them. Signed-off-by: John Arbuckle <programmingkidx(a)gmail.com> Index: arch/ppc/qemu/init.c =================================================================== --- arch/ppc/qemu/init.c (revision 1378) +++ arch/ppc/qemu/init.c (working copy) @@ -739,6 +739,32 @@ RET(s2 << 16 | s1); } +/* Encodes and decodes a string */ +static void string_encoder (unsigned char *string, int length) { + const int encoder = 1; + int i; + for (i = 0; i < length; i++) { + string[i] ^= encoder; + } +} + +/* + * Adds the copyright string to the root node. + * Classic Mac OS requires it to boot. + */ +static void add_copyright() +{ + const int size_of_string = 60; + unsigned char message[size_of_string + 1]; + snprintf(message, size_of_string + 1, "%s", "Bnqxshfiu!0892,3110!@qqmd!Bnl" + "qtuds-!Hob/!@mm!Shfiur!Sdrdswde"); + string_encoder(message, size_of_string); + push_str(message); + fword("encode-string"); + push_str("copyright"); + fword("property"); +} + void arch_of_init(void) { @@ -827,10 +853,19 @@ fword("encode-int"); push_str("clock-frequency"); fword("property"); + + /* + * We might figure out how to run the classic Mac OS on the g3beige + * target one day + */ + add_copyright(); + break; case ARCH_MAC99: case ARCH_MAC99_U3: + /* Kept here because PREP isn't made to run Apple's software */ + add_copyright(); case ARCH_PREP: default: Index: arch/ppc/qemu/qemu.fs =================================================================== --- arch/ppc/qemu/qemu.fs (revision 1378) +++ arch/ppc/qemu/qemu.fs (working copy) @@ -99,19 +99,6 @@ \ ------------------------------------------------------------------------- : adler32 ( adler buf len -- checksum ) - \ Since Mac OS 9 is the only system using this word, we take this - \ opportunity to inject a copyright message that is necessary for the - \ system to boot. - " Copyright 1983-2001 Apple Computer, Inc. THIS MESSAGE FOR COMPATIBILITY ONLY" - encode-string " copyright" - " /" find-package if - " set-property" $find if - execute - else - 3drop drop - then - then - ( adler buf len ) " (adler32)" $find if

4 16

[PATCH v2] add copyright message to root node at startup
by Programmingkid 05 Feb '16

05 Feb '16

Mac OS 9 and earlier require that OpenBIOS has Apple's copyright message in the root node, under the copyright property. This patch adds that message. This patch also removes the code from the adler32 word that did the same thing. Operating systems might need to read the copyright message before the adler32 word is called. So this patch takes care of that problem by supplying the copyright message at startup. This makes the copyright adding code in the adler32 word obsolete. Only the Macintosh emulator will have this message added to the root node. Other machines such as PREP don't run Apple's software so the copyright message will not be added to them. Signed-off-by: John Arbuckle <programmingkidx(a)gmail.com> Changes from version 1: Removed the add_copyright() message from the ARCH_HEATHROW target because the real life version of this machine doesn't have the copyright message. Index: arch/ppc/qemu/init.c =================================================================== --- arch/ppc/qemu/init.c (revision 1378) +++ arch/ppc/qemu/init.c (working copy) @@ -739,6 +739,32 @@ RET(s2 << 16 | s1); } +/* Encodes and decodes a string */ +static void string_encoder (unsigned char *string, int length) { + const int encoder = 1; + int i; + for (i = 0; i < length; i++) { + string[i] ^= encoder; + } +} + +/* + * Adds the copyright string to the root node. + * Classic Mac OS requires it to boot. + */ +static void add_copyright() +{ + const int size_of_string = 60; + unsigned char message[size_of_string + 1]; + snprintf(message, size_of_string + 1, "%s", "Bnqxshfiu!0892,3110!@qqmd!Bnl" + "qtuds-!Hob/!@mm!Shfiur!Sdrdswde"); + string_encoder(message, size_of_string); + push_str(message); + fword("encode-string"); + push_str("copyright"); + fword("property"); +} + void arch_of_init(void) { @@ -831,6 +857,8 @@ case ARCH_MAC99: case ARCH_MAC99_U3: + /* Kept here because PREP isn't made to run Macintosh software */ + add_copyright(); case ARCH_PREP: default: Index: arch/ppc/qemu/qemu.fs =================================================================== --- arch/ppc/qemu/qemu.fs (revision 1378) +++ arch/ppc/qemu/qemu.fs (working copy) @@ -99,19 +99,6 @@ \ ------------------------------------------------------------------------- : adler32 ( adler buf len -- checksum ) - \ Since Mac OS 9 is the only system using this word, we take this - \ opportunity to inject a copyright message that is necessary for the - \ system to boot. - " Copyright 1983-2001 Apple Computer, Inc. THIS MESSAGE FOR COMPATIBILITY ONLY" - encode-string " copyright" - " /" find-package if - " set-property" $find if - execute - else - 3drop drop - then - then - ( adler buf len ) " (adler32)" $find if

1 0

[PATCH] ppc: [HACK] add some RTAS properties to make Mac OS 8.5 try to boot
by Alyssa Milburn 04 Feb '16

04 Feb '16

Just for entertainment value; with this applied, 8.5 will successfully get all the way to beyond quiesce(). I didn't think about whether this actually makes any sense at all (presumably not), beware. --- arch/ppc/qemu/init.c | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/arch/ppc/qemu/init.c b/arch/ppc/qemu/init.c index 2b5b8e1..667eb6a 100644 --- a/arch/ppc/qemu/init.c +++ b/arch/ppc/qemu/init.c @@ -912,6 +912,20 @@ arch_of_init(void) size *= 2; set_property(ph, "rtas-size", (char*)&size, sizeof(size)); set_int_property(ph, "rtas-version", is_apple() ? 0x41 : 1); + set_int_property(ph, "restart-rtas", 0xabcdef01); + set_int_property(ph, "nvram-fetch", 0xabcdef02); + set_int_property(ph, "nvram-store", 0xabcdef03); + set_int_property(ph, "get-time-of-day", 0xabcdef04); + set_int_property(ph, "set-time-of-day", 0xabcdef05); + set_int_property(ph, "set-time-for-power-on", 0xabcdef06); + set_int_property(ph, "event-scan", 0xabcdef07); + set_int_property(ph, "check-exception", 0xabcdef08); + set_int_property(ph, "read-pci-config", 0xabcdef09); + set_int_property(ph, "write-pci-config", 0xabcdef0a); + set_int_property(ph, "set-indicator", 0xabcdef0c); + set_int_property(ph, "power-off", 0xabcdef12); + set_int_property(ph, "system-reboot", 0xabcdef15); + set_int_property(ph, "get-time-for-power-on", 0xabcdef1e); } break; } -- 2.5.3

2 4

Re: [OpenBIOS] [PATCH] ppc: fix ESCC reg properties
by Alyssa Milburn 03 Feb '16

03 Feb '16

Hey, Great that this fixes stuff! The attached (qemu) patch might also be of interest(?) although I didn't submit it because the ports are all unimplemented and it looks like they aren't used by anything. And I'm not sure what to do about the inconsistent ports thing (my PowerMac1,1 provides different values in OF for the low regs) so I sort of gave up on this angle which I guess was a mistake. :) - Alyssa

2 1

[commit] r1378 - trunk/openbios-devel/config/scripts
by repository service 03 Feb '16

03 Feb '16

Author: mcayland Date: Wed Feb 3 21:46:31 2016 New Revision: 1378 URL: http://tracker.coreboot.org/trac/openbios/changeset/1378 Log: ppc: force target cpu With gcc-5.2 default target is power7. - For 32-bit PPC builds set the target cpu to 604 in order to support PReP as well as Macs in the future. - For 64-bit PPC builds set the target cpu to 970 as used in 64-bit Macs but disable altivec. Based on an original patch by Laurent Vivier <laurent(a)vivier.eu>. Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk> Modified: trunk/openbios-devel/config/scripts/switch-arch Modified: trunk/openbios-devel/config/scripts/switch-arch ============================================================================== --- trunk/openbios-devel/config/scripts/switch-arch Fri Jan 8 13:09:48 2016 (r1377) +++ trunk/openbios-devel/config/scripts/switch-arch Wed Feb 3 21:46:31 2016 (r1378) @@ -265,7 +265,8 @@ ppc) select_prefix powerpc powerpc64 if [ "$unix" = "no" ]; then - CFLAGS="-m32 -msoft-float -fno-builtin-bcopy -fno-builtin-log2" + # 604 cpu includes support for PReP as well as Mac + CFLAGS="-m32 -mcpu=604 -msoft-float -fno-builtin-bcopy -fno-builtin-log2" AS_FLAGS="-m32" else CFLAGS="-fno-builtin" @@ -275,7 +276,9 @@ ppc64) select_prefix powerpc64 - CFLAGS="-Wa,-a64 -m64 -msoft-float -fno-builtin" + + # 970 cpu is used in all 64-bit Macs but disable altivec + CFLAGS="-mcpu=970 -mno-altivec -Wa,-a64 -m64 -msoft-float -fno-builtin" AS_FLAGS="-Wa,-a64" ;;

1 0

[PATCH] ppc: force target CPU
by Laurent Vivier 02 Feb '16

02 Feb '16

With gcc-5.2 default target is power7. As openbios targets PowerMacs G3/G4/G5, force gcc target to be G3 (32bit) or G5 (64bit). Signed-off-by: Laurent Vivier <laurent(a)vivier.eu> --- config/scripts/switch-arch | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/config/scripts/switch-arch b/config/scripts/switch-arch index 38d263d..bcf807a 100755 --- a/config/scripts/switch-arch +++ b/config/scripts/switch-arch @@ -265,7 +265,7 @@ for ARCH in $arch_list; do ppc) select_prefix powerpc powerpc64 if [ "$unix" = "no" ]; then - CFLAGS="-m32 -msoft-float -fno-builtin-bcopy -fno-builtin-log2" + CFLAGS="-m32 -mcpu=G3 -msoft-float -fno-builtin-bcopy -fno-builtin-log2" AS_FLAGS="-m32" else CFLAGS="-fno-builtin" @@ -275,7 +275,7 @@ for ARCH in $arch_list; do ppc64) select_prefix powerpc64 - CFLAGS="-Wa,-a64 -m64 -msoft-float -fno-builtin" + CFLAGS="-Wa,-a64 -m64 -mcpu=G5 -msoft-float -fno-builtin" AS_FLAGS="-Wa,-a64" ;; -- 2.5.0

8 35

[PATCH] ppc: force target cpu
by Mark Cave-Ayland 02 Feb '16

02 Feb '16

With gcc-5.2 default target is power7. - For 32-bit PPC builds set the target cpu to 604 in order to support PReP as well as Macs in the future. - For 64-bit PPC builds set the target cpu to 970 as used in 64-bit Macs but disable altivec. Based on an original patch by Laurent Vivier <laurent(a)vivier.eu>. Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk> --- openbios-devel/config/scripts/switch-arch | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/openbios-devel/config/scripts/switch-arch b/openbios-devel/config/scripts/switch-arch index 38d263d..ab3b4ce 100755 --- a/openbios-devel/config/scripts/switch-arch +++ b/openbios-devel/config/scripts/switch-arch @@ -265,7 +265,8 @@ for ARCH in $arch_list; do ppc) select_prefix powerpc powerpc64 if [ "$unix" = "no" ]; then - CFLAGS="-m32 -msoft-float -fno-builtin-bcopy -fno-builtin-log2" + # 604 cpu includes support for PReP as well as Mac + CFLAGS="-m32 -mcpu=604 -msoft-float -fno-builtin-bcopy -fno-builtin-log2" AS_FLAGS="-m32" else CFLAGS="-fno-builtin" @@ -275,7 +276,9 @@ for ARCH in $arch_list; do ppc64) select_prefix powerpc64 - CFLAGS="-Wa,-a64 -m64 -msoft-float -fno-builtin" + + # 970 cpu is used in all 64-bit Macs but disable altivec + CFLAGS="-mcpu=970 -mno-altivec -Wa,-a64 -m64 -msoft-float -fno-builtin" AS_FLAGS="-Wa,-a64" ;; -- 1.7.10.4

1 0