Remove the obsolete terse version of do_mii()
PATCH 2/2 Remove the obsolete terse version of do_mii()
We now have more useful version of do_mii() and everybody use it. Grald Van Baren says
Quoted 8 lines
When I originally wrote the mii command 6(!) years ago, I wrote a verbose version that printed human readable decomposition of the flags, etc., and a terse one that didn't print as much stuff and thus had a smaller memory footprint.
It sounds like the terse version has withered and died, apparently people are only using the verbose version (which is very understandable, I do myself).
---
common/cmd_mii.c | 139 ------------------------------------------------------ 1 files changed, 0 insertions(+), 139 deletions(-)
common/cmd_mii.c +0 -139
@@ -29,143 +29,6 @@#include <command.h> #include <miiphy.h> -#ifdef CONFIG_TERSE_MII -/* - * Display values from last command. - */ -uint last_op; -uint last_addr; -uint last_data; -uint last_reg; - -/* - * MII device/info/read/write - * - * Syntax: - * mii device {devname} - * mii info {addr} - * mii read {addr} {reg} - * mii write {addr} {reg} {data} - */ -int do_mii (cmd_tbl_t * cmdtp, int flag, int argc, char *argv[]) -{ - char op; - unsigned char addr, reg; - unsigned short data; - int rcode = 0; - char *devname; - - if (argc < 2) { - printf ("Usage:\n%s\n", cmdtp->usage); - return 1; - } - -#if defined(CONFIG_8xx) || defined(CONFIG_MCF52x2) - mii_init (); -#endif - - /* - * We use the last specified parameters, unless new ones are - * entered. - */ - op = last_op; - addr = last_addr; - data = last_data; - reg = last_reg; - - if ((flag & CMD_FLAG_REPEAT) == 0) { - op = argv[1][0]; - if (argc >= 3) - addr = simple_strtoul (argv[2], NULL, 16); - if (argc >= 4) - reg = simple_strtoul (argv[3], NULL, 16); - if (argc >= 5) - data = simple_strtoul (argv[4], NULL, 16); - } - - /* use current device */ - devname = miiphy_get_current_dev(); - - /* - * check device/read/write/list. - */ - if (op == 'i') { - unsigned char j, start, end; - unsigned int oui; - unsigned char model; - unsigned char rev; - - /* - * Look for any and all PHYs. Valid addresses are 0..31. - */ - if (argc >= 3) { - start = addr; end = addr + 1; - } else { - start = 0; end = 31; - } - - for (j = start; j < end; j++) { - if (miiphy_info (devname, j, &oui, &model, &rev) == 0) { - printf ("PHY 0x%02X: " - "OUI = 0x%04X, " - "Model = 0x%02X, " - "Rev = 0x%02X, " - "%3dbase%s, %s\n", - j, oui, model, rev, - miiphy_speed (devname, j), - miiphy_is_1000base_x (devname, j) - ? "X" : "T", - (miiphy_duplex (devname, j) == FULL) - ? "FDX" : "HDX"); - } - } - } else if (op == 'r') { - if (miiphy_read (devname, addr, reg, &data) != 0) { - puts ("Error reading from the PHY\n"); - rcode = 1; - } else { - printf ("%04X\n", data & 0x0000FFFF); - } - } else if (op == 'w') { - if (miiphy_write (devname, addr, reg, data) != 0) { - puts ("Error writing to the PHY\n"); - rcode = 1; - } - } else if (op == 'd') { - if (argc == 2) - miiphy_listdev (); - else - miiphy_set_current_dev (argv[2]); - } else { - printf ("Usage:\n%s\n", cmdtp->usage); - return 1; - } - - /* - * Save the parameters for repeats. - */ - last_op = op; - last_addr = addr; - last_data = data; - last_reg = reg; - - return rcode; -} - -/***************************************************/ - -U_BOOT_CMD( - mii, 5, 1, do_mii, - "mii - MII utility commands\n", - "device - list available devices\n" - "mii device <devname> - set current device\n" - "mii info <addr> - display MII PHY info\n" - "mii read <addr> <reg> - read MII PHY <addr> register <reg>\n" - "mii write <addr> <reg> <data> - write MII PHY <addr> register <reg>\n" -); - -#else /* ! CONFIG_TERSE_MII ================================================= */ - typedef struct _MII_reg_desc_t { ushort regno; char * name;
@@ -601,5 +464,3 @@ U_BOOT_CMD("mii dump <addr> <reg> - pretty-print <addr> <reg> (0-5 only)\n" "Addr and/or reg may be ranges, e.g. 2-7.\n" ); - -#endif /* CONFIG_TERSE_MII */
PATCH 2/2 Remove the obsolete terse version of do_mii()
Quoted 8 lines · Shinya Kuribayashi
We now have more useful version of
do_mii()and everybody use it. Grald Van Baren saysQuoted 5 lines
When I originally wrote the mii command 6(!) years ago, I wrote a verbose version that printed human readable decomposition of the flags, etc., and a terse one that didn't print as much stuff and thus had a smaller memory footprint.
Quoted 6 lines
Quoted 3 lines
It sounds like the terse version has withered and died, apparently people are only using the verbose version (which is very understandable, I do myself).
Signed-off-by Shinya Kuribayashi <[email protected]>Signed-off-by Gerald Van Baren <[email protected]>
Hi Shinya,
Thanks for the patch to clean this up. While I agree with the patch, I would appreciate respinning it:
* I should be an "Acked by" rather than a signed off by since I didn't do anything to generate the patch itself.
* I would also offer the following summary rather than quoting my original email (including the 6(!) year reference, which will look kinda funny in 2012 ;-):
Remove the "terse" version of the mii command, eliminating an #ifdef and apparently unused code. The space savings of the "terse" version is not significant and it isn't worth the extra maintenance effort to keep it.
Thanks,
gvb
PATCH 2/2 Remove the obsolete terse version of do_mii()
Quoted 4 lines · gvb.uboot
Thanks for the patch to clean this up. While I agree with the patch, I would appreciate respinning it:
* I should be an "Acked by" rather than a signed off by since I didn't do anything to generate the patch itself.
As you are the original writer, IMHO you could/should signed-off-by :-) But it was my bad to add your sign without asking you. I'll resubmit the patch later with your acked-by.
Quoted 7 lines
* I would also offer the following summary rather than quoting my original email (including the 6(!) year reference, which will look kinda funny in 2012 ;-):
Remove the "terse" version of the mii command, eliminating an #ifdef and apparently unused code. The space savings of the "terse" version is not significant and it isn't worth the extra maintenance effort to keep it.
Thanks for your kind summary.
Shinya
PATCH 2/2 Remove the obsolete terse version of do_mii()
Quoted 9 lines · Shinya Kuribayashi
Quoted 4 lines · gvb.uboot
Thanks for the patch to clean this up. While I agree with the patch, I would appreciate respinning it:
* I should be an "Acked by" rather than a signed off by since I didn't do anything to generate the patch itself.As you are the original writer, IMHO you could/should signed-off-by :-) But it was my bad to add your sign without asking you. I'll resubmit the patch later with your acked-by.
I figure that an acked-by is more appropriate than a signed-off-by because, while I approve of the patch, I didn't actually contribute to it directly (I contributed the anti-patch, that is, the code that is being removed... but that already has my signed-off-by on it).
Quoted 3 lines
Thanks for your kind summary.
Shinya
Thanks for rerolling,
gvb
PATCH 1/1 Remove the "terse" version of do_mii()
In reply to Shinya KuribayashiRemove the "terse" version of the mii command, eliminating an #ifdef and apparently unused code. The space savings of the "terse" version is not significant and it isn't worth the extra maintenance effort to keep it.
---
common/cmd_mii.c | 139 ------------------------------------------------------ 1 files changed, 0 insertions(+), 139 deletions(-)
common/cmd_mii.c +0 -139
@@ -29,143 +29,6 @@#include <command.h> #include <miiphy.h> -#ifdef CONFIG_TERSE_MII -/* - * Display values from last command. - */ -uint last_op; -uint last_addr; -uint last_data; -uint last_reg; - -/* - * MII device/info/read/write - * - * Syntax: - * mii device {devname} - * mii info {addr} - * mii read {addr} {reg} - * mii write {addr} {reg} {data} - */ -int do_mii (cmd_tbl_t * cmdtp, int flag, int argc, char *argv[]) -{ - char op; - unsigned char addr, reg; - unsigned short data; - int rcode = 0; - char *devname; - - if (argc < 2) { - printf ("Usage:\n%s\n", cmdtp->usage); - return 1; - } - -#if defined(CONFIG_8xx) || defined(CONFIG_MCF52x2) - mii_init (); -#endif - - /* - * We use the last specified parameters, unless new ones are - * entered. - */ - op = last_op; - addr = last_addr; - data = last_data; - reg = last_reg; - - if ((flag & CMD_FLAG_REPEAT) == 0) { - op = argv[1][0]; - if (argc >= 3) - addr = simple_strtoul (argv[2], NULL, 16); - if (argc >= 4) - reg = simple_strtoul (argv[3], NULL, 16); - if (argc >= 5) - data = simple_strtoul (argv[4], NULL, 16); - } - - /* use current device */ - devname = miiphy_get_current_dev(); - - /* - * check device/read/write/list. - */ - if (op == 'i') { - unsigned char j, start, end; - unsigned int oui; - unsigned char model; - unsigned char rev; - - /* - * Look for any and all PHYs. Valid addresses are 0..31. - */ - if (argc >= 3) { - start = addr; end = addr + 1; - } else { - start = 0; end = 31; - } - - for (j = start; j < end; j++) { - if (miiphy_info (devname, j, &oui, &model, &rev) == 0) { - printf ("PHY 0x%02X: " - "OUI = 0x%04X, " - "Model = 0x%02X, " - "Rev = 0x%02X, " - "%3dbase%s, %s\n", - j, oui, model, rev, - miiphy_speed (devname, j), - miiphy_is_1000base_x (devname, j) - ? "X" : "T", - (miiphy_duplex (devname, j) == FULL) - ? "FDX" : "HDX"); - } - } - } else if (op == 'r') { - if (miiphy_read (devname, addr, reg, &data) != 0) { - puts ("Error reading from the PHY\n"); - rcode = 1; - } else { - printf ("%04X\n", data & 0x0000FFFF); - } - } else if (op == 'w') { - if (miiphy_write (devname, addr, reg, data) != 0) { - puts ("Error writing to the PHY\n"); - rcode = 1; - } - } else if (op == 'd') { - if (argc == 2) - miiphy_listdev (); - else - miiphy_set_current_dev (argv[2]); - } else { - printf ("Usage:\n%s\n", cmdtp->usage); - return 1; - } - - /* - * Save the parameters for repeats. - */ - last_op = op; - last_addr = addr; - last_data = data; - last_reg = reg; - - return rcode; -} - -/***************************************************/ - -U_BOOT_CMD( - mii, 5, 1, do_mii, - "mii - MII utility commands\n", - "device - list available devices\n" - "mii device <devname> - set current device\n" - "mii info <addr> - display MII PHY info\n" - "mii read <addr> <reg> - read MII PHY <addr> register <reg>\n" - "mii write <addr> <reg> <data> - write MII PHY <addr> register <reg>\n" -); - -#else /* ! CONFIG_TERSE_MII ================================================= */ - typedef struct _MII_reg_desc_t { ushort regno; char * name;
@@ -601,5 +464,3 @@ U_BOOT_CMD("mii dump <addr> <reg> - pretty-print <addr> <reg> (0-5 only)\n" "Addr and/or reg may be ranges, e.g. 2-7.\n" ); - -#endif /* CONFIG_TERSE_MII */
Wolfgang Denk
AppliedPATCH 2/2 Remove the obsolete terse version of do_mii()
In reply to Shinya KuribayashiQuoted 8 lines · In message <[email protected]> you wrote:
We now have more useful version of
do_mii()and everybody use it. Grald Van Baren saysQuoted 5 lines
When I originally wrote the mii command 6(!) years ago, I wrote a verbose version that printed human readable decomposition of the flags, etc., and a terse one that didn't print as much stuff and thus had a smaller memory footprint.
Quoted 6 lines
Quoted 3 lines
It sounds like the terse version has withered and died, apparently people are only using the verbose version (which is very understandable, I do myself).
Signed-off-by Shinya Kuribayashi <[email protected]>Signed-off-by Gerald Van Baren <[email protected]>
Applied, thanks.
Best regards,
Wolfgang Denk
Signature · 5 lines
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
I think there's a world market for about five computers.
-- attr. Thomas J. Watson (Chairman of the Board, IBM), 1943PATCH 2/2 Remove the obsolete terse version of do_mii()
Quoted 12 lines · Wolfgang Denk
Quoted 11 lines · In message <[email protected]> you wrote:
We now have more useful version of
do_mii()and everybody use it. Grald Van Baren saysQuoted 8 lines
When I originally wrote the mii command 6(!) years ago, I wrote a verbose version that printed human readable decomposition of the flags, etc., and a terse one that didn't print as much stuff and thus had a smaller memory footprint.
It sounds like the terse version has withered and died, apparently people are only using the verbose version (which is very understandable, I do myself).
Quoted 8 lines
Quoted 2 lines
Signed-off-by Shinya Kuribayashi <[email protected]>Signed-off-by Gerald Van Baren <[email protected]>Applied, thanks.
Best regards,
Wolfgang Denk
It isn't a big deal, but Shinya re-rolled this with a better description and my "acked by" rather than "SOB".
Best regards,
gvb
PATCH 2/2 Remove the obsolete terse version of do_mii()
In reply to Wolfgang DenkQuoted 12 lines · Wolfgang Denk
Quoted 11 lines · In message <[email protected]> you wrote:
We now have more useful version of
do_mii()and everybody use it. Grald Van Baren saysQuoted 8 lines
When I originally wrote the mii command 6(!) years ago, I wrote a verbose version that printed human readable decomposition of the flags, etc., and a terse one that didn't print as much stuff and thus had a smaller memory footprint.
It sounds like the terse version has withered and died, apparently people are only using the verbose version (which is very understandable, I do myself).
Quoted 8 lines
Quoted 2 lines
Signed-off-by Shinya Kuribayashi <[email protected]>Signed-off-by Gerald Van Baren <[email protected]>Applied, thanks.
Best regards,
Wolfgang Denk
...here
<http://article.gmane.org/gmane.comp.boot-loaders.u-boot/34682/match=kuribayashi>
gvb