On Thu, Apr 24, 2008 at 9:45 AM, Andre Schwarz
<[EMAIL PROTECTED]> wrote:
> {MIIM_VSC8601_EPHY_CON,MIIM_VSC8601_EPHY_CON_INIT_SKEW,NULL},
> +#if defined(CFG_VSC8601_SKEW_TX) && defined(CFG_VSC8601_SKEW_RX)
> + {MIIM_EXT_PAGE_ACCESS,1,NULL},
> +#define VSC8101_SKEW (CFG_VSC8601_SKEW_TX<<14)|(CFG_VSC8601_SKEW_RX<<12)
> + {MIIM_VSC8601_SKEW_CTRL,VSC8101_SKEW,NULL},
> + {MIIM_EXT_PAGE_ACCESS,0,NULL},
> +#endif
> #endif
I'm not sure this is the best solution to this. It seems like it
wouldn't scale well. Either we need to set a bit somewhere that the
phy driver can read (and thereby determine how to configure the skew),
or we need to set the value to write in the board config file. I'm
partial to the first solution, as it encapsulates the information
inside the code that deals with it.
[...]
> /* Broadcom BCM54xx -- taken from linux sungem_phy */
> #define MIIM_BCM54xx_AUXSTATUS 0x19
> #define MIIM_BCM54xx_AUXSTATUS_LINKMODE_MASK 0x0700
> @@ -163,6 +165,8 @@
> /* Vitesse VSC8601 Extended PHY Control Register 1 */
> #define MIIM_VSC8601_EPHY_CON 0x17
> #define MIIM_VSC8601_EPHY_CON_INIT_SKEW 0x1120
> +#define MIIM_VSC8601_SKEW_CTRL 0x1c
> +#define MIIM_VSC8601_EPHY_CON_INIT_SKEW 0x1120
Am I crazy, or did you just doubly define MIIM_VSC8601_EPHY_CON_INIT_SKEW?
-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference
Don't miss this year's exciting event. There's still time to save $100.
Use priority code J8TL2D2.
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
_______________________________________________
U-Boot-Users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/u-boot-users
