Re: [EXT] Re: [PATCH v5 2/2] misc: nxp-sr1xx: UWB driver support for sr1xx series chip

[Greg Kroah-Hartman]

On Fri, Oct 07, 2022 at 07:34:25PM +0530, Manjunatha Venkatesh wrote:
> 
> On 9/14/2022 8:39 PM, Arnd Bergmann wrote:
> > Caution: EXT Email
> > 
> > On Wed, Sep 14, 2022, at 4:29 PM, Manjunatha Venkatesh wrote:
> > 
> > > NXP has SR1XX family of UWB Subsystems (UWBS) devices. SR1XX SOCs
> > > are FiRa Compliant. SR1XX SOCs are flash less devices and they need
> > > Firmware Download on every device boot. More details on the SR1XX Family
> > > can be found athttps://www.nxp.com/products/:UWB-TRIMENSION
> > > 
> > > The sr1xx driver work the SR1XX Family of UWBS, and uses UWB Controller
> > > Interface (UCI).  The corresponding details are available in the FiRa
> > > Consortium Website (https://www.firaconsortium.org/).
> > I know nothing about UWB, so I have no idea if the user interface
> > you propose here makes sense. My guess is that there is a good chance
> > that there are other implementations of UWB that would not work
> > with this specific driver interface, so you probably need a
> > slightly higher-level abstraction.
> > 
> > We had an older subsystem that was called UWB and that got removed
> > a while ago:
> > 
> > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/staging/uwb?id=caa6772db4c1deb5d9add48e95d6eab50699ee5e
> > 
> > Is that the same UWB or something completely different?
> Basically, it is SPI device driver which supports UCI(Ultra-wide band
> Command Interface) packet structure. It is not same as in mentioned link.

Why isn't this just a normal SPI driver and you do the "UCI" commands
from userspace through the device node there?

I know I asked this before, but I can't remember the answer, sorry, so
please include that in the changelog information when you resubmit.

thanks,

greg k-h