[PATCH] staging: tidspbridge: remove code referred by OPT_ZERO_COPY_LOADER

Remove code referred by OPT_ZERO_COPY_LOADER since it is not used. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/dynload/cload.c | 60 +-- 1 files changed, 20 insertions(+), 40 deletions(-) diff --git a/drivers/staging/tidspbridge/dynload/cload.c b

[PATCH] staging: tidspbridge: disable all peripherals at bridge_brd_stop

DSP Bridge needs to disable the peripheral clocks when switches to BRD_STOPPED since that would prevent the domain to enter in OFF state. Signed-off-by: Ernesto Ramos Signed-off-by: Omar Ramirez Luna --- drivers/staging/tidspbridge/core/tiomap3430.c |4 ++-- 1 files changed, 2 insertions

[PATCH] staging:ti dspbridge: Replace find_lcm with lcm kernel func

Resendig this patch since it was missed in the last merge... Remove find_lcm within nldr.c and use standard kernel function lcm(). Signed-off-by: Ernesto Ramos Signed-off-by: Omar Ramirez Luna --- drivers/staging/tidspbridge/rmgr/nldr.c | 17 ++--- 1 files changed, 2 insertions

[PATCH] staging:ti dspbridge: remove cmm_xlator_delete wrapper

Resending this patch since it was missed in the last merge... Remove unnecessary cmm_xlator_delete function and use kfree() kernel function directly. Signed-off-by: Ernesto Ramos Signed-off-by: Omar Ramirez Luna --- .../staging/tidspbridge/include/dspbridge/cmm.h| 19

[PATCH] staging: tidspbridge: disable all peripherals at bridge_brd_stop

DSP Bridge needs to disable the peripheral clocks when switches to BRD_STOPPED since that would prevent the domain to enter in OFF state. Signed-off-by: Ernesto Ramos Signed-off-by: Omar Ramirez Luna --- drivers/staging/tidspbridge/core/tiomap3430.c |5 +++-- 1 files changed, 3 insertions

[PATCH] staging:ti dspbridge: use processor handle from context instead of user's

Make sure dspbridge driver uses a valid processor handle by using the handle stored in process context. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/pmgr/dspapi.c | 50 + 1 files changed, 29 insertions(+), 21 deletions(-) diff --git a/drivers

[PATCH] staging:ti dspbridge: use node id instead of kernel address

Use idr kernel library to send/receive node ids to the user instead of kernel address. This id will be use to access the node handles at the kernel side, if id does not match to any handle error -EFAULT is returned. Signed-off-by: Ernesto Ramos --- .../staging/tidspbridge/include/dspbridge

[PATCH] staging:ti dspbridge: avoid errors if stream id is zero

As 'zero' can be a perfectly good id, it can be picked up as a NULL from userspace, avoid issues in API and user apps if stream handle is zero. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/pmgr/dspapi.c |8 ++-- 1 files changed, 6 insertions(+), 2 deletion

[PATCH] staging:ti dspbridge: use stream id instead of kernel address

Send stream ids to the user instead of handles, then when the id is coming from user dspbridge can retrive the handle using id and avoid using invalid handles. Signed-off-by: Ernesto Ramos --- .../staging/tidspbridge/include/dspbridge/drv.h|5 +- .../include/dspbridge/resourcecleanup.h

[PATCH] staging:ti dspbridge: avoid errors if node handle is zero

As 'zero' can be a perfectly good id, it can be picked up as a NULL from userspace, avoid issues in API and user apps if node handle is zero. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/pmgr/dspapi.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) di

[PATCH 0/5] use idr kernel library instead of handles

will make sure the handle is valid by using the handle stored in process context. Ernesto Ramos (5): staging:ti dspbridge: use node id instead of kernel address staging:ti dspbridge: avoid errors if node handle is zero staging:ti dspbridge: use processor handle from context instead of

[PATCH] staging:ti dspbridge: avoid possible NULL dereference panic

When dsp_notifications array is received from user, dspbridge verifies the array has valid pointers and dsp_notification structures. However, these structures contain pointers that need to be checked for valid handles. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/pmgr/dspapi.c

[PATCH] staging:ti dspbridge: remove find_gcf from nldr.c

