viewport_live Update for A650IS Port

na560

viewport_live Update for A650IS Port

« on: 19 / July / 2014, 15:23:14 »

Advertisements

After rebuilding a used A650IS I noticed slow motion detection and then found that vid_get _viewport_live_fb was not implemented. Dug through RAM dumps and eventually found the right addresses. A working vid_get_viewport_live_fb follows for the A650 sub 100d.

Code: [Select]

void *vid_get_viewport_live_fb()
{
//    return (void*)0;
    void **fb=(void **)0x21D8;
    unsigned char buff = *((unsigned char*)0x208C);
    if (buff == 0) {
        buff = 2;
    }
    else {
        buff--;
    }
   return fb[buff];
}

Logged

reyalp

11671

Re: viewport_live Update for A650IS Port

« Reply #1 on: 19 / July / 2014, 15:25:03 »

Thanks for that. Have you tested to verify that MD performance is improved? See http://chdk.wikia.com/wiki/Testing#MD_tune.bas

Logged

Don't forget what the H stands for.

na560

Re: viewport_live Update for A650IS Port

« Reply #2 on: 20 / July / 2014, 07:21:15 »

While rebuilding the A650 I clumsily tore the AF LED ribbon so can't use MD_Tune. However, I did test it with DataGhost's motion detector speed test. So relatively speaking with the test running on my laptop driving a separate display, 1.3.0-3496 gave detect times ranging from 130 to 320ms with a median of 210ms. With the new vid_get_viewport_live_fb, detect times varied only from 110 to 150ms with a130ms median.
Glad you asked.

Logged

reyalp

11671

Re: viewport_live Update for A650IS Port

« Reply #3 on: 20 / July / 2014, 14:53:38 »

Quote from: na560 on 20 / July / 2014, 07:21:15

While rebuilding the A650 I clumsily tore the AF LED ribbon so can't use MD_Tune. However, I did test it with DataGhost's motion detector speed test. So relatively speaking with the test running on my laptop driving a separate display, 1.3.0-3496 gave detect times ranging from 130 to 320ms with a median of 210ms. With the new vid_get_viewport_live_fb, detect times varied only from 110 to 150ms with a130ms median.
Glad you asked.

Thanks. I've checked this in on both the 1.2 and 1.3 branches.

Logged

Don't forget what the H stands for.