Re: [U-Boot] [PATCH 5/7] Add mdio command for new PHY infrastructure

2011-03-30 Thread Detlev Zundel
Hi Andy, The new mdio command doesn't have all of the features of the mii command, but it provides the necessary read/write primitives, and allows users to interact with 10G PHYs, and other PHYs which use Clause 45 of 802.3. This means that the mdio command requires a Device Address

Re: [U-Boot] [PATCH 5/7] Add mdio command for new PHY infrastructure

2011-03-30 Thread Andy Fleming
On Mar 30, 2011, at 6:55 AM, Detlev Zundel wrote: Hi Andy, +static int extract_range(char *input, int *plo, int *phi) +{ +char * end; +*plo = simple_strtol(input, end, 0); +if (end == input) +return -1; + +if (*end == '-') { What about the case of

Re: [U-Boot] [PATCH 5/7] Add mdio command for new PHY infrastructure

2011-03-30 Thread Mike Frysinger
On Tue, Mar 29, 2011 at 3:30 PM, Andy Fleming wrote: +        busname  addr\n +        busname:addr\n why bother ? pick one and go with it [busname:]addr -mike ___ U-Boot mailing list U-Boot@lists.denx.de

[U-Boot] [PATCH 5/7] Add mdio command for new PHY infrastructure

2011-03-29 Thread Andy Fleming
The new mdio command doesn't have all of the features of the mii command, but it provides the necessary read/write primitives, and allows users to interact with 10G PHYs, and other PHYs which use Clause 45 of 802.3. This means that the mdio command requires a Device Address argument, though for