Re: [PATCH v3 0/3] upstream console_countdown related work

2017-03-09 Thread Sascha Hauer
On Fri, Mar 10, 2017 at 07:05:20AM +0100, Oleksij Rempel wrote: > changes v3: > - make console_countdown_timeout_abort static bool > > changes v2: > - add my Signed-of-by > > Marc Kleine-Budde (3): > console_countdown: width to of countdown to 4 digits > console_countdown: add possibility t

Re: [PATCH 2/2] usb: fastboot: umount device before writing to it

2017-03-09 Thread Sascha Hauer
On Wed, Mar 08, 2017 at 10:24:02AM +0100, Oleksij Rempel wrote: > From: Sascha Hauer > > When a fastboot target refers to a device and we are going to write > to it, umount it beforehand, so that we do not end up with corrupt > data after writing. > > Signed-off-by: Sascha Hauer > --- > driver

Re: [PATCH 2/2] video: some framebuffer such as efi does support enable/disable

2017-03-09 Thread Sascha Hauer
On Thu, Mar 09, 2017 at 10:57:21AM +0100, Jean-Christophe PLAGNIOL-VILLARD wrote: > so allow them to do not pass this function via fbops > and indicate they are always on. > > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD > --- > drivers/video/fb.c | 29 - > 1 file

Re: [PATCH 1/2] parameter: fix read only int support

2017-03-09 Thread Sascha Hauer
On Thu, Mar 09, 2017 at 10:57:20AM +0100, Jean-Christophe PLAGNIOL-VILLARD wrote: > pass PARAM_FLAG_RO flag for read only it > so we can not change them > > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD > --- > lib/parameter.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Appli

[PATCH] video: call fb_[en|dis]able instead of fops directly

2017-03-09 Thread Sascha Hauer
We have fb_enable and fb_disable which handle enabling of a framebuffer, so use it instead of calling into the ops directly. This gets the enable count straight. Signed-off-by: Sascha Hauer --- drivers/video/fb.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers

Re: [PATCH 3/3] fbcon: use __iowrite{32/64} to speedup the framebuffer console scrolling

2017-03-09 Thread Sascha Hauer
On Thu, Mar 09, 2017 at 11:31:11AM +0100, Jean-Christophe PLAGNIOL-VILLARD wrote: > this devide the time by 4 on x86 > > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD > --- > drivers/video/fbconsole.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/video/fb

[PATCH] fixup! clk: No-op CLK_OF_DECLARE if not enabled

2017-03-09 Thread Andrey Smirnov
--- Sascha, let's try and see if this fixes the build issues you were having with the original patch. If you'd rather I spin a v2 of the original patch, let me know. Thanks, Andrey Smirnov include/linux/clk.h | 27 +-- 1 file changed, 21 insertions(+), 6 deletions(-) d

[PATCH v3 2/3] console_countdown: add possibility to abort countdown by external commands

2017-03-09 Thread Oleksij Rempel
From: Marc Kleine-Budde This patch makes it possible to abort a console countdown by an external command, for example when fastboot is used. This requires additional modifications in the external commands, a call to "console_countdown_abort()" has to be inserted. Signed-off-by: Marc Kleine-Budde

[PATCH v3 0/3] upstream console_countdown related work

2017-03-09 Thread Oleksij Rempel
changes v3: - make console_countdown_timeout_abort static bool changes v2: - add my Signed-of-by Marc Kleine-Budde (3): console_countdown: width to of countdown to 4 digits console_countdown: add possibility to abort countdown by external commands fastboot: abort autoboot timeout when

[PATCH v3 1/3] console_countdown: width to of countdown to 4 digits

2017-03-09 Thread Oleksij Rempel
From: Marc Kleine-Budde This patch increases the displayed width of the countdown to 4 digits, otherwise waiting for more then 99 seconds doesn't look good. Signed-off-by: Marc Kleine-Budde Signed-off-by: Oleksij Rempel --- common/console_countdown.c | 4 ++-- 1 file changed, 2 insertions(+),

[PATCH v3 3/3] fastboot: abort autoboot timeout when fastboot gadget is activated

