cmd: bind: Try to improve the (un)bind help

While it may sound totally obvious for the regular U-Boot developer to
get the parameters of the bind/unbind commands from the output of 'dm
tree', it did not felt straightforward to me until I was explicitly
told to look there. And even when I knew the command, I did not make a
direct link between the arguments of this command and the columns
returned by 'dm tree'.

Several of us lost a lot of time because of that, I would like to kindly
help other users by slightly improving this textual line. Unfortunately,
because of how this string is used (like within the 'help' command) I
cannot detail much more, but at least the pointer is there.

While we add this message, we can also imply CMD_DM when we enable
CMD_BIND so the debug message does not lead to an unknown command. This
way the 'dm' command will likely be there unless explicitly disabled.

Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Mattijs Korpershoek <mkorpershoek@baylibre.com>
Link: https://lore.kernel.org/r/20231010090304.49335-3-miquel.raynal@bootlin.com
Signed-off-by: Mattijs Korpershoek <mkorpershoek@baylibre.com>

diff --git a/cmd/Kconfig b/cmd/Kconfig
index 859e305634b725f81b441089b6458b0f9d265ca3..ac1644282a9fc34997448a08b9b05bafe1d8ea80 100644 (file)
--- a/cmd/Kconfig
+++ b/cmd/Kconfig
@@ -977,6 +977,7 @@ config CMD_BCB
 config CMD_BIND
        bool "bind/unbind - Bind or unbind a device to/from a driver"
        depends on DM
+       imply CMD_DM
        help
          Bind or unbind a device to/from a driver from the command line.
          This is useful in situations where a device may be handled by several

diff --git a/cmd/bind.c b/cmd/bind.c
index 4d1b7885e60a812efdb0592845f3b481ece70c4e..be0d4d2a7115806efb02e920a15a5ba58d9d57cf 100644 (file)
--- a/cmd/bind.c
+++ b/cmd/bind.c
@@ -246,6 +246,8 @@ U_BOOT_CMD(
        "Bind a device to a driver",
        "<node path> <driver>\n"
        "bind <class> <index> <driver>\n"
+       "Use 'dm tree' to list all devices registered in the driver model,\n"
+       "their path, class, index and current driver.\n"
 );
 
 U_BOOT_CMD(
@@ -254,4 +256,6 @@ U_BOOT_CMD(
        "<node path>\n"
        "unbind <class> <index>\n"
        "unbind <class> <index> <driver>\n"
+       "Use 'dm tree' to list all devices registered in the driver model,\n"
+       "their path, class, index and current driver.\n"
 );