Rockbox.org home
release
dev builds
extras
themes manual
wiki
device status forums
mailing lists
IRC bugs
patches
dev guide



Rockbox mail archive

Subject: Re: Improving simulator

Re: Improving simulator

From: Christian Gmeiner <christian.gmeiner_at_gmail.com>
Date: Thu, 10 May 2007 10:57:05 +0000

2007/5/10, Dave Chapman <dave_at_dchapman.com>:
> Christian Gmeiner wrote:
> > The big plus point is that we dont need to care about the simulator
> > with #ifdef's anymore as the simulator is a normal target.
>
> I don't see how that will happen - won't it just mean replacing "#ifdef
> SIMULATOR" with "#ifdef SDL_TARGET" ?

No you are wrong.. we will use the target tree feature:
firmware/target/sdl/*

Here we implement, for instance, a lcd driver based on sdl, audio
driver based on sdl,...

>
> Looking at the current code, there seem to be a lot of #ifdef SIMULATOR
> checks which are not needed - e.g. lines of the form:
>
> #if defined(CPU_COLDFIRE) && !defined(SIMULATOR)
>
> can just be changed to:
>
> #ifdef CPU_COLDFIRE
>
> - the CONFIG_CPU macro in the config files aren't defined for the simulator.
>
> Another problem is simply hardware features of targets which are not
> implemented in the sim - mainly the MAS on Archos devices, radio,
> recording, plus various other things like HAVE_PWM_BACKLIGHT_FADING.
> How would your proposal deal with these?

I dont want to write an emulator, I want to integrate the simulator
better into the current source! These are two _very_ different things.
So... whren we build a simulator for target X5, we will get a sdl
based executable (in combination with the bootloader), which has the
same size of lcd as x5, has the same buttons as x5, has recording if
supported - like our current uisimulator.

We can run now rockbox on our pc with sdl.. we will use sdl to play
sound and so on. But you have now the same core (threading,..) as we
would compile rockbox for a target, which can help debugging.

>
> I'm not convinced that a radical change in architecture is needed - the
> time might be better spent just implementing more features in the
> current sim, and ensuring #ifdef SIMULATOR is only being used when it
> absolutely has to be.

Believe me that nobody wants to check, if this ifdef is okay or not...
its much easier when there are none or a very small amout of these
#ifdef SIMULATOR.

Christian
Received on 2007-05-10


Page was last modified "Jan 10 2012" The Rockbox Crew
aaa