From patchwork Wed Sep 27 07:26:33 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Thomas Zimmermann X-Patchwork-Id: 13400242 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 87F08E810DA for ; Wed, 27 Sep 2023 07:48:03 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:Subject:Cc :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=oIbzzRbEXXW9mNZGcYQqSuaH2lsrV54Tu8fk+clZLio=; b=01EOOkW+lAIUoJ u6S6u5uQbas/zCTGDVAJChnjX6hPD5MWLGvSDKFCfZHTCUFmimdRnT+q0Aq6tRa7gmPjaND9E4rY3 WE8HLJUWSAV164qHX25KsZPzFPtRji6nm8hXdHt0LlwUD7yLUpujp+w8t8LswZ6xhBLdigKs/iY7I nyQBEQITr+nbAonQk95ziCB4DueM3J7qGXGq2nxKe+V9FIpkc51sFYNWD+va87Egid0WwNbLFjYzF DU0t89sEuDahEZP/QzbNe9CyK9d3WOYJ2l+UUmyyBN5i6wMXk+GRyVbwzqy7x66xQ5hJdorAjY0e7 /v9kghw1xlza1OXmTGZw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qlPGj-000DkB-1s; Wed, 27 Sep 2023 07:47:37 +0000 Received: from smtp-out1.suse.de ([2001:67c:2178:6::1c]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qlPGZ-000DYZ-2g for linux-arm-kernel@lists.infradead.org; Wed, 27 Sep 2023 07:47:29 +0000 Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by smtp-out1.suse.de (Postfix) with ESMTPS id D86C221873; Wed, 27 Sep 2023 07:47:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1695800844; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=60Z1BjKIJ4+wGz82oT0iRpCCahrgRXOSIFEA+P9XjPQ=; b=Rtaz7SByafcrHUiZ6N4kbs5VxRG4isNa5QHvUiB0BBTSKedn2hPFSyJbU5cN1VZ2DCiTvn mGBg0fj9hFIUvGHg6WrKSSgoxnpTPi6Pq/c1nfGwN2e0tq7SX6saMDz7z9jvz9w48HWhYS BVLg/Cr8Hf6sv2uopQMtz8+zjGv7luA= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1695800844; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=60Z1BjKIJ4+wGz82oT0iRpCCahrgRXOSIFEA+P9XjPQ=; b=h8sBohLiXB2A1LBrQ8cgdFElh7N9h1oKS0IjzzKoTyOOEyZsIkeSOnTLxr+64RBvDVzrxq iA222wVIEFUO5EDw== Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by imap2.suse-dmz.suse.de (Postfix) with ESMTPS id 8D0B11338F; Wed, 27 Sep 2023 07:47:24 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id B5+NIQzeE2XvUQAAMHmgww (envelope-from ); Wed, 27 Sep 2023 07:47:24 +0000 From: Thomas Zimmermann To: deller@gmx.de, javierm@redhat.com, sam@ravnborg.org, arnd@arndb.de, daniel@ffwll.ch Cc: linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, linux-nvidia@lists.surfsouth.com, linux-omap@vger.kernel.org, linux-parisc@vger.kernel.org, Thomas Zimmermann Subject: [PATCH 00/46] fbdev: Init fb_ops with helpers for I/O memory Date: Wed, 27 Sep 2023 09:26:33 +0200 Message-ID: <20230927074722.6197-1-tzimmermann@suse.de> X-Mailer: git-send-email 2.42.0 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230927_004728_178354_16FCE46F X-CRM114-Status: GOOD ( 13.53 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Initialize struct fb_ops for drivers for hardware with framebuffers in I/O-memory ranges with the respective helper macros. Also select the appropriate Kconfig dependencies. The patchset is part of a larger effort to modularize the fbdev core and make it more adaptable. Most of these drivers do not initialize the fb_read, fb_write and fb_mmap callbacks in fb_ops. By leaving the callback pointers to NULL, they rely on the fbdev core to invoke the I/O-memory helpers by default. This default makes it impossible to remove the I/O-memory helpers on systems that don't need them. Setting the pointers explicitly will allow for the removal of the default. If a callback in fb_ops is unset, the operation will then be unsupported. Initializing struct fb_ops via helpers macros will also allow for a fine-grained setup, depending on Kconfig options. For example, it will be possible to leave out file I/O if FB_DEVICE has not been set. This patchset contains the majority of fbdev driver work. The updated drivers now initialize fb_ops with __FB_DEFAULT_IOMEM_OPS_RDWR, __FB_DEFAULT_IOMEM_OPS_DRAW and/or __FB_DEFAULT_IOMEM_OPS_MMAP if possible. Some drivers now use FB_DEFAULT_IOMEM_OPS, which sets all fields correctly. In Kconfig, each driver selects FB_IOMEM_FOPS to get the helpers for I/O memory. Drivers that use _RDWR, _DRAW and _MMAP unconditionally select FB_IOMEM_HELPERS, which selects the correct modules automatically. Thomas Zimmermann (46): fbdev: Provide I/O-memory helpers as module fbdev/68328fb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/amba-clcd: Initialize fb_ops to fbdev I/O-memory helpers fbdev/amifb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/arkfb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/atafb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/atyfb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/au1100fb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/cirrusfb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/cobalt-lcd: Initialize fb_ops to fbdev I/O-memory helpers fbdev/controlfb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/cyber2000fb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/dnfb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/ep93xx-fb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/gbefb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/hgafb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/hitfb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/hpfb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/i810fb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/imsttfb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/intelfb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/matroxfb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/neofb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/nvidiafb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/omapfb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/pm2fb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/pm3fb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/pvr2fb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/radeon: Initialize fb_ops to fbdev I/O-memory helpers fbdev/rivafb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/s1d13xxxfb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/s3fb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/sa1100fb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/savagefb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/sisfb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/sm501fb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/sm712fb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/stifb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/sunxvr500: Initialize fb_ops to fbdev I/O-memory helpers fbdev/tdfxfb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/tgafb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/tridentfb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/vermilionfb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/vga16fb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/viafb: Initialize fb_ops to fbdev I/O-memory helpers fbdev/vt8623fb: Initialize fb_ops to fbdev I/O-memory helpers drivers/video/fbdev/68328fb.c | 5 +- drivers/video/fbdev/Kconfig | 93 +++++++++++-------- drivers/video/fbdev/amba-clcd.c | 5 +- drivers/video/fbdev/amifb.c | 2 + drivers/video/fbdev/arkfb.c | 2 + drivers/video/fbdev/atafb.c | 2 + drivers/video/fbdev/aty/atyfb_base.c | 3 + drivers/video/fbdev/aty/radeon_base.c | 2 + drivers/video/fbdev/au1100fb.c | 8 +- drivers/video/fbdev/cirrusfb.c | 2 + drivers/video/fbdev/cobalt_lcdfb.c | 2 + drivers/video/fbdev/controlfb.c | 5 +- drivers/video/fbdev/core/Kconfig | 6 ++ drivers/video/fbdev/core/Makefile | 3 +- drivers/video/fbdev/core/fb_io_fops.c | 3 + drivers/video/fbdev/cyber2000fb.c | 2 + drivers/video/fbdev/dnfb.c | 2 + drivers/video/fbdev/ep93xx-fb.c | 5 +- drivers/video/fbdev/gbefb.c | 7 +- drivers/video/fbdev/hgafb.c | 2 + drivers/video/fbdev/hitfb.c | 4 +- drivers/video/fbdev/hpfb.c | 2 + drivers/video/fbdev/i810/i810_main.c | 2 + drivers/video/fbdev/imsttfb.c | 2 + drivers/video/fbdev/intelfb/intelfbdrv.c | 4 +- drivers/video/fbdev/matrox/matroxfb_base.c | 2 + drivers/video/fbdev/matrox/matroxfb_crtc2.c | 4 +- drivers/video/fbdev/neofb.c | 2 + drivers/video/fbdev/nvidia/nvidia.c | 2 + drivers/video/fbdev/omap2/omapfb/Kconfig | 4 +- .../video/fbdev/omap2/omapfb/omapfb-main.c | 5 +- drivers/video/fbdev/pm2fb.c | 2 + drivers/video/fbdev/pm3fb.c | 2 + drivers/video/fbdev/pvr2fb.c | 14 +-- drivers/video/fbdev/riva/fbdev.c | 2 + drivers/video/fbdev/s1d13xxxfb.c | 25 +++-- drivers/video/fbdev/s3fb.c | 2 + drivers/video/fbdev/sa1100fb.c | 5 +- drivers/video/fbdev/savage/savagefb_driver.c | 6 +- drivers/video/fbdev/sis/sis_main.c | 4 +- drivers/video/fbdev/sm501fb.c | 4 + drivers/video/fbdev/sm712fb.c | 1 + drivers/video/fbdev/stifb.c | 2 + drivers/video/fbdev/sunxvr500.c | 2 + drivers/video/fbdev/tdfxfb.c | 6 +- drivers/video/fbdev/tgafb.c | 2 + drivers/video/fbdev/tridentfb.c | 2 + drivers/video/fbdev/vermilion/vermilion.c | 5 +- drivers/video/fbdev/vga16fb.c | 2 + drivers/video/fbdev/via/viafbdev.c | 2 + drivers/video/fbdev/vt8623fb.c | 2 + 51 files changed, 185 insertions(+), 101 deletions(-) base-commit: e1973de2c4516e9130157e538014e79c8aa57b41 prerequisite-patch-id: 0aa359f6144c4015c140c8a6750be19099c676fb prerequisite-patch-id: c67e5d886a47b7d0266d81100837557fda34cb24 prerequisite-patch-id: cbc453ee02fae02af22fbfdce56ab732c7a88c36 Acked-by: Javier Martinez Canillas