Maíra Canal <mca...@igalia.com> writes:

Hello Maíra,

> The array of BOs that are lookup at the start of exec doesn't need
> to be instantiated as drm_gem_dma_object, as it doesn't benefit
> from its attributes. So, simplify the code by replacing the array of
> drm_gem_dma_object for an array of drm_gem_object in the struct
> vc4_exec_info.
>
> Suggested-by: Melissa Wen <m...@igalia.com>
> Signed-off-by: Maíra Canal <mca...@igalia.com>
> Reviewed-by: André Almeida <andrealm...@igalia.com>
> ---

The change looks good to me and it certainly makes the code simpler.

Reviewed-by: Javier Martinez Canillas <javi...@redhat.com>

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat

Reply via email to