Re: [PATCH] b44: power down PHY when interface down

2007-07-01 Thread Michael Buesch
On Sunday 01 July 2007 17:00:06 Lennert Buytenhek wrote: > A multimeter should do the trick, but I would advise against this > if you're not totally comfortable with hacking hardware. Ok, the resistor on the board is 100ohm, which is too big according to the docs of the extender. So what I tried i

Re: [PATCH] b44: power down PHY when interface down

2007-07-01 Thread Michael Buesch
On Sunday 01 July 2007 17:00:06 Lennert Buytenhek wrote: > On Sun, Jul 01, 2007 at 12:23:16PM +0200, Michael Buesch wrote: > > > > More or less. You can't add the resistances like that, since the > > > bus isolation chip buffers the IDSEL signal, but it is correct that > > > if the host's IDSEL r

Re: [PATCH] b44: power down PHY when interface down

2007-07-01 Thread Lennert Buytenhek
On Sun, Jul 01, 2007 at 12:23:16PM +0200, Michael Buesch wrote: > > More or less. You can't add the resistances like that, since the > > bus isolation chip buffers the IDSEL signal, but it is correct that > > if the host's IDSEL resistor is larger than a certain value, the > > combination of the

Re: [PATCH] b44: power down PHY when interface down

2007-07-01 Thread Török Edvin
On 7/1/07, Michael Buesch <[EMAIL PROTECTED]> wrote: > > You patch is also damaged here. > Check your MUA settings. > [Sorry about that, resending again] When the interface is down (or driver removed), the BroadCom 44xx card remains powered on, and both its MAC and PHY is using up power. Thi

Re: [PATCH] b44: power down PHY when interface down

2007-07-01 Thread Török Edvin
On 6/30/07, Matthew Garrett <[EMAIL PROTECTED]> wrote: On Sat, Jun 30, 2007 at 07:44:59AM -0700, Arjan van de Ven wrote: > Matthew Garrett wrote: > >Do you still get link beat detection when the phy is powered down? No. As for link detection on the switch connected to this card: If I have WoL t

Re: [PATCH] b44: power down PHY when interface down

2007-07-01 Thread Michael Buesch
On Sunday 01 July 2007 14:49:23 Török Edvin wrote: > On 6/30/07, Stephen Hemminger <[EMAIL PROTECTED]> wrote: > > > > This is a non-standard formatting for comments, please follow > > Coding style: > > Thanks, I have updated the patch. > > When the interface is down (or driver removed), the Broad

Re: [PATCH] b44: power down PHY when interface down

2007-07-01 Thread Török Edvin
On 6/30/07, Stephen Hemminger <[EMAIL PROTECTED]> wrote: This is a non-standard formatting for comments, please follow Coding style: Thanks, I have updated the patch. When the interface is down (or driver removed), the BroadCom 44xx card remains powered on, and both its MAC and PHY is using u

Re: [PATCH] b44: power down PHY when interface down

2007-07-01 Thread Michael Buesch
On Sunday 01 July 2007 01:17:34 Lennert Buytenhek wrote: > More or less. You can't add the resistances like that, since the > bus isolation chip buffers the IDSEL signal, but it is correct that > if the host's IDSEL resistor is larger than a certain value, the > combination of the resistive coupli

Re: PM policy, hotplug, power saving (was Re: [PATCH] b44: power down PHY when interface down)

2007-06-30 Thread Kyle Moffett
On Jun 30, 2007, at 12:42:06, Jeff Garzik wrote: Definitely matters. Switch renegotiation can take a while, and you must take into account the common case of interface bouncing (immediate down, then up). Hoards actively complained the few times we experimented with this, because of e.g. D

Re: [PATCH] b44: power down PHY when interface down

2007-06-30 Thread Lennert Buytenhek
On Sun, Jul 01, 2007 at 12:24:40AM +0200, Michael Buesch wrote: > > > Hm, I was going to measure the real power advantage with a > > > PCI-extender card. But my B44B0 card doesn't seem to work in > > > that extender card. It works perfectly fine sticked directly into > > > the motherboard, though,

Re: [PATCH] b44: power down PHY when interface down

