]> git.baikalelectronics.ru Git - kernel.git/commit
net: 6pack: Fix tx timeout and slot time
authorGuenter Roeck <linux@roeck-us.net>
Thu, 9 Sep 2021 03:57:43 +0000 (20:57 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Thu, 16 Sep 2021 19:52:06 +0000 (12:52 -0700)
commit81e802ef65d9b51aba50fc1f639147ff3af4a403
tree7176048edd93285918f8de3617c4afe9ee6a7778
parent514c99b8a72f58820e1cb0e15e6121043ab00d21
net: 6pack: Fix tx timeout and slot time

tx timeout and slot time are currently specified in units of HZ.  On
Alpha, HZ is defined as 1024.  When building alpha:allmodconfig, this
results in the following error message.

  drivers/net/hamradio/6pack.c: In function 'sixpack_open':
  drivers/net/hamradio/6pack.c:71:41: error:
   unsigned conversion from 'int' to 'unsigned char'
   changes value from '256' to '0'

In the 6PACK protocol, tx timeout is specified in units of 10 ms and
transmitted over the wire:

    https://www.linux-ax25.org/wiki/6PACK

Defining a value dependent on HZ doesn't really make sense, and
presumably comes from the (very historical) situation where HZ was
originally 100.

Note that the SIXP_SLOTTIME use explicitly is about 10ms granularity:

        mod_timer(&sp->tx_t, jiffies + ((when + 1) * HZ) / 100);

and the SIXP_TXDELAY walue is sent as a byte over the wire.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/net/hamradio/6pack.c