[U-Boot] [PATCH 1/2] Add ntel PCIE PRO/1000 Network Driver support
Zang Roy-R61911
tie-fei.zang at freescale.com
Tue Jul 21 13:47:36 CEST 2009
> -----Original Message-----
> From: Ben Warren [mailto:biggerbadderben at gmail.com]
> Sent: Tuesday, July 21, 2009 14:17 PM
> To: Zang Roy-R61911
> Cc: Kumar Gala; U-Boot-Denx
> Subject: Re: [U-Boot] [PATCH 1/2] Add ntel PCIE PRO/1000
> Network Driver support
>
> Hi Roy,
> Roy Zang wrote:
> > From: Roy Zang <tie-fei.zang at freescale.com>
> >
> > Based on Intel PRO/1000 Network Driver 7.3.20-k2
> > Add Intel E1000 PCIE card support. The following cards are added:
> > INTEL_82571EB_COPPER
> > INTEL_82571EB_FIBER,
> > INTEL_82571EB_SERDES
> > INTEL_82571EB_QUAD_COPPER
> > INTEL_82571PT_QUAD_COPPER
> > INTEL_82571EB_QUAD_FIBER
> > INTEL_82571EB_QUAD_COPPER_LOWPROFILE
> > INTEL_82571EB_SERDES_DUAL
> > INTEL_82571EB_SERDES_QUAD
> > INTEL_82572EI_COPPER
> > INTEL_82572EI_FIBER
> > INTEL_82572EI_SERDES
> > INTEL_82572EI
> > INTEL_82573E
> > INTEL_82573E_IAMT
> > INTEL_82573L
> > INTEL_82546GB_QUAD_COPPER_KSP3
> > INTEL_80003ES2LAN_COPPER_DPT
> > INTEL_80003ES2LAN_SERDES_DPT
> > INTEL_80003ES2LAN_COPPER_SPT
> > INTEL_80003ES2LAN_SERDES_SPT
> >
> > 82571EB_COPPER dual ports,
> > 82572EI single port,
> > 82572EI_COPPER single port PCIE cards
> > and
> > 82545EM_COPPER,
> > 82541GI_LF
> > pci cards are tested on P2020 board, MPC8536DS
> > and MPC8544DS boards.
> >
> > Signed-off-by: Roy Zang <tie-fei.zang at freescale.com>
> >
> I have no objections to any of the content. A few themes, though:
>
> 1. Please ensure that the multi-line comments are formatted
> correctly.
> Lines 2+ should have a space in the first character.
Do you find this issue in the patch?
> 2. If you insist on including dead code (i.e. #if 0), please
> provide a
> comment on why it's there
Thanks for the understanding. The dead code in the previous file helped
me track the code.
That is why I left them. I will add the comment and remove the
unnecessary one.
> 3. I have a hard time believing that most of this is necessary. Sure
> it's a complicated chip and you're pulling it from Intel's
> driver, but
> 3000+ lines is too much. Please try to remove stuff that's
> not needed.
I will double check it.
Do not believe Intel's code. It take me two days to clean the code
style.
I will resend it after the code is ready.
Thanks.
Roy
More information about the U-Boot
mailing list