2017-03-09 Thread Oleksij Rempel
From: Marc Kleine-Budde This patch adds a call to "console_countdown_abort()" to abort a currently or upcoming running console timeout. Signed-off-by: Marc Kleine-Budde Signed-off-by: Oleksij Rempel --- drivers/usb/gadget/f_fastboot.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/dri

Re: e1000 driver hangs during probe

2017-03-09 Thread Andrey Smirnov
On Thu, Mar 9, 2017 at 1:18 PM, Uwe Kleine-König wrote: > Hello, > > I work with an i.MX6 based hardware that has an i210 pcie device with > barebox 2016.11.0 (but there didn't happen anything relevant since them > if I'm not mistaken). When enabling all necessary things to make the > e1000 bind,

Re: [PATCH 1/3] console_countdown: width to of countdown to 4 digits

2017-03-09 Thread Sam Ravnborg
Hi Oleksij. On Thu, Mar 09, 2017 at 06:05:43PM +0100, Oleksij Rempel wrote: > From: Marc Kleine-Budde > > This patch increases the displayed width of the countdown to 4 digits, > otherwise waiting for more then 99 seconds doesn't look good. > > Signed-off-by: Marc Kleine-Budde Since this was

e1000 driver hangs during probe

2017-03-09 Thread Uwe Kleine-König
Hello, I work with an i.MX6 based hardware that has an i210 pcie device with barebox 2016.11.0 (but there didn't happen anything relevant since them if I'm not mistaken). When enabling all necessary things to make the e1000 bind, the driver seems to hang. The problem is the following code in e1000

Re: [PATCH v2 2/3] console_countdown: add possibility to abort countdown by external commands

2017-03-09 Thread Marc Kleine-Budde
On 03/09/2017 06:58 PM, Oleksij Rempel wrote: > From: Marc Kleine-Budde > > This patch makes it possible to abort a console countdown by an external > command, for example when fastboot is used. This requires additional > modifications in the external commands, a call to "console_countdown_abort(

[PATCH v2 2/3] console_countdown: add possibility to abort countdown by external commands

2017-03-09 Thread Oleksij Rempel
From: Marc Kleine-Budde This patch makes it possible to abort a console countdown by an external command, for example when fastboot is used. This requires additional modifications in the external commands, a call to "console_countdown_abort()" has to be inserted. Signed-off-by: Marc Kleine-Budde

[PATCH v2 1/3] console_countdown: width to of countdown to 4 digits

2017-03-09 Thread Oleksij Rempel
From: Marc Kleine-Budde This patch increases the displayed width of the countdown to 4 digits, otherwise waiting for more then 99 seconds doesn't look good. Signed-off-by: Marc Kleine-Budde Signed-off-by: Oleksij Rempel --- common/console_countdown.c | 4 ++-- 1 file changed, 2 insertions(+),

[PATCH v2 3/3] fastboot: abort autoboot timeout when fastboot gadget is activated

2017-03-09 Thread Oleksij Rempel
From: Marc Kleine-Budde This patch adds a call to "console_countdown_abort()" to abort a currently or upcoming running console timeout. Signed-off-by: Marc Kleine-Budde Signed-off-by: Oleksij Rempel --- drivers/usb/gadget/f_fastboot.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/dri

Re: [PATCH 2/5] efi: fix lds for secure boot support

2017-03-09 Thread Lucas Stach
Am Donnerstag, den 09.03.2017, 15:34 +0100 schrieb Jean-Christophe PLAGNIOL-VILLARD: > everythink need to be aligned to 4096 Why? The commit message isn't really telling anything. Regards, Lucas > > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD > --- > arch/x86/mach-efi/elf_ia32_efi.lds.S

[PATCH 1/3] console_countdown: width to of countdown to 4 digits

2017-03-09 Thread Oleksij Rempel
From: Marc Kleine-Budde This patch increases the displayed width of the countdown to 4 digits, otherwise waiting for more then 99 seconds doesn't look good. Signed-off-by: Marc Kleine-Budde --- common/console_countdown.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/c

[PATCH 3/3] fastboot: abort autoboot timeout when fastboot gadget is activated

