[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