netdev - Re: [PATCH] bfin_mac: Fix MDIO clock frequency

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:	Tue, 29 Jan 2008 22:05:44 +0200 (EET)
From:	Kalle Pokki <kalle.pokki@....com>
To:	Bryan Wu <bryan.wu@...log.com>
cc:	netdev@...r.kernel.org
Subject: Re: [PATCH] bfin_mac: Fix MDIO clock frequency

On Wed, 30 Jan 2008, Bryan Wu wrote:

> 
> On Tue, 2008-01-29 at 20:27 +0800, Kalle Pokki wrote:
> > The clock divisor is set to all ones at reset. 
> > --- 
> >  drivers/net/bfin_mac.c |    2 +- 
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/drivers/net/bfin_mac.c b/drivers/net/bfin_mac.c 
> > index eb97175..c199633 100644 
> > --- a/drivers/net/bfin_mac.c 
> > +++ b/drivers/net/bfin_mac.c 
> > @@ -425,7 +425,7 @@ static int mii_probe(struct net_device *dev) 
> >   
> >         /* MDC  = 2.5 MHz */ 
> >         sysctl = bfin_read_EMAC_SYSCTL(); 
> > -       sysctl |= SET_MDCDIV(24); 
> > +       sysctl = (sysctl & ~MDCDIV) | SET_MDCDIV(24); 
> 
> Good catch, I will apply this. 
> Actually, another patch will modified this operation, you patch can not
> be applied cleanly:
> 
> >From e347fc2fdb8d8ecd8fa563929fcfa51759d5ac1c Mon Sep 17 00:00:00 2001
> From: Bryan Wu <bryan.wu@...log.com>
> Date: Fri, 11 Jan 2008 15:17:03 +0800
> Subject: [PATCH] [Blackfin] EMAC driver: define MDC_CLK=2.5MHz and caculate mdc_div according to SCLK.
> 
> Signed-off-by: Bryan Wu <bryan.wu@...log.com>
> ---
>  drivers/net/bfin_mac.c |   16 ++++++++++++----
>  1 files changed, 12 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/net/bfin_mac.c b/drivers/net/bfin_mac.c
> index 4006a5d..ee39819 100644
> --- a/drivers/net/bfin_mac.c
> +++ b/drivers/net/bfin_mac.c
> @@ -412,20 +412,26 @@ static void bf537_adjust_link(struct net_device *dev)
>  	spin_unlock_irqrestore(&lp->lock, flags);
>  }
>  
> +/* MDC  = 2.5 MHz */
> +#define MDC_CLK 2500000
> +
>  static int mii_probe(struct net_device *dev)
>  {
>  	struct bf537mac_local *lp = netdev_priv(dev);
>  	struct phy_device *phydev = NULL;
>  	unsigned short sysctl;
>  	int i;
> +	u32 sclk, mdc_div;
>  
>  	/* Enable PHY output early */
>  	if (!(bfin_read_VR_CTL() & PHYCLKOE))
>  		bfin_write_VR_CTL(bfin_read_VR_CTL() | PHYCLKOE);
>  
> -	/* MDC  = 2.5 MHz */
> +	sclk = get_sclk();
> +	mdc_div = ((sclk / MDC_CLK) / 2) - 1;
> +
>  	sysctl = bfin_read_EMAC_SYSCTL();
> -	sysctl |= SET_MDCDIV(24);
> +	sysctl |= SET_MDCDIV(mdc_div);
>  	bfin_write_EMAC_SYSCTL(sysctl);
>  
>  	/* search for connect PHY device */
> @@ -477,8 +483,10 @@ static int mii_probe(struct net_device *dev)
>  	lp->phydev = phydev;
>  
>  	printk(KERN_INFO "%s: attached PHY driver [%s] "
> -	       "(mii_bus:phy_addr=%s, irq=%d)\n",
> -	       DRV_NAME, phydev->drv->name, phydev->dev.bus_id, phydev->irq);
> +	       "(mii_bus:phy_addr=%s, irq=%d, mdc_clk=%dHz(mdc_div=%d)"
> +	       "@sclk=%dMHz)\n",
> +	       DRV_NAME, phydev->drv->name, phydev->dev.bus_id, phydev->irq,
> +	       MDC_CLK, mdc_div, sclk/1000000);
>  
>  	return 0;
>  }
> -- 
> 1.5.3.4
> 
> Do you mind I modified you patch to against this one and send out as a patch set to upstream?

No, I don't mind.
--
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