From nobody Thu Apr 9 08:10:18 2026 Received: from mail-24416.protonmail.ch (mail-24416.protonmail.ch [109.224.244.16]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6352C3148A6 for ; Tue, 10 Mar 2026 06:41:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=109.224.244.16 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773124912; cv=none; b=dfnOPqOSUoIKKGuVdv671rFPwTGaPZy2/yRnuT3hrD/XqH1sHMHA0Vnpe2LiJePMJprQ7CzBhMIJQjaewkPEqMI/85ZEf3g+GCYLbxjnPXoaG/6wT7XYNyaiuSoxx9G8BpM07Q6GKpHZoDk3g31DX0QOQw5T7dmBj7zkSSwrVqc= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773124912; c=relaxed/simple; bh=dYlycPl8p/x1gwyJn8ULpzMUW2NWNjW7cVDYT72HC9s=; h=Date:To:From:Cc:Subject:Message-ID:MIME-Version:Content-Type; b=SRJByA3zJbQ0quo+67X9raIeT7NjaIIXQvcEX1kuOzw/45kZbgXgxif7jgRqSUSCMkl97idlHmU0Zgva3RVO7dJNGXzCq3Be3N0MdvB+ewu2IqyQAuP3qf3IERIm4g/l2j1pWhMPYANDLGOle69gDb8iN5CtcPvQat6u0R4/1lU= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=pm.me; spf=pass smtp.mailfrom=pm.me; dkim=pass (2048-bit key) header.d=pm.me header.i=@pm.me header.b=W7xnpFmx; arc=none smtp.client-ip=109.224.244.16 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=pm.me Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=pm.me Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=pm.me header.i=@pm.me header.b="W7xnpFmx" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=pm.me; s=protonmail3; t=1773124908; x=1773384108; bh=M6MVz99apPoJz3VJsJLHJVaKn3OU4ENXUSbQXnLReTE=; h=Date:To:From:Cc:Subject:Message-ID:Feedback-ID:From:To:Cc:Date: Subject:Reply-To:Feedback-ID:Message-ID:BIMI-Selector; b=W7xnpFmxT4+cYcexITuG6Nh/H0Qz8e307MsCr52I9+6jd5XHfxyZJin2RYdtzWZrK Gt0n0J1ffxopbYw/TWDcfpyDplNEt3g23jUPnH/5/6gKm9l77sZep9nv0z+rmUZ8us tEZBuK02xgcp2S6i6aFXxc/zQ0ZPJnIm85/+thjHKSNvaWW6O6yIM7w+phvDDHcx9l vX/sCUxCdooAx7pUrvg8wkFE9JCMGFgpuc7i4p2LGmMb+R3dWTIIyIU0PUv7DtL9Sj oQP54+Ak/kQbJdmOnJkr3+N74fh+P3Q0F5xNnDl4l0QHNEJqLWxVtl9++UXv/RVq8x VMmc8lJn5zx/g== Date: Tue, 10 Mar 2026 06:41:43 +0000 To: Ferenc Bakonyi , Helge Deller From: Hardik Phalet Cc: Shuah Khan , Brigham Campbell , Thomas Zimmermann , linux-nvidia@lists.surfsouth.com, linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Hardik Phalet Subject: [PATCH] fbdev/hga: Request memory region before ioremap Message-ID: <20260310064124.602848-1-hardik.phalet@pm.me> Feedback-ID: 166659585:user:proton X-Pm-Message-ID: d4777026df3fa38f46012efaf1cb841e71c5cd25 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" The driver calls ioremap() on the HGA video memory at 0xb0000 without first reserving the physical address range via request_mem_region(). This leaves the kernel resource tree incomplete and can cause silent conflicts with other drivers claiming the same range. Add a request_mem_region() call before ioremap() in hga_card_detect() and release the region in all error paths and in hgafb_remove(). Signed-off-by: Hardik Phalet --- drivers/video/fbdev/hgafb.c | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/drivers/video/fbdev/hgafb.c b/drivers/video/fbdev/hgafb.c index 14418aa3791a..ceca6cc2c928 100644 --- a/drivers/video/fbdev/hgafb.c +++ b/drivers/video/fbdev/hgafb.c @@ -284,9 +284,16 @@ static int hga_card_detect(void) =20 hga_vram_len =3D 0x08000; =20 + if (!request_mem_region(0xb0000, hga_vram_len, "hgafb")) { + pr_err("hgafb: cannot reserve video memory at 0xb0000\n"); + return -EBUSY; + } + hga_vram =3D ioremap(0xb0000, hga_vram_len); - if (!hga_vram) + if (!hga_vram) { + release_mem_region(0xb0000, hga_vram_len); return -ENOMEM; + } =20 if (request_region(0x3b0, 12, "hgafb")) release_io_ports =3D 1; @@ -348,6 +355,7 @@ static int hga_card_detect(void) } return 0; error: + release_mem_region(0xb0000, hga_vram_len); if (release_io_ports) release_region(0x3b0, 12); if (release_io_port) @@ -619,6 +627,7 @@ static void hgafb_remove(struct platform_device *pdev) } =20 iounmap(hga_vram); + release_mem_region(0xb0000, hga_vram_len); =20 if (release_io_ports) release_region(0x3b0, 12); --=20 2.53.0