On Tue, Mar 20, 2007 at 10:29:12PM +0300, Valentine Barshak wrote:

> diff -ruN linux.orig/drivers/net/ibm_emac/ibm_emac_core.c 
> linux/drivers/net/ibm_emac/ibm_emac_core.c
> --- linux.orig/drivers/net/ibm_emac/ibm_emac_core.c   2007-03-16 
> 18:03:51.000000000 +0300
> +++ linux/drivers/net/ibm_emac/ibm_emac_core.c        2007-03-18 
> 18:53:08.000000000 +0300
> @@ -86,7 +86,8 @@
>  static u32 busy_phy_map;
>  
>  #if defined(CONFIG_IBM_EMAC_PHY_RX_CLK_FIX) && \
> -    (defined(CONFIG_405EP) || defined(CONFIG_440EP) || defined(CONFIG_440GR))
> +    (defined(CONFIG_405EP) || defined(CONFIG_440EP) || defined(CONFIG_440GR) 
> || \
> +     defined(CONFIG_440EPX) || defined(CONFIG_440GRX))

This is getting out of control. Please add a config option for this,
and add "select" statements to the Kconfig file where any of the above
CONFIG_4.* are defined. 

>  /* 405EP has "EMAC to PHY Control Register" (CPC0_EPCTL) which can help us
>   * with PHY RX clock problem.
>   * 440EP/440GR has more sane SDR0_MFR register implementation than 440GX, 
> which
> diff -ruN linux.orig/drivers/net/ibm_emac/ibm_emac.h 
> linux/drivers/net/ibm_emac/ibm_emac.h
> --- linux.orig/drivers/net/ibm_emac/ibm_emac.h        2007-03-16 
> 18:03:51.000000000 +0300
> +++ linux/drivers/net/ibm_emac/ibm_emac.h     2007-03-18 18:53:08.000000000 
> +0300
> @@ -26,7 +26,7 @@
>  #if !defined(CONFIG_405GP) && !defined(CONFIG_405GPR) && 
> !defined(CONFIG_405EP) && \
>      !defined(CONFIG_440GP) && !defined(CONFIG_440GX) && 
> !defined(CONFIG_440SP) && \
>      !defined(CONFIG_440EP) && !defined(CONFIG_NP405H) && 
> !defined(CONFIG_440SPE) && \
> -    !defined(CONFIG_440GR)
> +    !defined(CONFIG_440GR) && !defined(CONFIG_440EPX) && 
> !defined(CONFIG_440GRX)

Same here. I know you only added a couple more, but it's reached critical
mass (well, it did a while ago).



-Olof
_______________________________________________
Linuxppc-embedded mailing list
[email protected]
https://ozlabs.org/mailman/listinfo/linuxppc-embedded

Reply via email to