lkml.org 
[lkml]   [2020]   [Dec]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH v3 2/3] spi: Add SPI_NO_TX/RX support
On Thu, Dec 03, 2020 at 08:20:57AM +0000, Ardelean, Alexandru wrote:

> > > > @@ -43,5 +43,7 @@
> > > > #define SPI_TX_OCTAL 0x2000 /* transmit with 8 wires */
> > > > #define SPI_RX_OCTAL 0x4000 /* receive with 8 wires */
> > > > #define SPI_3WIRE_HIZ 0x8000 /* high impedance turnaround
> > */
> > > > +#define SPI_NO_TX 0x10000 /* no transmit wire */
> > > > +#define SPI_NO_RX 0x20000 /* no receive wire */

> > > Is it really material for uAPI?
> > > Perhaps we may have something like
> > > SPI_MODE_USER_MASK in uAPI and
> > > in internal headers

> Hmm, in a way this could make sense for some SPIDEVs as well, to set these flags and get an error if they try to TX with the NO_TX flag set.
> Not really sure about this.
> Initially I mechanically added these here as an inertia to the previous patch which is just unifying the headers.

> Any other opinions? Thoughts?
> Mark?

spidev is hacky at the best of times... It *is* probably better to only
have the usefully mainpulable modes exposed in uapi and then define the
rest internally though.
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2020-12-03 10:51    [W:0.066 / U:0.284 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site