staging: mmal-vchiq: If the VPU returns an error, don't negate it
authorDave Stevenson <dave.stevenson@raspberrypi.org>
Wed, 13 Feb 2019 12:51:03 +0000 (12:51 +0000)
committerpopcornmix <popcornmix@gmail.com>
Wed, 1 Jul 2020 15:32:52 +0000 (16:32 +0100)
There is an enum for the errors that the VPU can return.
port_parameter_get was negating that value, but also using -EINVAL
from the Linux error codes.
Pass the VPU error code as positive values. Should the function
need to pass a Linux failure, then return that as negative.

Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.org>
drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c

index 59a4d24..cfc861d 100644 (file)
@@ -1400,7 +1400,8 @@ static int port_parameter_get(struct vchiq_mmal_instance *instance,
                goto release_msg;
        }
 
-       ret = -rmsg->u.port_parameter_get_reply.status;
+       ret = rmsg->u.port_parameter_get_reply.status;
+
        /* port_parameter_get_reply.size includes the header,
         * whilst *value_size doesn't.
         */