Remove find_gcf from nldr.c and use kernel function gcd(). Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/rmgr/nldr.c | 20 ++-- 1 files changed, 2 insertions(+), 18 deletions(-) diff --git a/drivers/staging/tidspbridge/rmgr/nldr.c b/drivers/staging/tidspbridge

[PATCH 2/4] staging:ti dspbridge: proc_load/start should set IVA2 to OFF in case of failure

When a base image is being loaded or started and by some reason the process fails, the IVA2 should be switched OFF. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/rmgr/proc.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/staging/tidspbridge

[PATCH 1/4] staging:ti dspbridge: fix bridge_brd_stop so IVA2 is set OFF

right now, bridge_brd_stop is not changing the IVA2 power state to OFF since PM_PWSTST_IVA2 is not 0 after calling this function. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/core/tiomap3430.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a

[PATCH 4/4] staging:ti dspbridge: remove bridge_brd_delete function

From: Ernesto Ramos Remove bridge_brd_delete() function since it is doing the same that bridge_brd_stop(). Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/core/tiomap3430.c | 57 + 1 files changed, 1 insertions(+), 56 deletions(-) diff --git a/drivers

[PATCH 3/4] staging:ti dspbridge: make sure IVA2 is OFF when dev is created

Regardless the IVA2 power state before bridgedriver is installed, the driver must ensure that IVA2 power state is OFF when the device driver is created. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/core/tiomap3430.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions

[PATCH 0/4] staging:ti dspbridge: Fix IVA2 power OFF series

This series of patches fixes issues that prevent IVA2 to turn OFF when there are issues to load the base image and makes sure IVA2 switched OFF when dsp bridge driver is installed. Ernesto Ramos (4): staging:ti dspbridge: fix bridge_brd_stop so IVA2 is set OFF staging:ti dspbridge: proc_load

[PATCH 10/10] staging:ti dspbridge: remove DSP_FAILED macro definition

Since status succeeded is now 0 macro DSP_FAILED is not necessary anymore. This patch removes this define. Signed-off-by: Ernesto Ramos --- .../staging/tidspbridge/include/dspbridge/dbdefs.h |3 --- 1 files changed, 0 insertions(+), 3 deletions(-) diff --git a/drivers/staging/tidspbridge

[PATCH 02/10] staging:ti dspbridge: remove DSP_SUCCEEDED macro from pmgr

Since status succeeded is now 0 macro DSP_SUCCEEDED is not necessary anymore. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/pmgr/chnl.c |8 +- drivers/staging/tidspbridge/pmgr/cmm.c| 135 + drivers/staging/tidspbridge/pmgr/cod.c| 10

[PATCH 07/10] staging:ti dspbridge: remove DSP_FAILED macro from pmgr

Since status succeeded is 0, DSP_FAILED macro is not necessary anymore. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/pmgr/chnl.c |2 +- drivers/staging/tidspbridge/pmgr/cmm.c|2 +- drivers/staging/tidspbridge/pmgr/cod.c|8 +++--- drivers/staging/tidspbridge

[PATCH 05/10] staging:ti dspbridge: remove DSP_SUCCEEDED macro definition

Since status succeeded is now 0 macro DSP_SUCCEEDED is not necessary anymore. This patch removes this define. Signed-off-by: Ernesto Ramos --- .../staging/tidspbridge/include/dspbridge/dbdefs.h |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers/staging

[PATCH 08/10] staging:ti dspbridge: remove DSP_FAILED macro from rmgr

Since status succeeded is 0, DSP_FAILED macro is not necessary anymore. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/rmgr/dbdcd.c | 24 drivers/staging/tidspbridge/rmgr/disp.c | 28 drivers/staging/tidspbridge/rmgr/drv.c |8

[PATCH 01/10] staging:ti dspbridge: remove DSP_SUCCEEDED macro from core

Since status succeeded is now 0 macro DSP_SUCCEEDED is not necessary anymore. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/core/chnl_sm.c| 22 drivers/staging/tidspbridge/core/dsp-clock.c |4 +- drivers/staging/tidspbridge/core/io_sm.c | 47

[PATCH 06/10] staging:ti dspbridge: remove DSP_FAILED macro from core

Since status succeeded is 0, DSP_FAILED macro is not necessary anymore. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/core/chnl_sm.c| 20 +++--- drivers/staging/tidspbridge/core/io_sm.c | 64 ++-- drivers/staging/tidspbridge/core/msg_sm.c

[PATCH 09/10] staging:ti dspbridge: remove DSP_FAILED macro from services

Since status succeeded is 0, DSP_FAILED macro is not necessary anymore. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/services/cfg.c | 13 ++--- 1 files changed, 6 insertions(+), 7 deletions(-) diff --git a/drivers/staging/tidspbridge/services/cfg.c b/drivers/staging

[PATCH 00/10] staging:ti dspbridge: Remove DSP_FAILED and DSP_SUCCEEDED macros

This series of patches is targetted to remove macros DSP_FAILED and DSP_SUCCEEDED since bridge driver currently uses standard kernel error codes. Ernesto Ramos (10): staging:ti dspbridge: remove DSP_SUCCEEDED macro from core staging:ti dspbridge: remove DSP_SUCCEEDED macro from pmgr

[PATCH 04/10] staging:ti dspbridge: remove DSP_SUCCEEDED macro from services

Since status succeeded is now 0 macro DSP_SUCCEEDED is not necessary anymore. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/services/cfg.c | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/tidspbridge/services/cfg.c b/drivers

[PATCH 5/5] staging:ti dspbridge: replace simple_strtoul by strict_strtoul

Replace simple_strtoul by strict_strtoul in atoi function. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/rmgr/dbdcd.c |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/drivers/staging/tidspbridge/rmgr/dbdcd.c b/drivers/staging/tidspbridge/rmgr/dbdcd.c

[PATCH 3/5] staging:ti dspbridge: remove function delete_strm_mgr

Remove function delete_strm_mgr in strm.c and use kfree instead. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/rmgr/strm.c | 16 ++-- 1 files changed, 2 insertions(+), 14 deletions(-) diff --git a/drivers/staging/tidspbridge/rmgr/strm.c b/drivers/staging

[PATCH 2/5] staging:ti dspbridge: remove unnecessary check for NULL pointer in cmm.c

Remove unnecessary check for NULL pointer in cmm.c. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/pmgr/cmm.c |8 ++-- 1 files changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/staging/tidspbridge/pmgr/cmm.c b/drivers/staging/tidspbridge/pmgr/cmm.c index

[PATCH 4/5] staging:ti dspbridge: remove unnecessary volatile variables

Remove unnecessary volatile variables; use accessor functions __raw_readl/__raw_writel instead when applicable. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/core/tiomap3430.c |8 drivers/staging/tidspbridge/dynload/tramp.c |4 ++-- drivers/staging/tidspbridge

[PATCH 1/5] staging:ti dspbridge: remove unused typedef REG16

Remove unused typedef REG16 in dspbridge. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/core/_tiomap.h |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/drivers/staging/tidspbridge/core/_tiomap.h b/drivers/staging/tidspbridge/core/_tiomap.h index b754e49

[PATCH 0/5] staging:ti dspbridge: Checkpatch errors cleanup series

These patches are targetted to remove checkpatch errors within dspbridge driver. Ernesto Ramos (5): staging:ti dspbridge: remove unused typedef REG16 staging:ti dspbridge: remove unnecessary check for NULL pointer in cmm.c staging:ti dspbridge: remove function delete_strm_mgr

[PATCH] staging:ti dspbridge: avoid possible NULL dereference panic

When dsp_notifications array is received from user, dspbridge verifies the array has valid pointers and dsp_notification structures. However, these structures contain pointers that need to be checked for valid handles. Signed-off-by: Ernesto Ramos --- drivers/staging/tidspbridge/pmgr/dspapi.c

[PATCH 2/5] DSPBRIDGE: Remove unnecessary check for NULL pointer in cmm.c

Remove unnecessary check for NULL pointer in cmm.c. Signed-off-by: Ernesto Ramos --- drivers/dsp/bridge/pmgr/cmm.c |5 + 1 files changed, 1 insertions(+), 4 deletions(-) diff --git a/drivers/dsp/bridge/pmgr/cmm.c b/drivers/dsp/bridge/pmgr/cmm.c index 7aa4ca4..1134193 100644 --- a

