aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMiquel Raynal <miquel.raynal@bootlin.com>2023-10-10 11:03:03 +0200
committerMattijs Korpershoek <mkorpershoek@baylibre.com>2023-11-21 15:48:38 +0100
commit249a75d8e82b422639beedca3d7d945cd78869ba (patch)
treea131089ceccd3e98902c0417b09bb38e495c530b
parent9b63fcaec664780545318d923063bedd898a629c (diff)
downloadu-boot-249a75d8e82b422639beedca3d7d945cd78869ba.zip
u-boot-249a75d8e82b422639beedca3d7d945cd78869ba.tar.gz
u-boot-249a75d8e82b422639beedca3d7d945cd78869ba.tar.bz2
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>
-rw-r--r--cmd/Kconfig1
-rw-r--r--cmd/bind.c4
2 files changed, 5 insertions, 0 deletions
diff --git a/cmd/Kconfig b/cmd/Kconfig
index 859e305..ac16442 100644
--- 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 4d1b788..be0d4d2 100644
--- 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"
);