| Message ID | 20211011142215.9013-8-alexandru.tachici@analog.com (mailing list archive) |
|---|---|
| State | Changes Requested |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | net: phy: adin1100: Add initial support for ADIN1100 industrial PHY | expand |
| Context | Check | Description |
|---|---|---|
| netdev/cover_letter | success | Series has a cover letter |
| netdev/fixes_present | success | Fixes tag not required for -next series |
| netdev/patch_count | success | Link |
| netdev/tree_selection | success | Guessed tree name to be net-next |
| netdev/subject_prefix | success | Link |
| netdev/cc_maintainers | warning | 1 maintainers not CCed: f.fainelli@gmail.com |
| netdev/source_inline | success | Was 0 now: 0 |
| netdev/verify_signedoff | success | Signed-off-by tag matches author and committer |
| netdev/module_param | success | Was 0 now: 0 |
| netdev/build_32bit | success | Errors and warnings before: 0 this patch: 0 |
| netdev/kdoc | success | Errors and warnings before: 0 this patch: 0 |
| netdev/verify_fixes | success | No Fixes tag |
| netdev/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 15 lines checked |
| netdev/build_allmodconfig_warn | success | Errors and warnings before: 0 this patch: 0 |
| netdev/header_inline | success | No static functions without inline keyword in header files |
On Mon, Oct 11, 2021 at 05:22:14PM +0300, alexandru.tachici@analog.com wrote: > From: Alexandru Tachici <alexandru.tachici@analog.com> > > Add a tristate property to advertise desired transmit level. > > If the device supports the 2.4 Vpp operating mode for 10BASE-T1L, > as defined in 802.3gc, and the 2.4 Vpp transmit voltage operation > is desired, property should be set to 1. This property is used > to select whether Auto-Negotiation advertises a request to > operate the 10BASE-T1L PHY in increased transmit level mode. > > If property is set to 1, the PHY shall advertise a request > to operate the 10BASE-T1L PHY in increased transmit level mode. > If property is set to zero, the PHY shall not advertise > a request to operate the 10BASE-T1L PHY in increased transmit level mode. > > Signed-off-by: Alexandru Tachici <alexandru.tachici@analog.com> > --- > Documentation/devicetree/bindings/net/ethernet-phy.yaml | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/Documentation/devicetree/bindings/net/ethernet-phy.yaml b/Documentation/devicetree/bindings/net/ethernet-phy.yaml > index 2766fe45bb98..2bb3a96612a2 100644 > --- a/Documentation/devicetree/bindings/net/ethernet-phy.yaml > +++ b/Documentation/devicetree/bindings/net/ethernet-phy.yaml > @@ -77,6 +77,15 @@ properties: > description: > Maximum PHY supported speed in Mbits / seconds. > > + an-10base-t1l-2.4vpp: What does 'an' mean? > + description: | > + tristate, request/disable 2.4 Vpp operating mode. The values are: > + 0: Disable 2.4 Vpp operating mode. > + 1: Request 2.4 Vpp operating mode from link partner. > + Absence of this property will leave configuration to default values. > + $ref: "/schemas/types.yaml#/definitions/uint32" > + enum: [0, 1] What happened to this one doing the same thing?: https://lore.kernel.org/lkml/20201117201555.26723-3-dmurphy@ti.com/
On Mon, Oct 18, 2021 at 02:06:19PM -0500, Rob Herring wrote: > On Mon, Oct 11, 2021 at 05:22:14PM +0300, alexandru.tachici@analog.com wrote: > > From: Alexandru Tachici <alexandru.tachici@analog.com> > > > > Add a tristate property to advertise desired transmit level. > > > > If the device supports the 2.4 Vpp operating mode for 10BASE-T1L, > > as defined in 802.3gc, and the 2.4 Vpp transmit voltage operation > > is desired, property should be set to 1. This property is used > > to select whether Auto-Negotiation advertises a request to > > operate the 10BASE-T1L PHY in increased transmit level mode. > > > > If property is set to 1, the PHY shall advertise a request > > to operate the 10BASE-T1L PHY in increased transmit level mode. > > If property is set to zero, the PHY shall not advertise > > a request to operate the 10BASE-T1L PHY in increased transmit level mode. > > > > Signed-off-by: Alexandru Tachici <alexandru.tachici@analog.com> > > --- > > Documentation/devicetree/bindings/net/ethernet-phy.yaml | 9 +++++++++ > > 1 file changed, 9 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/net/ethernet-phy.yaml b/Documentation/devicetree/bindings/net/ethernet-phy.yaml > > index 2766fe45bb98..2bb3a96612a2 100644 > > --- a/Documentation/devicetree/bindings/net/ethernet-phy.yaml > > +++ b/Documentation/devicetree/bindings/net/ethernet-phy.yaml > > @@ -77,6 +77,15 @@ properties: > > description: > > Maximum PHY supported speed in Mbits / seconds. > > > > + an-10base-t1l-2.4vpp: > > What does 'an' mean? I assume, it is for Auto Negotiate. > > + description: | > > + tristate, request/disable 2.4 Vpp operating mode. The values are: > > + 0: Disable 2.4 Vpp operating mode. > > + 1: Request 2.4 Vpp operating mode from link partner. > > + Absence of this property will leave configuration to default values. > > + $ref: "/schemas/types.yaml#/definitions/uint32" > > + enum: [0, 1] > > What happened to this one doing the same thing?: > > https://lore.kernel.org/lkml/20201117201555.26723-3-dmurphy@ti.com/ This one was not really synced with the IEEE 802.3 standard. According to the standard, there is optional 10base-t1l specific 2.4 Vpp operating mode. To be able to operate in this mode, HW should be designed to do so. And other way around, if HW is designed for explosive environment, it should never operate in 2.4 Vpp mode. So, depending on this property, the ability of the link-partner and user space configuration, we may allow to auto negotiate this mode. The question is, should it actually be called "an-", since this property should limit automatic and manual link configuration Regards, Oleksij
diff --git a/Documentation/devicetree/bindings/net/ethernet-phy.yaml b/Documentation/devicetree/bindings/net/ethernet-phy.yaml index 2766fe45bb98..2bb3a96612a2 100644 --- a/Documentation/devicetree/bindings/net/ethernet-phy.yaml +++ b/Documentation/devicetree/bindings/net/ethernet-phy.yaml @@ -77,6 +77,15 @@ properties: description: Maximum PHY supported speed in Mbits / seconds. + an-10base-t1l-2.4vpp: + description: | + tristate, request/disable 2.4 Vpp operating mode. The values are: + 0: Disable 2.4 Vpp operating mode. + 1: Request 2.4 Vpp operating mode from link partner. + Absence of this property will leave configuration to default values. + $ref: "/schemas/types.yaml#/definitions/uint32" + enum: [0, 1] + broken-turn-around: $ref: /schemas/types.yaml#/definitions/flag description: