Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1554442

Re: [PATCH net-next v2] net: dsa: make "label" property optional for dsa2

From Jiri Pirko <jiri@resnulli.us>
Newsgroups linux.kernel
Subject Re: [PATCH net-next v2] net: dsa: make "label" property optional for dsa2
Date 2017-01-09 17:10 +0100
Message-ID <sXKGe-6kS-13@gated-at.bofh.it> (permalink)
References (1 earlier) <sXCIG-19v-7@gated-at.bofh.it> <sXJK9-5LV-25@gated-at.bofh.it> <sXJTQ-5Pf-27@gated-at.bofh.it> <sXKmS-5Z0-19@gated-at.bofh.it> <sXKGe-6kS-15@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Mon, Jan 09, 2017 at 05:00:19PM CET, andrew@lunn.ch wrote:
>> > No. That should be unique within one switch. In mlxsw we name it "p1",
>> > "p2", ...
>> >
>> > The final netdev names are:
>> > enp3s0np1, enp3s0np2, ...
>> 
>
>mlxsw are pci devices, so it follows this convention, i think:
>
> *   [P<domain>]p<bus>s<slot>[f<function>][n<phys_port_name>|d<dev_port>]
> *                                          PCI geographical location
>
>Our devices are not on PCI. So they won't follow this. I've no idea
>what they actually follow, since some are MDIO devices, some are SPI
>devices, some are memory mapped.

Got it. We just have to make sure udev names them appropriately. 


>
>I'm not against making the label option, but i do want to better
>understand what we get as a result, just to make sure it is sensible.
>
>Vivien, could you try a recent udev and see what happens?
>
>	Thanks
>		Andrew

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH net-next v2] net: dsa: make "label" property optional for dsa2 Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2017-01-09 00:20 +0100
  Re: [PATCH net-next v2] net: dsa: make "label" property optional for  dsa2 Andrew Lunn <andrew@lunn.ch> - 2017-01-09 00:40 +0100
    Re: [PATCH net-next v2] net: dsa: make "label" property optional for dsa2 Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2017-01-09 04:00 +0100
  Re: [PATCH net-next v2] net: dsa: make "label" property optional for  dsa2 Jiri Pirko <jiri@resnulli.us> - 2017-01-09 08:40 +0100
    Re: [PATCH net-next v2] net: dsa: make "label" property optional for dsa2 Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2017-01-09 16:10 +0100
      Re: [PATCH net-next v2] net: dsa: make "label" property optional for  dsa2 Jiri Pirko <jiri@resnulli.us> - 2017-01-09 16:20 +0100
        Re: [PATCH net-next v2] net: dsa: make "label" property optional for dsa2 Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2017-01-09 16:50 +0100
          Re: [PATCH net-next v2] net: dsa: make "label" property optional for  dsa2 Jiri Pirko <jiri@resnulli.us> - 2017-01-09 17:10 +0100
            Re: [PATCH net-next v2] net: dsa: make "label" property optional for  dsa2 Florian Fainelli <f.fainelli@gmail.com> - 2017-01-09 18:50 +0100
              Re: [PATCH net-next v2] net: dsa: make "label" property optional for  dsa2 Jiri Pirko <jiri@resnulli.us> - 2017-01-09 19:10 +0100
                Re: [PATCH net-next v2] net: dsa: make "label" property optional for  dsa2 Florian Fainelli <f.fainelli@gmail.com> - 2017-01-09 19:10 +0100
                Re: [PATCH net-next v2] net: dsa: make "label" property optional for  dsa2 Jiri Pirko <jiri@resnulli.us> - 2017-01-10 11:00 +0100
                Re: [PATCH net-next v2] net: dsa: make "label" property optional for  dsa2 Florian Fainelli <f.fainelli@gmail.com> - 2017-01-10 19:00 +0100
                Re: [PATCH net-next v2] net: dsa: make "label" property optional for  dsa2 Jiri Pirko <jiri@resnulli.us> - 2017-01-11 08:30 +0100
          Re: [PATCH net-next v2] net: dsa: make "label" property optional for  dsa2 Jiri Pirko <jiri@resnulli.us> - 2017-01-09 17:10 +0100
          Re: [PATCH net-next v2] net: dsa: make "label" property optional for  dsa2 Andrew Lunn <andrew@lunn.ch> - 2017-01-09 17:10 +0100

csiph-web