On Tue, 29 Jan 2008, Nate Case wrote:

> +static struct phy_driver bcm5482_driver = {
> +    .phy_id          = 0x0143bcb0,
> +     .phy_id_mask    = 0xfffffff0,

 Please check formatting above and also I am a bit curious as to why the 
ID is so different from the other ones -- the number is meant to be based 
on the OUI assigned to the manufacturer.  Otherwise your addition is fine.

  Maciej
--
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