From mboxrd@z Thu Jan 1 00:00:00 1970 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 35A63C54E64 for ; Thu, 28 Mar 2024 07:25:07 +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:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=vdW1WgDMy03adkz+sKuMESLk9EGiPdBB/TOKdaLEgQY=; b=efL7RUnt2lqALM 5T25VIw5VpfJbnQHymcra1lNmNVIvz9K5XduOHSg9W0u3UPHxZwQv9UR2a4acTshjl3CneUeSu0fN O4pVEKPbWDc8JX9Drf+U8AgMMjtxytbF+PYdNxHxGZ+FKr0tVWVbV2Foq5KipX7Xg3Wvl7CZanHB1 6HPdSw62Y2jMUbLDmxF3Dl92BltdP7yAB+87Fp7yDFqPurgVfQfBtzQTzC3H9bjgFI5DGRnVWANin taXHwuE7SmDRDS38VZGtjpxyWBBWzaR6wTdxbQXvbQU8XNVp3E9oNX00cXysa/cXlZujaB3dny79E xRkJ5QCAWkmYRqL0WTvQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rpk8I-0000000CrDT-2DOZ; Thu, 28 Mar 2024 07:25:06 +0000 Received: from mailrelay6-1.pub.mailoutpod3-cph3.one.com ([46.30.211.245]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rpk8A-0000000Cr9w-2ulD for linux-snps-arc@lists.infradead.org; Thu, 28 Mar 2024 07:25:03 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ravnborg.org; s=rsa2; h=in-reply-to:content-type:mime-version:references:message-id:subject:cc:to: from:date:from; bh=PSqQ0tUcd24FMa7LTSnOjznumgMgVETMUSxlbMzDyGI=; b=XIydy6S8dgWb8HmkJ3k6wh0/RlVPf3sfxGuBdSNfIpeWNXeiiQU+NfvXW/X9+NFT/8CUluyIj1pl+ xrwwoOUFkSlfym8kT0VFQfsy0VagKtqP4X/MgU5gKmNBVuaCDvDNqOqEYFbnQhfDL5GXlnCZ7zOOLc ACc2qPRYR2y/FggxpXGKjnAfJOLaHZSGUcN7XBMxLfcjuwdhhoZbx7t/fii5SYNl6+s+Zf54IFiMOw 1Ifmvqcxcp4yOqStqzLEFI/Zn9L0YdEqkFGxe7o5VsTDyZJXrUGfcC3o2KPGRUh4ip6cQwI1Ub2IQE VgtcDe2ontma++yvrzlfIO17dTktb5g== DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=ravnborg.org; s=ed2; h=in-reply-to:content-type:mime-version:references:message-id:subject:cc:to: from:date:from; bh=PSqQ0tUcd24FMa7LTSnOjznumgMgVETMUSxlbMzDyGI=; b=176d5uBvt7desytnCMCAugYv9YTukwo6PjTHPPtW6wVwX2G/GOqFtBLpCAxJdb/W2MOvfDlYuTnMn mQwb+ZADw== X-HalOne-ID: 135111f6-ecd4-11ee-9cc4-bd80189f112a Received: from ravnborg.org (2-105-2-98-cable.dk.customer.tdc.net [2.105.2.98]) by mailrelay6.pub.mailoutpod3-cph3.one.com (Halon) with ESMTPSA id 135111f6-ecd4-11ee-9cc4-bd80189f112a; Thu, 28 Mar 2024 07:23:31 +0000 (UTC) Date: Thu, 28 Mar 2024 08:23:29 +0100 From: Sam Ravnborg To: Thomas Zimmermann Cc: arnd@arndb.de, javierm@redhat.com, deller@gmx.de, sui.jingfeng@linux.dev, linux-arch@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-fbdev@vger.kernel.org, sparclinux@vger.kernel.org, linux-sh@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-parisc@vger.kernel.org, linux-mips@vger.kernel.org, linux-m68k@lists.linux-m68k.org, loongarch@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-snps-arc@lists.infradead.org, linux-kernel@vger.kernel.org, Vineet Gupta , Catalin Marinas , Will Deacon , Huacai Chen , WANG Xuerui , Geert Uytterhoeven , Thomas Bogendoerfer , "James E.J. Bottomley" , Michael Ellerman , Nicholas Piggin , Yoshinori Sato , Rich Felker , John Paul Adrian Glaubitz , "David S. Miller" , Andreas Larsson , Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , x86@kernel.org, "H. Peter Anvin" Subject: Re: [PATCH v2 3/3] arch: Rename fbdev header and source files Message-ID: <20240328072329.GA1573630@ravnborg.org> References: <20240327204450.14914-1-tzimmermann@suse.de> <20240327204450.14914-4-tzimmermann@suse.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20240327204450.14914-4-tzimmermann@suse.de> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240328_002459_039821_97161FD7 X-CRM114-Status: GOOD ( 36.42 ) X-BeenThere: linux-snps-arc@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux on Synopsys ARC Processors List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-snps-arc" Errors-To: linux-snps-arc-bounces+linux-snps-arc=archiver.kernel.org@lists.infradead.org Hi Thomas, On Wed, Mar 27, 2024 at 09:41:31PM +0100, Thomas Zimmermann wrote: > The per-architecture fbdev code has no dependencies on fbdev and can > be used for any video-related subsystem. Rename the files to 'video'. > Use video-sti.c on parisc as the source file depends on CONFIG_STI_CORE. > > Further update all includes statements, includ guards, and Makefiles. ^ missing 'e' > Also update a few strings and comments to refer to video instead of > fbdev. > > Signed-off-by: Thomas Zimmermann > Cc: Vineet Gupta > Cc: Catalin Marinas > Cc: Will Deacon > Cc: Huacai Chen > Cc: WANG Xuerui > Cc: Geert Uytterhoeven > Cc: Thomas Bogendoerfer > Cc: "James E.J. Bottomley" > Cc: Helge Deller > Cc: Michael Ellerman > Cc: Nicholas Piggin > Cc: Yoshinori Sato > Cc: Rich Felker > Cc: John Paul Adrian Glaubitz > Cc: "David S. Miller" > Cc: Andreas Larsson > Cc: Thomas Gleixner > Cc: Ingo Molnar > Cc: Borislav Petkov > Cc: Dave Hansen > Cc: x86@kernel.org > Cc: "H. Peter Anvin" If the patch is changed to use the Kbuild file to pick the generic variant of video.h then it is: Reviewed-by: Sam Ravnborg I also added an unrelated sparc comment, that can be addressed another time. Sam > --- > arch/arc/include/asm/fb.h | 8 -------- > arch/arc/include/asm/video.h | 8 ++++++++ > arch/arm/include/asm/fb.h | 6 ------ > arch/arm/include/asm/video.h | 6 ++++++ > arch/arm64/include/asm/fb.h | 10 ---------- > arch/arm64/include/asm/video.h | 10 ++++++++++ > arch/loongarch/include/asm/{fb.h => video.h} | 8 ++++---- > arch/m68k/include/asm/{fb.h => video.h} | 8 ++++---- > arch/mips/include/asm/{fb.h => video.h} | 12 ++++++------ > arch/parisc/include/asm/{fb.h => video.h} | 8 ++++---- > arch/parisc/video/Makefile | 2 +- > arch/parisc/video/{fbdev.c => video-sti.c} | 2 +- > arch/powerpc/include/asm/{fb.h => video.h} | 8 ++++---- > arch/powerpc/kernel/pci-common.c | 2 +- > arch/sh/include/asm/fb.h | 7 ------- > arch/sh/include/asm/video.h | 7 +++++++ > arch/sparc/include/asm/{fb.h => video.h} | 8 ++++---- > arch/sparc/video/Makefile | 2 +- > arch/sparc/video/{fbdev.c => video.c} | 4 ++-- > arch/x86/include/asm/{fb.h => video.h} | 8 ++++---- > arch/x86/video/Makefile | 2 +- > arch/x86/video/{fbdev.c => video.c} | 3 ++- > include/asm-generic/Kbuild | 2 +- > include/asm-generic/{fb.h => video.h} | 6 +++--- > include/linux/fb.h | 2 +- > 25 files changed, 75 insertions(+), 74 deletions(-) > delete mode 100644 arch/arc/include/asm/fb.h > create mode 100644 arch/arc/include/asm/video.h > delete mode 100644 arch/arm/include/asm/fb.h > create mode 100644 arch/arm/include/asm/video.h > delete mode 100644 arch/arm64/include/asm/fb.h > create mode 100644 arch/arm64/include/asm/video.h > rename arch/loongarch/include/asm/{fb.h => video.h} (86%) > rename arch/m68k/include/asm/{fb.h => video.h} (86%) > rename arch/mips/include/asm/{fb.h => video.h} (76%) > rename arch/parisc/include/asm/{fb.h => video.h} (68%) > rename arch/parisc/video/{fbdev.c => video-sti.c} (96%) > rename arch/powerpc/include/asm/{fb.h => video.h} (76%) > delete mode 100644 arch/sh/include/asm/fb.h > create mode 100644 arch/sh/include/asm/video.h > rename arch/sparc/include/asm/{fb.h => video.h} (89%) > rename arch/sparc/video/{fbdev.c => video.c} (86%) > rename arch/x86/include/asm/{fb.h => video.h} (77%) > rename arch/x86/video/{fbdev.c => video.c} (97%) > rename include/asm-generic/{fb.h => video.h} (96%) > > diff --git a/arch/arc/include/asm/fb.h b/arch/arc/include/asm/fb.h > deleted file mode 100644 > index 9c2383d29cbb9..0000000000000 > --- a/arch/arc/include/asm/fb.h > +++ /dev/null > @@ -1,8 +0,0 @@ > -/* SPDX-License-Identifier: GPL-2.0 */ > - > -#ifndef _ASM_FB_H_ > -#define _ASM_FB_H_ > - > -#include > - > -#endif /* _ASM_FB_H_ */ > diff --git a/arch/arc/include/asm/video.h b/arch/arc/include/asm/video.h > new file mode 100644 > index 0000000000000..8ff7263727fe7 > --- /dev/null > +++ b/arch/arc/include/asm/video.h > @@ -0,0 +1,8 @@ > +/* SPDX-License-Identifier: GPL-2.0 */ > + > +#ifndef _ASM_VIDEO_H_ > +#define _ASM_VIDEO_H_ > + > +#include > + > +#endif /* _ASM_VIDEO_H_ */ arch/arc/include/asm/video.h only exists to pick include/asm-generic/video.h. The simpler way to do this is to add a line to: arch/arc/include/asm/Kbuild: + generic-y += video.h While touching the file I suggest to move to the simpler way using the Kbuild file. The same goes for all the other video.h files that only picks the asm-generic variant. > diff --git a/arch/arm/include/asm/fb.h b/arch/arm/include/asm/fb.h > deleted file mode 100644 > index ce20a43c30339..0000000000000 > --- a/arch/arm/include/asm/fb.h > +++ /dev/null > @@ -1,6 +0,0 @@ > -#ifndef _ASM_FB_H_ > -#define _ASM_FB_H_ > - > -#include > - > -#endif /* _ASM_FB_H_ */ > diff --git a/arch/arm/include/asm/video.h b/arch/arm/include/asm/video.h > new file mode 100644 > index 0000000000000..f570565366e67 > --- /dev/null > +++ b/arch/arm/include/asm/video.h > @@ -0,0 +1,6 @@ > +#ifndef _ASM_VIDEO_H_ > +#define _ASM_VIDEO_H_ > + > +#include > + > +#endif /* _ASM_VIDEO_H_ */ > diff --git a/arch/arm64/include/asm/fb.h b/arch/arm64/include/asm/fb.h > deleted file mode 100644 > index 1a495d8fb2ce0..0000000000000 > --- a/arch/arm64/include/asm/fb.h > +++ /dev/null > @@ -1,10 +0,0 @@ > -/* SPDX-License-Identifier: GPL-2.0-only */ > -/* > - * Copyright (C) 2012 ARM Ltd. > - */ > -#ifndef __ASM_FB_H_ > -#define __ASM_FB_H_ > - > -#include > - > -#endif /* __ASM_FB_H_ */ > diff --git a/arch/arm64/include/asm/video.h b/arch/arm64/include/asm/video.h > new file mode 100644 > index 0000000000000..fe0e74983f4d9 > --- /dev/null > +++ b/arch/arm64/include/asm/video.h > @@ -0,0 +1,10 @@ > +/* SPDX-License-Identifier: GPL-2.0-only */ > +/* > + * Copyright (C) 2012 ARM Ltd. > + */ > +#ifndef __ASM_VIDEO_H_ > +#define __ASM_VIDEO_H_ > + > +#include > + > +#endif /* __ASM_VIDEO_H_ */ > diff --git a/arch/loongarch/include/asm/fb.h b/arch/loongarch/include/asm/video.h > similarity index 86% > rename from arch/loongarch/include/asm/fb.h > rename to arch/loongarch/include/asm/video.h > index 0b218b10a9ec3..9f76845f2d4fd 100644 > --- a/arch/loongarch/include/asm/fb.h > +++ b/arch/loongarch/include/asm/video.h > @@ -2,8 +2,8 @@ > /* > * Copyright (C) 2020-2022 Loongson Technology Corporation Limited > */ > -#ifndef _ASM_FB_H_ > -#define _ASM_FB_H_ > +#ifndef _ASM_VIDEO_H_ > +#define _ASM_VIDEO_H_ > > #include > #include > @@ -26,6 +26,6 @@ static inline void fb_memset_io(volatile void __iomem *addr, int c, size_t n) > } > #define fb_memset fb_memset_io > > -#include > +#include > > -#endif /* _ASM_FB_H_ */ > +#endif /* _ASM_VIDEO_H_ */ > diff --git a/arch/m68k/include/asm/fb.h b/arch/m68k/include/asm/video.h > similarity index 86% > rename from arch/m68k/include/asm/fb.h > rename to arch/m68k/include/asm/video.h > index 9941b7434b696..6cf2194c413d8 100644 > --- a/arch/m68k/include/asm/fb.h > +++ b/arch/m68k/include/asm/video.h > @@ -1,6 +1,6 @@ > /* SPDX-License-Identifier: GPL-2.0 */ > -#ifndef _ASM_FB_H_ > -#define _ASM_FB_H_ > +#ifndef _ASM_VIDEO_H_ > +#define _ASM_VIDEO_H_ > > #include > #include > @@ -27,6 +27,6 @@ static inline pgprot_t pgprot_framebuffer(pgprot_t prot, > } > #define pgprot_framebuffer pgprot_framebuffer > > -#include > +#include > > -#endif /* _ASM_FB_H_ */ > +#endif /* _ASM_VIDEO_H_ */ > diff --git a/arch/mips/include/asm/fb.h b/arch/mips/include/asm/video.h > similarity index 76% > rename from arch/mips/include/asm/fb.h > rename to arch/mips/include/asm/video.h > index d98d6681d64ec..007c106d980fd 100644 > --- a/arch/mips/include/asm/fb.h > +++ b/arch/mips/include/asm/video.h > @@ -1,5 +1,5 @@ > -#ifndef _ASM_FB_H_ > -#define _ASM_FB_H_ > +#ifndef _ASM_VIDEO_H_ > +#define _ASM_VIDEO_H_ > > #include > > @@ -13,8 +13,8 @@ static inline pgprot_t pgprot_framebuffer(pgprot_t prot, > > /* > * MIPS doesn't define __raw_ I/O macros, so the helpers > - * in don't generate fb_readq() and > - * fb_write(). We have to provide them here. > + * in don't generate fb_readq() and > + * fb_writeq(). We have to provide them here. > * > * TODO: Convert MIPS to generic I/O. The helpers below can > * then be removed. > @@ -33,6 +33,6 @@ static inline void fb_writeq(u64 b, volatile void __iomem *addr) > #define fb_writeq fb_writeq > #endif > > -#include > +#include > > -#endif /* _ASM_FB_H_ */ > +#endif /* _ASM_VIDEO_H_ */ > diff --git a/arch/parisc/include/asm/fb.h b/arch/parisc/include/asm/video.h > similarity index 68% > rename from arch/parisc/include/asm/fb.h > rename to arch/parisc/include/asm/video.h > index ed2a195a3e762..c5dff3223194a 100644 > --- a/arch/parisc/include/asm/fb.h > +++ b/arch/parisc/include/asm/video.h > @@ -1,6 +1,6 @@ > /* SPDX-License-Identifier: GPL-2.0 */ > -#ifndef _ASM_FB_H_ > -#define _ASM_FB_H_ > +#ifndef _ASM_VIDEO_H_ > +#define _ASM_VIDEO_H_ > > #include > > @@ -11,6 +11,6 @@ bool video_is_primary_device(struct device *dev); > #define video_is_primary_device video_is_primary_device > #endif > > -#include > +#include > > -#endif /* _ASM_FB_H_ */ > +#endif /* _ASM_VIDEO_H_ */ > diff --git a/arch/parisc/video/Makefile b/arch/parisc/video/Makefile > index 16a73cce46612..b5db5b42880f8 100644 > --- a/arch/parisc/video/Makefile > +++ b/arch/parisc/video/Makefile > @@ -1,3 +1,3 @@ > # SPDX-License-Identifier: GPL-2.0-only > > -obj-$(CONFIG_STI_CORE) += fbdev.o > +obj-$(CONFIG_STI_CORE) += video-sti.o > diff --git a/arch/parisc/video/fbdev.c b/arch/parisc/video/video-sti.c > similarity index 96% > rename from arch/parisc/video/fbdev.c > rename to arch/parisc/video/video-sti.c > index 540fa0c919d59..564661e87093c 100644 > --- a/arch/parisc/video/fbdev.c > +++ b/arch/parisc/video/video-sti.c > @@ -9,7 +9,7 @@ > > #include