Stephen Hemminger <[EMAIL PROTECTED]> :
[...]
> > +   struct {
> > +           u32 field;
> > +           unsigned int len;
> > +   } p[] = {
> > +           { GMII_PREAMBLE,        32 },   /* Preamble */
> > +           { GMII_ST,              2  },   /* ST */
> > +           { GMII_READ,            2  },   /* OP */
> > +           { phy_id,               5  },   /* PHYAD */
> > +           { phy_reg,              5  },   /* REGAD */
> > +           { 0x0000,               2  },   /* TA */
> > +           { 0x0000,               16 },   /* DATA */
> > +           { 0x0000,               1  }    /* IDLE */
> > +   };
> 
> This could be declared static const, since it doesn't change.

phy_id and phy_reg do change. It can be worked around but I see
no really nice solution. Any suggestion ?

-- 
Ueimor
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to