This is not an error code, however the way it is coded it doesn't expects a success (zero) either, return a positive number on this case given that this is not an error code. Signed-off-by: Omar Ramirez Luna <omar.ramirez@xxxxxx> --- arch/arm/plat-omap/include/dspbridge/dbdcd.h | 2 +- drivers/dsp/bridge/rmgr/dbdcd.c | 6 +++++- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/arch/arm/plat-omap/include/dspbridge/dbdcd.h b/arch/arm/plat-omap/include/dspbridge/dbdcd.h index 3bf3b4d..f7e4e79 100644 --- a/arch/arm/plat-omap/include/dspbridge/dbdcd.h +++ b/arch/arm/plat-omap/include/dspbridge/dbdcd.h @@ -123,7 +123,7 @@ extern dsp_status dcd_destroy_manager(IN struct dcd_manager *hdcd_mgr); * Returns: * 0: Success. * -EPERM: Unable to enumerate through the DCD database. - * DSP_SENUMCOMPLETE: Enumeration completed. This is not an error code. + * ENODATA: Enumeration completed. This is not an error code. * Requires: * DCD initialized. * uuid_obj is a valid pointer. diff --git a/drivers/dsp/bridge/rmgr/dbdcd.c b/drivers/dsp/bridge/rmgr/dbdcd.c index e0417a2..1731334 100644 --- a/drivers/dsp/bridge/rmgr/dbdcd.c +++ b/drivers/dsp/bridge/rmgr/dbdcd.c @@ -286,7 +286,11 @@ dsp_status dcd_enumerate_object(IN s32 cIndex, IN enum dsp_dcdobjtype obj_type, /* At the end of enumeration. Reset enum_refs. */ enum_refs = 0; - status = DSP_SENUMCOMPLETE; + /* + * TODO: Revisit, this is not an errror case but code + * expects non-zero value. + */ + status = ENODATA; } else { status = -EPERM; } -- 1.7.1 -- 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