remoteproc: pru: Fix wrong success return value for fw events
authorSuman Anna <s-anna@ti.com>
Wed, 7 Apr 2021 15:56:40 +0000 (10:56 -0500)
committerBjorn Andersson <bjorn.andersson@linaro.org>
Tue, 13 Apr 2021 20:43:26 +0000 (15:43 -0500)
commit1fe72bcfac087dba5ab52778e0646ed9e145cd32
treecf349cbca5e9613d462c8536da70ecf338b4600d
parent6d1f2803cb6b414c2e45fa64d1fdad6b581e1e88
remoteproc: pru: Fix wrong success return value for fw events

The irq_create_fwspec_mapping() returns a proper virq value on success
and 0 upon any failure. The pru_handle_intrmap() treats this as an error
and disposes all firmware event mappings correctly, but is returning
this incorrect value as is, letting the pru_rproc_start() interpret it
as a success and boot the PRU.

Fix this by returning an error value back upon any such failure. While
at this, revise the error trace to print some meaningful info about the
failed event.

Fixes: c75c9fdac66e ("remoteproc: pru: Add support for PRU specific interrupt configuration")
Signed-off-by: Suman Anna <s-anna@ti.com>
Reviewed-by: Mathieu Poirier <mathieu.poirier@linaro.org>
Link: https://lore.kernel.org/r/20210407155641.5501-3-s-anna@ti.com
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
drivers/remoteproc/pru_rproc.c