Re: [PATCH v2 2/4] gpiolib: Add code to support "active low" GPIOs

2017-06-01 Thread Andrey Smirnov
On Thu, Jun 1, 2017 at 10:28 PM, Sascha Hauer wrote: > On Thu, Jun 01, 2017 at 01:33:55PM -0700, Andrey Smirnov wrote: >> On Thu, Jun 1, 2017 at 12:19 AM, Sascha Hauer wrote: >> > On Tue, May 30, 2017 at 07:52:26AM -0700, Andrey Smirnov wrote: >> >> So far this particular aspect of various DT-bin

Re: [PATCH v2 2/4] gpiolib: Add code to support "active low" GPIOs

2017-06-01 Thread Sascha Hauer
On Thu, Jun 01, 2017 at 01:33:55PM -0700, Andrey Smirnov wrote: > On Thu, Jun 1, 2017 at 12:19 AM, Sascha Hauer wrote: > > On Tue, May 30, 2017 at 07:52:26AM -0700, Andrey Smirnov wrote: > >> So far this particular aspect of various DT-bindings has been handled > >> on a per-driver basis. With thi

Re: [PATCH v2 2/4] gpiolib: Add code to support "active low" GPIOs

2017-06-01 Thread Andrey Smirnov
On Thu, Jun 1, 2017 at 12:19 AM, Sascha Hauer wrote: > On Tue, May 30, 2017 at 07:52:26AM -0700, Andrey Smirnov wrote: >> So far this particular aspect of various DT-bindings has been handled >> on a per-driver basis. With this change, hopefully, we'll have a >> single place to handle necessary lo

[PATCH] console: only bidirectional consoles allow RATP

2017-06-01 Thread Aleksander Morgado
If a request to switch to RATP mode is received in an input-only console, ignore the request. This actually also avoids segfaulting later on: #0 0x in ?? () #1 0x0040b2a4 in console_send (r=, pkt=0x7fffd8ec, len=4) at common/ratp.c:102 #2 0x0042

Re: [PATCH] Documentation: remote-control: fix typo

2017-06-01 Thread Aleksander Morgado
On Thu, Jun 1, 2017 at 2:31 PM, Sascha Hauer wrote: >> >> Correctly reference `--port` instead of `--baudrate` when explaining >> >> how to specify the port. >> >> >> >> Signed-off-by: Aleksander Morgado >> >> --- >> >> Documentation/user/remote-control.rst | 7 +++ >> >> 1 file change

Re: [PATCH] Documentation: remote-control: fix typo

2017-06-01 Thread Sascha Hauer
On Thu, Jun 01, 2017 at 02:26:29PM +0200, Aleksander Morgado wrote: > On Thu, Jun 1, 2017 at 9:30 AM, Sascha Hauer wrote: > > On Tue, May 30, 2017 at 11:46:34PM +0200, Aleksander Morgado wrote: > >> Correctly reference `--port` instead of `--baudrate` when explaining > >> how to specify the

Re: [PATCH] Documentation: remote-control: fix typo

2017-06-01 Thread Aleksander Morgado
On Thu, Jun 1, 2017 at 9:30 AM, Sascha Hauer wrote: > On Tue, May 30, 2017 at 11:46:34PM +0200, Aleksander Morgado wrote: >> Correctly reference `--port` instead of `--baudrate` when explaining >> how to specify the port. >> >> Signed-off-by: Aleksander Morgado >> --- >> Documentation/user

Re: [PATCH v3 1/3] fs: add parseopt_b for loop mount option

2017-06-01 Thread Sascha Hauer
On Thu, Jun 01, 2017 at 12:37:28PM +0200, Philipp Zabel wrote: > Signed-off-by: Philipp Zabel > --- > No changes since v2. > --- > fs/parseopt.c | 26 ++ > fs/parseopt.h | 1 + > 2 files changed, 27 insertions(+) Applied, thanks Sascha > > diff --git a/fs/parseopt.c b

[PATCH v3 1/3] fs: add parseopt_b for loop mount option

2017-06-01 Thread Philipp Zabel
Signed-off-by: Philipp Zabel --- No changes since v2. --- fs/parseopt.c | 26 ++ fs/parseopt.h | 1 + 2 files changed, 27 insertions(+) diff --git a/fs/parseopt.c b/fs/parseopt.c index 12dbe1813c..8ff83019a7 100644 --- a/fs/parseopt.c +++ b/fs/parseopt.c @@ -2,6 +2,32 @@

[PATCH v3 3/3] commands: mount: mention file loop mounts in the help text

