Re: [PATCH v5] i40e: Look up MAC address in Open Firmware or IDPROM

2015-12-03 Thread David Miller
From: Sowmini Varadhan Date: Thu, 3 Dec 2015 10:48:23 -0500 > The patch here: http://patchwork.ozlabs.org/patch/540218/ > is marked "Awaiting Upstream", I think that means it has to first > show up in some other repo first (which one?)? This indication means that I

Re: [PATCH v5] i40e: Look up MAC address in Open Firmware or IDPROM

2015-12-03 Thread Sowmini Varadhan
Hi, The patch here: http://patchwork.ozlabs.org/patch/540218/ is marked "Awaiting Upstream", I think that means it has to first show up in some other repo first (which one?)? On addiitonal testing, we found a bug in the patch: if we did not find the macaddr from Open Firmwre or IDPROM (i.e.,

Re: [PATCH v5] i40e: Look up MAC address in Open Firmware or IDPROM

2015-12-03 Thread Jeff Kirsher
On Thu, 2015-12-03 at 10:48 -0500, Sowmini Varadhan wrote: > The patch here: http://patchwork.ozlabs.org/patch/540218/ > is marked "Awaiting Upstream", I think that means it has to first > show up in some other repo first (which one?)? > > On addiitonal testing, we found a bug in the patch: if >

Re: [PATCH v5] i40e: Look up MAC address in Open Firmware or IDPROM

2015-12-03 Thread Sowmini Varadhan
On (12/03/15 13:35), Jeff Kirsher wrote: > > I will drop your current patch in my next-queue tree (dev-queue branch) > and will await an updated patch.   sounds good. Let me work with Shannon and Andrew to make sure we cover all cases. --Sowmini -- To unsubscribe from this list: send the line

Re: [PATCH v5] i40e: Look up MAC address in Open Firmware or IDPROM

2015-11-04 Thread Andy Shevchenko
On Wed, Nov 4, 2015 at 10:06 PM, Sowmini Varadhan wrote: > On (11/04/15 21:59), Andy Shevchenko wrote: >> > See earlier response. So, if maintainer is okay I'm also okay with those and you may take my tag. -- With Best Regards, Andy Shevchenko -- To unsubscribe

RE: [PATCH v5] i40e: Look up MAC address in Open Firmware or IDPROM

2015-11-04 Thread Nelson, Shannon
> From: Sowmini Varadhan [mailto:sowmini.varad...@oracle.com] > Sent: Wednesday, November 04, 2015 11:40 AM > > > This is the i40e equivalent of commit c762dff24c06 ("ixgbe: Look up MAC > address in Open Firmware or IDPROM"). > > As with that fix, attempt to look up the MAC address in Open

RE: [PATCH v5] i40e: Look up MAC address in Open Firmware or IDPROM

2015-11-04 Thread Nelson, Shannon
> From: Andy Shevchenko [mailto:andy.shevche...@gmail.com] > Sent: Wednesday, November 04, 2015 11:59 AM > > On Wed, Nov 4, 2015 at 9:39 PM, Sowmini Varadhan > wrote: > > > > This is the i40e equivalent of commit c762dff24c06 ("ixgbe: Look up MAC > > address in Open

Re: [PATCH v5] i40e: Look up MAC address in Open Firmware or IDPROM

2015-11-04 Thread Andy Shevchenko
On Thu, Nov 5, 2015 at 12:53 AM, Nelson, Shannon wrote: >> From: Andy Shevchenko [mailto:andy.shevche...@gmail.com] >> Sent: Wednesday, November 04, 2015 11:59 AM >> >> On Wed, Nov 4, 2015 at 9:39 PM, Sowmini Varadhan >> wrote: >> > >> >

Re: [PATCH v5] i40e: Look up MAC address in Open Firmware or IDPROM

2015-11-04 Thread Andy Shevchenko
On Wed, Nov 4, 2015 at 9:39 PM, Sowmini Varadhan wrote: > > This is the i40e equivalent of commit c762dff24c06 ("ixgbe: Look up MAC > address in Open Firmware or IDPROM"). > > As with that fix, attempt to look up the MAC address in Open Firmware > on systems that

Re: [PATCH v5] i40e: Look up MAC address in Open Firmware or IDPROM

2015-11-04 Thread Sowmini Varadhan
On (11/04/15 21:59), Andy Shevchenko wrote: > > Usually the structure of kernel doc is something like following > > /** > * func - summary > * @paramx: desc > * > * Description: > * Long description in many lines and / or paragraphs > * > * Returns: > * 0 on success or errno otherwise. >

[PATCH v5] i40e: Look up MAC address in Open Firmware or IDPROM

2015-11-04 Thread Sowmini Varadhan
This is the i40e equivalent of commit c762dff24c06 ("ixgbe: Look up MAC address in Open Firmware or IDPROM"). As with that fix, attempt to look up the MAC address in Open Firmware on systems that support it, and use IDPROM on SPARC if no OF address is found. In the case of the i40e there is an