[U-Boot] [PATCH v2 0/6] Universal PHY Infrastructure

Detlev Zundel dzu at denx.de
Wed Apr 6 14:07:53 CEST 2011


Hi Andy,

> This second version cleans up all checkpatch errors that I reasonably could,
> and addresses most of the comments from the first round of reviews.

  WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt
  #1299: FILE: drivers/net/tsec.c:1740:
  +static int tsec_send(struct eth_device *dev, volatile void *packet, int length)

Ah I see.  This is needed because include/net.h prescribes this as the
interface.  Oh well, we should clean this up at some later point, so
it's ok from my perspective to ignore this and at least stay consistent.
  
  WARNING: do not add new typedefs
  #4271: FILE: include/phy.h:389:
  +typedef enum {

I know that we have lots of typedefs, but we should not add new ones.
Actually not using this typedef for an enum-type would make the sources
easier for me to read as I don't have to do this extra lookup step.
  
  WARNING: do not add new typedefs
  #6304: FILE: arch/powerpc/include/asm/fsl_enet.h:31:
  +typedef struct tsec_mii_mng {

I'd also appreciate if you use "struct tsec_mii_mng" explicitely instead
of adding another typedef.
  
Overall I agree, checkpatch is much happier now ;)

Cheers
  Detlev

-- 
Cyberwar is certainly not a myth. But you haven't seen it yet, despite
the attacks on Estonia. Cyberwar is warfare in cyberspace. And warfare
involves massive death and destruction. When you see it, you'll know it.
                           -- Bruce Schneier, Nov. 2007
--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu at denx.de


More information about the U-Boot mailing list