From 8983bd77509e066c033f82fdf60a41f98db5001f Mon Sep 17 00:00:00 2001 From: Sam Lantinga Date: Sat, 12 Feb 2005 18:01:31 +0000 Subject: [PATCH] Date: Wed, 26 Jan 2005 13:37:09 GMT From: Peter Naulls Subject: RISC OS SDL Patches Sam, I've attached a diff of the latest changes to libSDL for RISC OS support. These changes are by Alan Buckley and myself. The most significant of these are: Optimised assembler blit rountines - I've attached the file src/video/riscos/SDL_riscosASM.s which is needed for this. Move to using /dev/dsp instead of its own audio implementation. This means that src/audio/riscos/SDL_drenderer.c should be removed Typo fixes. Mainly correct spelling of "RISC OS", but some from elsewhere too. --- README.RISCOS | 39 +- configure.in | 12 +- include/SDL_syswm.h | 6 +- src/audio/SDL_audiodev.c | 2 +- src/audio/riscos/.cvsignore | 6 - src/audio/riscos/Makefile.am | 9 - src/audio/riscos/SDL_drenderer.c | 382 ------------------- src/audio/riscos/SDL_drenderer.h | 36 -- src/joystick/riscos/SDL_sysjoystick.c | 4 +- src/thread/linux/clone.S | 2 +- src/timer/riscos/SDL_systimer.c | 6 +- src/video/riscos/Makefile.am | 1 + src/video/riscos/SDL_riscosASM.s | 121 ++++++ src/video/riscos/SDL_riscosFullScreenVideo.c | 235 ++++++++++-- src/video/riscos/SDL_riscosevents.c | 16 +- src/video/riscos/SDL_riscosmouse.c | 2 +- src/video/riscos/SDL_riscossprite.c | 2 +- src/video/riscos/SDL_riscostask.c | 12 +- src/video/riscos/SDL_riscostask.h | 2 +- src/video/riscos/SDL_riscosvideo.c | 13 +- src/video/riscos/SDL_wimppoll.c | 8 +- src/video/riscos/SDL_wimpvideo.c | 6 +- 22 files changed, 397 insertions(+), 525 deletions(-) delete mode 100644 src/audio/riscos/.cvsignore delete mode 100644 src/audio/riscos/Makefile.am delete mode 100644 src/audio/riscos/SDL_drenderer.c delete mode 100644 src/audio/riscos/SDL_drenderer.h create mode 100644 src/video/riscos/SDL_riscosASM.s diff --git a/README.RISCOS b/README.RISCOS index ee7f71670..7c251c775 100644 --- a/README.RISCOS +++ b/README.RISCOS @@ -1,37 +1,39 @@ Readme for RISC OS port of SDL ============================== -This document last updated on 25th June 2004 +This document last updated on 26th January 2005 -This is a RISC OS port of the Simple Direct Media Layer (SDL) by Alan Buckley. +This is a RISC OS port of the Simple Direct Media Layer (SDL) by Alan Buckley with contributions from Peter Naulls. Details of the SDL can be found at http://www.libsdl.org. -The source code including the RISC OS version can be obtained from http://www.libsdl.org. +The source code including the RISC OS version can be obtained from +http://www.libsdl.org. RISC OS makefiles, pre built libraries and many games and applications compiled for RISC OS using this library can be downloaded from The Unix Porting Project at http://www.chocky.org/unix/index.html. This is released under the LGPL see the file COPYING for details. -RISCOS port of SDL runtime information +RISC OS port of SDL runtime information ====================================== Runtime requirements -------------------- -This library currently needs a minimum of RISCOS 3.6. The source code for the library (and a lot of the programs built with it) also need long file names. +This library currently needs a minimum of RISC OS 3.6. The source code for the library (and a lot of the programs built with it) also need long file names. -To use the audio you also need 16 bit sound and to have installed the Digital Render module by Andreas Dehmel version 0.51 available from his web site: - http://home.t-online.de/~zarquon +To use the audio you also need 16 bit sound and to have installed the DigitalRender module by Andreas Dehmel version 0.51 available from his +web site: http://home.t-online.de/~zarquon +This is loaded when needed by UnixLib. -Note: As most programs ported from other OS's use high resolution graphics and a memory back buffer a machine with a StrongARM processor and 1 or 2MB of VRAM (or a better machine) is recomended. +Note: As most programs ported from other OSes use high resolution graphics and a memory back buffer a machine with a StrongARM processor and 1 or 2MB of VRAM (or a better machine) is recomended. -RISCOS runtime parameters +RISC OS runtime parameters ------------------------- -Several environmental variables have been defined to make porting programs easier (i.e. By setting these variable you do not need to have source code differences between OS's). +Several environmental variables have been defined to make porting programs easier (i.e. By setting these variable you do not need to have source code differences between OSes). They are all defined on an application basis. @@ -41,9 +43,9 @@ The used below is found as follows: The variables are: -SDL$$TaskName - The name of the task for RISCOS. If omitted then is used for the task name, +SDL$$TaskName - The name of the task for RISC OS. If omitted then is used for the task name, -SDL$$BackBuffer - set to 1 to use a system memory back buffer for the screen in full screen mode. Some programs on other systems assume their is always a back buffer even though the SDL specification specifies this is not the case. The current RISCOS implementation uses direct writes to the screen if a hardware fullscreen is requested. +SDL$$BackBuffer - set to 1 to use a system memory back buffer for the screen in full screen mode. Some programs on other systems assume their is always a back buffer even though the SDL specification specifies this is not the case. The current RISC OS implementation uses direct writes to the screen if a hardware fullscreen is requested. SDL$$CloseAction - set the action for the close icon. Again as programs don't match the specification you can set this to 0 to remove the close icon from the main window for applications where this does not affect the program. @@ -51,7 +53,7 @@ SDL$$AudioBuffer - set the minimum size of the audio buffer in samples -RISCOS SDL port API notes +RISC OS SDL port API notes ========================= Current level of implementation @@ -59,11 +61,11 @@ Current level of implementation The following list is an overview of how much of the SDL is implemented. The areas match the main areas of the SDL. -video - Mostly done. Doesn't cover gamma, YUV-overlay or open gl. +video - Mostly done. Doesn't cover gamma, YUV-overlay or OpenGL. Window Manager - Mostly done. SetIcon/IconifyWindow not implemented. Events - Mostly done. Resize and some joystick events missing. Joystick - Currently assumes a single joystick with 4 buttons. -Audio - Requires Digital Renderer module. +Audio - Done CDROM - Not implemented. Threads - Done Timers - Done @@ -76,12 +78,12 @@ SDL API notes This section contains additional notes on some specific commands. SDL_SetVideoMode - On RISCOS a fullscreen mode directly accesses the screen. This can be modified by the environmental variable (SDL$$BackBuffer) or by using the SDL_SWSURFACE flag to write to an offscreen buffer that is updated using SDL_UpdateRects. + On RISC OS a fullscreen mode directly accesses the screen. This can be modified by the environmental variable (SDL$$BackBuffer) or by using the SDL_SWSURFACE flag to write to an offscreen buffer that is updated using SDL_UpdateRects. Open GL is not supported so SDL_OPENGL and SDL_OPENGLBLIT flags fail. SDL_RESIZEABLE and SDL_NOFRAME flags are not supported. SDL_SetColors - In a wimp mode the screen colours are not changed for a hardware palette instead the RISCOS sprite colour mapping is used to get the best matching colours. + In a wimp mode the screen colours are not changed for a hardware palette instead the RISC OS sprite colour mapping is used to get the best matching colours. SDL_CreateCursor Inverted colour is not supported. @@ -107,6 +109,3 @@ Notes on current implementation ------------------------------- Keyboard and mouse are polled so if too long a time is spent between a call to SDL_PumpEvents, functions that use it, or SDL_Delay events can be missed. - -The sound is sent to the digital renderer player in a seperate thread. If there is too long a delay between this thread being processed (possibly if a lot else is running in the Wimp) the sound may sound incorrect or dissappear. - diff --git a/configure.in b/configure.in index c850798bf..ff6ca5b4a 100644 --- a/configure.in +++ b/configure.in @@ -2644,7 +2644,7 @@ case "$target" in # MiNT does not define "unix" CFLAGS="$CFLAGS -Dunix" ;; - *-*-riscos) + *-riscos) ARCH=riscos # Set up files for the video library if test x$enable_video = xyes; then @@ -2653,8 +2653,11 @@ case "$target" in fi # Set up files for the audio library if test x$enable_audio = xyes; then - AUDIO_SUBDIRS="$AUDIO_SUBDIRS riscos" - AUDIO_DRIVERS="$AUDIO_DRIVERS riscos/libaudio_riscos.la" + CFLAGS="$CFLAGS -DOSS_SUPPORT" + AUDIO_SUBDIRS="$AUDIO_SUBDIRS dsp" + AUDIO_DRIVERS="$AUDIO_DRIVERS dsp/libaudio_dsp.la" + AUDIO_SUBDIRS="$AUDIO_SUBDIRS dma" + AUDIO_DRIVERS="$AUDIO_DRIVERS dma/libaudio_dma.la" fi # Set up files for the joystick library if test x$enable_joystick = xyes; then @@ -2684,7 +2687,7 @@ case "$target" in COPY_ARCH_SRC(src/timer, riscos, SDL_systimer.c) fi - CFLAGS="$CFLAGS -DENABLE_RISCOS -DDRENDERER_SUPPORT" + CFLAGS="$CFLAGS -DENABLE_RISCOS" SYSTEM_LIBS="$SYSTEM_LIBS -ljpeg -ltiff -lpng -lz" ;; @@ -2859,7 +2862,6 @@ src/audio/nas/Makefile src/audio/nto/Makefile src/audio/openbsd/Makefile src/audio/paudio/Makefile -src/audio/riscos/Makefile src/audio/sun/Makefile src/audio/ums/Makefile src/audio/windib/Makefile diff --git a/include/SDL_syswm.h b/include/SDL_syswm.h index 9149b23b4..f56a8e298 100644 --- a/include/SDL_syswm.h +++ b/include/SDL_syswm.h @@ -140,12 +140,12 @@ struct SDL_SysWMmsg { int pollBlock[64]; }; -/* The RISCOS custom window manager information structure */ +/* The RISC OS custom window manager information structure */ typedef struct SDL_SysWMinfo { SDL_version version; int wimpVersion; /* Wimp version running under */ - int taskHandle; /* The RISCOS task handle */ - int window; /* The RISCOS display window */ + int taskHandle; /* The RISC OS task handle */ + int window; /* The RISC OS display window */ } SDL_SysWMinfo; #elif defined(__QNXNTO__) diff --git a/src/audio/SDL_audiodev.c b/src/audio/SDL_audiodev.c index c3f3be33f..46c62a733 100644 --- a/src/audio/SDL_audiodev.c +++ b/src/audio/SDL_audiodev.c @@ -27,7 +27,7 @@ static char rcsid = /* Get the name of the audio device we use for output */ -#if defined(unix) || defined(__unix__) +#if defined(unix) || defined(__unix__) || defined(__riscos__) #include #include diff --git a/src/audio/riscos/.cvsignore b/src/audio/riscos/.cvsignore deleted file mode 100644 index 899d53557..000000000 --- a/src/audio/riscos/.cvsignore +++ /dev/null @@ -1,6 +0,0 @@ -Makefile.in -Makefile -.libs -*.o -*.lo -*.la diff --git a/src/audio/riscos/Makefile.am b/src/audio/riscos/Makefile.am deleted file mode 100644 index 1881e138d..000000000 --- a/src/audio/riscos/Makefile.am +++ /dev/null @@ -1,9 +0,0 @@ - -## Makefile.am for SDL using the RISC OS audio system - -noinst_LTLIBRARIES = libaudio_riscos.la -libaudio_riscos_la_SOURCES = $(SRCS) - -# The SDL audio driver sources -SRCS = SDL_drenderer.c \ - SDL_drenderer.h diff --git a/src/audio/riscos/SDL_drenderer.c b/src/audio/riscos/SDL_drenderer.c deleted file mode 100644 index 8e766a99d..000000000 --- a/src/audio/riscos/SDL_drenderer.c +++ /dev/null @@ -1,382 +0,0 @@ -/* - SDL - Simple DirectMedia Layer - Copyright (C) 1997-2004 Sam Lantinga - - This library is free software; you can redistribute it and/or - modify it under the terms of the GNU Library General Public - License as published by the Free Software Foundation; either - version 2 of the License, or (at your option) any later version. - - This library is distributed in the hope that it will be useful, - but WITHOUT ANY WARRANTY; without even the implied warranty of - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - Library General Public License for more details. - - You should have received a copy of the GNU Library General Public - License along with this library; if not, write to the Free - Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA - - Sam Lantinga - slouken@devolution.com -*/ - - -#include -#include -#include - -#include -#include "swis.h" - -#include "SDL_endian.h" -#include "SDL_audio.h" -#include "SDL_audio_c.h" -#include "SDL_audiomem.h" -#include "SDL_sysaudio.h" -#include "SDL_drenderer.h" - -#ifndef DISABLE_THREADS -#include -#endif - -#define DigitalRenderer_Activate 0x4F700 -#define DigitalRenderer_Deactivate 0x4F701 -#define DigitalRenderer_ReadState 0x4F705 -#define DigitalRenderer_NewSample 0x4F706 -#define DigitalRenderer_NumBuffers 0x4F709 -#define DigitalRenderer_StreamSamples 0x4F70A -#define DigitalRenderer_Stream16BitSamples 0x4F70B -#define DigitalRenderer_StreamStatistics 0x4F70C -#define DigitalRenderer_StreamFlags 0x4F70D -#define DigitalRenderer_Activate16 0x4F70F -#define DigitalRenderer_GetFrequency 0x4F710 - -static int FillBuffer; -extern SDL_AudioDevice *current_audio; - -extern int riscos_audiobuffer; /* Override for audio buffer size */ - -/* Audio driver functions */ - -static void DRenderer_CloseAudio(_THIS); -static int DRenderer_OpenAudio(_THIS, SDL_AudioSpec *spec); - -#ifndef DISABLE_THREADS -static void DRenderer_WaitAudio(_THIS); -static Uint8 *DRenderer_GetAudioBuf(_THIS); -static void DRenderer_PlayAudio(_THIS); -#endif - -/* Audio driver bootstrap functions */ - -/* Define following to dump stats to stdout */ -/* #define DUMP_AUDIO */ - - -static int Audio_Available(void) -{ - _kernel_swi_regs regs; - int available = 0; - - /* Use call to set buffers to also check if Module is loaded */ - regs.r[0] = 0; - if (_kernel_swi(DigitalRenderer_NumBuffers, ®s, ®s) == 0) available = 1; - - return(available); -} - -static void Audio_DeleteDevice(SDL_AudioDevice *device) -{ - free(device->hidden); - free(device); -} - -static SDL_AudioDevice *Audio_CreateDevice(int devindex) -{ - SDL_AudioDevice *this; - - /* Initialize all variables that we clean on shutdown */ - this = (SDL_AudioDevice *)malloc(sizeof(SDL_AudioDevice)); - if ( this ) { - memset(this, 0, (sizeof *this)); - this->hidden = (struct SDL_PrivateAudioData *) - malloc((sizeof *this->hidden)); - } - if ( (this == NULL) || (this->hidden == NULL) ) { - SDL_OutOfMemory(); - if ( this ) { - free(this); - } - return(0); - } - memset(this->hidden, 0, (sizeof *this->hidden)); - - /* Set the function pointers */ - this->OpenAudio = DRenderer_OpenAudio; - this->CloseAudio = DRenderer_CloseAudio; -#ifndef DISABLE_THREADS - this->GetAudioBuf = DRenderer_GetAudioBuf; - this->PlayAudio = DRenderer_PlayAudio; - this->WaitAudio = DRenderer_WaitAudio; -#endif - this->free = Audio_DeleteDevice; - - return this; -} - -AudioBootStrap DRENDERER_bootstrap = { - "drenderer", "RiscOS Digital Renderer Module", - Audio_Available, Audio_CreateDevice -}; - -/* Routine called to check and fill audio buffers if necessary */ -static Uint8 *buffer = NULL; - -#ifdef DISABLE_THREADS - -/* Buffer fill routine called during polling */ -void DRenderer_FillBuffers() -{ - SDL_AudioDevice *audio = current_audio; - - if ( !audio || ! audio->enabled ) - { - return; - } - - if ( ! audio->paused ) - { - _kernel_swi_regs regs; - /* Check filled buffers count */ - _kernel_swi(DigitalRenderer_StreamStatistics, ®s, ®s); - -#ifdef DUMP_AUDIO - if (regs.r[0] <= FillBuffer) - { - printf("Buffers in use %d\n", regs.r[0]); - } -#endif - - while (regs.r[0] <= FillBuffer && !audio->paused) - { - if ( audio->convert.needed ) - { - int silence; - if ( audio->convert.src_format == AUDIO_U8 ) - { - silence = 0x80; - } else { - silence = 0; - } - memset(audio->convert.buf, silence, audio->convert.len); - audio->spec.callback(audio->spec.userdata, - (Uint8 *)audio->convert.buf,audio->convert.len); - SDL_ConvertAudio(&audio->convert); -#if 0 - if ( audio->convert.len_cvt != audio->spec.size ) { - /* Uh oh... probably crashes here; */ - } -#endif - regs.r[0] = (int)audio->convert.buf; - regs.r[1] = audio->spec.samples * audio->spec.channels; - _kernel_swi(DigitalRenderer_Stream16BitSamples, ®s, ®s); - - } else - { - /* Fill buffer with silence */ - memset (buffer, 0, audio->spec.size); - - audio->spec.callback(audio->spec.userdata, - (Uint8 *)buffer, audio->spec.size); - - regs.r[0] = (int)buffer; - regs.r[1] = audio->spec.samples * audio->spec.channels; - _kernel_swi(DigitalRenderer_Stream16BitSamples, ®s, ®s); - } - /* Check if we have enough buffers yet */ - _kernel_swi(DigitalRenderer_StreamStatistics, ®s, ®s); - } - - } -} - -#endif - -/* Size of DMA buffer to use */ -#define DRENDERER_BUFFER_SIZE 512 - -/* Number of centiseconds of sound to buffer. - Hopefully more than the maximum time between calls to the - FillBuffers routine above (non-threaded) or the PlayAudio - routine below (threaded). -*/ - -#define DRENDERER_CSEC_TO_BUFFER 10 - -static void DeactivateAudio() -{ - _kernel_swi_regs regs; - - /* Close down the digital renderer */ - _kernel_swi(DigitalRenderer_Deactivate, ®s, ®s); - - if (buffer != NULL) - { - free(buffer); - buffer = NULL; - } -} - -static int DRenderer_OpenAudio(_THIS, SDL_AudioSpec *spec) -{ - _kernel_swi_regs regs; - int buffers_per_sample; - -#ifdef DUMP_AUDIO - printf("Request format %d\n", spec->format); - printf("Request freq %d\n", spec->freq); - printf("Samples %d\n", spec->samples); - printf("Channels %d\n", spec->channels); -#endif - - /* Only support signed 16bit format */ - spec->format = AUDIO_S16LSB; - - if (spec->samples < DRENDERER_BUFFER_SIZE) spec->samples = DRENDERER_BUFFER_SIZE; - - buffers_per_sample = spec->samples / DRENDERER_BUFFER_SIZE; - - if ((spec->samples % DRENDERER_BUFFER_SIZE) != 0) - { - buffers_per_sample++; - spec->samples = buffers_per_sample * DRENDERER_BUFFER_SIZE; - } - - - /* Set number of buffers to use - the following should give enough - data between calls to the sound polling. - */ - - if (riscos_audiobuffer == 0) - { - FillBuffer = (int)((double)DRENDERER_CSEC_TO_BUFFER / ((double)DRENDERER_BUFFER_SIZE * 100.0 / (double)spec->freq)) + 1; - } else FillBuffer = riscos_audiobuffer/DRENDERER_BUFFER_SIZE - buffers_per_sample; - - if (FillBuffer < buffers_per_sample) FillBuffer = buffers_per_sample; -#ifndef DISABLE_THREADS - if (buffers_per_sample < FillBuffer) - { - /* For the threaded version we are only called once per cycle - so the callback needs to give us the full data we need in - one go, rather than multiple calls as it the case for the - non threaded version */ - buffers_per_sample = FillBuffer; - spec->samples = buffers_per_sample * DRENDERER_BUFFER_SIZE; - } -#endif - SDL_CalculateAudioSpec(spec); - regs.r[0] = FillBuffer + buffers_per_sample; - -#ifdef DUMP_AUDIO - printf("Buffers per sample %d\n", buffers_per_sample); - printf("Fill buffer %d\n", FillBuffer); - printf("Time buffered (ms) %d\n",(int)((1000.0 * regs.r[0] * DRENDERER_BUFFER_SIZE)/(double)spec->freq)); -#endif - - if (_kernel_swi(DigitalRenderer_NumBuffers, ®s, ®s) != 0) - { - SDL_SetError("Can't set number of streaming sound buffers\n"); - return -1; - } - - /* Now initialise sound system */ - regs.r[0] = spec->channels; /* Number of channels */ - regs.r[1] = DRENDERER_BUFFER_SIZE; /* Samples length */ - regs.r[2] = spec->freq; /* frequency */ - regs.r[3] = 1; /* Restore previous handler on exit */ - - if (_kernel_swi(DigitalRenderer_Activate16, ®s, ®s) != 0) - { - SDL_SetError("Unable to activate digital renderer in 16 bit mode\n"); - return -1; - } - - if (_kernel_swi(DigitalRenderer_GetFrequency, ®s, ®s) == 0) - { - spec->freq = regs.r[0]; - } - - /* Ensure sound is deactivated if we exit without calling SDL_Quit */ - atexit(DeactivateAudio); - -#ifdef DUMP_AUDIO - printf("Got format %d\n", spec->format); - printf("Frequency %d\n", spec->freq); - printf("Samples %d\n", spec->samples); -#endif - - /* Set to fill buffer with zero if we don't get data to it fast enough */ - regs.r[0] = 1; - regs.r[1] = ~1; - _kernel_swi(DigitalRenderer_StreamFlags, ®s, ®s); - - buffer = (Uint8 *)malloc(sizeof(Uint8) * spec->size); - if (buffer == NULL) - { - SDL_OutOfMemory(); - return -1; - } - -#ifdef DISABLE_THREADS - /* Hopefully returning 2 will show success, but not start up an audio thread */ - return 2; -#else - /* Success and start audio thread */ - return 0; -#endif -} - -static void DRenderer_CloseAudio(_THIS) -{ - DeactivateAudio(); -} - -#ifndef DISABLE_THREADS - -/* Routines for threaded version of audio */ - -void DRenderer_WaitAudio(_THIS) -{ - _kernel_swi_regs regs; - int waiting = 1; - - while (waiting) - { - /* Check filled buffers count */ - _kernel_swi(DigitalRenderer_StreamStatistics, ®s, ®s); -#if 0 - if (regs.r[0] <= FillBuffer) - { - printf("Buffers in use %d\n", regs.r[0]); - } -#endif - if (regs.r[0] <= FillBuffer) waiting = 0; - else pthread_yield(); - } -} - -Uint8 *DRenderer_GetAudioBuf(_THIS) -{ - return buffer; -} - -void DRenderer_PlayAudio(_THIS) -{ - _kernel_swi_regs regs; - - regs.r[0] = (int)buffer; - regs.r[1] = current_audio->spec.samples * current_audio->spec.channels; - _kernel_swi(DigitalRenderer_Stream16BitSamples, ®s, ®s); -} - -#endif diff --git a/src/audio/riscos/SDL_drenderer.h b/src/audio/riscos/SDL_drenderer.h deleted file mode 100644 index 750c32130..000000000 --- a/src/audio/riscos/SDL_drenderer.h +++ /dev/null @@ -1,36 +0,0 @@ -/* - SDL - Simple DirectMedia Layer - Copyright (C) 1997-2004 Sam Lantinga - - This library is free software; you can redistribute it and/or - modify it under the terms of the GNU Library General Public - License as published by the Free Software Foundation; either - version 2 of the License, or (at your option) any later version. - - This library is distributed in the hope that it will be useful, - but WITHOUT ANY WARRANTY; without even the implied warranty of - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - Library General Public License for more details. - - You should have received a copy of the GNU Library General Public - License along with this library; if not, write to the Free - Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA - - Sam Lantinga - slouken@devolution.com -*/ - - -#ifndef _SDL_drenderer_h -#define _SDL_drenderer_h - -#include "SDL_sysaudio.h" - -/* Hidden "this" pointer for the video functions */ -#define _THIS SDL_AudioDevice *this - -struct SDL_PrivateAudioData { - unsigned char *audio_buf; -}; - -#endif /* _SDL_drenderer_h */ diff --git a/src/joystick/riscos/SDL_sysjoystick.c b/src/joystick/riscos/SDL_sysjoystick.c index ca570a520..cc102a940 100644 --- a/src/joystick/riscos/SDL_sysjoystick.c +++ b/src/joystick/riscos/SDL_sysjoystick.c @@ -21,7 +21,7 @@ */ /* - RISCOS - Joystick support by Alan Buckley (alan_baa@hotmail.com) - 10 April 2003 + RISC OS - Joystick support by Alan Buckley (alan_baa@hotmail.com) - 10 April 2003 Note: Currently assumes joystick is present if joystick module is loaded and that there is one joystick with four buttons. @@ -78,7 +78,7 @@ const char *SDL_SYS_JoystickName(int index) { if (index == 0) { - return "RISCOS Joystick 0"; + return "RISC OS Joystick 0"; } SDL_SetError("No joystick available with that index"); diff --git a/src/thread/linux/clone.S b/src/thread/linux/clone.S index 5abebce16..75d5a3bac 100644 --- a/src/thread/linux/clone.S +++ b/src/thread/linux/clone.S @@ -1,7 +1,7 @@ /* Taken with thanks from LinuxThreads 0.6 */ -/* This is no longer necessary with glibc-2.1, which has it's own clone() */ +/* This is no longer necessary with glibc-2.1, which has its own clone() */ #ifdef linux /* Look to see if glibc is available, and if so, what version */ #include diff --git a/src/timer/riscos/SDL_systimer.c b/src/timer/riscos/SDL_systimer.c index b5aeca271..b1c699d46 100644 --- a/src/timer/riscos/SDL_systimer.c +++ b/src/timer/riscos/SDL_systimer.c @@ -99,8 +99,8 @@ void SDL_Delay (Uint32 ms) } else is_event_thread = 1; #endif -/*TODO: Next version of unixlib may allow us to use usleep here */ -/* for non event threads */ + /*TODO: Next version of Unixlib may allow us to use usleep here */ + /* for non event threads */ /* Set the timeout interval - Linux only needs to do this once */ then = SDL_GetTicks(); @@ -222,7 +222,7 @@ void SDL_SYS_TimerQuit(void) int SDL_SYS_StartTimer(void) { - SDL_SetError("Internal logic error: RISCOS uses threaded timer"); + SDL_SetError("Internal logic error: RISC OS uses threaded timer"); return(-1); } diff --git a/src/video/riscos/Makefile.am b/src/video/riscos/Makefile.am index 91c77fcf8..4b7217fcf 100644 --- a/src/video/riscos/Makefile.am +++ b/src/video/riscos/Makefile.am @@ -6,6 +6,7 @@ libvideo_riscos_la_SOURCES = $(CGX_SRCS) # The SDL RISC OS video driver sources CGX_SRCS = \ + SDL_riscosASM.s \ SDL_riscosevents.c \ SDL_riscosevents_c.h \ SDL_riscosFullScreenVideo.c \ diff --git a/src/video/riscos/SDL_riscosASM.s b/src/video/riscos/SDL_riscosASM.s new file mode 100644 index 000000000..08cd64fb0 --- /dev/null +++ b/src/video/riscos/SDL_riscosASM.s @@ -0,0 +1,121 @@ +Date: Wed, 26 Jan 2005 13:37:09 GMT +From: Peter Naulls +To: slouken@libsdl.org +Subject: RISC OS SDL Patches + +; +; SDL - Simple DirectMedia Layer +; Copyright (C) 1997-2004 Sam Lantinga +; +; This library is free software; you can redistribute it and/or +; modify it under the terms of the GNU Library General Public +; License as published by the Free Software Foundation; either +; version 2 of the License, or (at your option) any later version. +; +; This library is distributed in the hope that it will be useful, +; but WITHOUT ANY WARRANTY; without even the implied warranty of +; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU +; Library General Public License for more details. +; +; You should have received a copy of the GNU Library General Public +; License along with this library; if not, write to the Free +; Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA +; +; Sam Lantinga +; slouken@devolution.com +; +; Assembler routines for RISC OS display +; + + AREA |C$$CODE| + + EXPORT |RISCOS_Put32| + +; Display 32bpp to 32bpp, 1:1 +; +; Code provided by Adrain Lees +; +; entry a1 -> destination +; a2 = dest width in pixels +; a3 = dest line length in bytes +; a4 = dest height in scanlines +; arg5 -> source +; arg6 = byte offset from end of source line to start of next + +Arg5 * 10*4 +Arg6 * Arg5+4 + +RISCOS_Put32 ROUT + STMFD sp!,{a2,v1-v6,sl,fp,lr} + LDR ip,[sp,#Arg5] + MOV lr,a1 + B ucp64lp + +00 ;tail strip of 1-15 pixels + + LDR v1,[ip],#4 +01 SUBS a2,a2,#1 + STR v1,[lr],#4 + LDRHI v1,[ip],#4 + BHI %01 + B %02 + +ucp64end ADDS a2,a2,#16 + BNE %00 + +02 SUBS a4,a4,#1 ;height-- + LDRHI v1,[sp,#Arg6] + LDRHI a2,[sp] ;reload width + BLS %03 + + ;move to start of next scanline + + ADD lr,a1,a3 + ADD a1,a1,a3 + ADD ip,ip,v1 + +ucp64lp SUBS a2,a2,#16 + BLO ucp64end + + PLD [ip,#64] + + LDR v1,[ip],#4 + LDR v2,[ip],#4 + LDR v3,[ip],#4 + LDR v4,[ip],#4 + LDR v5,[ip],#4 + LDR v6,[ip],#4 + LDR sl,[ip],#4 + LDR fp,[ip],#4 + STR v1,[lr],#4 + STR v2,[lr],#4 + STR v3,[lr],#4 + STR v4,[lr],#4 + STR v5,[lr],#4 + STR v6,[lr],#4 + STR sl,[lr],#4 + STR fp,[lr],#4 + + PLD [ip,#64] + + LDR v1,[ip],#4 + LDR v2,[ip],#4 + LDR v3,[ip],#4 + LDR v4,[ip],#4 + LDR v5,[ip],#4 + LDR v6,[ip],#4 + LDR sl,[ip],#4 + LDR fp,[ip],#4 + STR v1,[lr],#4 + STR v2,[lr],#4 + STR v3,[lr],#4 + STR v4,[lr],#4 + STR v5,[lr],#4 + STR v6,[lr],#4 + STR sl,[lr],#4 + STR fp,[lr],#4 + + B ucp64lp + +03 LDMFD sp!,{a2,v1-v6,sl,fp,pc} + diff --git a/src/video/riscos/SDL_riscosFullScreenVideo.c b/src/video/riscos/SDL_riscosFullScreenVideo.c index 3ad8ca6d4..e18cbd3ed 100644 --- a/src/video/riscos/SDL_riscosFullScreenVideo.c +++ b/src/video/riscos/SDL_riscosFullScreenVideo.c @@ -21,10 +21,10 @@ */ /* - File added by Alan Buckley (alan_baa@hotmail.com) for RISCOS compatability + File added by Alan Buckley (alan_baa@hotmail.com) for RISC OS compatability 27 March 2003 - Implements RISCOS full screen display. + Implements RISC OS full screen display. */ #include @@ -69,10 +69,17 @@ void FULLSCREEN_SetupBanks(_THIS); /* SDL video device functions for fullscreen mode */ static int FULLSCREEN_SetColors(_THIS, int firstcolor, int ncolors, SDL_Color *colors); static int FULLSCREEN_FlipHWSurface(_THIS, SDL_Surface *surface); -static void FULLSCREEN_UpdateRects(_THIS, int numrects, SDL_Rect *rects); void FULLSCREEN_SetWMCaption(_THIS, const char *title, const char *icon); extern int RISCOS_GetWmInfo(_THIS, SDL_SysWMinfo *info); +/* UpdateRects variants */ +static void FULLSCREEN_UpdateRects(_THIS, int numrects, SDL_Rect *rects); +static void FULLSCREEN_UpdateRectsMemCpy(_THIS, int numrects, SDL_Rect *rects); +static void FULLSCREEN_UpdateRects8bpp(_THIS, int numrects, SDL_Rect *rects); +static void FULLSCREEN_UpdateRects16bpp(_THIS, int numrects, SDL_Rect *rects); +static void FULLSCREEN_UpdateRects32bpp(_THIS, int numrects, SDL_Rect *rects); +static void FULLSCREEN_UpdateRectsOS(_THIS, int numrects, SDL_Rect *rects); + /* Local helper functions */ static int cmpmodes(const void *va, const void *vb); static int FULLSCREEN_AddMode(_THIS, int bpp, int w, int h); @@ -85,7 +92,11 @@ void FULLSCREEN_BuildModeList(_THIS); /* Following variable is set up in riskosTask.c */ extern int riscos_backbuffer; /* Create a back buffer in system memory for full screen mode */ +/* Following is used to create a sprite back buffer */ +extern unsigned char *WIMP_CreateBuffer(int width, int height, int bpp); +/* Fast assembler copy */ +extern void RISCOS_Put32(void *to, int pixels, int pitch, int rows, void *from, int src_skip_bytes); SDL_Surface *FULLSCREEN_SetVideoMode(_THIS, SDL_Surface *current, int width, int height, int bpp, Uint32 flags) @@ -180,11 +191,14 @@ SDL_Surface *FULLSCREEN_SetVideoMode(_THIS, SDL_Surface *current, if (create_back_buffer) { /* If not double buffered we may need to create a memory - ** back buffer to simulate processing on other OS's. + ** back buffer to simulate processing on other OSes. ** This is turned on by setting the enviromental variable - ** SDL$$BackBuffer to 1 - */ - this->hidden->bank[0] = malloc(height * current->pitch); + ** SDL$$BackBuffer >= 1 + */ + if (riscos_backbuffer == 3) + this->hidden->bank[0] = WIMP_CreateBuffer(width, height, bpp); + else + this->hidden->bank[0] = malloc(height * current->pitch); if (this->hidden->bank[0] == 0) { RISCOS_RestoreWimpMode(); @@ -195,21 +209,29 @@ SDL_Surface *FULLSCREEN_SetVideoMode(_THIS, SDL_Surface *current, current->flags &= ~SDL_HWSURFACE; } - /* Store address of allocated screen bank to be freed later */ - if (this->hidden->alloc_bank) free(this->hidden->alloc_bank); - if (create_back_buffer) - { - this->hidden->alloc_bank = this->hidden->bank[0]; - } else - this->hidden->alloc_bank = 0; - - // Clear both banks to black - memset(this->hidden->bank[0], 0, height * current->pitch); - memset(this->hidden->bank[1], 0, height * current->pitch); + /* Store address of allocated screen bank to be freed later */ + if (this->hidden->alloc_bank) free(this->hidden->alloc_bank); + if (create_back_buffer) + { + this->hidden->alloc_bank = this->hidden->bank[0]; + if (riscos_backbuffer == 3) + { + this->hidden->bank[0] += 60; /* Start of sprite data */ + if (bpp == 8) this->hidden->bank[0] += 2048; /* 8bpp sprite have palette first */ + } + } else + this->hidden->alloc_bank = 0; + + // Clear both banks to black + memset(this->hidden->bank[0], 0, height * current->pitch); + memset(this->hidden->bank[1], 0, height * current->pitch); this->hidden->current_bank = 0; current->pixels = this->hidden->bank[0]; + /* Have to set the screen here, so SetDeviceMode will pick it up */ + this->screen = current; + /* Reset device functions for the wimp */ FULLSCREEN_SetDeviceMode(this); @@ -222,10 +244,45 @@ SDL_Surface *FULLSCREEN_SetVideoMode(_THIS, SDL_Surface *current, /* Reset any device functions that have been changed because we have run in WIMP mode */ void FULLSCREEN_SetDeviceMode(_THIS) { + /* Update rects is different if we have a backbuffer */ + + if (riscos_backbuffer && (this->screen->flags & SDL_DOUBLEBUF) == 0) + { + switch(riscos_backbuffer) + { + case 2: /* ARM code full word copy */ + switch(this->screen->format->BytesPerPixel) + { + case 1: /* 8bpp modes */ + this->UpdateRects = FULLSCREEN_UpdateRects8bpp; + break; + case 2: /* 15/16bpp modes */ + this->UpdateRects = FULLSCREEN_UpdateRects16bpp; + break; + case 4: /* 32 bpp modes */ + this->UpdateRects = FULLSCREEN_UpdateRects32bpp; + break; + + default: /* Just default to the memcpy routine */ + this->UpdateRects = FULLSCREEN_UpdateRectsMemCpy; + break; + } + break; + + case 3: /* Use OS sprite plot routine */ + this->UpdateRects = FULLSCREEN_UpdateRectsOS; + break; + + default: /* Old but safe memcpy */ + this->UpdateRects = FULLSCREEN_UpdateRectsMemCpy; + break; + } + } else + this->UpdateRects = FULLSCREEN_UpdateRects; /* Default do nothing implementation */ + if (this->SetColors == FULLSCREEN_SetColors) return; /* Already set up */ this->SetColors = FULLSCREEN_SetColors; - this->UpdateRects = FULLSCREEN_UpdateRects; this->FlipHWSurface = FULLSCREEN_FlipHWSurface; @@ -322,11 +379,14 @@ static int FULLSCREEN_FlipHWSurface(_THIS, SDL_Surface *surface) return(0); } +/* Nothing to do if we are writing direct to hardware */ static void FULLSCREEN_UpdateRects(_THIS, int numrects, SDL_Rect *rects) { - if (riscos_backbuffer && (this->screen->flags & SDL_DOUBLEBUF) == 0) - { - /* If not double buffered copy rectangles to main screen now */ +} + +/* Safe but slower Memory copy from our allocated back buffer */ +static void FULLSCREEN_UpdateRectsMemCpy(_THIS, int numrects, SDL_Rect *rects) +{ int j; char *to, *from; int pitch = this->screen->pitch; @@ -344,9 +404,138 @@ static void FULLSCREEN_UpdateRects(_THIS, int numrects, SDL_Rect *rects) } rects++; } +} + +/* Use optimized assembler memory copy. Deliberately copies extra columns if + necessary to ensure the rectangle is word aligned. */ +static void FULLSCREEN_UpdateRects8bpp(_THIS, int numrects, SDL_Rect *rects) +{ + int j; + char *to, *from; + int pitch = this->screen->pitch; + int width_bytes; + int src_skip_bytes; + + for (j = 0; j < numrects; j++) + { + from = this->hidden->bank[0] + rects->x + rects->y * pitch; + to = this->hidden->bank[1] + rects->x + rects->y * pitch; + width_bytes = rects->w; + if ((int)from & 3) + { + int extra = ((int)from & 3); + from -= extra; + to -= extra; + width_bytes += extra; + } + if (width_bytes & 3) width_bytes += 4 - (width_bytes & 3); + src_skip_bytes = pitch - width_bytes; + + RISCOS_Put32(to, (width_bytes >> 2), pitch, (int)rects->h, from, src_skip_bytes); + rects++; + } +} + +/* Use optimized assembler memory copy. Deliberately copies extra columns if + necessary to ensure the rectangle is word aligned. */ +static void FULLSCREEN_UpdateRects16bpp(_THIS, int numrects, SDL_Rect *rects) +{ + int j; + char *to, *from; + int pitch = this->screen->pitch; + int width_bytes; + int src_skip_bytes; + + for (j = 0; j < numrects; j++) + { + from = this->hidden->bank[0] + (rects->x << 1) + rects->y * pitch; + to = this->hidden->bank[1] + (rects->x << 1) + rects->y * pitch; + width_bytes = (((int)rects->w) << 1); + if ((int)from & 3) + { + from -= 2; + to -= 2; + width_bytes += 2; + } + if (width_bytes & 3) width_bytes += 2; + src_skip_bytes = pitch - width_bytes; + + RISCOS_Put32(to, (width_bytes >> 2), pitch, (int)rects->h, from, src_skip_bytes); + rects++; + } +} + +/* Use optimized assembler memory copy. 32 bpp modes are always word aligned */ +static void FULLSCREEN_UpdateRects32bpp(_THIS, int numrects, SDL_Rect *rects) +{ + int j; + char *to, *from; + int pitch = this->screen->pitch; + int width; + + for (j = 0; j < numrects; j++) + { + from = this->hidden->bank[0] + (rects->x << 2) + rects->y * pitch; + to = this->hidden->bank[1] + (rects->x << 2) + rects->y * pitch; + width = (int)rects->w ; + + RISCOS_Put32(to, width, pitch, (int)rects->h, from, pitch - (width << 2)); + rects++; } } +/* Use operating system sprite plots. Currently this is much slower than the + other variants however accelerated sprite plotting can be seen on the horizon + so this prepares for it. */ +static void FULLSCREEN_UpdateRectsOS(_THIS, int numrects, SDL_Rect *rects) +{ + _kernel_swi_regs regs; + _kernel_oserror *err; + int j; + int y; + + regs.r[0] = 28 + 512; + regs.r[1] = (unsigned int)this->hidden->alloc_bank; + regs.r[2] = (unsigned int)this->hidden->alloc_bank+16; + regs.r[5] = 0; + + for (j = 0; j < numrects; j++) + { + y = this->screen->h - rects->y; /* top of clipping region */ + _kernel_oswrch(24); /* Set graphics clip region */ + _kernel_oswrch((rects->x << this->hidden->xeig) & 0xFF); /* left */ + _kernel_oswrch(((rects->x << this->hidden->xeig) >> 8) & 0xFF); + _kernel_oswrch(((y - rects->h) << this->hidden->yeig) & 0xFF); /* bottom */ + _kernel_oswrch((((y - rects->h) << this->hidden->yeig)>> 8) & 0xFF); + _kernel_oswrch(((rects->x + rects->w - 1) << this->hidden->xeig) & 0xFF); /* right */ + _kernel_oswrch((((rects->x + rects->w - 1)<< this->hidden->xeig) >> 8) & 0xFF); + _kernel_oswrch(((y-1) << this->hidden->yeig) & 0xFF); /* top */ + _kernel_oswrch((((y-1) << this->hidden->yeig) >> 8) & 0xFF); + + regs.r[3] = 0; + regs.r[4] = 0; + + if ((err = _kernel_swi(OS_SpriteOp, ®s, ®s)) != 0) + { + printf("OS_SpriteOp failed \n%s\n",err->errmess); + } + + rects++; + + /* Reset to full screen clipping */ + _kernel_oswrch(24); /* Set graphics clip region */ + _kernel_oswrch(0); /* left */ + _kernel_oswrch(0); + _kernel_oswrch(0); /* bottom */ + _kernel_oswrch(0); + _kernel_oswrch(((this->screen->w-1) << this->hidden->xeig) & 0xFF); /* right */ + _kernel_oswrch((((this->screen->w-1) << this->hidden->xeig) >> 8) & 0xFF); + _kernel_oswrch(((this->screen->h-1) << this->hidden->yeig) & 0xFF); /* top */ + _kernel_oswrch((((this->screen->h-1) << this->hidden->yeig) >> 8) & 0xFF); + } +} + + int FULLSCREEN_SetColors(_THIS, int firstcolor, int ncolors, SDL_Color *colors) { _kernel_swi_regs regs; @@ -573,7 +762,7 @@ int FULLSCREEN_ToggleFromWimp(_THIS) { char *buffer = this->hidden->alloc_bank; /* This is start of sprite data */ /* Support back buffer mode only */ - riscos_backbuffer = 1; + if (riscos_backbuffer == 0) riscos_backbuffer = 1; FULLSCREEN_SetupBanks(this); diff --git a/src/video/riscos/SDL_riscosevents.c b/src/video/riscos/SDL_riscosevents.c index 39804f962..184766c78 100644 --- a/src/video/riscos/SDL_riscosevents.c +++ b/src/video/riscos/SDL_riscosevents.c @@ -21,7 +21,7 @@ */ /* - File added by Alan Buckley (alan_baa@hotmail.com) for RISCOS compatability + File added by Alan Buckley (alan_baa@hotmail.com) for RISC OS compatability 27 March 2003 Implements keyboard setup, event pump and keyboard and mouse polling @@ -43,10 +43,10 @@ #include "kernel.h" #include "swis.h" -/* The translation table from a RISCOS internal key numbers to a SDL keysym */ +/* The translation table from a RISC OS internal key numbers to a SDL keysym */ static SDLKey RO_keymap[SDLK_LAST]; -/* RISCOS Key codes */ +/* RISC OS Key codes */ #define ROKEY_SHIFT 0 #define ROKEY_CTRL 1 #define ROKEY_ALT 2 @@ -90,7 +90,7 @@ void FULLSCREEN_PumpEvents(_THIS) RISCOS_PollKeyboard(); RISCOS_PollMouse(this); #ifdef DISABLE_THREADS - DRenderer_FillBuffers(); +// DRenderer_FillBuffers(); if (SDL_timer_running) RISCOS_CheckTimer(); #endif } @@ -262,9 +262,11 @@ void RISCOS_PollMouseHelper(_THIS, int fullscreen) Sint16 new_x = regs.r[0]; /* Initialy get as OS units */ Sint16 new_y = regs.r[1]; -/* Discard mouse events until the let go of the mouse after starting */ - if (starting && regs.r[2] != 0) return; - else starting = 0; + /* Discard mouse events until they let go of the mouse after starting */ + if (starting && regs.r[2] != 0) + return; + else + starting = 0; if (new_x != last_x || new_y != last_y || last_buttons != regs.r[2]) { diff --git a/src/video/riscos/SDL_riscosmouse.c b/src/video/riscos/SDL_riscosmouse.c index ebd061940..a75e9b879 100644 --- a/src/video/riscos/SDL_riscosmouse.c +++ b/src/video/riscos/SDL_riscosmouse.c @@ -21,7 +21,7 @@ */ /* - File added by Alan Buckley (alan_baa@hotmail.com) for RISCOS compatability + File added by Alan Buckley (alan_baa@hotmail.com) for RISC OS compatability 27 March 2003 Implements mouse cursor shape definitions and positioning diff --git a/src/video/riscos/SDL_riscossprite.c b/src/video/riscos/SDL_riscossprite.c index 612149397..5537486d3 100644 --- a/src/video/riscos/SDL_riscossprite.c +++ b/src/video/riscos/SDL_riscossprite.c @@ -21,7 +21,7 @@ */ /* - File added by Alan Buckley (alan_baa@hotmail.com) for RISCOS compatability + File added by Alan Buckley (alan_baa@hotmail.com) for RISC OS compatability 27 March 2003 Implements Sprite plotting code for wimp display.window diff --git a/src/video/riscos/SDL_riscostask.c b/src/video/riscos/SDL_riscostask.c index c07c8fd6f..d02789092 100644 --- a/src/video/riscos/SDL_riscostask.c +++ b/src/video/riscos/SDL_riscostask.c @@ -21,7 +21,7 @@ */ /* - This file added by Alan Buckley (alan_baa@hotmail.com) to support RISCOS + This file added by Alan Buckley (alan_baa@hotmail.com) to support RISC OS 26 March 2003 File includes routines for: @@ -44,7 +44,7 @@ pthread_t main_thread; #endif -/* RISCOS variables */ +/* RISC OS variables */ static int task_handle = 0; static int wimp_version = 0; @@ -84,7 +84,7 @@ static void dump_mode() /****************************************************************** - Initialise as RISCOS Wimp task + Initialise as RISC OS Wimp task *******************************************************************/ @@ -169,7 +169,7 @@ void RISCOS_ExitTask() there is a variable set up in the form SDL$$TaskName in which case the value of this variable will be used. - Now also gets other riscos configuration varibles + Now also gets other RISC OS configuration varibles SDL$$BackBuffer - set to 1 to use a system memory backbuffer in fullscreen mode so updates wait until a call to SDL_UpdateRects. (default 0) This is required for programmes where they have assumed this is @@ -226,7 +226,7 @@ int RISCOS_GetTaskName(char *task_name) if (*p) { - /* Read variables that effect the RISCOS SDL engine for this task */ + /* Read variables that effect the RISC OS SDL engine for this task */ env_var = malloc(strlen(p) + 18); /* 18 is larger than the biggest variable name */ if (env_var) { @@ -246,7 +246,7 @@ int RISCOS_GetTaskName(char *task_name) strcat(env_var, "$BackBuffer"); env_val = getenv(env_var); - if (env_val && strcmp(env_val,"1") == 0) riscos_backbuffer = 1; + if (env_val) riscos_backbuffer = atoi(env_val); strcpy(env_var, "SDL$"); strcat(env_var, p); diff --git a/src/video/riscos/SDL_riscostask.h b/src/video/riscos/SDL_riscostask.h index 55efd4945..82ee88ecc 100644 --- a/src/video/riscos/SDL_riscostask.h +++ b/src/video/riscos/SDL_riscostask.h @@ -21,7 +21,7 @@ */ /* - This file added by Alan Buckley (alan_baa@hotmail.com) to support RISCOS + This file added by Alan Buckley (alan_baa@hotmail.com) to support RISC OS 26 March 2003 */ diff --git a/src/video/riscos/SDL_riscosvideo.c b/src/video/riscos/SDL_riscosvideo.c index 17e089c3d..fe0b6d0d0 100644 --- a/src/video/riscos/SDL_riscosvideo.c +++ b/src/video/riscos/SDL_riscosvideo.c @@ -21,10 +21,10 @@ */ /* - File added by Alan Buckley (alan_baa@hotmail.com) for RISCOS compatability + File added by Alan Buckley (alan_baa@hotmail.com) for RISC OS compatability 23 March 2003 - Implements RISCOS display device management. + Implements RISC OS display device management. Routines for full screen and wimp modes are split into other source files. */ @@ -83,7 +83,7 @@ static int RISCOS_LockHWSurface(_THIS, SDL_Surface *surface); static void RISCOS_UnlockHWSurface(_THIS, SDL_Surface *surface); static void RISCOS_FreeHWSurface(_THIS, SDL_Surface *surface); -/* RISCOS driver bootstrap functions */ +/* RISC OS driver bootstrap functions */ static int RISCOS_Available(void) { @@ -150,16 +150,11 @@ static SDL_VideoDevice *RISCOS_CreateDevice(int devindex) /* Set other entries for fullscreen mode */ FULLSCREEN_SetDeviceMode(device); -/* Turn off unixlib file name translation - we hope people have initialised - the video system before they try to read any files */ -/* __riscosify_control = __RISCOSIFY_NO_PROCESS; -*//* We may be able to eventually replace our processing of filenames with the correct flags above*/ - return device; } VideoBootStrap RISCOS_bootstrap = { - RISCOSVID_DRIVER_NAME, "RISCOS video driver", + RISCOSVID_DRIVER_NAME, "RISC OS video driver", RISCOS_Available, RISCOS_CreateDevice }; diff --git a/src/video/riscos/SDL_wimppoll.c b/src/video/riscos/SDL_wimppoll.c index 624ea7fb6..266ad0af2 100644 --- a/src/video/riscos/SDL_wimppoll.c +++ b/src/video/riscos/SDL_wimppoll.c @@ -21,7 +21,7 @@ */ /* - File added by Alan Buckley (alan_baa@hotmail.com) for RISCOS compatability + File added by Alan Buckley (alan_baa@hotmail.com) for RISC OS compatability 27 March 2003 Implements Pumping of events and WIMP polling @@ -33,6 +33,7 @@ #include "SDL_events_c.h" #include "SDL_riscosvideo.h" #include "SDL_riscosevents_c.h" +#include "SDL_riscosmouse_c.h" #include "SDL_timer_c.h" #include "memory.h" @@ -61,8 +62,6 @@ extern void WIMP_PollMouse(_THIS); extern void RISCOS_PollKeyboard(); #ifdef DISABLE_THREADS -extern void DRenderer_FillBuffers(); - /* Timer running function */ extern void RISCOS_CheckTimer(); @@ -85,7 +84,6 @@ void WIMP_PumpEvents(_THIS) if (hasFocus) RISCOS_PollKeyboard(); if (mouseInWindow) WIMP_PollMouse(this); #ifdef DISABLE_THREADS - DRenderer_FillBuffers(); if (SDL_timer_running) RISCOS_CheckTimer(); #endif } @@ -329,8 +327,6 @@ void RISCOS_BackgroundTasks(void) WIMP_Poll(current_video, 0); } #ifdef DISABLE_THREADS - /* Keep sound buffers running */ - DRenderer_FillBuffers(); if (SDL_timer_running) RISCOS_CheckTimer(); #endif } diff --git a/src/video/riscos/SDL_wimpvideo.c b/src/video/riscos/SDL_wimpvideo.c index f710d7bb6..3d564e5f3 100644 --- a/src/video/riscos/SDL_wimpvideo.c +++ b/src/video/riscos/SDL_wimpvideo.c @@ -21,10 +21,10 @@ */ /* - File added by Alan Buckley (alan_baa@hotmail.com) for RISCOS compatability + File added by Alan Buckley (alan_baa@hotmail.com) for RISC OS compatability 27 March 2003 - Implements RISCOS wimp display. + Implements RISC OS Wimp display. */ #include @@ -486,7 +486,7 @@ int WIMP_ToggleFromFullScreen(_THIS) WIMP_SetDeviceMode(this); WIMP_SetupPlotInfo(this); - riscos_backbuffer = 1; + if (riscos_backbuffer == 0) riscos_backbuffer = 1; if (buffer && old_alloc_bank) free(old_alloc_bank);