net: phy: micrel: fix KSZ9031 clock skew for values greater 0ps
authorAndreas Pretzsch <apr@cn-eng.de>
Thu, 29 Nov 2018 19:04:53 +0000 (20:04 +0100)
committerJoe Hershberger <joe.hershberger@ni.com>
Thu, 24 Jan 2019 17:35:28 +0000 (11:35 -0600)
commit3b4cda34d48ab997e788be5fb9cbd5151f1e7c2e
tree1d5c504144e9f470542a6105e182da29791eb89c
parentbe09f5bc7c0c3f0d0bc7b88d23dae883bfa4e014
net: phy: micrel: fix KSZ9031 clock skew for values greater 0ps

For KSZ9021, all skew register fields are 4-bit wide.
For KSZ9031, the clock skew register fields are 5-bit wide.

The common code in ksz90x1_of_config_group calculating the combined
register value checks if the requested value is above the maximum
and uses this maximum if so. The calculation of this maximum uses
the register width, but the check itself does not. It uses a hardcoded
value of 0xf, which is too low in case of the 5-bit clock (0x1f).
This detail was probably lost during driver unification.

Effect (only for KSZ9031 clock skews): For values greater 900 (== 0ps),
this silently results in 1860 (== +960ps) instead of the requested one.

Fix the check by using the bit width instead of hardcoded value(s).

Signed-off-by: Andreas Pretzsch <apr@cn-eng.de>
Acked-by: Joe Hershberger <joe.hershberger@ni.com>
drivers/net/phy/micrel_ksz90x1.c