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 kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 939A8C678D4 for ; Fri, 3 Mar 2023 12:41:19 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 02E3E6B0072; Fri, 3 Mar 2023 07:41:19 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id EF9456B0073; Fri, 3 Mar 2023 07:41:18 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id D73246B0074; Fri, 3 Mar 2023 07:41:18 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id C37F66B0072 for ; Fri, 3 Mar 2023 07:41:18 -0500 (EST) Received: from smtpin24.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay08.hostedemail.com (Postfix) with ESMTP id 8FD28141263 for ; Fri, 3 Mar 2023 12:41:18 +0000 (UTC) X-FDA: 80527547436.24.81C4A3E Received: from out4-smtp.messagingengine.com (out4-smtp.messagingengine.com [66.111.4.28]) by imf11.hostedemail.com (Postfix) with ESMTP id 8866D4000C for ; Fri, 3 Mar 2023 12:41:15 +0000 (UTC) Authentication-Results: imf11.hostedemail.com; dkim=pass header.d=arndb.de header.s=fm1 header.b=hhOAXcZA; dkim=pass header.d=messagingengine.com header.s=fm1 header.b=p+1grLR0; spf=pass (imf11.hostedemail.com: domain of arnd@arndb.de designates 66.111.4.28 as permitted sender) smtp.mailfrom=arnd@arndb.de; dmarc=none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1677847275; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=miW7z/m1o3Z+JHrWEAUIQgA5k8V0IlAjl3B3oQaEa/8=; b=sQo1EsuXeYbIPqT2125r3TyPlfta6pX6OkP+jU1T7GTin2tJx5C4YOsy1/ViiSMSWeRd/o 6fAAaMSrx8araHIOu3J8VTwO21UTO+JruVmXeIjTajcQNZA+ii19tTk/OpGf+QK0LSOQWr 4dGgbhXNGPDGKTlC6oppOB0CmoJ6aHs= ARC-Authentication-Results: i=1; imf11.hostedemail.com; dkim=pass header.d=arndb.de header.s=fm1 header.b=hhOAXcZA; dkim=pass header.d=messagingengine.com header.s=fm1 header.b=p+1grLR0; spf=pass (imf11.hostedemail.com: domain of arnd@arndb.de designates 66.111.4.28 as permitted sender) smtp.mailfrom=arnd@arndb.de; dmarc=none ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1677847275; a=rsa-sha256; cv=none; b=G9fjZAawFx5BU7f9wms43JTEZRjkCb52/5ajZX0NdN71a9dmNcEP99ECQGdz0l0NonoZNn +n5kzGc6yYndKc5MV6DIqtE3JTvPr6hPRFZ4DAifsbXN6xGSzf+0IxTGXkjJApIvCdvups SqzVR0sfNB7VTEKd/YfIzxFhoAojX48= Received: from compute6.internal (compute6.nyi.internal [10.202.2.47]) by mailout.nyi.internal (Postfix) with ESMTP id EF03D5C00B5; Fri, 3 Mar 2023 07:41:14 -0500 (EST) Received: from imap51 ([10.202.2.101]) by compute6.internal (MEProxy); Fri, 03 Mar 2023 07:41:14 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arndb.de; h=cc :cc:content-type:content-type:date:date:from:from:in-reply-to :in-reply-to:message-id:mime-version:references:reply-to:sender :subject:subject:to:to; s=fm1; t=1677847274; x=1677933674; bh=mi W7z/m1o3Z+JHrWEAUIQgA5k8V0IlAjl3B3oQaEa/8=; b=hhOAXcZA+/RUyzOHaZ T5ezvIZOpsKpWta+5rrs5rURKmZEvgPA6cw83GAsNk5jXNmWJNr/9JH0bGjrXp2V WH+LWyzvgh+Lmd4E1BP7NdQbPTUnbwYE6lsNzFqF6jiYIGm7T5+2Avki93acIgjk f9MAOtwpiiRYBRnujRdlHupSoNtrEjUG1Zk4z5/DevTsGYhYnGe9RFYZf8F5W537 NeYcoSCLsfzCCNoZJBcvFV+X42im7M6vpQgJRK8gnRM4OKHB6JfskOzoovAMu5eN tTDriPdyV0XTLTJtZZn185F3N2o01MhpcOE9gK+BHgaLR4waa3wwNfma4dlQ8Cyn ZVZw== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-type:content-type:date:date :feedback-id:feedback-id:from:from:in-reply-to:in-reply-to :message-id:mime-version:references:reply-to:sender:subject :subject:to:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm1; t=1677847274; x=1677933674; bh=miW7z/m1o3Z+J HrWEAUIQgA5k8V0IlAjl3B3oQaEa/8=; b=p+1grLR0ZatZ8NvlWEJIT2nBMwBaT JFJtPp4pdg9dZqck9YoxP5C4aMEZVQPW9OwIhuuJan0SIVnBAPm0atu/L5rBPctC JZPTRQevuPvq3TxG8iR2ts7EanRMNBbBpcBAuMIBWGc0Hu4MUUfNlqKH2zFWMqt5 nxq3TezzlrAdE24np88ZQYJv3suX54SIYlhI7DPOfWKexOdBzmURa+3FKrsztG9c GuXOJHGjep6q4SYY9vMsFb7z5W2p36nywwTmZryx9RQ4CaiVfYBc5fJYtYHawrZC 6E7bLbibTxM/arg414VxOX9I9r0LEICnX1UH7mhKm0NDsBWkm+nNd0psg== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvhedrudelledgfeekucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepofgfggfkjghffffhvfevufgtsehttdertderredtnecuhfhrohhmpedftehr nhguuceuvghrghhmrghnnhdfuceorghrnhgusegrrhhnuggsrdguvgeqnecuggftrfgrth htvghrnhepffehueegteeihfegtefhjefgtdeugfegjeelheejueethfefgeeghfektdek teffnecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomheprg hrnhgusegrrhhnuggsrdguvg X-ME-Proxy: Feedback-ID: i56a14606:Fastmail Received: by mailuser.nyi.internal (Postfix, from userid 501) id 1679EB60089; Fri, 3 Mar 2023 07:41:14 -0500 (EST) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.9.0-alpha0-183-gbf7d00f500-fm-20230220.001-gbf7d00f5 Mime-Version: 1.0 Message-Id: In-Reply-To: <20230303102817.212148-2-bhe@redhat.com> References: <20230303102817.212148-1-bhe@redhat.com> <20230303102817.212148-2-bhe@redhat.com> Date: Fri, 03 Mar 2023 13:40:53 +0100 From: "Arnd Bergmann" To: "Baoquan He" , linux-kernel@vger.kernel.org Cc: Linux-Arch , linux-mm@kvack.org, "Geert Uytterhoeven" , "Luis Chamberlain" , "Christoph Hellwig" , "Thomas Bogendoerfer" , "Helge Deller" , "Serge Semin" , "Florian Fainelli" , "Huacai Chen" , "Jiaxun Yang" , linux-mips@vger.kernel.org Subject: Re: [PATCH v3 1/2] mips: add including Content-Type: text/plain X-Rspam-User: X-Rspamd-Server: rspam04 X-Rspamd-Queue-Id: 8866D4000C X-Stat-Signature: na976xtboxgszi4xy8kid6yrbexzjqur X-HE-Tag: 1677847275-87389 X-HE-Meta: U2FsdGVkX196apIjX6BC1aPT9799EO4gyPYRLRlvaLLKWjlpvmzk1b+jUbDcvpLAfZ/9pd3Z3tR7wdWpoNF64brOl5AiBmCLTrRlN96sDZc7Lu+50+UWAxwmjdiy1Y23LFZ3cYkhczbHbqbux6at5YnKAp1H69tIanSuT9ouKmLrYbJuxctLpS0RAF42EcpQyPxGzS2WUc+zvF558qd8xnOqLGg09RV/S2lSe+d6g/RWTyqwAgTeYohXhtTR9WYFykH2ocSsmOLyrhra8kp3mZhzFckduJuH5sxSEW1IwJfcIVKlgZeLOPRRRS/r8WL92AXYvQ2qgeBu0fYJYHXBoyj0dxBmS4/yddfBaz3UUHQ7wWMjNfyTgLiCLqEPGyV43n+dzu25UTP845cTwKLktUY7LSZESlTwvNqPEkxFqre7lFWLFYiNFujtL3EWUrAImbiCl+2ET6WGI1A9e5602ryU9a7jls8AaDf9m6iZhAEcPP9j+o3QMkZDKqQazaxn97lYAqgclUYB2fP0NW4V8Tq3QwB1ksgRdmoUCu5Ub1EIKjE3SMPo+3QGErNYVEA1t9lRL37KSgGYC5zAGmYrmw2qNiv79M5pRAUDkGdWs3CAIr7/yDGWTql9a/JIMzrOm6zivesUwYzB+fW0cgdr0TIKHVFKeEBl19zghK2/5DnNDslqWB+HXz2tvQsfRzcU7pcNfh2wI2vadb3NrsVx/Osw8yh1sMN6nVzQhXjIQamqi62SXLrFGiPIcOs7AtVFNo6HDQKHYGfemvQPrA57vI3Q35S3wuAaYR6aZXqk3cxxL4wXlN4eEFSEKnpEkE9QX+jh5qDhODSaD5IRfZQ1qHbRa4v2+o7VRerRcmT3OPbeK+9L8U+AhGW6L99G4+gtFjprCly9hpgwHQSAU3/v+rqKabZgTqGpHtUZL4OK+5kKLfT5mcLmIejhW6s+aCOl29fEpflSPti1GtwaYhO IbUDcyIu 4oBvh3VwvitYOjYPhHiKuIVokNmE/1vXj9u7cnFZolL/TdpK0WYzDVcz19bIqL2CNPI1vYE8+/j41v7M1soGZh+eTmfwXNaiM83xKZ95mFyLzrvmIkLGGFuzd3lcnzTloaAUNsevSz6YbGwFcBJfmD0YDkZZa8hSrLKywTDENXHAAYSAjRGN4vibbAuRIGKM/Rp/qIz/KP0KUxj+98LSnv4r6HunsfmnNHdzaWLSp1BM2DZ7rE7kgvfbpal5yUkp37ArQ8r1Te/vHmcFoKjMRXej6XppAu4lVxNf+Z7VClwCnyitolh2VP0x7UmAVoFgwORHszeEey8bky4q75lwSNpL5urqhFIhNxRHlbUKA5pUb2tRabHqlWW8ze25H17fs0RYy2fhK015Liq/P99RIvoWtING0rmIEQwnO6GKrf1+Ki1KzVOOmtKKDL5UvTI4mZtIf X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Fri, Mar 3, 2023, at 11:28, Baoquan He wrote: > With the adding, some default ioremap_xx methods defined in > asm-generic/io.h can be used. E.g the default ioremap_uc() returning > NULL. > > Signed-off-by: Baoquan He > Cc: Thomas Bogendoerfer > Cc: Geert Uytterhoeven > Cc: Helge Deller > Cc: Serge Semin > Cc: Florian Fainelli > Cc: Huacai Chen > Cc: Jiaxun Yang > Cc: linux-mips@vger.kernel.org Reviewed-by: Arnd Bergmann I think this is all good. I had look at what cleanups we could do as follow-ups: > +#define phys_to_virt phys_to_virt > static inline void * phys_to_virt(unsigned long address) > { > return __va(address); This is the same as the asm-generic version, so the mips definition is no longer needed. > @@ -359,6 +360,27 @@ __BUILD_MEMORY_PFX(__raw_, q, u64, 0) > __BUILD_MEMORY_PFX(__mem_, q, u64, 0) > #endif > > +#define readb readb > +#define readw readw > +#define readl readl > +#define writeb writeb > +#define writew writew > +#define writel writel > + > +#ifdef CONFIG_64BIT > +#define readq readq > +#define writeq writeq > +#define __raw_readq __raw_readq > +#define __raw_writeq __raw_writeq > +#endif > + > +#define __raw_readb __raw_readb > +#define __raw_readw __raw_readw > +#define __raw_readl __raw_readl > +#define __raw_writeb __raw_writeb > +#define __raw_writew __raw_writew > +#define __raw_writel __raw_writel The mips code defines the __raw variants with slightly different semantics on both barriers and byteswap, which makes it impractical to share any of the above. > +#define memset_io memset_io > static inline void memset_io(volatile void __iomem *addr, unsigned > char val, int count) > { > memset((void __force *) addr, val, count); > } > +#define memcpy_fromio memcpy_fromio > static inline void memcpy_fromio(void *dst, const volatile void > __iomem *src, int count) > { > memcpy(dst, (void __force *) src, count); > } > +#define memcpy_toio memcpy_toio These are again the same as the generic version Arnd