Received: by 2002:a05:6358:3188:b0:123:57c1:9b43 with SMTP id q8csp1735680rwd; Wed, 17 May 2023 00:05:48 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ45aeO/ODD3TvXC6Jw8EtGkN0Eezj7sqH4LSQcQnwkjSp5CNAdWT0KPmxHagkkfs/NLdqwG X-Received: by 2002:a17:902:70c9:b0:1ad:d1ab:9cc9 with SMTP id l9-20020a17090270c900b001add1ab9cc9mr18227492plt.12.1684307148491; Wed, 17 May 2023 00:05:48 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1684307148; cv=none; d=google.com; s=arc-20160816; b=E/MDrbcP2O1GkWumD2S6c2vcXysobyOJKwdfhHQDtxaXaN/xGjr5tLunZdy/wshNHo Qc5x0Eg7thqLK5kJI+e0P1cPKetsCYFonphdT30Jji9k8pUqcmQHp4/pq8nnkD4G9fBl rxGyeVlP6LTv/YJALZvQ7G6ScOJYID93DoRkREV3uKUdYwYJy0/cL6epreQidlbGUGx5 nf1UJ74P+6SE5aG14Y+fO8hWlciLDPZyFisoNDs6tpD1KX1HAQsg9boaYK8nEPkeU/8H rOfkrObJGrgn6q/X/q+mGRLtBryezF03Mi0T8Yf5c7HqB+7DB4SsZF1kTmt3xUTmZ5Xt qqCQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:subject:cc:to:from :date:references:in-reply-to:message-id:mime-version:user-agent :feedback-id:dkim-signature:dkim-signature; bh=kl6TNT66P/y7iDeev/8upXtGKNlx3dVK/bGgkTY44/g=; b=UdI5FmGc7d2r1ku8BJSBiaIGCYYh7gZgEVO3u1ie1w/BaQQePIx53Wyli8zx2M+Zpw 4M7GKoyhDH/Ly+0Y5EfCJwQSg0v7AuGy9NVXQNEtvgUCQ4Aj8sIrKay7s4fJNE9FGoo2 ggKlTCdjRtnVmYYlYaHFlXRdrgrYoBMLuDVhtSRlK9DgJweYgxR03bxP0FaNwWEwvm0u cIzaX3oi58I2lFv1Pir3gFIeQOxAItqlBnoCRXucUPjSi6cuhhp86i+XU09F356BoNYH G/fWxv8PkdRuTM2Cbps7Zao6Uznz2ukpZK+Naf28wQ2x6x4pISSiR1HtvzJ0fIfPbvYk iKIQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@arndb.de header.s=fm3 header.b=dDepjuPQ; dkim=pass header.i=@messagingengine.com header.s=fm1 header.b=PipUvu0e; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id u1-20020a170902e5c100b001a923bc3af0si21512815plf.544.2023.05.17.00.05.36; Wed, 17 May 2023 00:05:48 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@arndb.de header.s=fm3 header.b=dDepjuPQ; dkim=pass header.i=@messagingengine.com header.s=fm1 header.b=PipUvu0e; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229844AbjEQGke (ORCPT + 99 others); Wed, 17 May 2023 02:40:34 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55208 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229525AbjEQGkc (ORCPT ); Wed, 17 May 2023 02:40:32 -0400 Received: from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com [66.111.4.27]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C87A0CE; Tue, 16 May 2023 23:40:30 -0700 (PDT) Received: from compute6.internal (compute6.nyi.internal [10.202.2.47]) by mailout.nyi.internal (Postfix) with ESMTP id A5A755C01A5; Wed, 17 May 2023 02:40:27 -0400 (EDT) Received: from imap51 ([10.202.2.101]) by compute6.internal (MEProxy); Wed, 17 May 2023 02:40:27 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arndb.de; h=cc :cc:content-transfer-encoding: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=fm3; t= 1684305627; x=1684392027; bh=kl6TNT66P/y7iDeev/8upXtGKNlx3dVK/bG gkTY44/g=; b=dDepjuPQJpgpVcowQGAp9w9CSr8sS4dhN4qAyaBed7q08tauAKA 2JVAJPtCONiJyrmQqEUDNDyvKkkGeMzzhHDVORpiBuFBcnRJmRiuV+vrF/Q962QP tZwalgxXh/0ZtE5RzAQyER7HXq2ngLal+K8eZUWX3QL2QYR9DRqkxLBYYxmrxLTq 64M/MrtC2X9tkI1WAaym1SGbvlp91I3xuHdeiuKRR5hlOX2x2AtteDfccnsqxqwJ 5CsPP24TnqrV0sjE0Aer95/M7NgZ/up3BzTEkEeVQPH60OzUIgZszVz57EPxY3uP rMFGvmypNQBvgzdz2XSHXKsu8XnFmL/y3xg== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :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= 1684305627; x=1684392027; bh=kl6TNT66P/y7iDeev/8upXtGKNlx3dVK/bG gkTY44/g=; b=PipUvu0ehXleAfyypA9PUQLNwaI2g6c2IRoo+aylUllBX3EY2ES XQz2+3ZvfVPqLVVOuoLtuZG5s50Y8MCOYOdsFo/jNk28UhjAfVdQ61BwzRlup4nj pxrQhR8Sko0agy4sNPUNV050rnNxwWEC6HwXpXcL0Gnp3RZYTS/m/1SBL4/Xp/ye IgLTQRhiINQ1HktO5BigEHjzcMyiB0MNjp7QYTfUAEwoR4VQcb1OUIXO1Ui78Xrn XD/fIpYfQ28Uh9F/uYXo9iHfxWSVXBZ61FNlcd0gsf8qZk/4tPvCQh8DWor2I4C6 fJ0T9s6BC45qjosZzAf68noItbmhkrszv6g== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvhedrfeeitddguddtgecutefuodetggdotefrod ftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfgh necuuegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmd enucfjughrpefofgggkfgjfhffhffvvefutgfgsehtqhertderreejnecuhfhrohhmpedf tehrnhguuceuvghrghhmrghnnhdfuceorghrnhgusegrrhhnuggsrdguvgeqnecuggftrf grthhtvghrnhepgfekueelgeeigefhudduledtkeefffejueelheelfedutedttdfgveeu feefieegnecuffhomhgrihhnpehkvghrnhgvlhdrohhrghenucevlhhushhtvghrufhiii gvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpegrrhhnugesrghrnhgusgdruggv X-ME-Proxy: Feedback-ID: i56a14606:Fastmail Received: by mailuser.nyi.internal (Postfix, from userid 501) id 21F70B60086; Wed, 17 May 2023 02:40:25 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.9.0-alpha0-431-g1d6a3ebb56-fm-20230511.001-g1d6a3ebb Mime-Version: 1.0 Message-Id: <4e026f08-d733-4b01-ab47-e921d041e74e@app.fastmail.com> In-Reply-To: References: <20230516153109.514251-1-arnd@kernel.org> <20230516153109.514251-6-arnd@kernel.org> Date: Wed, 17 May 2023 08:39:53 +0200 From: "Arnd Bergmann" To: "Fabio Estevam" , "Arnd Bergmann" , "Martin Kaiser" Cc: soc@kernel.org, "Russell King" , "Bartosz Golaszewski" , "Hartley Sweeten" , "Alexander Sverdlin" , "Andre Przywara" , "Shawn Guo" , "Sascha Hauer" , "Pengutronix Kernel Team" , "NXP Linux Team" , "Vladimir Zapolskiy" , "Aaro Koskinen" , "Janusz Krzysztofik" , "Tony Lindgren" , "Andrew Lunn" , "Sebastian Hesselbarth" , "Gregory Clement" , "Daniel Mack" , "Haojian Zhuang" , "Robert Jarzmik" , "Viresh Kumar" , "Shiraz Hashim" , "Sudeep Holla" , "Lorenzo Pieralisi" , "Linus Walleij" , "Michael Turquette" , "Stephen Boyd" , "Greg Kroah-Hartman" , "Alan Stern" , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Linux-OMAP , linux-clk@vger.kernel.org, linux-usb@vger.kernel.org Subject: Re: [PATCH 05/13] ARM: imx: remove unused mx25_revision() Content-Type: text/plain;charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW, RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, May 17, 2023, at 01:34, Fabio Estevam wrote: > On Tue, May 16, 2023 at 12:32=E2=80=AFPM Arnd Bergmann wrote: >> >> From: Arnd Bergmann >> >> This function has no prototype and no callers: >> >> arm/mach-imx/cpu-imx25.c:43:5: error: no previous prototype for 'mx25= _revision' [-Werror=3Dmissing-prototypes] >> >> Signed-off-by: Arnd Bergmann > > Martin Kaiser sent a patch adding a user for this function: > > https://lore.kernel.org/linux-arm-kernel/20220815190748.102664-2-marti= n@kaiser.cx/ > > It would be better to apply Martin's patch instead of removing mx25_re= vision(). I think either way is ok to address the warning. If we wanted to do this properly, the mx{25,27,31,35,5}_revision functions could all be removed and the logic hooked up to imx_set_soc_revision() in the same way that they already use mxc_set_cpu_type() for drivers/soc/imx/soc-imx.c. I'll leave it up to you, if you want to merge Martin's patches or a replacement for the soc-imx driver through the imx tree for 6.5, I'll drop my patch from this series, otherwise I'll keep it for now and we can still do it better at later point. Arnd