2007-06-30 Thread Michael Buesch
On Sunday 01 July 2007 00:03:01 Lennert Buytenhek wrote: > On Sat, Jun 30, 2007 at 11:53:25PM +0200, Michael Buesch wrote: > > > > When the interface is down (or driver removed), the BroadCom 44xx card > > > remains > > > powered on, and both its MAC and PHY is using up power. > > > This patch ma

Re: [PATCH] b44: power down PHY when interface down

2007-06-30 Thread Lennert Buytenhek
On Sat, Jun 30, 2007 at 11:53:25PM +0200, Michael Buesch wrote: > > When the interface is down (or driver removed), the BroadCom 44xx card > > remains > > powered on, and both its MAC and PHY is using up power. > > This patch makes the driver issue a MAC_CTRL_PHY_PDOWN when the interface > > is h

Re: [PATCH] b44: power down PHY when interface down

2007-06-30 Thread Michael Buesch
On Saturday 30 June 2007 13:47:35 Török Edvin wrote: > When the interface is down (or driver removed), the BroadCom 44xx card remains > powered on, and both its MAC and PHY is using up power. > This patch makes the driver issue a MAC_CTRL_PHY_PDOWN when the interface > is halted, and does a partial

Re: PM policy, hotplug, power saving (was Re: [PATCH] b44: power down PHY when interface down)

2007-06-30 Thread Stephen Hemminger
On Sat, 30 Jun 2007 12:42:06 -0400 Jeff Garzik <[EMAIL PROTECTED]> wrote: > Arjan van de Ven wrote: > > Matthew Garrett wrote: > >> Do you still get link beat detection when the phy is powered down? > > > does that matter? > > If the interface is down, nic drivers aren't expected to detect link..

PM policy, hotplug, power saving (was Re: [PATCH] b44: power down PHY when interface down)

2007-06-30 Thread Jeff Garzik
Arjan van de Ven wrote: Matthew Garrett wrote: Do you still get link beat detection when the phy is powered down? does that matter? If the interface is down, nic drivers aren't expected to detect link... if userspace wants to find link status it should have the interface up. Definitely ma

Re: [PATCH] b44: power down PHY when interface down

2007-06-30 Thread Lennert Buytenhek
On Sat, Jun 30, 2007 at 04:19:23PM +0100, Matthew Garrett wrote: > I'd agree that there's a need for a state where we power down as much as > possible (even at the cost of functionality), but where possible it > would also be nice to offer a state where the mac is powered down and > the phy lef

Re: [PATCH] b44: power down PHY when interface down

2007-06-30 Thread Matthew Garrett
On Sat, Jun 30, 2007 at 07:44:59AM -0700, Arjan van de Ven wrote: > Matthew Garrett wrote: > >Do you still get link beat detection when the phy is powered down? > > > does that matter? > If the interface is down, nic drivers aren't expected to detect > link... if userspace wants to find link statu

Re: [PATCH] b44: power down PHY when interface down

2007-06-30 Thread Arjan van de Ven
Matthew Garrett wrote: On Sat, Jun 30, 2007 at 02:47:35PM +0300, Török Edvin wrote: When the interface is down (or driver removed), the BroadCom 44xx card remains powered on, and both its MAC and PHY is using up power. This patch makes the driver issue a MAC_CTRL_PHY_PDOWN when the interface is

Re: [PATCH] b44: power down PHY when interface down

2007-06-30 Thread Matthew Garrett
On Sat, Jun 30, 2007 at 02:47:35PM +0300, Török Edvin wrote: > When the interface is down (or driver removed), the BroadCom 44xx card > remains > powered on, and both its MAC and PHY is using up power. > This patch makes the driver issue a MAC_CTRL_PHY_PDOWN when the interface > is halted, and doe

[PATCH] b44: power down PHY when interface down

2007-06-30 Thread Török Edvin
When the interface is down (or driver removed), the BroadCom 44xx card remains powered on, and both its MAC and PHY is using up power. This patch makes the driver issue a MAC_CTRL_PHY_PDOWN when the interface is halted, and does a partial chip reset turns off the activity LEDs too. Applies to 2.6