* [PATCH] DSPBRIDGE: Fix declaration and initialization of sync objects.
@ 2010-04-19 22:51 Guzman Lugo, Fernando
2010-04-19 22:58 ` Deepak Chitriki
0 siblings, 1 reply; 3+ messages in thread
From: Guzman Lugo, Fernando @ 2010-04-19 22:51 UTC (permalink / raw)
To: linux-omap
Cc: Hiroshi Doyu, Ameya Palande, Ramirez Luna, Omar, Felipe Contreras
>From 2edd80839e6d3bbe6e92914c3097a6dedca23784 Mon Sep 17 00:00:00 2001
From: Fernando Guzman Lugo <x0095840@ti.com>
Date: Sun, 18 Apr 2010 00:32:01 -0500
Subject: [PATCH] DSPBRIDGE: Fix declaration and initialization of sync objects.
This patch fixes declaration of completions for sync in
drv_interface and also initialize sync->done object in
msg_sm.c
Signed-off-by: Fernando Guzman Lugo <x0095840@ti.com>
---
drivers/dsp/bridge/rmgr/drv_interface.c | 4 ++--
drivers/dsp/bridge/wmd/msg_sm.c | 6 +++---
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/dsp/bridge/rmgr/drv_interface.c b/drivers/dsp/bridge/rmgr/drv_interface.c
index d62e508..1dd635e 100644
--- a/drivers/dsp/bridge/rmgr/drv_interface.c
+++ b/drivers/dsp/bridge/rmgr/drv_interface.c
@@ -98,8 +98,8 @@ static int tc_wordswapon; /* Default value is always false */
static atomic_t bridge_cref; /* number of bridge open handles */
static struct workqueue_struct *bridge_rec_queue;
static struct work_struct bridge_recovery_work;
-static DECLARE_COMPLETION_ONSTACK(bridge_comp);
-static DECLARE_COMPLETION_ONSTACK(bridge_open_comp);
+static DECLARE_COMPLETION(bridge_comp);
+static DECLARE_COMPLETION(bridge_open_comp);
static bool recover;
#endif
diff --git a/drivers/dsp/bridge/wmd/msg_sm.c b/drivers/dsp/bridge/wmd/msg_sm.c
index 19e3cee..6f00071 100644
--- a/drivers/dsp/bridge/wmd/msg_sm.c
+++ b/drivers/dsp/bridge/wmd/msg_sm.c
@@ -185,8 +185,8 @@ dsp_status bridge_msg_create_queue(struct msg_mgr *hmsg_mgr,
if (DSP_SUCCEEDED(status)) {
msg_q->sync_done = kzalloc(sizeof(struct sync_object),
GFP_KERNEL);
- if (msg_q->sync_event)
- sync_init_event(msg_q->sync_event);
+ if (msg_q->sync_done)
+ sync_init_event(msg_q->sync_done);
else
status = DSP_EMEMORY;
}
@@ -194,7 +194,7 @@ dsp_status bridge_msg_create_queue(struct msg_mgr *hmsg_mgr,
if (DSP_SUCCEEDED(status)) {
msg_q->sync_done_ack = kzalloc(sizeof(struct sync_object),
GFP_KERNEL);
- if (msg_q->sync_event)
+ if (msg_q->sync_done_ack)
sync_init_event(msg_q->sync_done_ack);
else
status = DSP_EMEMORY;
--
1.6.0.4
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] DSPBRIDGE: Fix declaration and initialization of sync objects.
2010-04-19 22:51 [PATCH] DSPBRIDGE: Fix declaration and initialization of sync objects Guzman Lugo, Fernando
@ 2010-04-19 22:58 ` Deepak Chitriki
2010-04-21 19:50 ` Omar Ramirez Luna
0 siblings, 1 reply; 3+ messages in thread
From: Deepak Chitriki @ 2010-04-19 22:58 UTC (permalink / raw)
To: Guzman Lugo, Fernando
Cc: linux-omap, Hiroshi Doyu, Ameya Palande, Ramirez Luna, Omar,
Felipe Contreras
Guzman Lugo, Fernando wrote:
> From 2edd80839e6d3bbe6e92914c3097a6dedca23784 Mon Sep 17 00:00:00 2001
> From: Fernando Guzman Lugo <x0095840@ti.com>
> Date: Sun, 18 Apr 2010 00:32:01 -0500
> Subject: [PATCH] DSPBRIDGE: Fix declaration and initialization of sync objects.
>
> This patch fixes declaration of completions for sync in
> drv_interface and also initialize sync->done object in
> msg_sm.c
>
> Signed-off-by: Fernando Guzman Lugo <x0095840@ti.com>
> ---
> drivers/dsp/bridge/rmgr/drv_interface.c | 4 ++--
> drivers/dsp/bridge/wmd/msg_sm.c | 6 +++---
> 2 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/dsp/bridge/rmgr/drv_interface.c b/drivers/dsp/bridge/rmgr/drv_interface.c
> index d62e508..1dd635e 100644
> --- a/drivers/dsp/bridge/rmgr/drv_interface.c
> +++ b/drivers/dsp/bridge/rmgr/drv_interface.c
> @@ -98,8 +98,8 @@ static int tc_wordswapon; /* Default value is always false */
> static atomic_t bridge_cref; /* number of bridge open handles */
> static struct workqueue_struct *bridge_rec_queue;
> static struct work_struct bridge_recovery_work;
> -static DECLARE_COMPLETION_ONSTACK(bridge_comp);
> -static DECLARE_COMPLETION_ONSTACK(bridge_open_comp);
> +static DECLARE_COMPLETION(bridge_comp);
> +static DECLARE_COMPLETION(bridge_open_comp);
> static bool recover;
> #endif
>
> diff --git a/drivers/dsp/bridge/wmd/msg_sm.c b/drivers/dsp/bridge/wmd/msg_sm.c
> index 19e3cee..6f00071 100644
> --- a/drivers/dsp/bridge/wmd/msg_sm.c
> +++ b/drivers/dsp/bridge/wmd/msg_sm.c
> @@ -185,8 +185,8 @@ dsp_status bridge_msg_create_queue(struct msg_mgr *hmsg_mgr,
> if (DSP_SUCCEEDED(status)) {
> msg_q->sync_done = kzalloc(sizeof(struct sync_object),
> GFP_KERNEL);
> - if (msg_q->sync_event)
> - sync_init_event(msg_q->sync_event);
> + if (msg_q->sync_done)
> + sync_init_event(msg_q->sync_done);
> else
> status = DSP_EMEMORY;
> }
> @@ -194,7 +194,7 @@ dsp_status bridge_msg_create_queue(struct msg_mgr *hmsg_mgr,
> if (DSP_SUCCEEDED(status)) {
> msg_q->sync_done_ack = kzalloc(sizeof(struct sync_object),
> GFP_KERNEL);
> - if (msg_q->sync_event)
> + if (msg_q->sync_done_ack)
> sync_init_event(msg_q->sync_done_ack);
> else
> status = DSP_EMEMORY;
>
Acked-by: Deepak Chitriki <deepak.chitriki@ti.com>
Deepak
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] DSPBRIDGE: Fix declaration and initialization of sync objects.
2010-04-19 22:58 ` Deepak Chitriki
@ 2010-04-21 19:50 ` Omar Ramirez Luna
0 siblings, 0 replies; 3+ messages in thread
From: Omar Ramirez Luna @ 2010-04-21 19:50 UTC (permalink / raw)
To: Chitriki Rudramuni, Deepak
Cc: Guzman Lugo, Fernando, linux-omap, Hiroshi Doyu, Ameya Palande,
Felipe Contreras
On 4/19/2010 5:58 PM, Chitriki Rudramuni, Deepak wrote:
> Guzman Lugo, Fernando wrote:
>> From 2edd80839e6d3bbe6e92914c3097a6dedca23784 Mon Sep 17 00:00:00 2001
>> From: Fernando Guzman Lugo<x0095840@ti.com>
>> Date: Sun, 18 Apr 2010 00:32:01 -0500
>> Subject: [PATCH] DSPBRIDGE: Fix declaration and initialization of sync objects.
>>
>> This patch fixes declaration of completions for sync in
>> drv_interface and also initialize sync->done object in
>> msg_sm.c
>>
>> Signed-off-by: Fernando Guzman Lugo<x0095840@ti.com>
[...]
> Acked-by: Deepak Chitriki<deepak.chitriki@ti.com>
>
> Deepak
Pushed to dspbridge
- omar
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-04-21 19:50 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-04-19 22:51 [PATCH] DSPBRIDGE: Fix declaration and initialization of sync objects Guzman Lugo, Fernando
2010-04-19 22:58 ` Deepak Chitriki
2010-04-21 19:50 ` Omar Ramirez Luna
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox