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: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Make sure dspbridge driver uses a valid processor handle by
using the handle stored in process context.
Signed-off-by: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

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: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

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: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

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: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Remove find_gcf from nldr.c and use kernel function
gcd().
Signed-off-by: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Remove bridge_brd_delete() function since it is
doing the same that bridge_brd_stop().
Signed-off-by: NErnesto Ramos <ernesto@ernesto-desktop.(none)>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

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: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

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: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

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: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Since status succeeded is now 0 macro DSP_FAILED
is not necessary anymore. This patch removes this
define.
Signed-off-by: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Since status succeeded is 0, DSP_FAILED macro
is not necessary anymore.
Signed-off-by: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Since status succeeded is 0, DSP_FAILED macro
is not necessary anymore.
Signed-off-by: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Since status succeeded is 0, DSP_FAILED macro
is not necessary anymore.
Signed-off-by: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Since status succeeded is 0, DSP_FAILED macro
is not necessary anymore.
Signed-off-by: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Since status succeeded is now 0 macro DSP_SUCCEEDED
is not necessary anymore. This patch removes this
define.
Signed-off-by: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Since status succeeded is now 0 macro DSP_SUCCEEDED
is not necessary anymore.
Signed-off-by: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Since status succeeded is now 0 macro DSP_SUCCEEDED
is not necessary anymore.
Signed-off-by: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Since status succeeded is now 0 macro DSP_SUCCEEDED
is not necessary anymore.
Signed-off-by: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Since status succeeded is now 0 macro DSP_SUCCEEDED
is not necessary anymore.
Signed-off-by: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Replace simple_strtoul by strict_strtoul in atoi function.
Signed-off-by: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Remove unnecessary volatile variables; use accessor
functions __raw_readl/__raw_writel instead when applicable.
Signed-off-by: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Remove function delete_strm_mgr in strm.c and
use kfree instead.
Signed-off-by: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Remove unnecessary check for NULL pointer in cmm.c.
Signed-off-by: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Remove unused typedef REG16 in dspbridge.
Signed-off-by: NErnesto Ramos <ernesto@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

This patch removes a couple of macros that are not
being used
Signed-off-by: NArmando Uribe <x0095078@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

This patch changes preprocesing macros to static inline
funcions. Also the function is_equal_uuid (IS_EQUAL_UUID) now uses the
memcmp function.
Signed-off-by: NArmando Uribe <x0095078@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

This patch removes those macros that are used to
perform casts
Signed-off-by: NArmando Uribe <x0095078@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

This patch removes an unused magic number from the cod_manager
structure.
Signed-off-by: NArmando Uribe <x0095078@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

This patch removes a redundant macro from cod.c simply used
to verify that a field was not NULL
Signed-off-by: NArmando Uribe <x0095078@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

This patch changes the macros to static inline function used
in cload as well adds two definition of const to avoid using magic
numbers
Signed-off-by: NArmando Uribe <x0095078@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

This patch change to macros to static inline functions, these
macros are used to perform write/read operations in shared memory
Signed-off-by: NArmando Uribe <x0095078@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

This patch removes 4 redundant macros used to perform
read/write operations in shared memory region from io_sm.c.
Signed-off-by: NArmando Uribe <x0095078@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

This patch removes redundant wrappers from io_sm.c file
Signed-off-by: NArmando Uribe <x0095078@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Remove yet another custom definition header
Signed-off-by: NNishanth Menon <nm@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Remove OUT modifier which makes no sense for linux kernel
Signed-off-by: NNishanth Menon <nm@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

OPTIONAL modifier makes no sense in linux kernel
Signed-off-by: NNishanth Menon <nm@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

IN modifier does not exist. remove it
Signed-off-by: NNishanth Menon <nm@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Signed-off-by: NNishanth Menon <nm@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>

Remove custom globaltypes.h header
Signed-off-by: NNishanth Menon <nm@ti.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>