2017-03-09 Thread Oleksij Rempel
From: Marc Kleine-Budde This patch adds a call to "console_countdown_abort()" to abort a currently or upcoming running console timeout. Signed-off-by: Marc Kleine-Budde --- drivers/usb/gadget/f_fastboot.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/usb/gadget/f_fastboot.c b/

[PATCH 2/3] console_countdown: add possibility to abort countdown by external commands

2017-03-09 Thread Oleksij Rempel
From: Marc Kleine-Budde This patch makes it possible to abort a console countdown by an external command, for example when fastboot is used. This requires additional modifications in the external commands, a call to "console_countdown_abort()" has to be inserted. Signed-off-by: Marc Kleine-Budde

Re: [PATCH 0/2] efivarfs: rework the filesystem to make it human readable

2017-03-09 Thread Jean-Christophe PLAGNIOL-VILLARD
Hi, Sascha is this one ok? Best Regards, J. On 16:27 Fri 03 Mar , Jean-Christophe PLAGNIOL-VILLARD wrote: > Hi, > > efivarfs: rework the filesystem to make it human readable > > today we use - > > such as Boot-8be4df61-93ca-11d2-aa0d-00e098032b8c > > now we will use a differen

[PATCH 4/5] boot: if we are in secure boot mode

2017-03-09 Thread Jean-Christophe PLAGNIOL-VILLARD
request confirmation before booting an unsigned image with a default timeout Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD --- commands/go.c | 7 +++ common/Kconfig| 8 common/Makefile | 1 + common/bootm.c| 7 +++ common/secure_boot.c | 43

[PATCH 3/5] efi: fix secure and setup mode report

2017-03-09 Thread Jean-Christophe PLAGNIOL-VILLARD
Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD --- drivers/efi/efi-device.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/efi/efi-device.c b/drivers/efi/efi-device.c index 6ed7f12b3..7029bfb31 100644 --- a/drivers/efi/efi-device.c +++ b/drivers/efi/efi-device.

[PATCH 5/5] efi: enable sercure boot support

2017-03-09 Thread Jean-Christophe PLAGNIOL-VILLARD
This will ensure that we just start secured binary without user confirmation But for now on we only support EFI correctly signed image to start Later will allow both. Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD --- arch/x86/Kconfig | 1 + common/efi/efi-image.c | 1 + drivers/efi/

[PATCH 2/5] efi: fix lds for secure boot support

2017-03-09 Thread Jean-Christophe PLAGNIOL-VILLARD
everythink need to be aligned to 4096 Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD --- arch/x86/mach-efi/elf_ia32_efi.lds.S | 10 +++--- arch/x86/mach-efi/elf_x86_64_efi.lds.S | 10 ++ arch/x86/mach-efi/include/mach/barebox.lds.h | 14 +- include/asm-gene

[PATCH 1/5] efi: add more security related guid for the efivars

2017-03-09 Thread Jean-Christophe PLAGNIOL-VILLARD
Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD --- common/efi-guid.c | 6 ++ include/efi.h | 18 ++ 2 files changed, 24 insertions(+) diff --git a/common/efi-guid.c b/common/efi-guid.c index 71aa21ddd..01b02bbb1 100644 --- a/common/efi-guid.c +++ b/common/efi-guid.c @@

[PATCH 0/5] EFI Secure boot support

2017-03-09 Thread Jean-Christophe PLAGNIOL-VILLARD
Hi, This patch serie is the first one to add the secure boot support to barebox on efi For now on this will allow you to execute only properly signed EFI Application. And request confirmation for non signed binary other than EFI Later will add non

Re: [PATCH 08/20] clk: No-op CLK_OF_DECLARE if not enabled

2017-03-09 Thread Sascha Hauer
Hi Andrey, On Wed, Mar 08, 2017 at 02:08:57PM -0800, Andrey Smirnov wrote: > Instead of wrapping each defenition of CLK_OF_DECLARE hook with > preprocessor guards, change the definition of CLK_OF_DECLARE to expand > into no-op if COMMON_CLK_OF_PROVIDER is not enabled. > > Signed-off-by: Andrey Sm

[PATCH 3/3] fbcon: use __iowrite{32/64} to speedup the framebuffer console scrolling

