All of lore.kernel.org
 help / color / mirror / Atom feed
From: Aaron Sierra <asierra@xes-inc.com>
To: intel-wired-lan@osuosl.org
Subject: [Intel-wired-lan] [PATCH 1/5] igb: Remove GS40G specific defines/functions
Date: Fri, 6 Nov 2015 12:45:40 -0600 (CST)	[thread overview]
Message-ID: <745006060.48564.1446835540714.JavaMail.zimbra@xes-inc.com> (raw)
In-Reply-To: <1446797097.3515.5.camel@intel.com>

----- Original Message -----
> From: "Jeff Kirsher" <jeffrey.t.kirsher@intel.com>
> Sent: Friday, November 6, 2015 2:04:57 AM
> 
> On Tue, 2015-11-03 at 12:37 -0600, Aaron Sierra wrote:
> > The I210 internal PHY can be accessed just as well with the access
> > functions shared by 82580, I350, and I354 devices. A side effect of
> > relying on the common functions, is that I210 cable length support
> > is folded back into the common case which effectively reverts the
> > following commit:
> > 
> > ??? commit 59f301046b276f87483b3afa3201a4273def06a9
> > ??? Author: Carolyn Wyborny <carolyn.wyborny@intel.com>
> > ??? Date:?? Wed Oct 10 04:42:59 2012 +0000
> > 
> > ??? igb: Update get cable length function for i210/i211
> > 
> > Cc: Carolyn Wyborny <carolyn.wyborny@intel.com>
> > Signed-off-by: Aaron Sierra <asierra@xes-inc.com>
> > ---
> > ?drivers/net/ethernet/intel/igb/e1000_82575.c | 13 ++---
> > ?drivers/net/ethernet/intel/igb/e1000_i210.c? |? 5 +-
> > ?drivers/net/ethernet/intel/igb/e1000_i210.h? |? 2 +-
> > ?drivers/net/ethernet/intel/igb/e1000_phy.c?? | 82 +-----------------
> > ----------
> > ?drivers/net/ethernet/intel/igb/e1000_phy.h?? | 15 +----
> > ?5 files changed, 11 insertions(+), 106 deletions(-)
> 
> Other than patch 4 of your series, the rest of the series touch code
> which we "share" with our various other drivers. ?We would need you to
> assign copyright back to Intel for patches 1, 2, 3 & 5 so that would
> retain copyright on the changes so that we can re-license the changes
> for our other drivers.

That isn't a problem. Do you need a statement any more explicit or
official than that?

-Aaron

  reply	other threads:[~2015-11-06 18:45 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <596383518.303713.1446574778623.JavaMail.zimbra@xes-inc.com>
2015-11-03 18:37 ` [Intel-wired-lan] [PATCH 1/5] igb: Remove GS40G specific defines/functions Aaron Sierra
2015-11-06  8:04   ` Jeff Kirsher
2015-11-06 18:45     ` Aaron Sierra [this message]
2015-12-03  3:39     ` Brown, Aaron F

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=745006060.48564.1446835540714.JavaMail.zimbra@xes-inc.com \
    --to=asierra@xes-inc.com \
    --cc=intel-wired-lan@osuosl.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.