lkml.org 
[lkml]   [2018]   [Nov]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Subject[PATCH 0/4] serdev: make serdev_device_write() more usable
Date
This series make the synchronous serdev_device_write() helper more
usable by

1) allowing drivers to pass a zero timeout to indicate that they
want to wait forever;

2) returning the number of bytes actually written (buffered)
if the helper is interrupted;

3) make the helper use interruptible wait so that the helper can
be used on behalf of user space.

Finally, the two write functions are documented using kernel-doc.

Turns out I was using the wrong timeout for two gnss drivers that
expected the helper to wait indefinitely. I've fixed up those separately
(by using MAX_SCHEDULE_TIMEOUT for now), but for the helper to be usable
when using flow control we really want it to be interruptible.

Besides the two gnss drivers, there's currently only one other in-kernel
user of this helper and that driver (rave-sp) uses a non-zero timeout
and doesn't check the return value and therefore does not need to be
updated.

Note that this series depends on the two above mentioned GNSS fixes
(submitted for v4.20-rc3).

Johan


Johan Hovold (4):
serdev: use zero to indicate infinite write timeout
serdev: make synchronous write return bytes written
serdev: make synchronous write helper interruptible
serdev: document the write functions using kernel-doc

drivers/gnss/serial.c | 2 +-
drivers/gnss/sirf.c | 2 +-
drivers/tty/serdev/core.c | 70 +++++++++++++++++++++++++++++++++++----
include/linux/serdev.h | 2 +-
4 files changed, 66 insertions(+), 10 deletions(-)

--
2.19.1

\
 
 \ /
  Last update: 2018-11-14 16:11    [W:0.070 / U:6.336 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site