[PATCH 3/5] DSPBRIDGE: Remove function delete_strm_mgr

Remove function delete_strm_mgr in strm.c and use kfree instead. Signed-off-by: Ernesto Ramos --- drivers/dsp/bridge/rmgr/strm.c | 16 ++-- 1 files changed, 2 insertions(+), 14 deletions(-) diff --git a/drivers/dsp/bridge/rmgr/strm.c b/drivers/dsp/bridge/rmgr/strm.c index e537ee8

[PATCH 1/5] DSPBRIDGE: Remove unused typedefs.

Remove unused typedefs in dspbridge. Signed-off-by: Ernesto Ramos --- arch/arm/plat-omap/include/dspbridge/dbtype.h |1 - drivers/dsp/bridge/core/_tiomap.h |2 -- 2 files changed, 0 insertions(+), 3 deletions(-) diff --git a/arch/arm/plat-omap/include/dspbridge/dbtype.h b

[PATCH 5/5] DSPBRIDGE: Replace simple_strtoul by strict_strtoul

Replace simple_strtoul by strict_strtoul in atoi function. Signed-off-by: Ernesto Ramos --- drivers/dsp/bridge/rmgr/dbdcd.c |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/drivers/dsp/bridge/rmgr/dbdcd.c b/drivers/dsp/bridge/rmgr/dbdcd.c index e014600..ae85dc2

[PATCH 4/5] DSPBRIDGE: Remove unnecessary volatile variables

Remove unnecessary volatile variables; use accessor functions __raw_readl/__raw_writel instead when applicable. Signed-off-by: Ernesto Ramos --- drivers/dsp/bridge/core/tiomap3430.c |8 drivers/dsp/bridge/dynload/tramp.c |4 ++-- drivers/dsp/bridge/pmgr/cmm.c|7

[PATCHv3] DSPBRIDGE: Include missing info in MMU Fault debugging trace

match low addresses in the hash table and are not useful to print. Any address below DYNEXTBASE is not valid for the DSP. Signed-off-by: Ernesto Ramos --- drivers/dsp/bridge/core/io_sm.c | 72 +-- 1 files changed, 54 insertions(+), 18 deletions(-) diff --git a

[PATCH 1/5v3] DSPBRIDGE: Use node id instead of kernel address

Use idr kernel library to send/receive node ids to the user instead of kernel address. This id will be use to access the node handles at the kernel side, if id does not match to any handle DSP error DSP_EHANDlE is returned. Signed-off-by: Ernesto Ramos --- arch/arm/plat-omap/include/dspbridge

[PATCH 5/5v3] DSPBRIDGE: Avoid errors if stream id is zero

As 'zero' can be a perfectly good id, it can be picked up as a NULL from userspace, avoid issues in API and user apps if stream handle is zero. Signed-off-by: Ernesto Ramos --- drivers/dsp/bridge/pmgr/wcd.c |8 ++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git

[PATCH 3/5v3] DSPBRIDGE: Use processor handle from context instead of user's.

Make sure dspbridge driver uses a valid processor handle by using the handle stored in process context. Signed-off-by: Ernesto Ramos --- drivers/dsp/bridge/pmgr/wcd.c | 56 - 1 files changed, 33 insertions(+), 23 deletions(-) diff --git a/drivers/dsp

[PATCH 4/5v3] DSPBRIDGE: Use stream id instead of kernel address.

Send stream ids to the user instead of handles, then when the id is coming from user dspbridge can retrive the handle using id and avoid using invalid handles. Signed-off-by: Ernesto Ramos --- arch/arm/plat-omap/include/dspbridge/drv.h |5 +- .../plat-omap/include/dspbridge

[PATCH 2/5v3] DSPBRIDGE: Avoid errors if node handle is zero

As 'zero' can be a perfectly good id, it can be picked up as a NULL from userspace, avoid issues in API and user apps if node handle is zero. Signed-off-by: Ernesto Ramos --- drivers/dsp/bridge/pmgr/wcd.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/d