Message ID | 1425227620-6656-8-git-send-email-sjg@chromium.org |
---|---|
State | RFC |
Delegated to: | Simon Glass |
Headers | show |
Hi Simon, On Sun, Mar 1, 2015 at 10:33 AM, Simon Glass <sjg@chromium.org> wrote: > > This function is not supported with driver model. Again, sorry about this... I hadn't yet addressed ARM with DM_ETH enabled. > Note: There is really no reason for this as far as I can see. At some point > we should bring it back. At this point I don't see much value in listing the Ethernet devices in bdinfo. It seems like a bit of a spurious thing to display and as such I didn't feel like rewriting it to be compatible with DM or implementing the functions bdinfo's print_eths() uses. Also, it is only supported on ARM and Microblaze. The inconsistency also led to my decision to drop it. > Signed-off-by: Simon Glass <sjg@chromium.org> Acked-by: Joe Hershberger <joe.hershberger@ni.com>
diff --git a/common/cmd_bdinfo.c b/common/cmd_bdinfo.c index b4cce25..f16d5c7 100644 --- a/common/cmd_bdinfo.c +++ b/common/cmd_bdinfo.c @@ -377,7 +377,7 @@ static int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, print_num("-> size", bd->bi_dram[i].size); } -#if defined(CONFIG_CMD_NET) +#if defined(CONFIG_CMD_NET) && !defined(CONFIG_DM_ETH) print_eths(); #endif printf("baudrate = %u bps\n", gd->baudrate);
This function is not supported with driver model. Note: There is really no reason for this as far as I can see. At some point we should bring it back. Signed-off-by: Simon Glass <sjg@chromium.org> --- common/cmd_bdinfo.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)