On Tue, Jul 14 2026 at 21:24, Haofeng Li wrote: > - iounmap(regs); > + /* Only OF path uses of_iomap(); ACPI/platform uses devm mapping. */ Instead of this comment and conditional unmap the OF specific map side should just use devm_of_iomap(), no? > + if (is_of_node(fwnode)) > + iounmap(regs); > return error; > }