Am Tue, Feb 04, 2025 at 09:34:09AM -0600 schrieb Rob Herring: > On Tue, Feb 04, 2025 at 02:09:15PM +0100, Dimitri Fedrau wrote: > > Add property tx-amplitude-100base-tx-percent in the device tree bindings > > for configuring the tx amplitude of 100BASE-TX PHYs. Modifying it can be > > necessary to compensate losses on the PCB and connector, so the voltages > > measured on the RJ45 pins are conforming. > > > > Signed-off-by: Dimitri Fedrau <dimitri.fedrau@xxxxxxxxxxxx> > > --- > > Documentation/devicetree/bindings/net/ethernet-phy.yaml | 7 +++++++ > > 1 file changed, 7 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/net/ethernet-phy.yaml b/Documentation/devicetree/bindings/net/ethernet-phy.yaml > > index 2c71454ae8e362e7032e44712949e12da6826070..04f42961035f273990fdf4368ad1352397fc3774 100644 > > --- a/Documentation/devicetree/bindings/net/ethernet-phy.yaml > > +++ b/Documentation/devicetree/bindings/net/ethernet-phy.yaml > > @@ -232,6 +232,13 @@ properties: > > PHY's that have configurable TX internal delays. If this property is > > present then the PHY applies the TX delay. > > > > + tx-amplitude-100base-tx-percent: > > + description: | > > Don't need '|' if no formatting to preserve. > Wasn't sure about the '|', because some properties use it(rx-internal-delay-ps or tx-internal-delay-ps) others not. Is it wrong there ? I don't see any reason to preserve formatting there. Anyway will remove the '|' as you proposed. Thanks for explaining, didn't know. Best regards, Dimitri Fedrau [...]
