Re: [PATCH] docs: ABI: sysfs-tty: close times are in hundredths of a second

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 24. 08. 23, 9:18, Simon Arlott wrote:
The times for close_delay and closing_wait are in hundredths of a
second, not milliseconds. Fix the documentation instead of trying
to use millisecond values (which would have to be rounded).

Signed-off-by: Simon Arlott <simon@xxxxxxxxxxx>
---
If you'd prefer, I can fold the second part of this into my previous
patch which shouldn't have documented it as milliseconds in the first
place (but I copied it from the other entry).

  Documentation/ABI/testing/sysfs-tty | 12 ++++++++----
  1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/Documentation/ABI/testing/sysfs-tty b/Documentation/ABI/testing/sysfs-tty
index e04e322af568..6ee878771f51 100644
--- a/Documentation/ABI/testing/sysfs-tty
+++ b/Documentation/ABI/testing/sysfs-tty
@@ -87,7 +87,8 @@ What:		/sys/class/tty/ttyS<x>/close_delay
  Date:		October 2012
  Contact:	Alan Cox <alan@xxxxxxxxxxxxxxx>
  Description:
-		 Show the closing delay time for this port in ms.
+		 Show the closing delay time for this port in hundredths
+		 of a second.
These sysfs values expose the TIOCGSERIAL interface via
  		 sysfs rather than via ioctls.
@@ -96,7 +97,8 @@ What:		/sys/class/tty/ttyS<x>/closing_wait
  Date:		October 2012
  Contact:	Alan Cox <alan@xxxxxxxxxxxxxxx>
  Description:
-		 Show the close wait time for this port in ms.
+		 Show the close wait time for this port in hundredths of
+		 a second.
These sysfs values expose the TIOCGSERIAL interface via
  		 sysfs rather than via ioctls.

Could you send these two hunks as a separate patch? It's correct regardless of your other patch.

And I would use "centiseconds" instead, which is used (IMO) in these cases.

thanks,
--
js
suse labs




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux