On Fri, Aug 22, 2025 at 10:41:00AM +0800, Zhang Heng wrote: > Remove the redundant return Missing period. ... > iomap_err: > kfree(dd); > pci_set_drvdata(pdev, NULL); I'm wondering if this also being redundant. I know about some corner cases, so is this one of them, or can it be removed as well? > - return rv; > done: > return rv; -- With Best Regards, Andy Shevchenko