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 lists.ozlabs.org (lists.ozlabs.org [112.213.38.117]) (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 A83F9C77B7F for ; Fri, 12 May 2023 02:43:36 +0000 (UTC) Received: from boromir.ozlabs.org (localhost [IPv6:::1]) by lists.ozlabs.org (Postfix) with ESMTP id 4QHY3q16QLz3fQG for ; Fri, 12 May 2023 12:43:35 +1000 (AEST) Authentication-Results: lists.ozlabs.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=ellerman.id.au header.i=@ellerman.id.au header.a=rsa-sha256 header.s=201909 header.b=dIps/oqn; dkim-atps=neutral Received: from gandalf.ozlabs.org (gandalf.ozlabs.org [150.107.74.76]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 4QHY2w2MVsz3bhC for ; Fri, 12 May 2023 12:42:48 +1000 (AEST) Authentication-Results: lists.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=ellerman.id.au header.i=@ellerman.id.au header.a=rsa-sha256 header.s=201909 header.b=dIps/oqn; dkim-atps=neutral Received: from authenticated.ozlabs.org (localhost [127.0.0.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mail.ozlabs.org (Postfix) with ESMTPSA id 4QHY2v2286z4x5R; Fri, 12 May 2023 12:42:46 +1000 (AEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ellerman.id.au; s=201909; t=1683859368; bh=UHonVxCxl9tE/BE4saKmOTqDfW/Z+XH0atlSFKqQKFw=; h=From:To:Cc:Subject:In-Reply-To:References:Date:From; b=dIps/oqn68QXS/wtfWcyekxrrBV800VlhYXyckk0yyHlNuAi/OANA1HtUmq/Dm9HY RCiAcMCb25X2GkBordupXcyG4V4fbQ21NRVicTRxpC+ul9+wr55ARw5deXskUIusKV A89LRqgyTf2iZog4EKY6eRpyMZI2Og/axtJ7SqtKHDnysjUjgr1MLwuvMgrcsChKxW Tn2qFReh+IIS6pyD5m3dl/jn19fXV3tTJtc0MPNh86ctlPKzO7xmR2cz77y5pDu1Lj MfpgFo/YdkymlLAJvrkNWgdTIxMES90GDKurKIxsqVbTZxQfmtWNicbivLkrkYQk0o 7shnTEqgaAJpA== From: Michael Ellerman To: Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= Subject: Re: [PATCH] powerpc: Drop MPC5200 LocalPlus bus FIFO driver In-Reply-To: <20230509082153.jcy2qpvyd44bjaee@pengutronix.de> References: <20221228145129.31700-1-u.kleine-koenig@pengutronix.de> <20230412150524.ojsvukh47hing6db@pengutronix.de> <87zg7cabf6.fsf@mpe.ellerman.id.au> <20230413061642.kqkor4wkt7lp2mhp@pengutronix.de> <20230509082153.jcy2qpvyd44bjaee@pengutronix.de> Date: Fri, 12 May 2023 12:42:46 +1000 Message-ID: <873542l17d.fsf@mail.lhotse> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Anatolij Gustschin , linux-kernel@vger.kernel.org, Nicholas Piggin , Andy Shevchenko , linuxppc-dev@lists.ozlabs.org Errors-To: linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org Sender: "Linuxppc-dev" Uwe Kleine-K=C3=B6nig writes: > On Thu, Apr 13, 2023 at 08:16:42AM +0200, Uwe Kleine-K=C3=B6nig wrote: >> While mpc5200b.dtsi contains a device that this driver can bind to, the >> only purpose of a bound device is to be used by the four exported functi= ons >> mpc52xx_lpbfifo_submit(), mpc52xx_lpbfifo_abort(), mpc52xx_lpbfifo_poll() >> and mpc52xx_lpbfifo_start_xfer(). However there are no callers to this >> function and so the driver is effectively superfluous and can be deleted. >> Also drop some defines and a struct from that are unused >> now together with the declarations of the four mentioned functions. >>=20 >> Signed-off-by: Uwe Kleine-K=C3=B6nig >> --- >> On Thu, Apr 13, 2023 at 10:11:25AM +1000, Michael Ellerman wrote: >> > Uwe Kleine-K=C3=B6nig writes: >> > > On Wed, Dec 28, 2022 at 03:51:29PM +0100, Uwe Kleine-K=C3=B6nig wrot= e: >> > >> The four exported functions mpc52xx_lpbfifo_submit(), >> > >> mpc52xx_lpbfifo_abort(), mpc52xx_lpbfifo_poll(), and >> > >> mpc52xx_lpbfifo_start_xfer() are not used. So they can be dropped a= nd the >> > >> definitions needed to call them can be moved into the driver file. >> > >>=20 >> > >> Signed-off-by: Uwe Kleine-K=C3=B6nig >> > > >> > > I never got feedback about this driver and it has not appeared in ne= xt >> > > up to now. Did it fell through the cracks? >> >=20 >> > Yeah. I was hoping someone would explain what's going on with the >> > driver. >> >=20 >> > Presumably there are some out-of-tree drivers that use the routines >> > provided by this driver? >>=20 >> I googled for the function names but the only related hits were >> references to this thread :-) >>=20 >> > I think rather than merging this patch, which keeps the code but makes >> > it completely useless, do you mind sending a patch to remove the whole >> > driver? Maybe that will get someone's attention. >>=20 >> fair enough, here it is. > > What is your thought about this patch? If you (also) think getting it > into next soon after a merge window closed, around now would be a good > opportunity to do so .. I was hoping someone would explain why we have a driver that does nothing useful :) But I guess it's too old for anyone to remember/care. So yeah I'll take this, we can always put it back if someone needs it. I base my next on rc2, so I'll pick this up next week. cheers