Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1487940
| From | Joel Stanley <joel@jms.id.au> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH net-next 6/7] net/faraday: Fix phy link irq on Aspeed G5 SoCs |
| Date | 2016-09-21 11:20 +0200 |
| Message-ID | <sjLR8-78t-23@gated-at.bofh.it> (permalink) |
| References | (1 earlier) <sjmSJ-7Pm-1@gated-at.bofh.it> <sjsbL-2RR-3@gated-at.bofh.it> <sjv9D-4MD-3@gated-at.bofh.it> <sjF8Z-2O1-5@gated-at.bofh.it> <sjLHs-74R-23@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Wed, Sep 21, 2016 at 6:33 PM, Benjamin Herrenschmidt <benh@kernel.crashing.org> wrote: > On Wed, 2016-09-21 at 11:32 +0930, Joel Stanley wrote: >> I had a look at the eval board schematic and it appears that the line >> has pull down resistors on it, explaining why the IRQ fires when it's >> configured to active low. Other machines re-use the pin pin as a GPIO. >> So yes, I will change this to a dt property in v2. That will mean >> dropping 4/7 "net/faraday: Avoid PHYSTS_CHG interrupt" as well. > > What line is it out of the PHY ? The PHY IRQ ? If yes then it's meant > to be telling you to go look at the PHY registers for a link status > change, but only works if the PHY has also been configured > appropriately... Yep, PHY IRQ. > Mostly we ignore those things in Linux and just poll the PHY. That's simpler. It's what we're doing on Aspeed systems when using NSCI already. The driver is already polling the PHY, I propose we mask out this interrupt for all systems. I gave that a run on my ast2500evb and it behaved itself. Cheers, Joe
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH net-next 6/7] net/faraday: Fix phy link irq on Aspeed G5 SoCs Joel Stanley <joel@jms.id.au> - 2016-09-20 08:40 +0200
Re: [PATCH net-next 6/7] net/faraday: Fix phy link irq on Aspeed G5 SoCs Benjamin Herrenschmidt <benh@kernel.crashing.org> - 2016-09-20 14:20 +0200
Re: [PATCH net-next 6/7] net/faraday: Fix phy link irq on Aspeed G5 SoCs Andrew Lunn <andrew@lunn.ch> - 2016-09-20 17:30 +0200
Re: [PATCH net-next 6/7] net/faraday: Fix phy link irq on Aspeed G5 SoCs Joel Stanley <joel@jms.id.au> - 2016-09-21 04:10 +0200
Re: [PATCH net-next 6/7] net/faraday: Fix phy link irq on Aspeed G5 SoCs Benjamin Herrenschmidt <benh@kernel.crashing.org> - 2016-09-21 11:10 +0200
Re: [PATCH net-next 6/7] net/faraday: Fix phy link irq on Aspeed G5 SoCs Joel Stanley <joel@jms.id.au> - 2016-09-21 11:20 +0200
Re: [PATCH net-next 6/7] net/faraday: Fix phy link irq on Aspeed G5 SoCs Benjamin Herrenschmidt <benh@kernel.crashing.org> - 2016-09-21 12:50 +0200
Re: [PATCH net-next 6/7] net/faraday: Fix phy link irq on Aspeed G5 SoCs Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> - 2016-09-20 14:20 +0200
csiph-web