Re: [PATCH 1/1 v1] cmd: gpio: Correct do_gpio() return value

2020-03-10 Thread Alex Kiernan
On Tue, Mar 10, 2020 at 12:37 PM Tom Rini wrote: > > On Tue, Mar 10, 2020 at 09:47:33AM +, Alex Kiernan wrote: > > On Sat, Feb 8, 2020 at 12:06 AM Tom Rini wrote: > > > > > > On Sun, Jan 05, 2020 at 08:10:56PM +0100, Luka Kovacic wrote: > > > > > > > Use the correct return value in function

Re: [PATCH 1/1 v1] cmd: gpio: Correct do_gpio() return value

2020-03-10 Thread Tom Rini
On Tue, Mar 10, 2020 at 09:47:33AM +, Alex Kiernan wrote: > On Sat, Feb 8, 2020 at 12:06 AM Tom Rini wrote: > > > > On Sun, Jan 05, 2020 at 08:10:56PM +0100, Luka Kovacic wrote: > > > > > Use the correct return value in function do_gpio() and update > > > commands documentation with the

Re: [PATCH 1/1 v1] cmd: gpio: Correct do_gpio() return value

2020-03-10 Thread Alex Kiernan
On Sat, Feb 8, 2020 at 12:06 AM Tom Rini wrote: > > On Sun, Jan 05, 2020 at 08:10:56PM +0100, Luka Kovacic wrote: > > > Use the correct return value in function do_gpio() and update > > commands documentation with the return values from command_ret_t enum. > > > > CMD_RET_SUCCESS is returned on

Re: [PATCH 1/1 v1] cmd: gpio: Correct do_gpio() return value

2020-02-07 Thread Tom Rini
On Sun, Jan 05, 2020 at 08:10:56PM +0100, Luka Kovacic wrote: > Use the correct return value in function do_gpio() and update > commands documentation with the return values from command_ret_t enum. > > CMD_RET_SUCCESS is returned on command success and CMD_RET_FAILURE is > returned on command

Re: [PATCH 1/1 v1] cmd: gpio: Correct do_gpio() return value

2020-02-02 Thread Simon Glass
Hi Tom, On Fri, 31 Jan 2020 at 13:59, Tom Rini wrote: > > On Thu, Jan 30, 2020 at 07:27:57PM -0700, Simon Glass wrote: > > Hi Tom. > > > > On Thu, 30 Jan 2020 at 11:52, Tom Rini wrote: > > > > > > On Wed, Jan 29, 2020 at 07:17:09PM -0700, Simon Glass wrote: > > > > Hi Tom, > > > > > > > > On

Re: [PATCH 1/1 v1] cmd: gpio: Correct do_gpio() return value

2020-01-31 Thread Tom Rini
On Thu, Jan 30, 2020 at 07:27:57PM -0700, Simon Glass wrote: > Hi Tom. > > On Thu, 30 Jan 2020 at 11:52, Tom Rini wrote: > > > > On Wed, Jan 29, 2020 at 07:17:09PM -0700, Simon Glass wrote: > > > Hi Tom, > > > > > > On Thu, 23 Jan 2020 at 14:12, Tom Rini wrote: > > > > > > > > On Thu, Jan 23,

Re: [PATCH 1/1 v1] cmd: gpio: Correct do_gpio() return value

2020-01-30 Thread Simon Glass
Hi Tom. On Thu, 30 Jan 2020 at 11:52, Tom Rini wrote: > > On Wed, Jan 29, 2020 at 07:17:09PM -0700, Simon Glass wrote: > > Hi Tom, > > > > On Thu, 23 Jan 2020 at 14:12, Tom Rini wrote: > > > > > > On Thu, Jan 23, 2020 at 10:04:05PM +0100, Luka Kovačič wrote: > > > > > > > Hello Tom, > > > > > >

Re: [PATCH 1/1 v1] cmd: gpio: Correct do_gpio() return value

2020-01-30 Thread Tom Rini
On Wed, Jan 29, 2020 at 07:17:09PM -0700, Simon Glass wrote: > Hi Tom, > > On Thu, 23 Jan 2020 at 14:12, Tom Rini wrote: > > > > On Thu, Jan 23, 2020 at 10:04:05PM +0100, Luka Kovačič wrote: > > > > > Hello Tom, > > > > > > thank you for feedback and review. I understand the implications. > > >

Re: [PATCH 1/1 v1] cmd: gpio: Correct do_gpio() return value

2020-01-29 Thread Simon Glass
Hi Tom, On Thu, 23 Jan 2020 at 14:12, Tom Rini wrote: > > On Thu, Jan 23, 2020 at 10:04:05PM +0100, Luka Kovačič wrote: > > > Hello Tom, > > > > thank you for feedback and review. I understand the implications. > > Would it make sense to document this somewhere to avoid any future > >

Re: [PATCH 1/1 v1] cmd: gpio: Correct do_gpio() return value

2020-01-23 Thread Tom Rini
On Thu, Jan 23, 2020 at 10:04:05PM +0100, Luka Kovačič wrote: > Hello Tom, > > thank you for feedback and review. I understand the implications. > Would it make sense to document this somewhere to avoid any future confusion? Yes, along with a standalone patch to update the document to use

Re: [PATCH 1/1 v1] cmd: gpio: Correct do_gpio() return value

2020-01-23 Thread Luka Kovačič
Hello Tom, thank you for feedback and review. I understand the implications. Would it make sense to document this somewhere to avoid any future confusion? Thanks, Luka On Thu, Jan 23, 2020 at 1:31 PM Tom Rini wrote: > > On Sun, Jan 05, 2020 at 08:10:56PM +0100, Luka Kovacic wrote: > > > Use

Re: [PATCH 1/1 v1] cmd: gpio: Correct do_gpio() return value

2020-01-23 Thread Tom Rini
On Sun, Jan 05, 2020 at 08:10:56PM +0100, Luka Kovacic wrote: > Use the correct return value in function do_gpio() and update > commands documentation with the return values from command_ret_t enum. > > CMD_RET_SUCCESS is returned on command success and CMD_RET_FAILURE is > returned on command

[PATCH 1/1 v1] cmd: gpio: Correct do_gpio() return value

2020-01-06 Thread Luka Kovacic
Use the correct return value in function do_gpio() and update commands documentation with the return values from command_ret_t enum. CMD_RET_SUCCESS is returned on command success and CMD_RET_FAILURE is returned on command failure. The command was returning the pin value, which caused confusion