S19 different? - MS2 development/coding for dummies

[luftvagon]

I have a similar compiler question:

I had to symlink the following:

lrwxrwxrwx 1 root root 20 Nov 28 09:34 m68hc11-elf-gcc -> /usr/bin/m68hc11-gcc
lrwxrwxrwx 1 root root 19 Nov 28 09:36 m68hc11-elf-nm -> /usr/bin/m68hc11-nm
lrwxrwxrwx 1 root root 24 Nov 28 09:36 m68hc11-elf-objcopy -> /usr/bin/m68hc11-objcopy
lrwxrwxrwx 1 root root 24 Nov 28 09:35 m68hc11-elf-objdump -> /usr/bin/m68hc11-objdump

To test the compiler, I compiled the microsquirt image without altering source code:

ms2extra# md5sum microsquirt.s19
089e2c212c4addc66076d896a6505b61 microsquirt.s19

ms2extra# md5sum ../microsquirt.s19
648f7f4a5ff8920143c45fe885156493 ../microsquirt.s19

I expected the s19 image to be unchanged, however it differes from the compiled version provided with ms2extra3.3.0. I did this twice, downloaded the firmware, make clean, make microsquirt, and twice I got the same md5sum hash.
The first couple of hundred bytes are the same, but then they start differing:

ms2extra# ls -al microsquirt.s19
-rwxr-xr-x 1 dev dev 351174 Nov 28 09:42 microsquirt.s19
ms2extra# ls -al ../microsquirt.s19
-rwxrwxrwx 1 dev dev 346248 Nov 21 08:13 ../microsquirt.s19

Am I doing something wrong?

[acab]

when compining - you get some errors?

[jsmcortina]

Why did you need the symlinks? Why weren't the build tools extracted to the correct directory?

James

[luftvagon]

These are debian build toos provided by gcc-m68hc1x package.
Here is the compiler log (afrer make clean):

Code: Select all

/tmp/ms2extra_3.3.0_release/ms2extra# make microsquirt
ls: cannot access c:/usr/bin/m68hc11-elf-gcc: No such file or directory
make make_microsquirt
ls: cannot access c:/usr/bin/m68hc11-elf-gcc: No such file or directory
make[1]: Entering directory `/tmp/ms2extra_3.3.0_release/ms2extra'
echo '#define MICROSQUIRT' > us_opt_c.h
echo '.set MICROSQUIRT, 1' > us_opt_asm.inc
grep -v "^#[02345]" ms2extra.ini | sed 's/^#1//' > microsquirt.ini
echo "/* DO NOT EDIT - this file is automatically generated during build */" > ms2_extra_defines.h
grep "^#define" ms2_extra_vars.h >> ms2_extra_defines.h
echo "; DO NOT EDIT - this file is automatically generated during build" > ms2_extra_defines.inc
grep "^#define" ms2_extra_vars.h | awk '{print ".equ",$2,",",$3};' >> ms2_extra_defines.inc
echo "/* DO NOT EDIT - this file is automatically generated during build */" > ms2_extra_main_vars.h
cat ms2_extra_vars.h | sed 's/extern //' | grep -v "^#" >> ms2_extra_main_vars.h
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_main.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_can.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_can_isr.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_ego.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_ign.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_ign_wheel.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_misc.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c premain.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_init.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c sigs.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_ign_in.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_inj.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_idle.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_user.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c serial.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c msii_flash_gcc.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_sermon.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_conferr.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_burner.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c isr_inj.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c isr_sci.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c isr_rtc.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_asm.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c isr_timerovf.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c isr_ign.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c crc32.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c interp.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -Wl,-defsym,vectors_addr=0xff80,-m,m68hc12elfb,-T,m68hc12elfb.x,--no-trampoline -o microsquirt.elf ms2_extra_main.o ms2_extra_can.o ms2_extra_can_isr.o ms2_extra_ego.o ms2_extra_ign.o ms2_extra_ign_wheel.o ms2_extra_misc.o premain.o ms2_extra_init.o sigs.o ms2_extra_ign_in.o ms2_extra_inj.o ms2_extra_idle.o ms2_extra_user.o serial.o msii_flash_gcc.o ms2_sermon.o ms2_conferr.o ms2_extra_burner.o isr_inj.o isr_sci.o isr_rtc.o ms2_extra_asm.o isr_timerovf.o isr_ign.o crc32.o interp.o
ms2_extra_main.o:(.rodata+0x5c00): warning: reference to a banked address [3c:a400] in the normal address space at 5c00
ms2_extra_main.o:(.rodata+0x5c02): warning: reference to a banked address [3c:a400] in the normal address space at 5c02
ms2_extra_main.o:(.rodata+0x5c06): warning: reference to a banked address [3c:ac00] in the normal address space at 5c06
ms2_extra_main.o:(.rodata+0x5c08): warning: reference to a banked address [3c:ac00] in the normal address space at 5c08
ms2_extra_main.o:(.rodata+0x5c0c): warning: reference to a banked address [3c:b400] in the normal address space at 5c0c
ms2_extra_main.o:(.rodata+0x5c0e): warning: reference to a banked address [3c:b400] in the normal address space at 5c0e
ms2_extra_main.o:(.rodata+0x5c12): warning: reference to a banked address [3c:b800] in the normal address space at 5c12
ms2_extra_main.o:(.rodata+0x5c14): warning: reference to a banked address [3c:b800] in the normal address space at 5c14
ms2_extra_ign_in.o:(.rodata+0x5ce1): warning: reference to a banked address [38:85c6] in the normal address space at 5ce1
ms2_extra_ign_in.o:(.rodata+0x5ce3): warning: reference to a banked address [38:85c6] in the normal address space at 5ce3
ms2_extra_ign_in.o:(.rodata+0x5ce5): warning: reference to a banked address [38:86af] in the normal address space at 5ce5
ms2_extra_ign_in.o:(.rodata+0x5ce7): warning: reference to a banked address [38:8a16] in the normal address space at 5ce7
ms2_extra_ign_in.o:(.rodata+0x5ce9): warning: reference to a banked address [38:8c37] in the normal address space at 5ce9
ms2_extra_ign_in.o:(.rodata+0x5ceb): warning: reference to a banked address [38:8df4] in the normal address space at 5ceb
ms2_extra_ign_in.o:(.rodata+0x5ced): warning: reference to a banked address [38:8fba] in the normal address space at 5ced
ms2_extra_ign_in.o:(.rodata+0x5cef): warning: reference to a banked address [38:911e] in the normal address space at 5cef
ms2_extra_ign_in.o:(.rodata+0x5cf1): warning: reference to a banked address [38:919a] in the normal address space at 5cf1
ms2_extra_ign_in.o:(.rodata+0x5cf3): warning: reference to a banked address [38:92aa] in the normal address space at 5cf3
ms2_extra_ign_in.o:(.rodata+0x5cf5): warning: reference to a banked address [38:933f] in the normal address space at 5cf5
ms2_extra_ign_in.o:(.rodata+0x5cf7): warning: reference to a banked address [38:93b4] in the normal address space at 5cf7
ms2_extra_ign_in.o:(.rodata+0x5cf9): warning: reference to a banked address [38:94a3] in the normal address space at 5cf9
ms2_extra_ign_in.o:(.rodata+0x5cfb): warning: reference to a banked address [38:956a] in the normal address space at 5cfb
ms2_extra_ign_in.o:(.rodata+0x5cfd): warning: reference to a banked address [38:9635] in the normal address space at 5cfd
ms2_extra_ign_in.o:(.rodata+0x5cff): warning: reference to a banked address [38:96ec] in the normal address space at 5cff
ms2_extra_ign_in.o:(.rodata+0x5d01): warning: reference to a banked address [38:9784] in the normal address space at 5d01
ms2_extra_ign_in.o:(.rodata+0x5d03): warning: reference to a banked address [38:98fe] in the normal address space at 5d03
ms2_extra_ign_in.o:(.rodata+0x5d05): warning: reference to a banked address [38:9958] in the normal address space at 5d05
ms2_extra_ign_in.o:(.rodata+0x5d07): warning: reference to a banked address [38:9a1a] in the normal address space at 5d07
ms2_extra_ign_in.o:(.rodata+0x5d09): warning: reference to a banked address [38:9a88] in the normal address space at 5d09
ms2_extra_ign_in.o:(.rodata+0x5d0b): warning: reference to a banked address [38:9b2b] in the normal address space at 5d0b
ms2_extra_ign_in.o:(.rodata+0x5d0d): warning: reference to a banked address [38:9d12] in the normal address space at 5d0d
ms2_extra_ign_in.o:(.rodata+0x5d0f): warning: reference to a banked address [38:9e60] in the normal address space at 5d0f
ms2_extra_ign_in.o:(.rodata+0x5d11): warning: reference to a banked address [38:bda9] in the normal address space at 5d11
ms2_extra_ign_in.o:(.rodata+0x5d13): warning: reference to a banked address [38:bda9] in the normal address space at 5d13
ms2_extra_ign_in.o:(.rodata+0x5d15): warning: reference to a banked address [38:9f33] in the normal address space at 5d15
ms2_extra_ign_in.o:(.rodata+0x5d17): warning: reference to a banked address [38:a056] in the normal address space at 5d17
ms2_extra_ign_in.o:(.rodata+0x5d19): warning: reference to a banked address [38:a8cc] in the normal address space at 5d19
ms2_extra_ign_in.o:(.rodata+0x5d1b): warning: reference to a banked address [38:a8cc] in the normal address space at 5d1b
ms2_extra_ign_in.o:(.rodata+0x5d1d): warning: reference to a banked address [38:a147] in the normal address space at 5d1d
ms2_extra_ign_in.o:(.rodata+0x5d1f): warning: reference to a banked address [38:a147] in the normal address space at 5d1f
ms2_extra_ign_in.o:(.rodata+0x5d21): warning: reference to a banked address [38:a147] in the normal address space at 5d21
ms2_extra_ign_in.o:(.rodata+0x5d23): warning: reference to a banked address [38:a147] in the normal address space at 5d23
ms2_extra_ign_in.o:(.rodata+0x5d25): warning: reference to a banked address [38:a147] in the normal address space at 5d25
ms2_extra_ign_in.o:(.rodata+0x5d27): warning: reference to a banked address [38:a147] in the normal address space at 5d27
ms2_extra_ign_in.o:(.rodata+0x5d29): warning: reference to a banked address [38:a147] in the normal address space at 5d29
ms2_extra_ign_in.o:(.rodata+0x5d2b): warning: reference to a banked address [38:a147] in the normal address space at 5d2b
ms2_extra_ign_in.o:(.rodata+0x5d2d): warning: reference to a banked address [38:a14a] in the normal address space at 5d2d
ms2_extra_ign_in.o:(.rodata+0x5d2f): warning: reference to a banked address [38:a3db] in the normal address space at 5d2f
ms2_extra_ign_in.o:(.rodata+0x5d31): warning: reference to a banked address [38:bda9] in the normal address space at 5d31
ms2_extra_ign_in.o:(.rodata+0x5d33): warning: reference to a banked address [38:bda9] in the normal address space at 5d33
ms2_extra_ign_in.o:(.rodata+0x5d35): warning: reference to a banked address [38:bda9] in the normal address space at 5d35
ms2_extra_ign_in.o:(.rodata+0x5d37): warning: reference to a banked address [38:bda9] in the normal address space at 5d37
ms2_extra_ign_in.o:(.rodata+0x5d39): warning: reference to a banked address [38:bda9] in the normal address space at 5d39
ms2_extra_ign_in.o:(.rodata+0x5d3b): warning: reference to a banked address [38:bda9] in the normal address space at 5d3b
ms2_extra_ign_in.o:(.rodata+0x5d3d): warning: reference to a banked address [38:bda9] in the normal address space at 5d3d
ms2_extra_ign_in.o:(.rodata+0x5d3f): warning: reference to a banked address [38:bda9] in the normal address space at 5d3f
ms2_extra_ign_in.o:(.rodata+0x5d41): warning: reference to a banked address [38:bda9] in the normal address space at 5d41
ms2_extra_ign_in.o:(.rodata+0x5d43): warning: reference to a banked address [38:bda9] in the normal address space at 5d43
ms2_extra_ign_in.o:(.rodata+0x5d45): warning: reference to a banked address [38:bda9] in the normal address space at 5d45
ms2_extra_ign_in.o:(.rodata+0x5d47): warning: reference to a banked address [38:bda9] in the normal address space at 5d47
ms2_extra_ign_in.o:(.rodata+0x5d49): warning: reference to a banked address [38:a51e] in the normal address space at 5d49
ms2_extra_ign_in.o:(.rodata+0x5d4b): warning: reference to a banked address [38:a727] in the normal address space at 5d4b
/usr/lib/gcc-lib/m68hc11/3.3.6-m68hc1x-20060122/m68hc12/mshort/crt1.o: In function `fatal':
crt0.s:(.install4+0xc026): warning: reference to a banked address [3d:8000] in the normal address space at c026
/usr/bin/m68hc11-elf-objdump -Sdh microsquirt.elf > microsquirt.dmp
/usr/bin/m68hc11-elf-objcopy --output-target=srec --only-section=.text3 --only-section=.rodata --only-section=.vectors --only-section=.eeprom --only-section=.lookup --change-section-lma .lookup=0x3Ca400 --only-section=.text3c --change-section-lma .text3c=0x3C8000 --only-section=.text3d --change-section-lma .text3d=0x3D8000 --only-section=.text3b --change-section-lma .text3b=0x3B8000 --only-section=.text3a --change-section-lma .text3a=0x3A8000 --only-section=.text39 --change-section-lma .text39=0x398000 --only-section=.text38 --change-section-lma .text38=0x388000 --only-section=.lookup --only-section=.text microsquirt.elf microsquirt.s19
/usr/bin/m68hc11-elf-nm microsquirt.elf | sort > microsquirt.map
make[1]: Leaving directory `/tmp/ms2extra_3.3.0_release/ms2extra'

[luftvagon]

Thanks James.

[acab]

The Debian S12 tools aren't what you need. Install the tools linked here.
i.e. http://www.msextra.com/tools/

James

Hmm, i install all tools to clean Debian 7.2.
And my compiled md5 summ not same like in original ;(

Code: Select all

root@msextra-dev:/home/ms/ms2extra_3.3.0_release/ms2extra# md5sum megasquirt2.s19
3dc903bad45c30a53e2f63b601041f7f  megasquirt2.s19

root@msextra-dev:/home/ms/ms2extra_3.3.0_release/ms2extra# md5sum ../megasquirt2.s19
21b4e23fc782d0134d2698b73d345b33  ../megasquirt2.s19

Code: Select all

root@msextra-dev:/home/ms/ms2extra_3.3.0_release/ms2extra# make clean
rm -f -rf release *~
make clean_all
make[1]: Entering directory `/home/ms/ms2extra_3.3.0_release/ms2extra'
rm -f -f megasquirt2.elf microsquirt.elf microsquirt-module.elf mspnp2.elf
rm -f -f megasquirt2.s19 microsquirt.s19 microsquirt-module.s19 mspnp2.s19
rm -f -f megasquirt2.dmp microsquirt.dmp microsquirt-module.dmp mspnp2.dmp
rm -f -f megasquirt2.map microsquirt.map microsquirt-module.map mspnp2.map
rm -f -f megasquirt2.ini microsquirt.ini microsquirt-module.ini mspnp2.ini
rm -f -f ms2_extra_main.o ms2_extra_can.o ms2_extra_can_isr.o ms2_extra_ego.o ms2_extra_ign.o ms2_extra_ign_wheel.o ms2_extra_misc.o premain.o ms2_extra_init.o sigs.o ms2_extra_ign_in.o ms2_extra_inj.o ms2_extra_idle.o ms2_extra_user.o serial.o msii_flash_gcc.o ms2_sermon.o ms2_conferr.o ms2_extra_burner.o isr_inj.o isr_sci.o isr_rtc.o ms2_extra_asm.o isr_timerovf.o isr_ign.o crc32.o interp.o
rm -f -f us_opt_c.h
rm -f -f us_opt_asm.inc ms2_extra_defines.h ms2_extra_defines.inc ms2_extra_main_vars.h
make[1]: Leaving directory `/home/ms/ms2extra_3.3.0_release/ms2extra'
root@msextra-dev:/home/ms/ms2extra_3.3.0_release/ms2extra# make
make make_megasquirt2
make[1]: Entering directory `/home/ms/ms2extra_3.3.0_release/ms2extra'
echo '// setup for Megasquirt' > us_opt_c.h
echo '; setup for Megasquirt' > us_opt_asm.inc
grep -v "^#[12345]" ms2extra.ini | sed 's/^#0//' > megasquirt2.ini
echo "/* DO NOT EDIT - this file is automatically generated during build */" > ms2_extra_defines.h
grep "^#define" ms2_extra_vars.h >> ms2_extra_defines.h
echo "; DO NOT EDIT - this file is automatically generated during build" > ms2_extra_defines.inc
grep "^#define" ms2_extra_vars.h | awk '{print ".equ",$2,",",$3};' >> ms2_extra_defines.inc
echo "/* DO NOT EDIT - this file is automatically generated during build */" > ms2_extra_main_vars.h
cat ms2_extra_vars.h | sed 's/extern //' | grep -v "^#" >> ms2_extra_main_vars.h
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_main.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_can.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_can_isr.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_ego.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_ign.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_ign_wheel.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_misc.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c premain.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_init.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c sigs.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_ign_in.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_inj.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_idle.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_user.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c serial.c
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c msii_flash_gcc.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_sermon.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_conferr.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_burner.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c isr_inj.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c isr_sci.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c isr_rtc.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c ms2_extra_asm.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c isr_timerovf.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c isr_ign.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c crc32.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -c interp.s
/usr/bin/m68hc11-elf-gcc -g -Wall -Werror -Wsign-compare -Wshadow -O -fomit-frame-pointer -m68hcs12 -mshort -msoft-reg-count=5 -mauto-incdec -fsigned-char -Wl,-defsym,vectors_addr=0xff80,-m,m68hc12elfb,-T,m68hc12elfb.x,--no-trampoline -o megasquirt2.elf ms2_extra_main.o ms2_extra_can.o ms2_extra_can_isr.o ms2_extra_ego.o ms2_extra_ign.o ms2_extra_ign_wheel.o ms2_extra_misc.o premain.o ms2_extra_init.o sigs.o ms2_extra_ign_in.o ms2_extra_inj.o ms2_extra_idle.o ms2_extra_user.o serial.o msii_flash_gcc.o ms2_sermon.o ms2_conferr.o ms2_extra_burner.o isr_inj.o isr_sci.o isr_rtc.o ms2_extra_asm.o isr_timerovf.o isr_ign.o crc32.o interp.o
ms2_extra_main.o:(.rodata+0x5c00): warning: reference to a banked address [3c:a400] in the normal address space at 5c00
ms2_extra_main.o:(.rodata+0x5c02): warning: reference to a banked address [3c:a400] in the normal address space at 5c02
ms2_extra_main.o:(.rodata+0x5c06): warning: reference to a banked address [3c:ac00] in the normal address space at 5c06
ms2_extra_main.o:(.rodata+0x5c08): warning: reference to a banked address [3c:ac00] in the normal address space at 5c08
ms2_extra_main.o:(.rodata+0x5c0c): warning: reference to a banked address [3c:b400] in the normal address space at 5c0c
ms2_extra_main.o:(.rodata+0x5c0e): warning: reference to a banked address [3c:b400] in the normal address space at 5c0e
ms2_extra_main.o:(.rodata+0x5c12): warning: reference to a banked address [3c:b800] in the normal address space at 5c12
ms2_extra_main.o:(.rodata+0x5c14): warning: reference to a banked address [3c:b800] in the normal address space at 5c14
ms2_extra_ign_in.o:(.rodata+0x5ce1): warning: reference to a banked address [38:858b] in the normal address space at 5ce1
ms2_extra_ign_in.o:(.rodata+0x5ce3): warning: reference to a banked address [38:858b] in the normal address space at 5ce3
ms2_extra_ign_in.o:(.rodata+0x5ce5): warning: reference to a banked address [38:8669] in the normal address space at 5ce5
ms2_extra_ign_in.o:(.rodata+0x5ce7): warning: reference to a banked address [38:899b] in the normal address space at 5ce7
ms2_extra_ign_in.o:(.rodata+0x5ce9): warning: reference to a banked address [38:8bab] in the normal address space at 5ce9
ms2_extra_ign_in.o:(.rodata+0x5ceb): warning: reference to a banked address [38:8d5b] in the normal address space at 5ceb
ms2_extra_ign_in.o:(.rodata+0x5ced): warning: reference to a banked address [38:8f1e] in the normal address space at 5ced
ms2_extra_ign_in.o:(.rodata+0x5cef): warning: reference to a banked address [38:9079] in the normal address space at 5cef
ms2_extra_ign_in.o:(.rodata+0x5cf1): warning: reference to a banked address [38:90f3] in the normal address space at 5cf1
ms2_extra_ign_in.o:(.rodata+0x5cf3): warning: reference to a banked address [38:9201] in the normal address space at 5cf3
ms2_extra_ign_in.o:(.rodata+0x5cf5): warning: reference to a banked address [38:9292] in the normal address space at 5cf5
ms2_extra_ign_in.o:(.rodata+0x5cf7): warning: reference to a banked address [38:9301] in the normal address space at 5cf7
ms2_extra_ign_in.o:(.rodata+0x5cf9): warning: reference to a banked address [38:93ee] in the normal address space at 5cf9
ms2_extra_ign_in.o:(.rodata+0x5cfb): warning: reference to a banked address [38:94ab] in the normal address space at 5cfb
ms2_extra_ign_in.o:(.rodata+0x5cfd): warning: reference to a banked address [38:9574] in the normal address space at 5cfd
ms2_extra_ign_in.o:(.rodata+0x5cff): warning: reference to a banked address [38:9623] in the normal address space at 5cff
ms2_extra_ign_in.o:(.rodata+0x5d01): warning: reference to a banked address [38:96b9] in the normal address space at 5d01
ms2_extra_ign_in.o:(.rodata+0x5d03): warning: reference to a banked address [38:9831] in the normal address space at 5d03
ms2_extra_ign_in.o:(.rodata+0x5d05): warning: reference to a banked address [38:9889] in the normal address space at 5d05
ms2_extra_ign_in.o:(.rodata+0x5d07): warning: reference to a banked address [38:9949] in the normal address space at 5d07
ms2_extra_ign_in.o:(.rodata+0x5d09): warning: reference to a banked address [38:99b5] in the normal address space at 5d09
ms2_extra_ign_in.o:(.rodata+0x5d0b): warning: reference to a banked address [38:9a56] in the normal address space at 5d0b
ms2_extra_ign_in.o:(.rodata+0x5d0d): warning: reference to a banked address [38:9c26] in the normal address space at 5d0d
ms2_extra_ign_in.o:(.rodata+0x5d0f): warning: reference to a banked address [38:9d70] in the normal address space at 5d0f
ms2_extra_ign_in.o:(.rodata+0x5d11): warning: reference to a banked address [38:bc4a] in the normal address space at 5d11
ms2_extra_ign_in.o:(.rodata+0x5d13): warning: reference to a banked address [38:bc4a] in the normal address space at 5d13
ms2_extra_ign_in.o:(.rodata+0x5d15): warning: reference to a banked address [38:9e41] in the normal address space at 5d15
ms2_extra_ign_in.o:(.rodata+0x5d17): warning: reference to a banked address [38:9f62] in the normal address space at 5d17
ms2_extra_ign_in.o:(.rodata+0x5d19): warning: reference to a banked address [38:a7b7] in the normal address space at 5d19
ms2_extra_ign_in.o:(.rodata+0x5d1b): warning: reference to a banked address [38:a7b7] in the normal address space at 5d1b
ms2_extra_ign_in.o:(.rodata+0x5d1d): warning: reference to a banked address [38:a049] in the normal address space at 5d1d
ms2_extra_ign_in.o:(.rodata+0x5d1f): warning: reference to a banked address [38:a049] in the normal address space at 5d1f
ms2_extra_ign_in.o:(.rodata+0x5d21): warning: reference to a banked address [38:a049] in the normal address space at 5d21
ms2_extra_ign_in.o:(.rodata+0x5d23): warning: reference to a banked address [38:a049] in the normal address space at 5d23
ms2_extra_ign_in.o:(.rodata+0x5d25): warning: reference to a banked address [38:a049] in the normal address space at 5d25
ms2_extra_ign_in.o:(.rodata+0x5d27): warning: reference to a banked address [38:a049] in the normal address space at 5d27
ms2_extra_ign_in.o:(.rodata+0x5d29): warning: reference to a banked address [38:a049] in the normal address space at 5d29
ms2_extra_ign_in.o:(.rodata+0x5d2b): warning: reference to a banked address [38:a049] in the normal address space at 5d2b
ms2_extra_ign_in.o:(.rodata+0x5d2d): warning: reference to a banked address [38:a04c] in the normal address space at 5d2d
ms2_extra_ign_in.o:(.rodata+0x5d2f): warning: reference to a banked address [38:a2d8] in the normal address space at 5d2f
ms2_extra_ign_in.o:(.rodata+0x5d31): warning: reference to a banked address [38:bc4a] in the normal address space at 5d31
ms2_extra_ign_in.o:(.rodata+0x5d33): warning: reference to a banked address [38:bc4a] in the normal address space at 5d33
ms2_extra_ign_in.o:(.rodata+0x5d35): warning: reference to a banked address [38:bc4a] in the normal address space at 5d35
ms2_extra_ign_in.o:(.rodata+0x5d37): warning: reference to a banked address [38:bc4a] in the normal address space at 5d37
ms2_extra_ign_in.o:(.rodata+0x5d39): warning: reference to a banked address [38:bc4a] in the normal address space at 5d39
ms2_extra_ign_in.o:(.rodata+0x5d3b): warning: reference to a banked address [38:bc4a] in the normal address space at 5d3b
ms2_extra_ign_in.o:(.rodata+0x5d3d): warning: reference to a banked address [38:bc4a] in the normal address space at 5d3d
ms2_extra_ign_in.o:(.rodata+0x5d3f): warning: reference to a banked address [38:bc4a] in the normal address space at 5d3f
ms2_extra_ign_in.o:(.rodata+0x5d41): warning: reference to a banked address [38:bc4a] in the normal address space at 5d41
ms2_extra_ign_in.o:(.rodata+0x5d43): warning: reference to a banked address [38:bc4a] in the normal address space at 5d43
ms2_extra_ign_in.o:(.rodata+0x5d45): warning: reference to a banked address [38:bc4a] in the normal address space at 5d45
ms2_extra_ign_in.o:(.rodata+0x5d47): warning: reference to a banked address [38:bc4a] in the normal address space at 5d47
ms2_extra_ign_in.o:(.rodata+0x5d49): warning: reference to a banked address [38:a41b] in the normal address space at 5d49
ms2_extra_ign_in.o:(.rodata+0x5d4b): warning: reference to a banked address [38:a61b] in the normal address space at 5d4b
/usr/lib/gcc-lib/m68hc11-elf/3.3.6-9s12x-20120502/m68hc12/mshort/crt1.o:(.install4+0x1): warning: Reference to the far symbol `main' using a wrong relocation may result in incorrect execution
/usr/lib/gcc-lib/m68hc11-elf/3.3.6-9s12x-20120502/m68hc12/mshort/crt1.o: In function `fatal':
crt0.s:(.install4+0xc026): warning: reference to a banked address [3d:8000] in the normal address space at c026
/usr/bin/m68hc11-elf-objdump -Sdh megasquirt2.elf > megasquirt2.dmp
/usr/bin/m68hc11-elf-objcopy --output-target=srec --only-section=.text3 --only-section=.rodata --only-section=.vectors --only-section=.eeprom --only-section=.lookup --change-section-lma .lookup=0x3Ca400 --only-section=.text3c --change-section-lma .text3c=0x3C8000 --only-section=.text3d --change-section-lma .text3d=0x3D8000 --only-section=.text3b --change-section-lma .text3b=0x3B8000 --only-section=.text3a --change-section-lma .text3a=0x3A8000 --only-section=.text39 --change-section-lma .text39=0x398000 --only-section=.text38 --change-section-lma .text38=0x388000 --only-section=.lookup --only-section=.text megasquirt2.elf megasquirt2.s19
/usr/bin/m68hc11-elf-nm megasquirt2.elf | sort > megasquirt2.map
make[1]: Leaving directory `/home/ms/ms2extra_3.3.0_release/ms2extra'

[jsmcortina]

Correct gcc versions?

Code: Select all

$ /usr/bin/m68hc11-elf-gcc --version
m68hc11-elf-gcc (GCC) 3.3.6-9s12x-20120502

Code: Select all

$ /usr/bin/m68hc11-elf-as --version
GNU assembler (GNU Binutils) 2.23

(I have a slightly older version installed on my Linux box.)

James

[acab]

May be same...

Code: Select all

root@msextra-dev:/home/ms/ms2extra_3.3.0_release/ms2extra# /usr/bin/m68hc11-elf-gcc --version
m68hc11-elf-gcc (GCC) 3.3.6-9s12x-20120502
Copyright (C) 2003 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Code: Select all

root@msextra-dev:/home/ms/ms2extra_3.3.0_release/ms2extra# /usr/bin/m68hc11-elf-as --version
GNU assembler (GNU Binutils) 2.23
Copyright 2012 Free Software Foundation, Inc.
This program is free software; you may redistribute it under the terms of
the GNU General Public License version 3 or later.
This program has absolutely no warranty.
This assembler was configured for a target of `m68hc11-elf'.

