[PATCH] cmd: host: fix duplicate device_unbind and wrong error message in unbind

[Naveen Kumar Chaudhary]

do_host_unbind()'s error handler for device_unbind() prints the
misleading message "Cannot attach file" and then calls device_unbind()
a second time on the same device, both of which look like copy-paste
mistakes left over from neighbouring code.

Remove the duplicate device_unbind() call and report the correct
failure with the device name.

Fixes: 952018117ab ("dm: sandbox: Switch over to using the new host uclass")
Signed-off-by: Naveen Kumar Chaudhary <naveen.osdev at gmail.com>
---
 cmd/host.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/cmd/host.c b/cmd/host.c
index e03576b4d2d..47a59f5ff20 100644
--- a/cmd/host.c
+++ b/cmd/host.c
@@ -132,10 +132,7 @@ static int do_host_unbind(struct cmd_tbl *cmdtp, int flag, int argc,
 
 	ret = device_unbind(dev);
 	if (ret) {
-		printf("Cannot attach file\n");
-		ret = device_unbind(dev);
-		if (ret)
-			printf("Cannot unbind device '%s'\n", dev->name);
+		printf("Cannot unbind device '%s'\n", dev->name);
 		return CMD_RET_FAILURE;
 	}
 
-- 
2.43.0