Hi Sam, Thanks for the review.
You are right that the cast to (void __iomem *) makes it look like memory,
but the resource is explicitly initialized as I/O ports a few lines earlier:
vga_res.flags = IORESOURCE_IO;
Since the resource flag is IORESOURCE_IO (targeting the legacy VGA ports),
I used devm_request_region() instead of devm_request_mem_region().
Best regards,
Swaraj