[luftvagon]

I just compiled the tools based on your instructions:

Code: Select all

# /usr/bin/m68hc11-elf-gcc --version
m68hc11-elf-gcc (GCC) 3.3.6-9s12x-20120502
Copyright (C) 2003 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Code: Select all

 
/usr/bin/m68hc11-elf-as --version
GNU assembler (GNU Binutils) 2.23.1
Copyright 2012 Free Software Foundation, Inc.
This program is free software; you may redistribute it under the terms of
the GNU General Public License version 3 or later.
This program has absolutely no warranty.
This assembler was configured for a target of `m68hc11-elf'.

Code: Select all

 
ms2extra# md5sum microsquirt.s19
fd0965d4611a21e35072f740160b74e8  microsquirt.s19
ms2extra# ls -al microsquirt.s19
-rwxr-xr-x 1 dev dev 346264 Nov 28 17:24 microsquirt.s19

Code: Select all

 
ms2extra# md5sum ../microsquirt.s19
648f7f4a5ff8920143c45fe885156493  ../microsquirt.s19
ms2extra# ls -al ../microsquirt.s19
-rwxrwxrwx 1 root root 346248 Nov 21 08:13 ../microsquirt.s19

It's a lot closer byte wise.

[jsmcortina]

Did you try the provided binaries?

James

[luftvagon]

They would not execute for me.
Linux dev 2.6.35.4-rscloud #8 SMP Mon Sep 20 15:54:33 UTC 2010 x86_64 GNU/Linux

[luftvagon]

James,

I emailed you a elf dmp file from the non modified source to compare versus your 3.3.0 dmp file, to see if there is a difference in the instruction set.
Please let me know if and what is the difference with your 3.3.0 microsquirt dmp file.

Thank you.

[jsmcortina]

Jean dropped me a mail on this too and highlighted the one line difference between the output. I made a change in the linker a year back to produce more optimal code which impacts that one line. It has no functional impact, but I can see the confusion.

Bizarrely I'd got the OLD version of the linker installed as my working toolchain. Future patch releases of MS2/Extra will be built using the same tools as I released.

James

[jsmcortina]

With the 2.23 version linker, these are the file sizes and md5sums I get.

megasquirt2.s19 347756 3dc903bad45c30a53e2f63b601041f7f
microsquirt.s19 346260 6c3b70ee96a1d474a603d9f30adf15b5
microsquirt-module.s19 346274 6332f4c7c742185aee0a89cb56edc76f
mspnp2.s19 346250 7d4037c41bc196d91882ade4fafbd373

That still looks to be slightly different to the numbers luftwagon posted above.

I've split these posts off from the main topic.

Hope that helps

James

[acab]

If i understand - its okay?

But, you tested compiled code?
If i remember, i get not working advance few month ago.
http://www.msextra.com/forums/viewtopic ... 91&t=50238

[jsmcortina]

If i understand - its okay?

But, you tested compiled code?

The difference is one single line of assembler and it has no operational impact.
Jean's emails have reminded me that the MS2 source needs one line changing to resolve the difference. It wasn't the the build tools that changed it.

James

[luftvagon]

James,

Is the dmp file I emailed you ok? I would really like to implement Fuel VE Switching based on CTS.
If my dmp file is ok, I can go ahead and do the code myself and test it... if not, no point of doing it.

I will buy you a pint of beer or 6 pack if you could compile the feature requested

[luftvagon]

Code: Select all

# md5sum megasquirt2.s19
3a6c78efed0893107d67a1795a99e4bc  megasquirt2.s19

Code: Select all

# md5sum ../megasquirt2.s19
21b4e23fc782d0134d2698b73d345b33  ../megasquirt2.s19

[luftvagon]

acab,

can you shoot me your megasquirt2.dmp file. i want to compare it to mine.

[acab]

James, did you test compiled code with new tools?

i get very bad problem few month ago...

http://www.msextra.com/forums/viewtopic ... 91&t=50238