[PATCH v2] test: dm: add button_cmd test
Tom Rini
trini at konsulko.com
Thu Mar 14 01:40:33 CET 2024
On Tue, Mar 05, 2024 at 02:51:03PM +0000, Caleb Connolly wrote:
> Add a test for the button_cmd feature. This validates that commands can
> be mapped to two buttons, that the correct command runs based on which
> button is pressed, that only 1 command is run, and that no command runs
> if button_cmd_0_name is wrong or unset.
>
> Additionally, fix a potential uninitialised variable use caught by these
> tests, the btn variable in get_button_cmd() is assumed to be null if
> button_get_by_label() fails, but it's actually used uninitialised in
> that case.
>
> CONFIG_BUTTON is now enabled automatically and was removed when running
> save_defconfig.
>
> Signed-off-by: Caleb Connolly <caleb.connolly at linaro.org>
> Fixes: e761035b6423 ("boot: add support for button commands")
This still leads to CI failures:
https://source.denx.de/u-boot/u-boot/-/jobs/798332
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20240313/df2ba9a6/attachment.sig>
More information about the U-Boot
mailing list