diff mbox series

[net-next,v1] dt-bindings: net: ethernet-controller: document label property

Message ID 20220209082820.2210753-1-o.rempel@pengutronix.de (mailing list archive)
State Not Applicable
Delegated to: Netdev Maintainers
Headers show
Series [net-next,v1] dt-bindings: net: ethernet-controller: document label property | expand

Checks

Context Check Description
netdev/tree_selection success Clearly marked for net-next
netdev/fixes_present success Fixes tag not required for -next series
netdev/subject_prefix success Link
netdev/cover_letter success Single patches do not need cover letters
netdev/patch_count success Link
netdev/header_inline success No static functions without inline keyword in header files
netdev/build_32bit success Errors and warnings before: 0 this patch: 0
netdev/cc_maintainers success CCed 5 of 5 maintainers
netdev/build_clang success Errors and warnings before: 0 this patch: 0
netdev/module_param success Was 0 now: 0
netdev/verify_signedoff success Signed-off-by tag matches author and committer
netdev/verify_fixes success No Fixes tag
netdev/build_allmodconfig_warn success Errors and warnings before: 0 this patch: 0
netdev/checkpatch success total: 0 errors, 0 warnings, 0 checks, 10 lines checked
netdev/kdoc success Errors and warnings before: 0 this patch: 0
netdev/source_inline success Was 0 now: 0

Commit Message

Oleksij Rempel Feb. 9, 2022, 8:28 a.m. UTC
"label" provides human readable name used on a box, board or schematic
to identify Ethernet port.

Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
---
 .../devicetree/bindings/net/ethernet-controller.yaml          | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Rob Herring Feb. 9, 2022, 3:34 p.m. UTC | #1
On Wed, Feb 9, 2022 at 2:28 AM Oleksij Rempel <o.rempel@pengutronix.de> wrote:
>
> "label" provides human readable name used on a box, board or schematic
> to identify Ethernet port.

Do you still need this?

> Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
> ---
>  .../devicetree/bindings/net/ethernet-controller.yaml          | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/net/ethernet-controller.yaml b/Documentation/devicetree/bindings/net/ethernet-controller.yaml
> index 34c5463abcec..817794e56227 100644
> --- a/Documentation/devicetree/bindings/net/ethernet-controller.yaml
> +++ b/Documentation/devicetree/bindings/net/ethernet-controller.yaml
> @@ -13,6 +13,10 @@ properties:
>    $nodename:
>      pattern: "^ethernet(@.*)?$"
>
> +  label:
> +    $ref: /schemas/types.yaml#/definitions/string
> +    description: Human readable label on a port of a box.
> +
>    local-mac-address:
>      description:
>        Specifies the MAC address that was assigned to the network device.
> --
> 2.30.2
>
Oleksij Rempel Feb. 9, 2022, 3:53 p.m. UTC | #2
On Wed, Feb 09, 2022 at 09:34:36AM -0600, Rob Herring wrote:
> On Wed, Feb 9, 2022 at 2:28 AM Oleksij Rempel <o.rempel@pengutronix.de> wrote:
> >
> > "label" provides human readable name used on a box, board or schematic
> > to identify Ethernet port.
> 
> Do you still need this?

Yes. No need to force label based name from kernel, but user space
should be able to use it to rename interface if needed or use it for
GUI. Otherwise user space will need to have date base of all boards to
provide this information.

Regards,
Oleksij
Rob Herring Feb. 11, 2022, 4:52 p.m. UTC | #3
On Wed, 09 Feb 2022 09:28:20 +0100, Oleksij Rempel wrote:
> "label" provides human readable name used on a box, board or schematic
> to identify Ethernet port.
> 
> Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
> ---
>  .../devicetree/bindings/net/ethernet-controller.yaml          | 4 ++++
>  1 file changed, 4 insertions(+)
> 

Applied, thanks!
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/net/ethernet-controller.yaml b/Documentation/devicetree/bindings/net/ethernet-controller.yaml
index 34c5463abcec..817794e56227 100644
--- a/Documentation/devicetree/bindings/net/ethernet-controller.yaml
+++ b/Documentation/devicetree/bindings/net/ethernet-controller.yaml
@@ -13,6 +13,10 @@  properties:
   $nodename:
     pattern: "^ethernet(@.*)?$"
 
+  label:
+    $ref: /schemas/types.yaml#/definitions/string
+    description: Human readable label on a port of a box.
+
   local-mac-address:
     description:
       Specifies the MAC address that was assigned to the network device.