> On Jan 25, 2021, at 03:45, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote: > > The devm_memremap() function never returns NULL, it returns error > pointers so the test needs to be fixed. Also we need to call > pci_release_regions() to avoid a memory leak. > > Fixes: be4f77ac6884 ("drm/vmwgfx: Cleanup fifo mmio handling") > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Thanks, Dan. I have a patch based on your report that fixes that too but it comes with a refactoring. I’d be happy to rebase it on top of yours just to get yours in before. If you’d like I can push your patch through drm-misc-next, if not: Reviewed-by: Zack Rusin <zackr@xxxxxxxxxx> z _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel