netdev - Re: [PATCH 1/4] forcedeth: add vitesse phy

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Thu, 24 May 2007 17:54:50 -0400
From:	Jeff Garzik <jeff@...zik.org>
To:	Ayaz Abdulla <aabdulla@...dia.com>
CC:	Manfred Spraul <manfred@...orfullife.com>,
	Andrew Morton <akpm@...l.org>, nedev <netdev@...r.kernel.org>
Subject: Re: [PATCH 1/4] forcedeth: add vitesse phy

Ayaz Abdulla wrote:
> This patch adds errata support for the vitesse phy.
> 
> Signed-off-by: Ayaz Abdulla <aabdulla@...dia.com>
> 
> 
> ------------------------------------------------------------------------
> 
> --- old/drivers/net/forcedeth.c	2007-05-01 15:30:53.000000000 -0400
> +++ new/drivers/net/forcedeth.c	2007-05-01 15:30:57.000000000 -0400
> @@ -550,6 +550,7 @@
>  /* PHY defines */
>  #define PHY_OUI_MARVELL	0x5043
>  #define PHY_OUI_CICADA	0x03f1
> +#define PHY_OUI_VITESSE	0x01c1
>  #define PHYID1_OUI_MASK	0x03ff
>  #define PHYID1_OUI_SHFT	6
>  #define PHYID2_OUI_MASK	0xfc00
> @@ -557,12 +558,29 @@
>  #define PHYID2_MODEL_MASK		0x03f0
>  #define PHY_MODEL_MARVELL_E3016		0x220
>  #define PHY_MARVELL_E3016_INITMASK	0x0300
> -#define PHY_INIT1	0x0f000
> -#define PHY_INIT2	0x0e00
> -#define PHY_INIT3	0x01000
> -#define PHY_INIT4	0x0200
> -#define PHY_INIT5	0x0004
> -#define PHY_INIT6	0x02000
> +#define PHY_CICADA_INIT1	0x0f000
> +#define PHY_CICADA_INIT2	0x0e00
> +#define PHY_CICADA_INIT3	0x01000
> +#define PHY_CICADA_INIT4	0x0200
> +#define PHY_CICADA_INIT5	0x0004
> +#define PHY_CICADA_INIT6	0x02000

Please split this into two patches:

1) rename PHY_INIT symbols to PHY_CICADA_INIT

2) add vitesse phy errata workarounds

otherwise, looks OK


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