Received-SPF: pass (google.com: domain of linux-kernel+bounces-119444-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Precedence: bulk
MIME-Version: 1.0
References: <20240325210045.153827-1-afd@ti.com> <2024032631-excursion-opposing-be36@gregkh>
In-Reply-To: <2024032631-excursion-opposing-be36@gregkh>
From: Robert Nelson <robertcnelson@gmail.com>
Date: Tue, 26 Mar 2024 11:19:54 -0500
Message-ID: <CAOCHtYjauA+BAxZJBMTaxxaMGcvipP9=ZPeWe4FiNFs_jpq6dg@mail.gmail.com>
Subject: Re: [PATCH] uio: pruss: Deprecate use of this driver
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Jason Kridner <jkridner@beagleboard.org>, 
	Matthijs van Duin <matthijsvanduin@gmail.com>, Drew Fustini <drew@beagleboard.org>
Cc: Andrew Davis <afd@ti.com>, Tony Lindgren <tony@atomide.com>, linux-omap@vger.kernel.org, 
	linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Tue, Mar 26, 2024 at 12:41=E2=80=AFAM Greg Kroah-Hartman
<gregkh@linuxfoundation.org> wrote:
>
> On Mon, Mar 25, 2024 at 04:00:45PM -0500, Andrew Davis wrote:
> > This UIO driver was used to control the PRU processors found on various
> > TI SoCs. It was created before the Remoteproc framework, but now with
> > that we have a standard way to program and manage the PRU processors.
> > The proper PRU Remoteproc driver should be used instead of this driver.
> > Mark this driver deprecated.
> >
> > The userspace tools to use this are no longer available, so also remove
> > those dead links from the Kconfig description.
> >
> > Signed-off-by: Andrew Davis <afd@ti.com>
> > ---
> >  drivers/uio/Kconfig | 10 ++--------
> >  1 file changed, 2 insertions(+), 8 deletions(-)
> >
> > diff --git a/drivers/uio/Kconfig b/drivers/uio/Kconfig
> > index 2e16c5338e5b1..358dc2d19b885 100644
> > --- a/drivers/uio/Kconfig
> > +++ b/drivers/uio/Kconfig
> > @@ -126,19 +126,13 @@ config UIO_FSL_ELBC_GPCM_NETX5152
> >         http://www.hilscher.com/netx
> >
> >  config UIO_PRUSS
> > -     tristate "Texas Instruments PRUSS driver"
> > +     tristate "Texas Instruments PRUSS driver (DEPRECATED)"
>
> This isn't going to do much, why not just delete the driver entirely if
> no one uses it?

CC'ing Matthijs one of our BeagleBoard community members who utilizes
and supports UIO on a number of community projects.

We know TI and Mainline in general do not like this UIO driver as it's
very open-ended.

While the remoteproc_pruss driver is now mainline (it has taken a long
time, since 3.14.x i I think TI first started this..)

There is a large user base of UIO examples that have been running
since 3.8.x and as a community we have made sure ( mostly Matthijs )
that these continue to operate on this driver in
v5.x/v6.x/lts/mainline branches.

We can always revert/etc..  But I'm hoping Matthijs will chime-in..

Regards,

--=20
Robert Nelson
https://rcn-ee.com/