diff mbox series

[1/1] cmd: fix long text for sound command

Message ID 20221204153750.79043-1-heinrich.schuchardt@canonical.com
State Superseded, archived
Delegated to: Tom Rini
Headers show
Series [1/1] cmd: fix long text for sound command | expand

Commit Message

Heinrich Schuchardt Dec. 4, 2022, 3:37 p.m. UTC
Make it clear that if only 1 parameter is provided this is the duration.

The ISO symbol for hertz is Hz.

Fixes: c0c88533fffd ("Sound: Add command for audio playback")
Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
---
 cmd/sound.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Simon Glass Dec. 4, 2022, 9:16 p.m. UTC | #1
On Mon, 5 Dec 2022 at 04:38, Heinrich Schuchardt
<heinrich.schuchardt@canonical.com> wrote:
>
> Make it clear that if only 1 parameter is provided this is the duration.
>
> The ISO symbol for hertz is Hz.
>
> Fixes: c0c88533fffd ("Sound: Add command for audio playback")
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
> ---
>  cmd/sound.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Simon Glass <sjg@chromium.org>
diff mbox series

Patch

diff --git a/cmd/sound.c b/cmd/sound.c
index f82f2aa670..20ac3f758e 100644
--- a/cmd/sound.c
+++ b/cmd/sound.c
@@ -86,5 +86,5 @@  U_BOOT_CMD(
 	sound, 4, 1, do_sound,
 	"sound sub-system",
 	"init - initialise the sound driver\n"
-	"sound play [len] [freq] - play a sound for len ms at freq hz\n"
+	"sound play [len [freq]] - play a sound for len ms at freq Hz\n"
 );