1. Patchew
  2. QEMU
  3. [Qemu-devel] [PATCH 0/2] Support multiple adlib sound cards
  4. View patch

[Qemu-devel] [PATCH 1/2] audio/fmopl: modify timer callback to give opaque and channel parameters in two arguments

Hervé Poussineau posted 2 patches 8 years, 7 months ago
[Qemu-devel] [PATCH 1/2] audio/fmopl: modify timer callback to give opaque and channel parameters in two arguments
Posted by Hervé Poussineau 8 years, 7 months ago 
Signed-off-by: Hervé Poussineau <hpoussin@reactos.org>
---
 hw/audio/adlib.c |  2 +-
 hw/audio/fmopl.c | 18 +++++++++++++-----
 hw/audio/fmopl.h |  7 ++++---
 3 files changed, 18 insertions(+), 9 deletions(-)

diff --git a/hw/audio/adlib.c b/hw/audio/adlib.c
index c6e0f10c16..be4203476a 100644
--- a/hw/audio/adlib.c
+++ b/hw/audio/adlib.c
@@ -130,7 +130,7 @@ static uint32_t adlib_read(void *opaque, uint32_t nport)
     return data;
 }
 
-static void timer_handler (int c, double interval_Sec)
+static void timer_handler (void *opaque, int c, double interval_Sec)
 {
     AdlibState *s = glob_adlib;
     unsigned n = c & 1;
diff --git a/hw/audio/fmopl.c b/hw/audio/fmopl.c
index 202f752c5d..5cfb6a96dd 100644
--- a/hw/audio/fmopl.c
+++ b/hw/audio/fmopl.c
@@ -788,14 +788,18 @@ static void OPLWriteReg(FM_OPL *OPL, int r, int v)
 				{
 					double interval = st2 ? (double)OPL->T[1]*OPL->TimerBase : 0.0;
 					OPL->st[1] = st2;
-					if (OPL->TimerHandler) (OPL->TimerHandler)(OPL->TimerParam+1,interval);
+                    if (OPL->TimerHandler) {
+                        (OPL->TimerHandler)(OPL->TimerParam, 1, interval);
+                    }
 				}
 				/* timer 1 */
 				if(OPL->st[0] != st1)
 				{
 					double interval = st1 ? (double)OPL->T[0]*OPL->TimerBase : 0.0;
 					OPL->st[0] = st1;
-					if (OPL->TimerHandler) (OPL->TimerHandler)(OPL->TimerParam+0,interval);
+                    if (OPL->TimerHandler) {
+                        (OPL->TimerHandler)(OPL->TimerParam, 0, interval);
+                    }
 				}
 			}
 			return;
@@ -1128,10 +1132,11 @@ void OPLDestroy(FM_OPL *OPL)
 
 /* ----------  Option handlers ----------       */
 
-void OPLSetTimerHandler(FM_OPL *OPL,OPL_TIMERHANDLER TimerHandler,int channelOffset)
+void OPLSetTimerHandler(FM_OPL *OPL, OPL_TIMERHANDLER TimerHandler,
+                        void *param)
 {
 	OPL->TimerHandler   = TimerHandler;
-	OPL->TimerParam = channelOffset;
+    OPL->TimerParam = param;
 }
 
 /* ---------- YM3812 I/O interface ---------- */
@@ -1197,6 +1202,9 @@ int OPLTimerOver(FM_OPL *OPL,int c)
 		}
 	}
 	/* reload timer */
-	if (OPL->TimerHandler) (OPL->TimerHandler)(OPL->TimerParam+c,(double)OPL->T[c]*OPL->TimerBase);
+    if (OPL->TimerHandler) {
+        (OPL->TimerHandler)(OPL->TimerParam, c,
+                            (double)OPL->T[c] * OPL->TimerBase);
+    }
 	return OPL->status>>7;
 }
diff --git a/hw/audio/fmopl.h b/hw/audio/fmopl.h
index fc9f16b58a..f4065f425c 100644
--- a/hw/audio/fmopl.h
+++ b/hw/audio/fmopl.h
@@ -3,7 +3,7 @@
 
 #include <stdint.h>
 
-typedef void (*OPL_TIMERHANDLER)(int channel,double interval_Sec);
+typedef void (*OPL_TIMERHANDLER)(void *param, int channel, double interval_Sec);
 
 /* !!!!! here is private section , do not access there member direct !!!!! */
 
@@ -87,13 +87,14 @@ typedef struct fm_opl_f {
 	uint8_t wavesel;
 	/* external event callback handler */
 	OPL_TIMERHANDLER  TimerHandler;		/* TIMER handler   */
-	int TimerParam;						/* TIMER parameter */
+    void *TimerParam; /* TIMER parameter */
 } FM_OPL;
 
 /* ---------- Generic interface section ---------- */
 FM_OPL *OPLCreate(int clock, int rate);
 void OPLDestroy(FM_OPL *OPL);
-void OPLSetTimerHandler(FM_OPL *OPL,OPL_TIMERHANDLER TimerHandler,int channelOffset);
+void OPLSetTimerHandler(FM_OPL *OPL, OPL_TIMERHANDLER TimerHandler,
+                        void *param);
 
 int OPLWrite(FM_OPL *OPL,int a,int v);
 unsigned char OPLRead(FM_OPL *OPL,int a);
-- 
2.11.0
Re: [Qemu-devel] [PATCH 1/2] audio/fmopl: modify timer callback to give opaque and channel parameters in two arguments
Posted by Philippe Mathieu-Daudé 8 years, 6 months ago 
Hi Hervé,

On 06/21/2017 01:34 AM, Hervé Poussineau wrote:
> Signed-off-by: Hervé Poussineau <hpoussin@reactos.org>
> ---
>   hw/audio/adlib.c |  2 +-
>   hw/audio/fmopl.c | 18 +++++++++++++-----
>   hw/audio/fmopl.h |  7 ++++---
>   3 files changed, 18 insertions(+), 9 deletions(-)
> 
> diff --git a/hw/audio/adlib.c b/hw/audio/adlib.c
> index c6e0f10c16..be4203476a 100644
> --- a/hw/audio/adlib.c
> +++ b/hw/audio/adlib.c
> @@ -130,7 +130,7 @@ static uint32_t adlib_read(void *opaque, uint32_t nport)
>       return data;
>   }
>   
> -static void timer_handler (int c, double interval_Sec)
> +static void timer_handler (void *opaque, int c, double interval_Sec)
>   {
>       AdlibState *s = glob_adlib;
>       unsigned n = c & 1;
> diff --git a/hw/audio/fmopl.c b/hw/audio/fmopl.c
> index 202f752c5d..5cfb6a96dd 100644
> --- a/hw/audio/fmopl.c
> +++ b/hw/audio/fmopl.c
> @@ -788,14 +788,18 @@ static void OPLWriteReg(FM_OPL *OPL, int r, int v)
>   				{
>   					double interval = st2 ? (double)OPL->T[1]*OPL->TimerBase : 0.0;
>   					OPL->st[1] = st2;
> -					if (OPL->TimerHandler) (OPL->TimerHandler)(OPL->TimerParam+1,interval);
> +                    if (OPL->TimerHandler) {
> +                        (OPL->TimerHandler)(OPL->TimerParam, 1, interval);
> +                    }
>   				}
>   				/* timer 1 */
>   				if(OPL->st[0] != st1)
>   				{
>   					double interval = st1 ? (double)OPL->T[0]*OPL->TimerBase : 0.0;
>   					OPL->st[0] = st1;
> -					if (OPL->TimerHandler) (OPL->TimerHandler)(OPL->TimerParam+0,interval);
> +                    if (OPL->TimerHandler) {
> +                        (OPL->TimerHandler)(OPL->TimerParam, 0, interval);
> +                    }
>   				}
>   			}
>   			return;
> @@ -1128,10 +1132,11 @@ void OPLDestroy(FM_OPL *OPL)
>   
>   /* ----------  Option handlers ----------       */
>   
> -void OPLSetTimerHandler(FM_OPL *OPL,OPL_TIMERHANDLER TimerHandler,int channelOffset)
> +void OPLSetTimerHandler(FM_OPL *OPL, OPL_TIMERHANDLER TimerHandler,
> +                        void *param)
>   {
>   	OPL->TimerHandler   = TimerHandler;
> -	OPL->TimerParam = channelOffset;
> +    OPL->TimerParam = param;
>   }
>   
>   /* ---------- YM3812 I/O interface ---------- */
> @@ -1197,6 +1202,9 @@ int OPLTimerOver(FM_OPL *OPL,int c)
>   		}
>   	}
>   	/* reload timer */
> -	if (OPL->TimerHandler) (OPL->TimerHandler)(OPL->TimerParam+c,(double)OPL->T[c]*OPL->TimerBase);
> +    if (OPL->TimerHandler) {
> +        (OPL->TimerHandler)(OPL->TimerParam, c,
> +                            (double)OPL->T[c] * OPL->TimerBase);
> +    }
>   	return OPL->status>>7;
>   }
> diff --git a/hw/audio/fmopl.h b/hw/audio/fmopl.h
> index fc9f16b58a..f4065f425c 100644
> --- a/hw/audio/fmopl.h
> +++ b/hw/audio/fmopl.h
> @@ -3,7 +3,7 @@
>   
>   #include <stdint.h>
>   
> -typedef void (*OPL_TIMERHANDLER)(int channel,double interval_Sec);
> +typedef void (*OPL_TIMERHANDLER)(void *param, int channel, double interval_Sec);
>   
>   /* !!!!! here is private section , do not access there member direct !!!!! */
>   
> @@ -87,13 +87,14 @@ typedef struct fm_opl_f {
>   	uint8_t wavesel;
>   	/* external event callback handler */
>   	OPL_TIMERHANDLER  TimerHandler;		/* TIMER handler   */
> -	int TimerParam;						/* TIMER parameter */
> +    void *TimerParam; /* TIMER parameter */

not very useful comment ;)

>   } FM_OPL;
>   
>   /* ---------- Generic interface section ---------- */
>   FM_OPL *OPLCreate(int clock, int rate);
>   void OPLDestroy(FM_OPL *OPL);
> -void OPLSetTimerHandler(FM_OPL *OPL,OPL_TIMERHANDLER TimerHandler,int channelOffset);
> +void OPLSetTimerHandler(FM_OPL *OPL, OPL_TIMERHANDLER TimerHandler,
> +                        void *param);
>   
>   int OPLWrite(FM_OPL *OPL,int a,int v);
>   unsigned char OPLRead(FM_OPL *OPL,int a);
>

Patchew 2.1-devel-b67e4c2

© 2016 - 2026 Red Hat, Inc.