From mboxrd@z Thu Jan 1 00:00:00 1970 From: Grant Likely Subject: Re: [PATCH] Allow MACB to connect to a higher addresses PHY. Date: Wed, 31 Mar 2010 00:08:28 -0600 Message-ID: References: <1269943112-6975-1-git-send-email-anders.darander@gmail.com> <20100331055306.GC4837@datarespons.se> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: "David S. Miller" , Ralf Baechle , Maxime Bizon , David Daney , Sekhar Nori , Anton Vorontsov , Andy Fleming , netdev@vger.kernel.org, linux-kernel@vger.kernel.org To: Anders Darander Return-path: In-Reply-To: <20100331055306.GC4837@datarespons.se> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Tue, Mar 30, 2010 at 11:53 PM, Anders Darander wrote: > * Grant Likely [100330 16:42]: > One question, if I add such an kernel parameter, would it be OK to > implement something like this in the macb-driver: > > if parameter =3D=3D SOME_DEFAULT > =A0 =A0 =A0 =A0phydev =3D phy_find_first(bp->mii_bus); > else > =A0 =A0 =A0 =A0phydev =3D bp->mii_bus->phy_map[parameter]; > > I.e. if the parameter is set to some default value, the old behaviour= is > retained, otherwise the specified parameter is used for the phy_addr? Absolutely. g.