Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754234AbaBRXN0 (ORCPT ); Tue, 18 Feb 2014 18:13:26 -0500 Received: from shards.monkeyblade.net ([149.20.54.216]:37087 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753129AbaBRXNU (ORCPT ); Tue, 18 Feb 2014 18:13:20 -0500 Date: Tue, 18 Feb 2014 18:13:17 -0500 (EST) Message-Id: <20140218.181317.1712776234149210938.davem@davemloft.net> To: grant.likely@linaro.org Cc: ben.dooks@codethink.co.uk, linux-kernel@lists.codethink.co.uk, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, linux-sh@vger.kernel.org, sergei.shtylyov@cogentembedded.com Subject: Re: [PATCH v2] of_mdio: fix phy interrupt passing From: David Miller In-Reply-To: <20140218161551.44A52C40517@trevor.secretlab.ca> References: <1392725818-558-1-git-send-email-ben.dooks@codethink.co.uk> <20140218161551.44A52C40517@trevor.secretlab.ca> X-Mailer: Mew version 6.5 on Emacs 24.1 / Mule 6.0 (HANACHIRUSATO) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.5.7 (shards.monkeyblade.net [149.20.54.216]); Tue, 18 Feb 2014 15:13:20 -0800 (PST) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Grant Likely Date: Tue, 18 Feb 2014 16:15:51 +0000 > On Tue, 18 Feb 2014 12:16:58 +0000, Ben Dooks wrote: >> The of_mdiobus_register_phy() is not setting phy->irq thus causing >> some drivers to incorrectly assume that the PHY does not have an >> IRQ associated with it. Not only do some drivers report no IRQ >> they do not install an interrupt handler for the PHY. >> >> Simplify the code setting irq and set the phy->irq at the same >> time so that we cover the following issues, which should cover >> all the cases the code will find: >> >> - Set phy->irq if node has irq property and mdio->irq is NULL >> - Set phy->irq if node has no irq and mdio->irq is not NULL >> - Leave phy->irq as PHY_POLL default if none of the above >> >> This fixes the issue: >> net eth0: attached PHY 1 (IRQ -1) to driver Micrel KSZ8041RNLI >> >> to the correct: >> net eth0: attached PHY 1 (IRQ 416) to driver Micrel KSZ8041RNLI >> >> Signed-off-by: Ben Dooks > > Looks okay to me > > Reviewed-by: Grant Likely Applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/