Re: [v9, 6/6] fsl/fman: Add FMan MAC driver

2015-12-08 Thread Andy Fleming
On Thu, Dec 3, 2015 at 1:19 AM, wrote: > From: Igal Liberman > > This patch adds the Ethernet MAC driver supporting the three > different types of MACs: dTSEC, tGEC and mEMAC. > > Signed-off-by: Igal Liberman

Re: [PATCH] net: phy: fix PHY_RUNNING in phy_state_machine

2015-08-23 Thread Andy Fleming
On Mon, Aug 17, 2015 at 2:18 PM, David Miller da...@davemloft.net wrote: From: shh@gmail.com Date: Fri, 14 Aug 2015 12:23:40 +0800 From: Shaohui Xie shaohui@freescale.com Currently, if phy state is PHY_RUNNING, we always register a CHANGE when phy works in polling or interrupt

Re: Marvell PHY driver fix

2008-02-22 Thread Andy Fleming
0 +#define MII_M_SOFTWARE_RESET 0x8000 You don't need to do this. There are already constants for both of those values. That's a standard register. MODULE_DESCRIPTION(Marvell PHY driver); MODULE_AUTHOR(Andy Fleming); @@ -141,12 +160,22 @@ static int

Re: [PATCH RFC] phylib: fix forced mode misbehaviour for aneg off case

2008-02-22 Thread Andy Fleming
On Feb 22, 2008, at 09:55, Anton Vorontsov wrote: When user disabled autonegotiation via ethtool, and no link is detected, phylib will place phy into forcing mode, and then will start calling phy_force_reduction(). This will break user expectations. For example, user asks for fixed speed

[PATCH v2.6.25] gianfar: don't pass NULL dev ptr to DMA ops

2008-02-18 Thread Andy Fleming
From: Becky Bruce [EMAIL PROTECTED] Change all dma op invocations in gianfar.c to actually pass in the device pointer. Currently, the value is ignored, but it will be used going forward as we implement archdata for 32-bit powerpc. Signed-off-by: Becky Bruce [EMAIL PROTECTED] Acked-by: Andy

[PATCH] Fix PHY Lib support for gianfar and ucc_geth

2008-02-05 Thread Andy Fleming
The PHY Lib now uses mutexes instead of spin_locks. ucc_geth and gianfar both grab the locks in their mdio_reset functions, so they need to use mutex_(un)lock instead. This was not caught until someone tested it on an SMP system. Signed-off-by: Andy Fleming [EMAIL PROTECTED] --- drivers/net

Re: [PATCH] PHYLIB: Locking fixes for PHY I/O potentially sleeping

2008-01-28 Thread Andy Fleming
Jeff, Dave, any chance we can get this one in for 2.6.25? It will allow a number of other drivers to start using PHY Lib. I'm sure Nate can resend if needed. On Jan 3, 2008, at 17:36, Nate Case wrote: PHY read/write functions can potentially sleep (e.g., a PHY accessed via I2C). The

Re: [PATCH] PHYLIB: Locking fixes for PHY I/O potentially sleeping

2008-01-21 Thread Andy Fleming
them from an interrupt context. * Use work queue for PHY state machine handling since it can potentially sleep * Change phydev lock from spinlock to mutex Signed-off-by: Nate Case [EMAIL PROTECTED] Ok, I've tested this on my system and it works. Acked-by: Andy Fleming [EMAIL

Re: [PATCH][v2] phylib: add module owner to the mii_bus structure

2008-01-14 Thread Andy Fleming
On Dec 28, 2007, at 11:31, Ionut Nicu wrote: Prevent unloading mii bus driver module when other modules have references to some phydevs on that bus. Added a new member (module owner) to struct mii_bus and added code to increment the mii bus owner module usage count on phy_connect and

Re: [PATCH]drivers/net/phy/: default return value in ioctl phy.c

2008-01-14 Thread Andy Fleming
abstraction layer. I 've tetsted this with the macb driver. Signed-off-by: Rini van Zetten [EMAIL PROTECTED] Acked-by: Andy Fleming [EMAIL PROTECTED] -- To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More majordomo info at http

Re: [PATCH] Stop phy code from returning success to unknown ioctls.

2007-12-04 Thread Andy Fleming
big Power Management:off Signed-off-by: David Woodhouse [EMAIL PROTECTED] d'oh! Acked-by: Andy Fleming [EMAIL PROTECTED] diff --git a/drivers/net/phy/phy.c b/drivers/net/phy/phy.c index 9bc1177..7c9e6e3 100644 --- a/drivers/net/phy/phy.c +++ b/drivers/net/phy/phy.c @@ -406,6 +406,9

Re: [PATCH,RFC] ep93xx_eth: conversion to phylib framework

2007-11-21 Thread Andy Fleming
is that we might want to have support for checking preamble suppression support in the PHY Lib, itself. Acked-by: Andy Fleming [EMAIL PROTECTED] - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More majordomo info at http

Re: [PATCH] phy: implement release function

2007-09-12 Thread Andy Fleming
function, thus we're getting rid of this badness. Also small hidden bug fixed, hope none other introduced. ;-) Signed-off-by: Anton Vorontsov [EMAIL PROTECTED] Acked-by: Andy Fleming [EMAIL PROTECTED] Andy - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message

Re: [PATCH] phy layer: fix genphy_setup_forced (don't reset)

2007-08-20 Thread Andy Fleming
On Aug 17, 2007, at 01:54, Domen Puncer wrote: Writing BMCR_RESET bit will reset MII_BMCR to default values. This is clearly not what we want. Signed-off-by: Domen Puncer [EMAIL PROTECTED] Acked-by: Andy Fleming [EMAIL PROTECTED] I could have sworn there was a patch that did this, already

[PATCH 1/2] Fix error checking in Vitesse IRQ config

2007-07-18 Thread Andy Fleming
phy_read() returns a negative number if there's an error, but the error-checking code in the Vitesse driver's config_intr function triggers if phy_read() returns non-zero. Correct that. Signed-off-by: Andy Fleming [EMAIL PROTECTED] --- I made a really stupid mistake in the 4 patches I sent out

Re: [PATCH 1/2] Fix error checking in Vitesse IRQ config

2007-07-18 Thread Andy Fleming
On Jul 18, 2007, at 02:00, pradeep singh wrote: On 7/18/07, Andy Fleming [EMAIL PROTECTED] wrote: - if (err) + if (err 0) return err; but would that mean, if phy_read returns 0 it is a success? Yes. phy_read() returns a 32-bit value

[PATCH 2.6.22-rc7 0/4] phy/gianfar: Fixes for gianfar and vitesse RGMII-ID support

2007-07-16 Thread Andy Fleming
://opensource.freescale.com/pub/scm/linux-2.6-85xx.git netdev Andy Fleming (4): Fix Vitesse 824x PHY interrupt acking Add phy-connection-type to gianfar nodes Fix Vitesse RGMII-ID support Fix RGMII-ID handling in gianfar Documentation/powerpc/booting-without-of.txt |6 +++ arch

[PATCH 1/4] phy: Fix Vitesse 824x PHY interrupt acking

2007-07-16 Thread Andy Fleming
condition. So rather than change the PHY Lib, we change the Vitesse driver so it always clears interrupts before disabling them. Further, the ack function only clears the interrupt if interrupts are enabled. Signed-off-by: Andy Fleming [EMAIL PROTECTED] Signed-off-by: York Sun [EMAIL PROTECTED] Acked

[PATCH 3/4] phy: Fix Vitesse RGMII-ID support

2007-07-16 Thread Andy Fleming
The Vitesse PHY on the 8641D needs to be set up with internal delay to work in RGMII mode. So we add skew when it is set to RGMII_ID mode. Signed-off-by: Andy Fleming [EMAIL PROTECTED] Signed-off-by: Haruki Dai [EMAIL PROTECTED] Signed-off-by: Haiying Wang [EMAIL PROTECTED] --- drivers/net/phy

[PATCH 2/4] gianfar: Add phy-connection-type to gianfar nodes

2007-07-16 Thread Andy Fleming
The TSEC/eTSEC automatically detect their PHY interface type, unless the type is RGMII-ID (RGMII with internal delay). In that situation, it just detects RGMII. In order to fix this, we need to pass in rgmii-id if that is the connection type. Signed-off-by: Andy Fleming [EMAIL PROTECTED

[PATCH 4/4] gianfar: Fix RGMII-ID handling in gianfar

2007-07-16 Thread Andy Fleming
on the 8641D HPCN board where the Vitesse PHY doesn't use the delay for RGMII. Signed-off-by: Andy Fleming [EMAIL PROTECTED] --- arch/powerpc/sysdev/fsl_soc.c |9 + drivers/net/gianfar.c | 12 +++- include/linux/fsl_devices.h |1 + 3 files changed, 21 insertions(+), 1

Re: [PATCH] MII bitbang driver should generate MII bus phy_mask dynamically

2007-06-11 Thread Andy Fleming
On Jun 11, 2007, at 02:53, Mark Zhan wrote: Current MII bitbang bus driver hard-codes the phy mask of mii_bus to ~0x09, which is actually specific to the FSL boards. This patch will make the bitbang driver to generate MII bus phy_mask dynamically, by the PHY irq info provided by the

Re: [PATCH] phylib: phy_ethtool_{sset,gset} -- check phydev for NULL

2007-05-21 Thread Andy Fleming
On May 11, 2007, at 05:58, Matvejchikov Ilya wrote: Signed-off-by: Matvejchikov Ilya [EMAIL PROTECTED] Acked-by: Andy Fleming [EMAIL PROTECTED] - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More majordomo info at http

Re: [PATCH 28/46] PHY: remove rwsem use from phy core

2007-04-27 Thread Andy Fleming
-by: Greg Kroah-Hartman [EMAIL PROTECTED] Acked-by: Andy Fleming [EMAIL PROTECTED] --- I think I copied that code from elsewhere without truly understanding it. *bows head in shame* As such, I have no objection to this patch unless someone says it breaks their board. :) drivers/net

Re: phylib usage

2007-04-14 Thread Andy Fleming
On Apr 13, 2007, at 09:53, David Hollis wrote: On Wed, 2007-04-11 at 16:42 -0500, Kim Phillips wrote: On Wed, 11 Apr 2007 03:03:56 +0200 Lennert Buytenhek [EMAIL PROTECTED] wrote: There may not be as much, but there definitely are still cases where there are devices that may have one of

Re: [patch 1/1] network: add the missing phy_device speed information to phy_mii_ioctl

2007-03-06 Thread Andy Fleming
On Mar 2, 2007, at 11:42, Shan Lu wrote: Changelog: Function `phy_mii_ioctl' returns physical device's information based on user requests. When requested to return the basic mode control register information (BMCR), the original implementation only returns the physical device's duplex

Re: [PATCH v2, resend] gianfar: don't duplicate gfar_error()

2007-02-15 Thread Andy Fleming
itself as gfar_{receive/transmit/error}() do it anyway. While at it, also clarify/cleanup debug messages in gfar_error()... Signed-off-by: Sergei Shtylyov [EMAIL PROTECTED] Acked-by: Andy Fleming [EMAIL PROTECTED] --- The patch survived netperf stressing on MPC8540ADS realtime kernel

Re: [PATCH][RESPIN] gianfar: Convert network devices to use struct device instead of class_device

2007-02-12 Thread Andy Fleming
On Feb 9, 2007, at 15:33, Kumar Gala wrote: Convert network devices to use struct device instead of class_device. Greg missed this one in his cleanup path. Signed-off-by: Kumar Gala [EMAIL PROTECTED] Acked-by: Andy Fleming [EMAIL PROTECTED] Jeff, please get this into 2.6.21

Re: [PATCH] RFC: Broadcom PHY forcing fix

2007-02-12 Thread Andy Fleming
On Jan 26, 2007, at 08:36, Maciej W. Rozycki wrote: Kumar, I've got a BCM5461 that requires this fix to be able to force the speeds on the PHY. Not sure if its needed on the other variants or not. The problem is the genphy_config_aneg resets the PHY when forcing the speed and once we

Re: [PATCH] phy devices: use same arg types

2007-02-09 Thread Andy Fleming
) Signed-off-by: Randy Dunlap [EMAIL PROTECTED] Acked-by: Andy Fleming [EMAIL PROTECTED] Oops, I thought I had fixed that before submission. --- drivers/net/phy/phy_device.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- linux-2620-pv.orig/drivers/net/phy/phy_device.c +++ linux

[PATCH 2.6.21] phylib: Add support for Marvell 88e1111S and 88e1145

2007-02-09 Thread Andy Fleming
From Andy Fleming [EMAIL PROTECTED] Changes include: * New support for 88e1145 * New support for 88e111s * Fixing 88e1101 driver to not match non-88e1101 PHYs * Increases in feature support across Marvell PHY product line * Fixes a bunch of whitespace issues found by Lindent Signed-off

Re: Generic PHY lib vs. locking

2006-12-28 Thread Andy Fleming
On Dec 21, 2006, at 22:07, Benjamin Herrenschmidt wrote: Hi Andy ! I've been looking at porting various drivers (EMAC, sungem, spider_net, ...) to the generic PHY stuff. However, I have one significant problem here. One of the things I've been trying to do lately with EMAC and that I plan to

Re: [PATCH] Add Broadcom PHY support

2006-12-05 Thread Andy Fleming
On Dec 5, 2006, at 00:03, Benjamin Herrenschmidt wrote: I believe that this fiber enabling can be done by defining config_init in the phy_driver struct. struct phy_driver { snip /* Called to initialize the PHY, * including after a reset */ int

Re: [patch 3/6] 2.6.18: sb1250-mac: Phylib IRQ handling fixes

2006-12-04 Thread Andy Fleming
On Nov 30, 2006, at 12:07, Maciej W. Rozycki wrote: On Mon, 23 Oct 2006, Maciej W. Rozycki wrote: I'm not too enthusiastic about requiring the ethernet drivers to call phy_disconnect in a separate thread after close is called. Assuming there's not some sort of squash work queue function

[PATCH 2.6.20] PHY: Add support for configuring the PHY connection interface

2006-12-01 Thread Andy Fleming
specified in BSP code\n); return -EINVAL; diff --git a/drivers/net/gianfar.c b/drivers/net/gianfar.c index a06d8d1..4fb9d96 100644 --- a/drivers/net/gianfar.c +++ b/drivers/net/gianfar.c @@ -9,7 +9,7 @@ * Author: Andy Fleming * Maintainer: Kumar Gala * - * Copyright (c) 2002-2004

[PATCH] Add support for configuring the PHY connection interface

2006-11-07 Thread Andy Fleming
/gianfar.c +++ b/drivers/net/gianfar.c @@ -9,7 +9,7 @@ * Author: Andy Fleming * Maintainer: Kumar Gala * - * Copyright (c) 2002-2004 Freescale Semiconductor, Inc. + * Copyright (c) 2002-2006 Freescale Semiconductor, Inc. * * This program is free software; you can redistribute it and/or modify

[PATCH] Add support for Marvell 88e1111S and 88e1145

2006-11-07 Thread Andy Fleming
0x4100 +#define MII_M_PHY_LED_COMBINE 0x411c + MODULE_DESCRIPTION(Marvell PHY driver); MODULE_AUTHOR(Andy Fleming); MODULE_LICENSE(GPL); @@ -64,7 +77,7 @@ static int marvell_config_intr(struct ph { int err; - if(phydev-interrupts == PHY_INTERRUPT_ENABLED

Re: [PATCH] Add support for configuring the PHY connection interface

2006-11-07 Thread Andy Fleming
On Nov 8, 2006, at 00:16, Kumar Gala wrote: On Nov 8, 2006, at 12:10 AM, Andy Fleming wrote: Most PHYs connect to an ethernet controller over a GMII or MII interface. However, a growing number are connected over different interfaces, such as RGMII or SGMII. The ethernet driver will tell

Re: [PATCH/RFC] add netpoll support for gianfar

2006-11-06 Thread Andy Fleming
On Nov 6, 2006, at 05:19, Vitaly Wool wrote: The patch inlined below adds NET_POLL_CONTROLLER support for gianfar network driver. drivers/net/gianfar.c | 34 ++ 1 file changed, 34 insertions(+) Signed-off-by: Vitaly Wool [EMAIL PROTECTED] Index:

Re: [patch 3/6] 2.6.18: sb1250-mac: Phylib IRQ handling fixes

2006-10-20 Thread Andy Fleming
:21.0 + @@ -7,6 +7,7 @@ * Author: Andy Fleming * * Copyright (c) 2004 Freescale Semiconductor, Inc. + * Copyright (c) 2006 Maciej W. Rozycki * * This program is free software; you can redistribute it and/or modify it * under the terms of the GNU General Public License

[PATCH] Fixed a number of bugs in the PHY Layer

2006-10-16 Thread Andy Fleming
* genphy_update_link is now exported * Added a fix from [EMAIL PROTECTED] which changes forcing so it only updates the link. Otherwise, it never tries the lower values, since it is always overwriting the speed/duplex values with the current ones, rather than the intended ones. * Fixed a

Re: RFC: new WIP version of au1000_eth.c phylib conversion (was Re: RFC: au1000_etc.c phylib rewrite)

2006-05-08 Thread Andy Fleming
On May 4, 2006, at 04:17, Herbert Valerio Riedel wrote: Hello, I've tried to adapt the PHY detection code to allow for dynamic runtime configuration (with fallback to search for the 2nd MAC PHY on the 1st MAC's MII bus), as well as selectable static PHY configuration through Kconfig (e.g.

Re: RFC: au1000_etc.c phylib rewrite

2006-05-04 Thread Andy Fleming
On May 3, 2006, at 11:34, Herbert Valerio Riedel wrote: hello * On Tue, 2006-05-02 at 11:20 -0500, Mark Schank wrote: At 08:23 AM 5/2/06 +0200, Herbert Valerio Riedel wrote: On Mon, 2006-05-01 at 15:09 -0500, Mark Schank wrote: The Cogent CSB655 used the Broadcom Dual Phy. They eventually

[PATCH] Fix locking in gianfar

2006-04-12 Thread Andy Fleming
Signed-off-by: Andy Fleming [EMAIL PROTECTED] --- drivers/net/gianfar.c | 56 +- drivers/net/gianfar.h | 67 +++ +- drivers/net/gianfar_ethtool.c | 20 +--- drivers/net/gianfar_sysfs.c | 24

[PATCH] Fix locking in gianfar

2006-04-06 Thread Andy Fleming
-off-by: Andy Fleming [EMAIL PROTECTED] From nobody Mon Sep 17 00:00:00 2001 From: Andrew Fleming [EMAIL PROTECTED] Date: Thu Apr 6 20:18:31 2006 -0500 Subject: [PATCH] * Split gianfar locks into two types: TX and RX * Made it so gfar_start() now clears RHALT * Fixed a bug where calling

Re: [PATCH] gianfar: add netpoll api support

2005-12-12 Thread Andy Fleming
On Dec 12, 2005, at 13:41, Dale Farnsworth wrote: In article [EMAIL PROTECTED] you write: On Mon, Dec 12, 2005 at 11:54:06AM -0700, Dale Farnsworth wrote: +#ifdef CONFIG_NET_POLL_CONTROLLER +/* + * Polling - used by netconsole and other diagnostic tools + * to allow network i/o with

Re: [PATCH] 3c59x: read current link status from phy

2005-09-09 Thread Andy Fleming
On Sep 9, 2005, at 05:10, Bogdan Costescu wrote: On Thu, 8 Sep 2005, Andy Fleming wrote: Is the cost of an extra read every minute really too high? You probably didn't look at the code. The MII registers are not exposed in the PCI space, they need to be accessed through a serial

Re: [PATCH] 3c59x: read current link status from phy

2005-09-08 Thread Andy Fleming
On Sep 8, 2005, at 10:42, Bogdan Costescu wrote: On Thu, 8 Sep 2005, Tommy Christensen wrote: Personally, I'd prefer the delay to be 10 seconds. If you sample every 60 seconds ? Teach Shannon how to do it ;-) If you mean to reduce the sampling period, there is a very good reason not

[PATCH]: PHY Layer fixup

2005-08-24 Thread Andy Fleming
This patch adds back the code that was taken out, thus re-enabling: * The PHY Layer to initialize without crashing * Drivers to actually connect to PHYs * The entire PHY Control Layer This patch is used by the gianfar driver, and other drivers which are in development. Signed-off-by: Andy

PHY Layer changes

2005-08-18 Thread Andy Fleming
. Certainly it's all used by the gianfar ethernet driver (once updated to use the PHY Layer). I'd be happy to work with you to get this code up to... um... code. Andy Fleming Freescale Open Source Team - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message

Re: [PATCH] Gianfar PHY Layer Update

2005-08-11 Thread Andy Fleming
Hmmm I just did an update, and tried applying first the PHY Layer patch, and then the Gianfar PHY Layer patch. It worked fine for me. Did you apply the the 7/28 patch: [PATCH] sysfs feature enablement for gianfar? Because that patch would screw up the PHY patch for gianfar (I sent

[PATCH] sysfs feature support in gianfar (II)

2005-08-03 Thread Andy Fleming
for gianfar. Signed-off-by: Andy Fleming [EMAIL PROTECTED] * Merged stashing/sysfs/exact match support into PHY tree --- commit c65b921e4720850c7532f95caf2dc9341b26efa2 tree 8822bc37587224d6dd54ea3aae070f45b123e55d parent d58caeb7c62c4f2b6c6f81ad6ee4537f0ba5ba60 author Andrew Fleming [EMAIL PROTECTED