Magic Lantern Forum

Developing Magic Lantern => General Development => Topic started by: solarisfan on August 23, 2012, 03:20:42 PM

Title: Build target 60D fails with undefined reference - beep.c requires SetNextASIFADC
Post by: solarisfan on August 23, 2012, 03:20:42 PM
Hello,

looks like

magic-lantern/platform/60D.111/stubs.S

misses something like

NSTUB(0xff052088, SetNextASIFADCBuffer)

(picked up from 550D branch)

will break

magic-lantern/src/beep.c

asif_rec_continue_cbr()
{
....
SetNextASIFADCBuffer(buf, WAV_BUF_SIZE);
....
}

does anyone know if this is a pure miss or beep needs target attention
(#ifdef for 60D)

cheers,
Frank

OUTPUT
----------

[ LD       ]   magiclantern
beep.o: In function `asif_rec_continue_cbr':
beep.c:(.text+0x3a4): undefined reference to `SetNextASIFADCBuffer'
beep.o: In function `asif_continue_cbr':
beep.c:(.text+0x4c4): undefined reference to `SetNextASIFDACBuffer'
collect2: ld returned 1 exit status
Title: Re: Build target 60D fails with undefined reference - beep.c requires SetNextASIFADC
Post by: nanomad on August 23, 2012, 06:14:33 PM
Yes last few commits are broken, you can either:
- Help a1ex find the functions on your camera model (He will tell you what to look for)
- Comment out the two offending calls
- Check out an older revision (I suggest the one tagged 2.3)
Title: Re: Build target 60D fails with undefined reference - beep.c requires SetNextASIFADC
Post by: 1% on August 23, 2012, 06:34:03 PM
It should break any camera where the wav recorder is used but the stub is missing. Need to look in fw to get the right addresses or just comment out the calls in beep.c
Title: Re: Build target 60D fails with undefined reference - beep.c requires SetNextASIFADC
Post by: nanomad on August 24, 2012, 10:51:30 AM
On the 1100D there is:
0xFF055C60 -> SetNextUINT8ASIFADCBuffer
0xFF055C8C -> SetNextINT16ASIFADCBuffer

0xFF055D18 -> SetNextUINT8ASIFDACBuffer
0xFF055D44 -> SetNextINT16ASIFDACBuffer

Which one are correct? 8 or 16 bit versions?
Title: Re: Build target 60D fails with undefined reference - beep.c requires SetNextASIFADC
Post by: a1ex on August 24, 2012, 01:20:36 PM
Solved, all stubs are on the repo now.