Jaunty SRU request - e1000e driver update for Hanksville NIC

[Andy Whitcroft]

On Wed, Jul 22, 2009 at 11:37:23AM +0200, Stefan Bader wrote:
> 
> 
> Forwarding the work of Tim with an updated SRU description. The
> final commit message would be modified to be a SAUCE patch and get
> the BugLink added.
> As the complete patch is quite big I am only attaching the
> differences between the stand-alone driver and the code added to
> ubuntu/
> 
> -- Stefan
> 
> https://bugs.launchpad.net/ubuntu/+bug/402890
> 
> SRU justification:
> 
> Impact: Hanksville NIC (82577/82588) is a Single-Port GbE ASIC
> supporting Intel 09' platforms (Piketon/Calpella) which time to
> market is ahead of Karmic released. The e1000e driver supporting
> Hanksville has been accepted by upstream which will show up in .31
> kernel.
> We're requesting a driver update for Jaunty, for Karmic it should be
> supported by nature.
> 
> Fix: The stand-alone e1000e driver which is compatible with .28
> kernel is now available on "http://sourceforge.net/projects/e1000/".
> To add support in Jaunty, we will take the stand-alone driver and
> remove all pci id's already covered by the in-kernel driver and also
> rename the module.
> 
> Testcase: Install the resulting kernel package on older and newer
> hardware. Currently supported ethernet models would run with the
> current driver and newer models with the modified one.
> 
> The following changes since commit 801a789933c36d801f81da087b20ea104a12f6bf:
>   Jerone Young (1):
>         UBUNTU: SAUCE: Fix Soltech TA12 volume hotkeys not sending
> key release in Jaunty
> 
> are available in the git repository at:
> 
>   git://kernel.ubuntu.com/rtg/ubuntu-jaunty e1000e-next
> 
> Tim Gardner (1):
>       UBUNTU: Added e1000e from sourceforge.
> 
>  debian/config/amd64/config             |    1 +
>  debian/config/armel/config.iop32x      |    1 +
>  debian/config/armel/config.ixp4xx      |    1 +
>  debian/config/armel/config.versatile   |    1 +
>  debian/config/i386/config              |    1 +
>  debian/config/lpia/config              |    1 +
>  ubuntu/Kconfig                         |    1 +
>  ubuntu/e1000e-next/BOM                 |    3 +
>  ubuntu/e1000e-next/Kconfig             |   20 +
>  ubuntu/e1000e-next/Makefile            |   37 +
>  ubuntu/e1000e-next/e1000.h             |  673 ++++
>  ubuntu/e1000e-next/e1000_80003es2lan.c | 1474 ++++++++
>  ubuntu/e1000e-next/e1000_80003es2lan.h |   95 +
>  ubuntu/e1000e-next/e1000_82571.c       | 1767 ++++++++++
>  ubuntu/e1000e-next/e1000_82571.h       |   53 +
>  ubuntu/e1000e-next/e1000_defines.h     | 1466 ++++++++
>  ubuntu/e1000e-next/e1000_ich8lan.c     | 3042 ++++++++++++++++
>  ubuntu/e1000e-next/e1000_ich8lan.h     |  167 +
>  ubuntu/e1000e-next/e1000_mac.c         | 1864 ++++++++++
>  ubuntu/e1000e-next/e1000_mac.h         |   77 +
>  ubuntu/e1000e-next/e1000_manage.c      |  365 ++
>  ubuntu/e1000e-next/e1000_manage.h      |   82 +
>  ubuntu/e1000e-next/e1000_nvm.c         |  594 ++++
>  ubuntu/e1000e-next/e1000_nvm.h         |   51 +
>  ubuntu/e1000e-next/e1000_phy.c         | 3137 +++++++++++++++++
>  ubuntu/e1000e-next/e1000_phy.h         |  238 ++
>  ubuntu/e1000e-next/e1000_regs.h        |  338 ++
>  ubuntu/e1000e-next/ethtool.c           | 2033 +++++++++++
>  ubuntu/e1000e-next/hw.h                |  701 ++++
>  ubuntu/e1000e-next/kcompat.c           |  480 +++
>  ubuntu/e1000e-next/kcompat.h           | 1704 +++++++++
>  ubuntu/e1000e-next/kcompat_ethtool.c   | 1169 +++++++
>  ubuntu/e1000e-next/netdev.c            | 5953 ++++++++++++++++++++++++++++++++
>  ubuntu/e1000e-next/param.c             |  466 +++
>  34 files changed, 28056 insertions(+), 0 deletions(-)
>  create mode 100644 ubuntu/e1000e-next/BOM
>  create mode 100644 ubuntu/e1000e-next/Kconfig
>  create mode 100644 ubuntu/e1000e-next/Makefile
>  create mode 100644 ubuntu/e1000e-next/e1000.h
>  create mode 100644 ubuntu/e1000e-next/e1000_80003es2lan.c
>  create mode 100644 ubuntu/e1000e-next/e1000_80003es2lan.h
>  create mode 100644 ubuntu/e1000e-next/e1000_82571.c
>  create mode 100644 ubuntu/e1000e-next/e1000_82571.h
>  create mode 100644 ubuntu/e1000e-next/e1000_defines.h
>  create mode 100644 ubuntu/e1000e-next/e1000_ich8lan.c
>  create mode 100644 ubuntu/e1000e-next/e1000_ich8lan.h
>  create mode 100644 ubuntu/e1000e-next/e1000_mac.c
>  create mode 100644 ubuntu/e1000e-next/e1000_mac.h
>  create mode 100644 ubuntu/e1000e-next/e1000_manage.c
>  create mode 100644 ubuntu/e1000e-next/e1000_manage.h
>  create mode 100644 ubuntu/e1000e-next/e1000_nvm.c
>  create mode 100644 ubuntu/e1000e-next/e1000_nvm.h
>  create mode 100644 ubuntu/e1000e-next/e1000_phy.c
>  create mode 100644 ubuntu/e1000e-next/e1000_phy.h
>  create mode 100644 ubuntu/e1000e-next/e1000_regs.h
>  create mode 100644 ubuntu/e1000e-next/ethtool.c
>  create mode 100644 ubuntu/e1000e-next/hw.h
>  create mode 100644 ubuntu/e1000e-next/kcompat.c
>  create mode 100644 ubuntu/e1000e-next/kcompat.h
>  create mode 100644 ubuntu/e1000e-next/kcompat_ethtool.c
>  create mode 100644 ubuntu/e1000e-next/netdev.c
>  create mode 100644 ubuntu/e1000e-next/param.c
> 
> ---

Ok, I had a look over this driver.  Seems to build and only produce
module aliases for the 6 new devices.  Looks good to me.  Seems
appropriately conservative for SRU.

ACK.

-apw