Re: [PATCH v2] net/phy: micrel: Reenable interrupts during resume

2015-07-31 Thread David Miller
From: Nathan Sullivan Date: Fri, 31 Jul 2015 09:27:06 -0500 > I believe I found the issue, we are using this PHY with cadence macb as > the MAC. The driver currently turns off the management port in > macb_reset_hw, which we have stopped with a local change since our hardware > typically has

Re: [PATCH v2] net/phy: micrel: Reenable interrupts during resume

2015-07-31 Thread Nathan Sullivan
On Fri, Jul 31, 2015 at 12:22:04AM -0700, David Miller wrote: > From: Nathan Sullivan > Date: Thu, 30 Jul 2015 18:09:05 -0500 > > > On Thu, Jul 30, 2015 at 10:00:34AM -0700, David Miller wrote: > >> From: Nathan Sullivan > >> Date: Thu, 30 Jul 2015 10:15:48 -0500 > >> > >> > Changes for V2:

Re: [PATCH v2] net/phy: micrel: Reenable interrupts during resume

2015-07-31 Thread David Miller
From: Nathan Sullivan Date: Thu, 30 Jul 2015 18:09:05 -0500 > On Thu, Jul 30, 2015 at 10:00:34AM -0700, David Miller wrote: >> From: Nathan Sullivan >> Date: Thu, 30 Jul 2015 10:15:48 -0500 >> >> > Changes for V2: Actually make sure it compiles this time. >> >> If V1 didn't compile, even for

Re: [PATCH v2] net/phy: micrel: Reenable interrupts during resume

2015-07-31 Thread Nathan Sullivan
On Fri, Jul 31, 2015 at 12:22:04AM -0700, David Miller wrote: From: Nathan Sullivan nathan.sulli...@ni.com Date: Thu, 30 Jul 2015 18:09:05 -0500 On Thu, Jul 30, 2015 at 10:00:34AM -0700, David Miller wrote: From: Nathan Sullivan nathan.sulli...@ni.com Date: Thu, 30 Jul 2015 10:15:48

Re: [PATCH v2] net/phy: micrel: Reenable interrupts during resume

2015-07-31 Thread David Miller
From: Nathan Sullivan nathan.sulli...@ni.com Date: Fri, 31 Jul 2015 09:27:06 -0500 I believe I found the issue, we are using this PHY with cadence macb as the MAC. The driver currently turns off the management port in macb_reset_hw, which we have stopped with a local change since our hardware

Re: [PATCH v2] net/phy: micrel: Reenable interrupts during resume

2015-07-31 Thread David Miller
From: Nathan Sullivan nathan.sulli...@ni.com Date: Thu, 30 Jul 2015 18:09:05 -0500 On Thu, Jul 30, 2015 at 10:00:34AM -0700, David Miller wrote: From: Nathan Sullivan nathan.sulli...@ni.com Date: Thu, 30 Jul 2015 10:15:48 -0500 Changes for V2: Actually make sure it compiles this time.

Re: [PATCH v2] net/phy: micrel: Reenable interrupts during resume

2015-07-30 Thread Nathan Sullivan
On Thu, Jul 30, 2015 at 10:00:34AM -0700, David Miller wrote: > From: Nathan Sullivan > Date: Thu, 30 Jul 2015 10:15:48 -0500 > > > Changes for V2: Actually make sure it compiles this time. > > If V1 didn't compile, even for you, then I have a big problem. > > And that problem is that you

Re: [PATCH v2] net/phy: micrel: Reenable interrupts during resume

2015-07-30 Thread Sergei Shtylyov
Hello. On 07/30/2015 06:15 PM, Nathan Sullivan wrote: The ksz9031 has a behavior where it will clear the interrupt enable bits when leaving power down. To work around this, make sure the interrupt bits are in the state they are expected to be when resuming. Signed-off-by: Nathan Sullivan

Re: [PATCH v2] net/phy: micrel: Reenable interrupts during resume

2015-07-30 Thread David Miller
From: Nathan Sullivan Date: Thu, 30 Jul 2015 10:15:48 -0500 > Changes for V2: Actually make sure it compiles this time. If V1 didn't compile, even for you, then I have a big problem. And that problem is that you didn't test this change at all. -- To unsubscribe from this list: send the line

Re: [PATCH v2] net/phy: micrel: Reenable interrupts during resume

2015-07-30 Thread Florian Fainelli
On 30/07/15 08:15, Nathan Sullivan wrote: > The ksz9031 has a behavior where it will clear the interrupt enable bits > when leaving power down. To work around this, make sure the interrupt > bits are in the state they are expected to be when resuming. > > Signed-off-by: Nathan Sullivan

[PATCH v2] net/phy: micrel: Reenable interrupts during resume

2015-07-30 Thread Nathan Sullivan
The ksz9031 has a behavior where it will clear the interrupt enable bits when leaving power down. To work around this, make sure the interrupt bits are in the state they are expected to be when resuming. Signed-off-by: Nathan Sullivan --- Changes for V2: Actually make sure it compiles this

[PATCH v2] net/phy: micrel: Reenable interrupts during resume

2015-07-30 Thread Nathan Sullivan
The ksz9031 has a behavior where it will clear the interrupt enable bits when leaving power down. To work around this, make sure the interrupt bits are in the state they are expected to be when resuming. Signed-off-by: Nathan Sullivan nathan.sulli...@ni.com --- Changes for V2: Actually make

Re: [PATCH v2] net/phy: micrel: Reenable interrupts during resume

2015-07-30 Thread Nathan Sullivan
On Thu, Jul 30, 2015 at 10:00:34AM -0700, David Miller wrote: From: Nathan Sullivan nathan.sulli...@ni.com Date: Thu, 30 Jul 2015 10:15:48 -0500 Changes for V2: Actually make sure it compiles this time. If V1 didn't compile, even for you, then I have a big problem. And that problem is

Re: [PATCH v2] net/phy: micrel: Reenable interrupts during resume

2015-07-30 Thread David Miller
From: Nathan Sullivan nathan.sulli...@ni.com Date: Thu, 30 Jul 2015 10:15:48 -0500 Changes for V2: Actually make sure it compiles this time. If V1 didn't compile, even for you, then I have a big problem. And that problem is that you didn't test this change at all. -- To unsubscribe from this

Re: [PATCH v2] net/phy: micrel: Reenable interrupts during resume

2015-07-30 Thread Florian Fainelli
On 30/07/15 08:15, Nathan Sullivan wrote: The ksz9031 has a behavior where it will clear the interrupt enable bits when leaving power down. To work around this, make sure the interrupt bits are in the state they are expected to be when resuming. Signed-off-by: Nathan Sullivan

Re: [PATCH v2] net/phy: micrel: Reenable interrupts during resume

2015-07-30 Thread Sergei Shtylyov
Hello. On 07/30/2015 06:15 PM, Nathan Sullivan wrote: The ksz9031 has a behavior where it will clear the interrupt enable bits when leaving power down. To work around this, make sure the interrupt bits are in the state they are expected to be when resuming. Signed-off-by: Nathan Sullivan