[U-Boot] [PATCH] cmd: mii: don't check address for 'device' subcommand

All mii operations require a valid PHY address except the 'device' command, which expects the PHY name rather than the address.
Signed-off-by: Hector Palacios hector.palacios@digi.com --- cmd/mii.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/cmd/mii.c b/cmd/mii.c index ce7b393eeaae..c0c42a851f90 100644 --- a/cmd/mii.c +++ b/cmd/mii.c @@ -313,7 +313,7 @@ static int do_mii(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) mask = simple_strtoul(argv[5], NULL, 16); }
- if (addrhi > 31) { + if (addrhi > 31 && strncmp(op, "de", 2)) { printf("Incorrect PHY address. Range should be 0-31\n"); return CMD_RET_USAGE; }

On Fri, Aug 17, 2018 at 6:11 AM Hector Palacios hector.palacios@digi.com wrote:
All mii operations require a valid PHY address except the 'device' command, which expects the PHY name rather than the address.
Signed-off-by: Hector Palacios hector.palacios@digi.com
Acked-by: Joe Hershberger joe.hershberger@ni.com

Hi Hector,
https://patchwork.ozlabs.org/patch/958780/ was applied to http://git.denx.de/?p=u-boot/u-boot-net.git
Thanks! -Joe
participants (3)
-
Hector Palacios
-
Joe Hershberger
-
Joe Hershberger