Hi Philipp, On 21 August 2017 at 11:56, Philipp Zabel <p.zabel@xxxxxxxxxxxxxx> wrote: > +struct etna_bo * > +etna_bo_from_handle(struct etna_device *dev, uint32_t handle, uint32_t size) > +{ > + struct etna_bo *bo = NULL; You can drop the NULL initialisation since it's always set by the lookup_bo() call. But with that, both patches are: Reviewed-by: Daniel Stone <daniels@xxxxxxxxxxxxx> Cheers, Daniel _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel