* [PATCH 1/1] cmd: fix long text for sound command
@ 2022-12-04 15:37 Heinrich Schuchardt
2022-12-04 21:16 ` Simon Glass
0 siblings, 1 reply; 2+ messages in thread
From: Heinrich Schuchardt @ 2022-12-04 15:37 UTC (permalink / raw)
To: u-boot; +Cc: Rajeshwari Shinde, Minkyu Kang, Simon Glass, Heinrich Schuchardt
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(-)
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"
);
--
2.37.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH 1/1] cmd: fix long text for sound command
2022-12-04 15:37 [PATCH 1/1] cmd: fix long text for sound command Heinrich Schuchardt
@ 2022-12-04 21:16 ` Simon Glass
0 siblings, 0 replies; 2+ messages in thread
From: Simon Glass @ 2022-12-04 21:16 UTC (permalink / raw)
To: Heinrich Schuchardt; +Cc: u-boot, Rajeshwari Shinde, Minkyu Kang
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>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-12-04 21:18 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-12-04 15:37 [PATCH 1/1] cmd: fix long text for sound command Heinrich Schuchardt
2022-12-04 21:16 ` Simon Glass
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.