Sent from my iPad On Feb 9, 2016, at 12:54, Andrew Lunn <and...@lunn.ch> wrote: >>> +static void igb_read_mac_addr_dts(struct e1000_hw *hw) >>> +{ >>> + const u8 *mac; >>> + struct device_node *dn; >>> + >>> + dn = of_find_compatible_node(NULL, NULL, "intel,i211"); > > Hi John > > Would this also work for the i210? The usability scenario of i210 and i211 seem similar enough. Although, I will not be able to test the i210, I will use it as the compatible keyword. > > If so, you normally use the compatible string for the first device > this works with. So maybe this should be changed to intel,i210? > > Thanks > Andrew
- [Intel-wired-lan] [next] igb: allow setting MAC address on ... John Holland
- Re: [Intel-wired-lan] [next] igb: allow setting MAC ad... Jeff Kirsher
- Re: [Intel-wired-lan] [next] igb: allow setting MA... Andrew Lunn
- Re: [Intel-wired-lan] [next] igb: allow settin... John Holland
- Re: [Intel-wired-lan] [next] igb: allow se... Andrew Lunn
- Re: [Intel-wired-lan] [next] igb: allow setting MA... Andrew Lunn
- Re: [Intel-wired-lan] [next] igb: allow settin... Shannon Nelson
- Re: [Intel-wired-lan] [next] igb: allow se... David Miller
- Re: [Intel-wired-lan] [next] igb: allow se... John Holland
- Re: [Intel-wired-lan] [next] igb: allow settin... John Holland
- Re: [Intel-wired-lan] [next] igb: allow setting MA... John Holland