Re: [PATCH] alx: add a simple AR816x/AR817x device driver

[Johannes Berg <johannes@sipsolutions.net>]

On Sun, 2013-06-16 at 11:49 +0200, Johannes Stezenbach wrote:

> > > Can you check if one of RX actually works? It seems TX doesn't, at least
> > > given the message, but it'd be good to know if RX actually worked? Just
> > > trying ping obviously wouldn't determine that, maybe try tcpdump?
> 
> I tried it but no packets are received, and no irqs generated.

Ok.

> > I'm assuming you did try the QCA driver and it works?
> 
> It worked at some time in the past, but I just tried again
> and now it doesn't work, it spits a large register dump
> periodically and then does task:alx_reinit.  I'll try
> to backtrack to find the working version, but give me
> some time.

Ouch. Which tree are you testing there, btw? I found a handful of trees
on github.

> (Usually I don't use eth on this maching except for doing
> bulk data copies to my laptop, which happens very rarely.
> For the most time I'm using ath9k_htc WLAN USB dongle.)
> 
> Another thing: suspend to disk fails (also happened with QCA driver):
> 
> Jun 16 00:54:32 abc kernel: [93820.564582] Suspending console(s) (use no_console_suspend to debug)
> Jun 16 00:54:32 abc kernel: [93820.568633] serial 00:0a: disabled
> Jun 16 00:54:32 abc kernel: [93820.568644] serial 00:0a: System wakeup disabled by ACPI
> Jun 16 00:54:32 abc kernel: [93820.569407] alx 0000:03:00.0: PHY SPD/DPLX unresolved: 0xffff
> Jun 16 00:54:32 abc kernel: [93820.569410] alx 0000:03:00.0: shutdown fail in suspend -22
> Jun 16 00:54:32 abc kernel: [93820.569417] pci_pm_freeze(): alx_suspend+0x0/0x75 returns -22
> Jun 16 00:54:32 abc kernel: [93820.569421] dpm_run_callback(): pci_pm_freeze+0x0/0x8f returns -22
> Jun 16 00:54:32 abc kernel: [93820.569427] PM: Device 0000:03:00.0 failed to freeze async: error -22
> 
> (I think it worked the first time but failed on second suspend)

Hmm. I don't really have a use for suspend/resume here, I'll poke at it
but ... maybe I should just remove WOL and kill the chip in suspend. :)

johannes