2017-03-09 Thread Jean-Christophe PLAGNIOL-VILLARD
this devide the time by 4 on x86 Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD --- drivers/video/fbconsole.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/video/fbconsole.c b/drivers/video/fbconsole.c index b261f1704..b5e951e23 100644 --- a/drivers/video/fbconso

[PATCH 2/3] x86: add __raw_read{b,w,l,q} and __raw_write{b,w,l,q}

2017-03-09 Thread Jean-Christophe PLAGNIOL-VILLARD
so we can use __iowrite{32/64} to speedup the framebuffer console scrolling Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD --- arch/x86/include/asm/io.h | 33 + 1 file changed, 33 insertions(+) diff --git a/arch/x86/include/asm/io.h b/arch/x86/include/asm/io.h i

[PATCH 1/3] import linux __iowrite32/64_copy support

2017-03-09 Thread Jean-Christophe PLAGNIOL-VILLARD
Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD --- include/linux/io.h | 27 + lib/Makefile | 1 + lib/iomap_copy.c | 70 ++ 3 files changed, 98 insertions(+) create mode 100644 include/linux/io.h create mode 10064

[PATCH 2/2] video: some framebuffer such as efi does support enable/disable

2017-03-09 Thread Jean-Christophe PLAGNIOL-VILLARD
so allow them to do not pass this function via fbops and indicate they are always on. Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD --- drivers/video/fb.c | 29 - 1 file changed, 24 insertions(+), 5 deletions(-) diff --git a/drivers/video/fb.c b/drivers/video/fb.c

[PATCH 1/2] parameter: fix read only int support

2017-03-09 Thread Jean-Christophe PLAGNIOL-VILLARD
pass PARAM_FLAG_RO flag for read only it so we can not change them Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD --- lib/parameter.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/parameter.c b/lib/parameter.c index 9f96d0760..65d6c7c0d 100644 --- a/lib/parameter.c

[PATCH 2/2] usb: imx: Fix setting of dev->priv

2017-03-09 Thread Sascha Hauer
Since fc10539f7d the usbmisc pointer must be stored in dev->priv, but dev->priv has never been set. Fix this. Fixes: fc10539f7d usb: imx: Make usb-misc multi instance safe Signed-off-by: Sascha Hauer --- drivers/usb/imx/imx-usb-misc.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) di

[PATCH 1/2] usb: gadget: print a meaningful error message

2017-03-09 Thread Sascha Hauer
We can only register one USB multi gadget, so check for it being already registered and print a meaningful error message if it is. Signed-off-by: Sascha Hauer --- drivers/usb/gadget/multi.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/usb/gadget/multi.c b/drivers/usb/gadget/m

[PATCH 2/2] usb: gadget: properly release f_multi_opts

2017-03-09 Thread Sascha Hauer
The usbgadget commands uses statically allocated f_multi_opts and passes this to usb_multi_register(). These f_multi_opts are of course no longer valid when we leave the usbgadget command. Luckily we do not use the data after we left the usbgadget command, so this never has been a problem. However,

Re: [PATCH 1/2] video: add EFI Graphics Output Protocol support

2017-03-09 Thread Jean-Christophe PLAGNIOL-VILLARD
> On Mar 9, 2017, at 4:46 PM, Michael Olbrich wrote: > > On Mon, Mar 06, 2017 at 06:04:10AM +0100, Jean-Christophe PLAGNIOL-VILLARD > wrote: >> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD >> --- >> drivers/video/Kconfig | 4 + >> drivers/video/Makefile | 2 + >> drivers/video/efi_gop.

Re: [PATCH 1/2] video: add EFI Graphics Output Protocol support

2017-03-09 Thread Michael Olbrich
On Mon, Mar 06, 2017 at 06:04:10AM +0100, Jean-Christophe PLAGNIOL-VILLARD wrote: > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD > --- > drivers/video/Kconfig | 4 + > drivers/video/Makefile | 2 + > drivers/video/efi_gop.c | 267 > > 3

[PATCH 1/2] usb: imx: role_registered contains role

2017-03-09 Thread Sascha Hauer
role_registered contains the role, it's not a bool indicating that a role is registered. Signed-off-by: Sascha Hauer --- drivers/usb/imx/chipidea-imx.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/usb/imx/chipidea-imx.c b/drivers/usb/imx/chipidea-imx.c index cc