Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1495890 > unrolled thread
| Started by | Mike Looijmans <mike.looijmans@topic.nl> |
|---|---|
| First post | 2016-10-05 16:10 +0200 |
| Last post | 2016-10-07 02:50 +0200 |
| Articles | 3 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH] devicetree: net: micrel-ksz90x1.txt: Properly explain skew settings Mike Looijmans <mike.looijmans@topic.nl> - 2016-10-05 16:10 +0200
Re: [PATCH] devicetree: net: micrel-ksz90x1.txt: Properly explain skew settings Florian Fainelli <f.fainelli@gmail.com> - 2016-10-06 13:10 +0200
Re: [PATCH] devicetree: net: micrel-ksz90x1.txt: Properly explain skew settings David Miller <davem@davemloft.net> - 2016-10-07 02:50 +0200
| From | Mike Looijmans <mike.looijmans@topic.nl> |
|---|---|
| Date | 2016-10-05 16:10 +0200 |
| Subject | [PATCH] devicetree: net: micrel-ksz90x1.txt: Properly explain skew settings |
| Message-ID | <soV3s-209-13@gated-at.bofh.it> |
The KSZ9031 skew registers contain an offset, the chip's default value
is "neutral" which does not add any skew. Programming a 0 into a skew
property will actually set it the maximal negative adjustment and not
to a neutral position as one would expect.
Explain this situation in the devicetree binding documentation and list
the settings that the chip considers neutral.
Changing the implementation to accept negative values would have been
a better solution, but would break existing configurations.
Signed-off-by: Mike Looijmans <mike.looijmans@topic.nl>
---
Documentation/devicetree/bindings/net/micrel-ksz90x1.txt | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/Documentation/devicetree/bindings/net/micrel-ksz90x1.txt b/Documentation/devicetree/bindings/net/micrel-ksz90x1.txt
index f9c32ad..c35b5b4 100644
--- a/Documentation/devicetree/bindings/net/micrel-ksz90x1.txt
+++ b/Documentation/devicetree/bindings/net/micrel-ksz90x1.txt
@@ -34,16 +34,17 @@ KSZ9031:
All skew control options are specified in picoseconds. The minimum
value is 0, and the maximum is property-dependent. The increment
- step is 60ps.
+ step is 60ps. The default value is the neutral setting, so setting
+ rxc-skew-ps=<0> actually results in -900 picoseconds adjustment.
Optional properties:
- Maximum value of 1860:
+ Maximum value of 1860, default value 900:
- rxc-skew-ps : Skew control of RX clock pad
- txc-skew-ps : Skew control of TX clock pad
- Maximum value of 900:
+ Maximum value of 900, default value 420:
- rxdv-skew-ps : Skew control of RX CTL pad
- txen-skew-ps : Skew control of TX CTL pad
--
1.9.1
[toc] | [next] | [standalone]
| From | Florian Fainelli <f.fainelli@gmail.com> |
|---|---|
| Date | 2016-10-06 13:10 +0200 |
| Subject | Re: [PATCH] devicetree: net: micrel-ksz90x1.txt: Properly explain skew settings |
| Message-ID | <speIN-7Nv-29@gated-at.bofh.it> |
| In reply to | #1495890 |
On 10/05/2016 07:03 AM, Mike Looijmans wrote: > The KSZ9031 skew registers contain an offset, the chip's default value > is "neutral" which does not add any skew. Programming a 0 into a skew > property will actually set it the maximal negative adjustment and not > to a neutral position as one would expect. > > Explain this situation in the devicetree binding documentation and list > the settings that the chip considers neutral. > > Changing the implementation to accept negative values would have been > a better solution, but would break existing configurations. > > Signed-off-by: Mike Looijmans <mike.looijmans@topic.nl> Reviewed-by: Florian Fainelli <f.fainelli@gmail.com> Thanks! -- Florian
[toc] | [prev] | [next] | [standalone]
| From | David Miller <davem@davemloft.net> |
|---|---|
| Date | 2016-10-07 02:50 +0200 |
| Subject | Re: [PATCH] devicetree: net: micrel-ksz90x1.txt: Properly explain skew settings |
| Message-ID | <sprwl-fG-3@gated-at.bofh.it> |
| In reply to | #1495890 |
From: Mike Looijmans <mike.looijmans@topic.nl> Date: Wed, 5 Oct 2016 16:03:08 +0200 > The KSZ9031 skew registers contain an offset, the chip's default value > is "neutral" which does not add any skew. Programming a 0 into a skew > property will actually set it the maximal negative adjustment and not > to a neutral position as one would expect. > > Explain this situation in the devicetree binding documentation and list > the settings that the chip considers neutral. > > Changing the implementation to accept negative values would have been > a better solution, but would break existing configurations. > > Signed-off-by: Mike Looijmans <mike.looijmans@topic.nl> Applied.
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web