[Patch 7/8] DSPBRIDGE: typedef cleanup -BRD_STATUS

2010-02-05 Thread Hebbar, Shivananda
From 75fabc1654f4671906f64c12a348aa38625d4f28 Mon Sep 17 00:00:00 2001
From: Shivananda Hebbar x0heb...@ti.com
Date: Thu, 28 Jan 2010 20:40:48 -0600
Subject: [PATCH] DSPBRIDGE: typedef cleanup - BRD_STATUS

This patch removes the typedef BRD_STATUS and replaces
it with the normal C type. Here it need not be u32 value.
So replacing it with int type.

Signed-off-by: Shivananda Hebbar x0heb...@ti.com
---
 arch/arm/plat-omap/include/dspbridge/brddefs.h |1 -
 arch/arm/plat-omap/include/dspbridge/wmd.h |2 +-
 drivers/dsp/bridge/rmgr/proc.c |   10 +-
 drivers/dsp/bridge/wmd/tiomap3430.c|4 ++--
 4 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/arch/arm/plat-omap/include/dspbridge/brddefs.h 
b/arch/arm/plat-omap/include/dspbridge/brddefs.h
index ca503e2..ff83166 100644
--- a/arch/arm/plat-omap/include/dspbridge/brddefs.h
+++ b/arch/arm/plat-omap/include/dspbridge/brddefs.h
@@ -32,7 +32,6 @@
 #define BRD_RETENTION 0x8   /* Retention mode */
 #define BRD_DSP_HIBERNATION 0x9   /* DSP initiated hibernation */
 #define BRD_ERROR  0xA   /* Board state is Error */
-   typedef u32 BRD_STATUS;
 
 /* BRD Object */
struct BRD_OBJECT;
diff --git a/arch/arm/plat-omap/include/dspbridge/wmd.h 
b/arch/arm/plat-omap/include/dspbridge/wmd.h
index 748b1c6..70f3970 100644
--- a/arch/arm/plat-omap/include/dspbridge/wmd.h
+++ b/arch/arm/plat-omap/include/dspbridge/wmd.h
@@ -244,7 +244,7 @@
  */
typedef DSP_STATUS(*
   WMD_BRD_STATUS) (struct WMD_DEV_CONTEXT *hDevContext,
-   OUT BRD_STATUS * pdwState);
+   int *pdwState);
 
 /*
  *   WMD_BRD_Read 
diff --git a/drivers/dsp/bridge/rmgr/proc.c b/drivers/dsp/bridge/rmgr/proc.c
index 439dbe8..5a33df7 100644
--- a/drivers/dsp/bridge/rmgr/proc.c
+++ b/drivers/dsp/bridge/rmgr/proc.c
@@ -808,7 +808,7 @@ DSP_STATUS PROC_GetState(void *hProcessor,
 {
DSP_STATUS status = DSP_SOK;
struct PROC_OBJECT *pProcObject = (struct PROC_OBJECT *)hProcessor;
-   BRD_STATUS brdStatus;
+   int brdStatus;
struct DEH_MGR *hDehMgr;
 
DBC_Require(cRefs  0);
@@ -949,7 +949,7 @@ DSP_STATUS PROC_Load(void *hProcessor, IN CONST s32 iArgc,
u32 dwExtEnd;
u32 uProcId;
 #ifdef CONFIG_BRIDGE_DEBUG
-   BRD_STATUS uBrdState;
+   int uBrdState;
 #endif
 
 #ifdef OPT_LOAD_TIME_INSTRUMENTATION
@@ -1442,7 +1442,7 @@ DSP_STATUS PROC_Start(void *hProcessor)
struct COD_MANAGER *hCodMgr;/* Code manager handle*/
u32 dwDspAddr;  /* Loaded code's entry point.*/
 #ifdef CONFIG_BRIDGE_DEBUG
-   BRD_STATUS uBrdState;
+   int uBrdState;
 #endif
