From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Sat, 12 Sep 2020 10:52:38 -0400 Subject: [PATCH 1/1] cmd/button: return button status In-Reply-To: <20200902200826.97874-1-xypron.glpk@gmx.de> References: <20200902200826.97874-1-xypron.glpk@gmx.de> Message-ID: <20200912145238.GA9159@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Wed, Sep 02, 2020 at 10:08:26PM +0200, Heinrich Schuchardt wrote: > To make the button command useful in a shell script it should return the > status of the button: > > * 0 (true) - pressed, on > * 1 (false) - not pressed, off > > The button command takes only one argument. Correct maxargs. > > Signed-off-by: Heinrich Schuchardt > Reviewed-by: Philippe Reynes > --- > cmd/button.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > -- > 2.28.0 > > diff --git a/cmd/button.c b/cmd/button.c > index 84ad1653c7..64c5a8fa04 100644 > --- a/cmd/button.c > +++ b/cmd/button.c > @@ -75,11 +75,11 @@ int do_button(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[]) > > ret = show_button_state(dev); > > - return 0; > + return !ret; > } > > U_BOOT_CMD( > - button, 4, 1, do_button, > + button, 2, 1, do_button, > "manage buttons", > " \tGet button state\n" > "button list\t\tShow a list of buttons" This in turn breaks the current button test, which I believe needs to be updated to match. Thanks. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 659 bytes Desc: not available URL: