Hi, >-----Original Message----- >From: linux-omap-owner@xxxxxxxxxxxxxxx [mailto:linux-omap- >owner@xxxxxxxxxxxxxxx] 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 <x0hebbar@xxxxxx> >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 <x0hebbar@xxxxxx> >--- > 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 majordomo@xxxxxxxxxxxxxxx >More majordomo info at http://vger.kernel.org/majordomo-info.html Regards, Fernando. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