DBC_Require(cRefs  0);
GT_1trace(PROC_DebugMask, GT_ENTER, Entered PROC_Start, args:\n\t
@@ -1538,7 +1538,7 @@ DSP_STATUS PROC_Stop(void *hProcessor)
u32 uNodeTabSize = 1;
u32 uNumNodes = 0;
u32 uNodesAllocated = 0;
-   BRD_STATUS uBrdState;
+   int uBrdState;
 
DBC_Require(cRefs  0);
GT_1trace(PROC_DebugMask, GT_ENTER, Entered PROC_Stop, args:\n\t
@@ -1730,7 +1730,7 @@ static DSP_STATUS PROC_Monitor(struct PROC_OBJECT 
*hProcObject)
struct PROC_OBJECT *pProcObject = (struct PROC_OBJECT *)hProcObject;
struct MSG_MGR *hMsgMgr;
 #ifdef CONFIG_BRIDGE_DEBUG
-   BRD_STATUS uBrdState;
+   int uBrdState;
 #endif
 
DBC_Require(cRefs  0);
diff --git a/drivers/dsp/bridge/wmd/tiomap3430.c 
b/drivers/dsp/bridge/wmd/tiomap3430.c
index 3b9052e..a17a9ff 100644
--- a/drivers/dsp/bridge/wmd/tiomap3430.c
+++ b/drivers/dsp/bridge/wmd/tiomap3430.c
@@ -93,7 +93,7 @@ static DSP_STATUS WMD_BRD_Read(struct WMD_DEV_CONTEXT 
*pDevContext,
 static DSP_STATUS WMD_BRD_Start(struct WMD_DEV_CONTEXT *pDevContext,
u32 dwDSPAddr);
 static DSP_STATUS WMD_BRD_Status(struct WMD_DEV_CONTEXT *pDevContext,
-   OUT BRD_STATUS *pdwState);
+   int *pdwState);
 static DSP_STATUS WMD_BRD_Stop(struct WMD_DEV_CONTEXT *pDevContext);
 static DSP_STATUS WMD_BRD_Write(struct WMD_DEV_CONTEXT *pDevContext,
IN u8 *pbHostBuf,
@@ -926,7 +926,7 @@ static DSP_STATUS WMD_BRD_Delete(struct WMD_DEV_CONTEXT 
*hDevContext)
  *  Returns the board status.
  */
 static DSP_STATUS WMD_BRD_Status(struct WMD_DEV_CONTEXT *hDevContext,
-OUT BRD_STATUS *pdwState)
+int *pdwState)
 {
struct WMD_DEV_CONTEXT *pDevContext = hDevContext;
*pdwState = pDevContext-dwBrdState;
-- 
1.6.0.4

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Patch 7/8] DSPBRIDGE: typedef cleanup -BRD_STATUS

2010-02-02 Thread Hebbar, Shivananda
From 9c41e6639d93696cf92c740c026d887ab77f16a9 Mon Sep 17 00:00:00 2001
From: Shivananda Hebbar x0heb...@ti.com
Date: Thu, 28 Jan 2010 20:40:48 -0600
Subject: [PATCH] DSPBRIDGE: Remove typedef BRD_STATUS and replace it with
   normal C type

This patch removes the typedef BRD_STATUS and replaces it with the normal C 
type. Here BRD_STATUS need not be u32 type.
So replacing it with a short int type.

Signed-off-by: Shivananda Hebbar x0heb...@ti.com
---
 arch/arm/plat-omap/include/dspbridge/brddefs.h |1 -
 arch/arm/plat-omap/include/dspbridge/wmd.h |2 +-
 drivers/dsp/bridge/rmgr/proc.c |   10 +-
 drivers/dsp/bridge/wmd/tiomap3430.c|4 ++--
 4 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/arch/arm/plat-omap/include/dspbridge/brddefs.h 
b/arch/arm/plat-omap/include/dspbridge/brddefs.h
index ca503e2..ff83166 100644
--- a/arch/arm/plat-omap/include/dspbridge/brddefs.h
+++ b/arch/arm/plat-omap/include/dspbridge/brddefs.h
@@ -32,7 +32,6 @@
 #define BRD_RETENTION 0x8   /* Retention mode */
 #define BRD_DSP_HIBERNATION 0x9   /* DSP initiated hibernation */
 #define BRD_ERROR  0xA   /* Board state is Error */
-   typedef u32 BRD_STATUS;
 
 /* BRD Object */
struct BRD_OBJECT;
diff --git a/arch/arm/plat-omap/include/dspbridge/wmd.h 
b/arch/arm/plat-omap/include/dspbridge/wmd.h
index 748b1c6..70f3970 100644
--- a/arch/arm/plat-omap/include/dspbridge/wmd.h
+++ b/arch/arm/plat-omap/include/dspbridge/wmd.h
@@ -244,7 +244,7 @@
  */
typedef DSP_STATUS(*
   WMD_BRD_STATUS) (struct WMD_DEV_CONTEXT *hDevContext,
-   OUT BRD_STATUS * pdwState);
+   int *pdwState);
 
 /*
  *   WMD_BRD_Read 
diff --git a/drivers/dsp/bridge/rmgr/proc.c b/drivers/dsp/bridge/rmgr/proc.c 
index 84f4cb5..a4bd721 100644
--- a/drivers/dsp/bridge/rmgr/proc.c
+++ b/drivers/dsp/bridge/rmgr/proc.c
@@ -808,7 +808,7 @@ DSP_STATUS PROC_GetState(DSP_HPROCESSOR hProcessor,  {
DSP_STATUS status = DSP_SOK;
struct PROC_OBJECT *pProcObject = (struct PROC_OBJECT *)hProcessor;
-   BRD_STATUS brdStatus;
+   int brdStatus;
struct DEH_MGR *hDehMgr;
 
DBC_Require(cRefs  0);
@@ -949,7 +949,7 @@ DSP_STATUS PROC_Load(DSP_HPROCESSOR hProcessor, IN CONST 
s32 iArgc,
u32 dwExtEnd;
u32 uProcId;
 #ifdef CONFIG_BRIDGE_DEBUG
-   BRD_STATUS uBrdState;
+   int uBrdState;
 #endif
 
 #ifdef OPT_LOAD_TIME_INSTRUMENTATION
@@ -1442,7 +1442,7 @@ DSP_STATUS PROC_Start(DSP_HPROCESSOR hProcessor)
struct COD_MANAGER *hCodMgr;/* Code manager handle*/
u32 dwDspAddr;  /* Loaded code's entry point.*/
 #ifdef CONFIG_BRIDGE_DEBUG
-   BRD_STATUS uBrdState;
+   int uBrdState;
 #endif
DBC_Require(cRefs  0);
GT_1trace(PROC_DebugMask, GT_ENTER, Entered PROC_Start, args:\n\t
@@ -1538,7 +1538,7 @@ DSP_STATUS PROC_Stop(DSP_HPROCESSOR hProcessor)
u32 uNodeTabSize = 1;
u32 uNumNodes = 0;
u32 uNodesAllocated = 0;
-   BRD_STATUS uBrdState;
+   int uBrdState;
 
DBC_Require(cRefs  0);
GT_1trace(PROC_DebugMask, GT_ENTER, Entered PROC_Stop, args:\n\t
@@ -1730,7 +1730,7 @@ static DSP_STATUS PROC_Monitor(struct PROC_OBJECT 
*hProcObject)
struct PROC_OBJECT *pProcObject = (struct PROC_OBJECT *)hProcObject;
struct MSG_MGR *hMsgMgr;
 #ifdef CONFIG_BRIDGE_DEBUG
-   BRD_STATUS uBrdState;
+   int uBrdState;
 #endif
 
DBC_Require(cRefs  0);
diff --git a/drivers/dsp/bridge/wmd/tiomap3430.c 
b/drivers/dsp/bridge/wmd/tiomap3430.c
index 3b9052e..a17a9ff 100644
--- a/drivers/dsp/bridge/wmd/tiomap3430.c
+++ b/drivers/dsp/bridge/wmd/tiomap3430.c
@@ -93,7 +93,7 @@ static DSP_STATUS WMD_BRD_Read(struct WMD_DEV_CONTEXT 
*pDevContext,  static DSP_STATUS WMD_BRD_Start(struct WMD_DEV_CONTEXT 
*pDevContext,
u32 dwDSPAddr);
 static DSP_STATUS WMD_BRD_Status(struct WMD_DEV_CONTEXT *pDevContext,
-   OUT BRD_STATUS *pdwState);
+   int *pdwState);
 static DSP_STATUS WMD_BRD_Stop(struct WMD_DEV_CONTEXT *pDevContext);  static 
DSP_STATUS WMD_BRD_Write(struct WMD_DEV_CONTEXT *pDevContext,
IN u8 *pbHostBuf,
@@ -926,7 +926,7 @@ static DSP_STATUS WMD_BRD_Delete(struct WMD_DEV_CONTEXT 
*hDevContext)
  *  Returns the board status.
  */
 static DSP_STATUS WMD_BRD_Status(struct WMD_DEV_CONTEXT *hDevContext,
-OUT BRD_STATUS *pdwState)
+int *pdwState)
 {
struct WMD_DEV_CONTEXT *pDevContext = hDevContext;
*pdwState = pDevContext-dwBrdState;
--
1.6.0.4

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [Patch 7/8] DSPBRIDGE: typedef cleanup -BRD_STATUS

2010-02-02 Thread Guzman Lugo, Fernando


Hi,

-Original Message-
From: linux-omap-ow...@vger.kernel.org [mailto:linux-omap-
ow...@vger.kernel.org] On Behalf Of Hebbar, Shivananda
Sent: Tuesday, February 02, 2010 6:44 PM
To: linux-omap
Subject: [Patch 7/8] DSPBRIDGE: typedef cleanup -BRD_STATUS

From 9c41e6639d93696cf92c740c026d887ab77f16a9 Mon Sep 17 00:00:00 2001
From: Shivananda Hebbar x0heb...@ti.com
Date: Thu, 28 Jan 2010 20:40:48 -0600
Subject: [PATCH] DSPBRIDGE: Remove typedef BRD_STATUS and replace it with
  normal C type

This patch removes the typedef BRD_STATUS and replaces it with the normal C
type. Here BRD_STATUS need not be u32 type.
So replacing it with a short int type.

Wrong description: your description says BRD_STATUS would be replace with 
short int but it is actually replace for int



Signed-off-by: Shivananda Hebbar x0heb...@ti.com
---
 arch/arm/plat-omap/include/dspbridge/brddefs.h |1 -
 arch/arm/plat-omap/include/dspbridge/wmd.h |2 +-
 drivers/dsp/bridge/rmgr/proc.c |   10 +-
 drivers/dsp/bridge/wmd/tiomap3430.c|4 ++--
 4 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/arch/arm/plat-omap/include/dspbridge/brddefs.h
b/arch/arm/plat-omap/include/dspbridge/brddefs.h
index ca503e2..ff83166 100644
--- a/arch/arm/plat-omap/include/dspbridge/brddefs.h
+++ b/arch/arm/plat-omap/include/dspbridge/brddefs.h
@@ -32,7 +32,6 @@
 #define BRD_RETENTION 0x8   /* Retention mode */
 #define BRD_DSP_HIBERNATION 0x9   /* DSP initiated hibernation */
 #define BRD_ERROR 0xA   /* Board state is Error */
-  typedef u32 BRD_STATUS;

 /* BRD Object */
   struct BRD_OBJECT;
diff --git a/arch/arm/plat-omap/include/dspbridge/wmd.h b/arch/arm/plat-
omap/include/dspbridge/wmd.h
index 748b1c6..70f3970 100644
--- a/arch/arm/plat-omap/include/dspbridge/wmd.h
+++ b/arch/arm/plat-omap/include/dspbridge/wmd.h
@@ -244,7 +244,7 @@
  */
typedef DSP_STATUS(*
  WMD_BRD_STATUS) (struct WMD_DEV_CONTEXT *hDevContext,
-  OUT BRD_STATUS * pdwState);
+  int *pdwState);

 /*
  *   WMD_BRD_Read 
diff --git a/drivers/dsp/bridge/rmgr/proc.c
b/drivers/dsp/bridge/rmgr/proc.c index 84f4cb5..a4bd721 100644
--- a/drivers/dsp/bridge/rmgr/proc.c
+++ b/drivers/dsp/bridge/rmgr/proc.c
@@ -808,7 +808,7 @@ DSP_STATUS PROC_GetState(DSP_HPROCESSOR hProcessor,  {
   DSP_STATUS status = DSP_SOK;
   struct PROC_OBJECT *pProcObject = (struct PROC_OBJECT *)hProcessor;
-  BRD_STATUS brdStatus;
+  int brdStatus;
   struct DEH_MGR *hDehMgr;

   DBC_Require(cRefs  0);
@@ -949,7 +949,7 @@ DSP_STATUS PROC_Load(DSP_HPROCESSOR hProcessor, IN
CONST s32 iArgc,
   u32 dwExtEnd;
   u32 uProcId;
 #ifdef CONFIG_BRIDGE_DEBUG
-  BRD_STATUS uBrdState;
+  int uBrdState;
 #endif

 #ifdef OPT_LOAD_TIME_INSTRUMENTATION
@@ -1442,7 +1442,7 @@ DSP_STATUS PROC_Start(DSP_HPROCESSOR hProcessor)
   struct COD_MANAGER *hCodMgr;/* Code manager handle*/
   u32 dwDspAddr;  /* Loaded code's entry point.*/
 #ifdef CONFIG_BRIDGE_DEBUG
-  BRD_STATUS uBrdState;
+  int uBrdState;
 #endif
   DBC_Require(cRefs  0);
   GT_1trace(PROC_DebugMask, GT_ENTER, Entered PROC_Start, args:\n\t
@@ -1538,7 +1538,7 @@ DSP_STATUS PROC_Stop(DSP_HPROCESSOR hProcessor)
   u32 uNodeTabSize = 1;
   u32 uNumNodes = 0;
   u32 uNodesAllocated = 0;
-  BRD_STATUS uBrdState;
+  int uBrdState;

   DBC_Require(cRefs  0);
   GT_1trace(PROC_DebugMask, GT_ENTER, Entered PROC_Stop, args:\n\t
@@ -1730,7 +1730,7 @@ static DSP_STATUS PROC_Monitor(struct PROC_OBJECT
*hProcObject)
   struct PROC_OBJECT *pProcObject = (struct PROC_OBJECT *)hProcObject;
   struct MSG_MGR *hMsgMgr;
 #ifdef CONFIG_BRIDGE_DEBUG
-  BRD_STATUS uBrdState;
+  int uBrdState;
 #endif

   DBC_Require(cRefs  0);
diff --git a/drivers/dsp/bridge/wmd/tiomap3430.c
b/drivers/dsp/bridge/wmd/tiomap3430.c
index 3b9052e..a17a9ff 100644
--- a/drivers/dsp/bridge/wmd/tiomap3430.c
+++ b/drivers/dsp/bridge/wmd/tiomap3430.c
@@ -93,7 +93,7 @@ static DSP_STATUS WMD_BRD_Read(struct WMD_DEV_CONTEXT
*pDevContext,  static DSP_STATUS WMD_BRD_Start(struct WMD_DEV_CONTEXT
*pDevContext,
   u32 dwDSPAddr);
 static DSP_STATUS WMD_BRD_Status(struct WMD_DEV_CONTEXT *pDevContext,
-  OUT BRD_STATUS *pdwState);
+  int *pdwState);
 static DSP_STATUS WMD_BRD_Stop(struct WMD_DEV_CONTEXT *pDevContext);
static DSP_STATUS WMD_BRD_Write(struct WMD_DEV_CONTEXT *pDevContext,
   IN u8 *pbHostBuf,
@@ -926,7 +926,7 @@ static DSP_STATUS WMD_BRD_Delete(struct WMD_DEV_CONTEXT
*hDevContext)
  *  Returns the board status.
  */
 static DSP_STATUS WMD_BRD_Status(struct WMD_DEV_CONTEXT *hDevContext,
-   OUT BRD_STATUS *pdwState)
+   int