2017-06-01 Thread Philipp Zabel
Signed-off-by: Philipp Zabel --- No changes since v2. --- commands/mount.c | 1 + 1 file changed, 1 insertion(+) diff --git a/commands/mount.c b/commands/mount.c index f05e23b227..4cf1179b7b 100644 --- a/commands/mount.c +++ b/commands/mount.c @@ -117,6 +117,7 @@ BAREBOX_CMD_HELP_TEXT("With -a t

[PATCH v3 2/3] fs: add cdev_create_loop and cdev_remove_loop for loop mount option

2017-06-01 Thread Philipp Zabel
Allow to create a loopback cdev from a file. Signed-off-by: Philipp Zabel --- Changes since v1: - Renamed cdev_create_from_file to cdev_create_loop - Added cdev_remove_loop, which replaces cdev_close and also removes the cdev and frees its memories. - Actually increment loopno with each cal

Re: [PATCH 4/5] fs: add cdev_create_from_file for loop mount option

2017-06-01 Thread Philipp Zabel
On Thu, 2017-06-01 at 08:31 +0200, Sascha Hauer wrote: > Hi Philipp, > > On Tue, May 30, 2017 at 04:09:46PM +0200, Philipp Zabel wrote: > > Allow to create a loopback cdev from a file. > > > > Signed-off-by: Philipp Zabel > > --- > > fs/devfs-core.c | 81 > > ++

[PATCH v2 1/3] fs: add parseopt_b for loop mount option

2017-06-01 Thread Philipp Zabel
Signed-off-by: Philipp Zabel --- No changes since v1. --- fs/parseopt.c | 26 ++ fs/parseopt.h | 1 + 2 files changed, 27 insertions(+) diff --git a/fs/parseopt.c b/fs/parseopt.c index 12dbe1813c..8ff83019a7 100644 --- a/fs/parseopt.c +++ b/fs/parseopt.c @@ -2,6 +2,32 @@

[PATCH v2 3/3] commands: mount: mention file loop mounts in the help text

2017-06-01 Thread Philipp Zabel
Signed-off-by: Philipp Zabel --- No changes since v1. --- commands/mount.c | 1 + 1 file changed, 1 insertion(+) diff --git a/commands/mount.c b/commands/mount.c index f05e23b227..4cf1179b7b 100644 --- a/commands/mount.c +++ b/commands/mount.c @@ -117,6 +117,7 @@ BAREBOX_CMD_HELP_TEXT("With -a t

[PATCH v2 2/3] fs: add cdev_create_from_file for loop mount option

2017-06-01 Thread Philipp Zabel
Allow to create a loopback cdev from a file. Signed-off-by: Philipp Zabel --- Changes since v1: - Add cdev file operation to close the backing file descriptor on umount. --- fs/devfs-core.c | 89 fs/fs.c | 21 ++--- incl

Re: [PATCH 2/2] sandbox: --stdin and --stdout allow max one bidirectional console

2017-06-01 Thread Sascha Hauer
On Thu, Jun 01, 2017 at 10:28:42AM +0200, Aleksander Morgado wrote: > > On Wed, May 31, 2017 at 06:12:41PM +0200, Aleksander Morgado wrote: > >> Allow up to one bidirectional FIFO/file based console in addition to > >> the default stdin/stdout console that is always registered. > >> > >> We avoid o

Re: [PATCH 2/2] sandbox: --stdin and --stdout allow max one bidirectional console

2017-06-01 Thread Aleksander Morgado
> On Wed, May 31, 2017 at 06:12:41PM +0200, Aleksander Morgado wrote: >> Allow up to one bidirectional FIFO/file based console in addition to >> the default stdin/stdout console that is always registered. >> >> We avoid opening the FIFO files while parsing the options because the >> whole logic may

Re: [PATCH 2/2] sandbox: --stdin and --stdout allow max one bidirectional console

2017-06-01 Thread Sascha Hauer
Hi Aleksander, On Wed, May 31, 2017 at 06:12:41PM +0200, Aleksander Morgado wrote: > Allow up to one bidirectional FIFO/file based console in addition to > the default stdin/stdout console that is always registered. > > We avoid opening the FIFO files while parsing the options because the > whole

Re: [PATCH] Documentation: remote-control: fix typo

2017-06-01 Thread Sascha Hauer
On Tue, May 30, 2017 at 11:46:34PM +0200, Aleksander Morgado wrote: > Correctly reference `--port` instead of `--baudrate` when explaining > how to specify the port. > > Signed-off-by: Aleksander Morgado > --- > Documentation/user/remote-control.rst | 7 +++ > 1 file changed, 3 inserti

Re: [PATCH v2 2/4] gpiolib: Add code to support "active low" GPIOs

2017-06-01 Thread Sascha Hauer
On Tue, May 30, 2017 at 07:52:26AM -0700, Andrey Smirnov wrote: > So far this particular aspect of various DT-bindings has been handled > on a per-driver basis. With this change, hopefully, we'll have a > single place to handle necessary logic inversions and eventually > would be able to migrate ex