[U-Boot] [PATCH] sunxi: Add defconfig and DTS file for Allwinner R16 EVB (Parrot)

2016-06-24 Thread Quentin Schulz
, an audio/microphone jack, a camera CSI port, 2 sets of 22 GPIOs and an accelerometer. The DTS file is identical to the one submitted to the upstream kernel. Signed-off-by: Quentin Schulz <quentin.sch...@free-electrons.com> --- arch/arm/dts/Makefile | 3 +- arch/arm/dts/sun

Re: [U-Boot] Ethernet via USB on Sinlinx SinA33

2016-04-11 Thread Quentin Schulz
Hi, On 11/04/2016 13:07, Hans de Goede wrote: > Hi, > > On 11-04-16 12:25, Quentin Schulz wrote: >> Hi, >> >> On 06/04/2016 17:24, Hans de Goede wrote: >>> Hi, >>> >>> On 29-03-16 08:38, Quentin Schulz wrote: >>>>

Re: [U-Boot] Ethernet via USB on Sinlinx SinA33

2016-04-11 Thread Quentin Schulz
Hi, On 06/04/2016 17:24, Hans de Goede wrote: > Hi, > > On 29-03-16 08:38, Quentin Schulz wrote: >> Hi, >> >> On 28/03/2016 14:56, Hans de Goede wrote: >>> Hi, >>> >>> On 25-03-16 18:14, Quentin Schulz wrote: >>>> Hi,

[U-Boot] Ethernet via USB on Sinlinx SinA33

2016-03-25 Thread Quentin Schulz
Hi, I am trying to get Ethernet to work through the USB port of the Sinlinx SinA33 on U-Boot to use TFTP to get the kernel and dtb files. However, I am getting 'data abort' when using dhcp or tftp after adding: #define CONFIG_USB_HOST_ETHER #define CONFIG_USB_ETHER_ASIX to

Re: [U-Boot] Ethernet via USB on Sinlinx SinA33

2016-03-29 Thread Quentin Schulz
Hi, On 28/03/2016 14:56, Hans de Goede wrote: > Hi, > > On 25-03-16 18:14, Quentin Schulz wrote: >> Hi, >> >> I am trying to get Ethernet to work through the USB port of the Sinlinx >> SinA33 on U-Boot to use TFTP to get the kernel and dtb files. >> >

[U-Boot] Ethernet on Colombus (sun6i-a31-colombus)

2016-05-19 Thread Quentin Schulz
Hi, I put U-Boot for the Colombus on an SD Card but could not make Ethernet work. In U-Boot v2015.10, v2016.01 and v2016.05, Ethernet port is not detected. In U-Boot v2016.03, the Ethernet port is detected but cannot be used. I get the following error: => dhcp ethernet@01c3 Waiting for PHY

[U-Boot] [PATCH] sunxi: clk: fix N formula for CPUX clocks

2017-09-26 Thread Quentin Schulz
("sunxi: clk: add basic clocks for A83T") Signed-off-by: Quentin Schulz <quentin.sch...@free-electrons.com> --- arch/arm/include/asm/arch-sunxi/clock_sun8i_a83t.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/include/asm/arch-sunxi/clock_sun8i_a83t.h b/

Re: [U-Boot] [PATCH 10/14] env: Initialise all the environments

2017-11-28 Thread Quentin Schulz
return 0; > + if (!drv->init || !drv->init()) > + gd->env_has_init |= BIT(drv->location); > > - debug("%s: Environment %s failed to init (err=%d)\n", __func__, > + debug("%s: Environment %s init

Re: [U-Boot] [RFC PATCH 04/10] env: Pass additional parameters to the env lookup function

2017-11-24 Thread Quentin Schulz
gt; > - return 0; > + return -ENODEV; > } > > int env_init(void) > { > - struct env_driver *drv = env_driver_lookup(); > + struct env_driver *drv; > int ret = -ENOENT; > + int prio; > + >

Re: [U-Boot] [PATCH v3 3/5] docs: Document verified-boot for sunxi a64

2017-12-13 Thread Quentin Schulz
ff133cc0156828d48c8efb96987fa45f885761d22a25dc1 > + Verifying Hash Integrity ... sha256+ OK > + Booting using the fdt blob at 0x40710f24 > + Uncompressing Kernel Image ... OK > + Loading Device Tree to 49ffa000, end 49fff347 ... OK > + > +Starting kernel ... > +>8 > + > +Please pay attention to the lines that start with "Verifying Hash Integrity". > + > +"Verifying Hash Integrity ... sha256,rsa2048:dev+ OK" means the signature > check > +passed. > + > +"Verifying Hash Integrity ... sha256+ OK" (2 times) means the hash check > passed > +for kernel and DTB. > + > +If they are not displayed, the Verified Boot is not working. > + > +-- > +Jagan Teki <ja...@amarulasolutions.com> > +13 Dec 2017 > What's specific to sunxi boards in this README? We already have an extensive documentation in doc/uImage.FIT/. Could a few parts of your README make it to this documentation instead if it isn't already there? Thanks, Quentin -- Quentin Schulz, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

[U-Boot] [RESEND][PATCH] cmd: nvedit: add whitelist option for env import

2018-05-15 Thread Quentin Schulz
a handful of variables from an other, unsecure, environment without completely losing control of U-Boot. Signed-off-by: Quentin Schulz <quentin.sch...@bootlin.com> --- cmd/nvedit.c | 66 1 file changed, 62 insertions(+), 4 deletions(-)

Re: [U-Boot] [RESEND][PATCH] cmd: nvedit: add whitelist option for env import

2018-05-16 Thread Quentin Schulz
Hi Joe, On Tue, May 15, 2018 at 01:25:09PM -0500, Joe Hershberger wrote: > On Tue, May 15, 2018 at 5:03 AM, Quentin Schulz > <quentin.sch...@bootlin.com> wrote: > > While the `env export` can take as parameters variables to be exported, > > `env import` does not have such

Re: [U-Boot] [RESEND][PATCH] cmd: nvedit: add whitelist option for env import

2018-05-16 Thread Quentin Schulz
Hi Simon, On Tue, May 15, 2018 at 12:28:14PM -0600, Simon Glass wrote: > Hi Quentin, > > On 15 May 2018 at 20:03, Quentin Schulz <quentin.sch...@bootlin.com> wrote: > > While the `env export` can take as parameters variables to be exported, > > `env import` doe

Re: [U-Boot] [PATCH v2 1/2] cmd: nvedit: add whitelist option for env import

2018-05-21 Thread Quentin Schulz
On Mon, May 21, 2018 at 01:36:49PM +0100, Alex Kiernan wrote: > On Mon, May 21, 2018 at 1:06 PM Quentin Schulz <quentin.sch...@bootlin.com> > wrote: > > > Hi Alex, > > > On Mon, May 21, 2018 at 12:56:04PM +0100, Alex Kiernan wrote: > > > On Mon,

Re: [U-Boot] [PATCH v2 1/2] cmd: nvedit: add whitelist option for env import

2018-05-21 Thread Quentin Schulz
Hi Alex, On Mon, May 21, 2018 at 12:56:04PM +0100, Alex Kiernan wrote: > On Mon, May 21, 2018 at 9:02 AM Quentin Schulz <quentin.sch...@bootlin.com> > wrote: > > > Hi Stephen, > > > On Fri, May 18, 2018 at 10:00:27AM -0600, Stephen Warren wrote: > > >

Re: [U-Boot] [PATCH v2 1/2] cmd: nvedit: add whitelist option for env import

2018-05-21 Thread Quentin Schulz
On Mon, May 21, 2018 at 01:26:11PM +0100, Alex Kiernan wrote: > On Mon, May 21, 2018 at 1:06 PM Quentin Schulz <quentin.sch...@bootlin.com> > wrote: > > > Hi Alex, > > > On Mon, May 21, 2018 at 12:56:04PM +0100, Alex Kiernan wrote: > > > On Mon,

Re: [U-Boot] [PATCH v3 1/2] cmd: nvedit: env import can now import only variables passed as parameters

2018-05-25 Thread Quentin Schulz
Hi Lothar and Alex, On Fri, May 25, 2018 at 01:26:20PM +0200, Lothar Waßmann wrote: > Hi, > > On Fri, 25 May 2018 11:52:17 +0100 Alex Kiernan wrote: > > On Fri, May 25, 2018 at 9:38 AM Quentin Schulz <quentin.sch...@bootlin.com> > > wrote: > > > > > Whi

[U-Boot] [PATCH v3 1/2] cmd: nvedit: env import can now import only variables passed as parameters

2018-05-25 Thread Quentin Schulz
binary and load only a handful of variables from an other, unsecure, environment without completely losing control of U-Boot. Signed-off-by: Quentin Schulz <quentin.sch...@bootlin.com> --- v3: - migrate to env import addr size var... instead of env import -w addr size so that th

[U-Boot] [PATCH v3 2/2] test/py: add test for whitelist of variables while importing environment

2018-05-25 Thread Quentin Schulz
importing exported env, foo shall be empty, Any variable not passed as parameter should be left untouched. Two other tests are made to test that size cannot be '-' if the checksum protection is enabled. Signed-off-by: Quentin Schulz <quentin.sch...@bootlin.com> --- v3: - update whitelis

[U-Boot] [PATCH v2 1/2] cmd: nvedit: add whitelist option for env import

2018-05-18 Thread Quentin Schulz
a handful of variables from an other, unsecure, environment without completely losing control of U-Boot. Signed-off-by: Quentin Schulz <quentin.sch...@bootlin.com> --- v2: - use strdup instead of malloc + strcpy, - NULL-check the result of strdup, - add common exit path for freeing memory

[U-Boot] [PATCH v2 2/2] test/py: add test for whitelist of variables while importing environment

2018-05-18 Thread Quentin Schulz
exported env, foo shall be empty, Any variable not in whitelisted_vars should be left untouched. Signed-off-by: Quentin Schulz <quentin.sch...@bootlin.com> --- added in v2 test/py/tests/test_env.py | 40 1 file changed, 40 insertions(+) diff --git

Re: [U-Boot] [PATCH v2 1/2] cmd: nvedit: add whitelist option for env import

2018-05-21 Thread Quentin Schulz
Hi Lothar, On Sun, May 20, 2018 at 03:01:22PM +0200, Lothar Waßmann wrote: > Hi, > > On Fri, 18 May 2018 16:44:59 +0200 Quentin Schulz wrote: > > While the `env export` can take as parameters variables to be exported, > > `env import` does not have such a mechanism

Re: [U-Boot] [PATCH v2 2/2] test/py: add test for whitelist of variables while importing environment

2018-05-21 Thread Quentin Schulz
Hi Stephen, On Fri, May 18, 2018 at 10:04:05AM -0600, Stephen Warren wrote: > On 05/18/2018 08:45 AM, Quentin Schulz wrote: > > This tests that the importing of an environment with a specified > > whitelist works as intended. > > > > If the variable whitelisted_vars is

Re: [U-Boot] [PATCH v2 1/2] cmd: nvedit: add whitelist option for env import

2018-05-21 Thread Quentin Schulz
Hi Stephen, On Fri, May 18, 2018 at 10:00:27AM -0600, Stephen Warren wrote: > On 05/18/2018 08:44 AM, Quentin Schulz wrote: > > While the `env export` can take as parameters variables to be exported, > > `env import` does not have such a mechanism of variable selection. > >

Re: [U-Boot] [PATCH v4 2/2] test/py: add test for whitelist of variables while importing environment

2018-06-08 Thread Quentin Schulz
Hi Stephen, On Thu, Jun 07, 2018 at 05:21:19PM -0600, Stephen Warren wrote: > On 06/04/2018 03:47 AM, Quentin Schulz wrote: > > This tests that the importing of an environment with a specified > > whitelist works as intended. > > > > If there are variables passed as

Re: [U-Boot] [U-Boot, v4, 2/2] test/py: add test for whitelist of variables while importing environment

2018-06-13 Thread Quentin Schulz
Hi Tom, On Wed, Jun 13, 2018 at 11:43:32AM -0400, Tom Rini wrote: > On Mon, Jun 04, 2018 at 11:47:30AM +0200, Quentin Schulz wrote: > > > This tests that the importing of an environment with a specified > > whitelist works as intended. > > > > If there are

[U-Boot] [PATCH] net: designware: set the PS bit when resetting DMA bus in MII configuration

2018-06-04 Thread Quentin Schulz
, with which the existing driver was working fine. - One equipped with a MII PHY, where the current driver fails because the DMA reset times out. Note: Taken from https://www.spinics.net/lists/netdev/msg432578.html Signed-off-by: Quentin Schulz --- Note, it looks like dw_adjust_link[1] already

[U-Boot] [PATCH v4 1/2] cmd: nvedit: env import can now import only variables passed as parameters

2018-06-04 Thread Quentin Schulz
binary and load only a handful of variables from an other, unsecure, environment without completely losing control of U-Boot. Signed-off-by: Quentin Schulz Tested-by: Alex Kiernan --- v4: - add tested-by by Alex, v3: - migrate to env import addr size var... instead of env import -w addr

[U-Boot] [PATCH v4 2/2] test/py: add test for whitelist of variables while importing environment

2018-06-04 Thread Quentin Schulz
importing exported env, foo shall be empty, Any variable not passed as parameter should be left untouched. Two other tests are made to test that size cannot be '-' if the checksum protection is enabled. Signed-off-by: Quentin Schulz Reviewed-by: Simon Glass --- v4: - add reviewed-by by Simon

Re: [U-Boot] [PATCH v5 4/5] cmd: nvedit: env import can now import only variables passed as parameters

2018-07-02 Thread Quentin Schulz
Hi Wolfgang, On Fri, Jun 29, 2018 at 02:52:29PM +0200, Wolfgang Denk wrote: > Dear Quentin, > > In message <20180629121942.lm4qbfmm5ya7fsx2@qschulz> you wrote: > > > > > I suggest that this isNOT the default behaviour. I think most > > > peaople would expect that this command only

Re: [U-Boot] [PATCH v5 4/5] cmd: nvedit: env import can now import only variables passed as parameters

2018-07-02 Thread Quentin Schulz
Hi Wolfgang, On Mon, Jul 02, 2018 at 01:21:09PM +0200, Wolfgang Denk wrote: > Dear Quentin, > > In message <20180702092548.bciqnfyd7d2hv26x@qschulz> you wrote: > > > > > Hm I don't see what you mean. himport_r imports a set of new > > > name=value pairs (in different formats); however,

Re: [U-Boot] [U-Boot, v4, 2/2] test/py: add test for whitelist of variables while importing environment

2018-06-26 Thread Quentin Schulz
Hi all, On Wed, Jun 13, 2018 at 01:02:06PM -0600, Stephen Warren wrote: > On 06/13/2018 12:53 PM, Quentin Schulz wrote: > > Hi Tom, > > > > On Wed, Jun 13, 2018 at 11:43:32AM -0400, Tom Rini wrote: > > > On Mon, Jun 04, 2018 at 11:47:30AM +0200, Quentin Schulz wr

Re: [U-Boot] [U-Boot, v4, 2/2] test/py: add test for whitelist of variables while importing environment

2018-06-27 Thread Quentin Schulz
Hi Tom, On Tue, Jun 26, 2018 at 10:44:59AM -0400, Tom Rini wrote: > On Tue, Jun 26, 2018 at 02:41:40PM +0200, Quentin Schulz wrote: > > Hi all, > > > > On Wed, Jun 13, 2018 at 01:02:06PM -0600, Stephen Warren wrote: > > > On 06/13/2018 12:53 PM, Quenti

[U-Boot] [PATCH] cmd: nvedit: add whitelist option for env import

2018-01-04 Thread Quentin Schulz
a handful of variables from an other, unsecure, environment without completely losing control of U-Boot. Signed-off-by: Quentin Schulz <quentin.sch...@free-electrons.com> --- cmd/nvedit.c | 71 ++-- 1 file changed, 64 insertions(+), 7 del

Re: [U-Boot] [PATCH 00/11] Introduce variables whitelisting in environment

2018-01-03 Thread Quentin Schulz
Hi Simon, On 29/12/2017 04:13, Simon Glass wrote: > Hi Quentin, > > On 22 December 2017 at 14:13, Quentin Schulz > <quentin.sch...@free-electrons.com> wrote: >> This patch series is based on this[1] patch series from Maxime. >> >> This is an RFC. It's bee

Re: [U-Boot] [PATCH 03/11] env: add support for whitelisting variables from secondary environments

2018-01-03 Thread Quentin Schulz
d it to suits your needs? > It's pretty much everything I needed except the whitelisting of variables. While it's possible for env export to take a list of variables to export, for what I've seen, env import does not take a list of variables to import. It's needed since the env I'll import is u

[U-Boot] [PATCH 00/11] Introduce variables whitelisting in environment

2017-12-22 Thread Quentin Schulz
feedback on this patch series. Thanks, Quentin [1] https://patchwork.ozlabs.org/cover/842057/ Quentin Schulz (11): env: fix ret not being set and fails when no env could have been init lib: hashtable: support whitelisting env variables env: add support for whitelisting variables from secondary env

[U-Boot] [PATCH 01/11] env: fix ret not being set and fails when no env could have been init

2017-12-22 Thread Quentin Schulz
This patch is fixing things that most likely will be done by Maxime in the v2 of his patch series for multiple environments support, one way or the other. Signed-off-by: Quentin Schulz <quentin.sch...@free-electrons.com> --- env/env.c | 21 +++-- 1 file changed, 11 inse

[U-Boot] [PATCH 04/11] env: make nowhere an env medium like the others

2017-12-22 Thread Quentin Schulz
makes it possible to either enforce the value of some variables by using nowhere medium when using it as secondary environment or to completely enforce all the environment variables except a few from a secondary environment when using it as base environment. Signed-off-by: Quentin Schulz <quentin.

[U-Boot] [PATCH 03/11] env: add support for whitelisting variables from secondary environments

2017-12-22 Thread Quentin Schulz
V_VAR_WHITELIST_LIST in Kconfig. Signed-off-by: Quentin Schulz <quentin.sch...@free-electrons.com> --- env/Kconfig | 17 +++- env/common.c | 7 +++- env/env.c | 100 +++ include/environment.h | 6 +++- 4 fil

[U-Boot] [PATCH 02/11] lib: hashtable: support whitelisting env variables

2017-12-22 Thread Quentin Schulz
that are not in the loaded environment from the current environment. The remaining of the code will rightfully override any variable that already exists. Signed-off-by: Quentin Schulz <quentin.sch...@free-electrons.com> --- board/sunxi/board.c | 2 +- cmd/nvedit.c| 2 +- env/common.c

[U-Boot] [PATCH 08/11] env: add env_driver parameter to env_import_redund

2017-12-22 Thread Quentin Schulz
To prepare for an upcoming patch that uses the location of the driver used when doing an env_import_redund, add env_driver as an argument of env_import_redund so that it can be used within. Update all calls to this function and the prototype. Signed-off-by: Quentin Schulz <quentin.sch...@f

[U-Boot] [PATCH 10/11] env: introducing env_info struct for storing info per env

2017-12-22 Thread Quentin Schulz
to be used in include/asm-generic/global_data.h and thus needs to be put out of the #ifndef DO_DEPS_ONLY. Signed-off-by: Quentin Schulz <quentin.sch...@free-electrons.com> --- include/environment.h | 34 -- 1 file changed, 20 insertions(+), 14 deletions(-) diff

[U-Boot] [PATCH 09/11] env: make env_locations a global variable

2017-12-22 Thread Quentin Schulz
Make env_locations a global variable so that it can be used in env drivers. Signed-off-by: Quentin Schulz <quentin.sch...@free-electrons.com> --- env/env.c | 2 +- include/environment.h | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/env/env.c b/env/env.c

[U-Boot] [PATCH 07/11] env: gd flags without ENV_READY is enough to discriminate in

2017-12-22 Thread Quentin Schulz
and getting from env_get_default. Signed-off-by: Quentin Schulz <quentin.sch...@free-electrons.com> --- env/common.c | 3 --- env/env.c| 2 +- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/env/common.c b/env/common.c index c8d8993..75b8334 100644 --- a/env/common.c +++ b/env/co

[U-Boot] [PATCH 06/11] env: add env_driver to env_driver functions' arguments

2017-12-22 Thread Quentin Schulz
Might be interesting to get some infos about the driver (e.g. its location) when inside one of its functions. Signed-off-by: Quentin Schulz <quentin.sch...@free-electrons.com> --- env/eeprom.c | 6 +++--- env/env.c | 8 env/ext4.c| 4 ++-- env

[U-Boot] [PATCH 05/11] cmd: saveenv: enable the saveenv command when ENV_IS_NOWHERE is defined

2017-12-22 Thread Quentin Schulz
but another env medium is enabled too Now that it is possible to have multiple environments at the same time, nowhere included, enable the saveenv command when nowhere medium is enabled but accompanied by another medium on which the saveenv command is possible. Signed-off-by: Quentin Schulz

[U-Boot] [PATCH 11/11] env: store information about each environment in gd

2017-12-22 Thread Quentin Schulz
(only when using redundant env and it's set before being used). Signed-off-by: Quentin Schulz <quentin.sch...@free-electrons.com> --- common/board_r.c | 8 - env/common.c | 25 ++--- env/eeprom.c | 34 +++-

[U-Boot] [PATCH 1/2] spi: add support for ARM PL022 SPI controller

2018-08-07 Thread Quentin Schulz
This adds support for the ARM PL022 SPI controller for the standard variant (0x00041022) which has a 16bit wide and 8 locations deep TX/RX FIFO. A few parts were borrowed from the Linux kernel driver. Cc: Armando Visconti Cc: Vipin Kumar Signed-off-by: Quentin Schulz --- Based on "[PAT

[U-Boot] [PATCH 2/2] arm: spear: enable SSP (SPI controllers) clocks when PL022 is enabled

2018-08-07 Thread Quentin Schulz
There are three separate clocks for each SSP (SPI) controller in the SoC. These SPI controllers are ARM PL022 controllers. Let's enable the clocks when support for PL022 SPI controller is enabled. Signed-off-by: Quentin Schulz --- arch/arm/cpu/arm926ejs/spear/cpu.c | 3 +++ 1 file changed, 3

[U-Boot] [PATCH] arm: spear: enable GPIO3 and 4 clocks when the GPIO controller driver is built

2018-08-31 Thread Quentin Schulz
There are multiple GPIOs that can be used with the same driver depending on the CONFIG_GPIO_BASE defined in the controller driver. GPIO3 and 4 require a clock to be enabled so let's enable them when the driver is built. Signed-off-by: Quentin Schulz --- arch/arm/cpu/arm926ejs/spear/cpu.c

[U-Boot] [PATCH v6 2/6] test/py: return a RAM address different from 0 as it can be interpreted as NULL

2018-07-09 Thread Quentin Schulz
. Suggested-by: Stephen Warren Signed-off-by: Quentin Schulz Reviewed-by: Simon Glass Reviewed-by: Stephen Warren Tested-by: Stephen Warren --- added in v5 test/py/u_boot_utils.py | 6 ++ 1 file changed, 6 insertions(+) diff --git a/test/py/u_boot_utils.py b/test/py/u_boot_utils.py

[U-Boot] [PATCH v6 1/6] env: add the same prefix to error messages to make it detectable by tests

2018-07-09 Thread Quentin Schulz
Warren Signed-off-by: Quentin Schulz Reviewed-by: Simon Glass Reviewed-by: Stephen Warren Tested-by: Stephen Warren --- added in v5 cmd/nvedit.c | 12 env/common.c | 3 ++- 2 files changed, 10 insertions(+), 5 deletions(-) diff --git a/cmd/nvedit.c b/cmd/nvedit.c index ddc888a

[U-Boot] [PATCH v6 5/6] cmd: nvedit: env import can now import only variables passed as parameters

2018-07-09 Thread Quentin Schulz
or the overall behaviour. One of its use case could be to load a secure environment from the signed U-Boot binary and load only a handful of variables from an other, unsecure, environment without completely losing control of U-Boot. Signed-off-by: Quentin Schulz Tested-by: Alex Kiernan Tested-by: Stephen

[U-Boot] [PATCH v6 3/6] test/py: remove hacks for non-zero RAM base address in tests

2018-07-09 Thread Quentin Schulz
Some functions have different behaviour when the given address is 0 (assumed to be NULL by the function). find_ram_base() does not return 0 anymore so it's safe to remove those offsets. Suggested-by: Stephen Warren Signed-off-by: Quentin Schulz Reviewed-by: Stephen Warren Tested-by: Stephen

[U-Boot] [PATCH v6 6/6] test/py: add test for whitelist of variables while importing environment

2018-07-09 Thread Quentin Schulz
importing exported env, foo shall be empty ONLY if the -d option is passed to env import, otherwise foo shall be bar, Any variable not passed as parameter should be left untouched. Two other tests are made to test that size cannot be '-' if the checksum protection is enabled. Signed-off-by: Quentin

[U-Boot] [PATCH v6 4/6] hashtable: do not recreate whole hash table if vars are passed to himport_r

2018-07-09 Thread Quentin Schulz
if the H_NOCLEAR flag is not set. Let's clear the whole environment only if H_NOCLEAR and vars are not passed to himport_r. Let's remove variables that are in the current environment but not in the imported env only if the H_NOCLEAR flag is not passed. Suggested-by: Wolfgang Denk Signed-off-by: Quentin

Re: [U-Boot] [PATCH v6 1/6] env: add the same prefix to error messages to make it detectable by tests

2018-07-09 Thread Quentin Schulz
unrelated to this patch series (happened multiple times without my patch series) as far as I can tell. Quentin On Mon, Jul 09, 2018 at 07:16:25PM +0200, Quentin Schulz wrote: > The error message should start with `## Error: ` so that it's easily > detectable by tests without needing to have

Re: [U-Boot] [PATCH] fit: skip signature verification if board request

2018-04-17 Thread Quentin Schulz
Hi, On Wed, Apr 11, 2018 at 11:13:05PM +0800, Jun Nie wrote: > It may be unnecessary to check signature on unlocked board. > Get the hint from platform specific code to support secure boot > and non-secure boot with the same binary, so that boot is not > blocked if board is not locked and has no

Re: [U-Boot] [PATCH] cmd: nvedit: Add filtering during env import

2018-03-27 Thread Quentin Schulz
On Tue, Mar 27, 2018 at 01:39:25PM +0100, Alex Kiernan wrote: > On Tue, Mar 27, 2018 at 10:28 AM, Quentin Schulz > <quentin.sch...@bootlin.com> wrote: > > Hi Alex, > > > > On Tue, Mar 27, 2018 at 08:43:26AM +, Alex Kiernan wrote: > >> When importing

Re: [U-Boot] [PATCH] cmd: nvedit: Add filtering during env import

2018-03-27 Thread Quentin Schulz
On Tue, Mar 27, 2018 at 03:14:59PM +0200, Quentin Schulz wrote: > On Tue, Mar 27, 2018 at 01:39:25PM +0100, Alex Kiernan wrote: > > On Tue, Mar 27, 2018 at 10:28 AM, Quentin Schulz > > <quentin.sch...@bootlin.com> wrote: > > > Hi Alex, > > > > > >

Re: [U-Boot] [PATCH] cmd: nvedit: Add filtering during env import

2018-03-27 Thread Quentin Schulz
Hi Alex, On Tue, Mar 27, 2018 at 08:43:26AM +, Alex Kiernan wrote: > When importing variables allow size to be elided using '-' and then > support a list of variables which restricts what will be picked during > the import. > > Signed-off-by: Alex Kiernan I'm pretty

Re: [U-Boot] [PATCH] cmd: nvedit: add whitelist option for env import

2018-03-02 Thread Quentin Schulz
Hi all, On Thu, Jan 04, 2018 at 11:39:15AM +0100, Quentin Schulz wrote: > While the `env export` can take as parameters variables to be exported, > `env import` does not have such a mechanism of variable selection. > > Let's add a `-w` option that asks `env imp

[U-Boot] [PATCH v5 1/5] env: add the same prefix to error messages to make it detectable by tests

2018-06-29 Thread Quentin Schulz
Warren Signed-off-by: Quentin Schulz --- added in v5 cmd/nvedit.c | 12 env/common.c | 3 ++- 2 files changed, 10 insertions(+), 5 deletions(-) diff --git a/cmd/nvedit.c b/cmd/nvedit.c index ddc888a..70d7068 100644 --- a/cmd/nvedit.c +++ b/cmd/nvedit.c @@ -959,7 +959,8 @@ NXTARG

[U-Boot] [PATCH v5 4/5] cmd: nvedit: env import can now import only variables passed as parameters

2018-06-29 Thread Quentin Schulz
binary and load only a handful of variables from an other, unsecure, environment without completely losing control of U-Boot. Signed-off-by: Quentin Schulz Tested-by: Alex Kiernan --- v4: - add tested-by by Alex, v3: - migrate to env import addr size var... instead of env import -w addr

[U-Boot] [PATCH v5 5/5] test/py: add test for whitelist of variables while importing environment

2018-06-29 Thread Quentin Schulz
importing exported env, foo shall be empty, Any variable not passed as parameter should be left untouched. Two other tests are made to test that size cannot be '-' if the checksum protection is enabled. Signed-off-by: Quentin Schulz Reviewed-by: Simon Glass Reviewed-by: Stephen Warren --- v5

[U-Boot] [PATCH v5 3/5] test/py: remove hacks for non-zero RAM base address in tests

2018-06-29 Thread Quentin Schulz
Some functions have different behaviour when the given address is 0 (assumed to be NULL by the function). find_ram_base() does not return 0 anymore so it's safe to remove those offsets. Suggested-by: Stephen Warren Signed-off-by: Quentin Schulz --- added in v5 test/py/tests

[U-Boot] [PATCH v5 2/5] test/py: return a RAM address different from 0 as it can be interpreted as NULL

2018-06-29 Thread Quentin Schulz
. Suggested-by: Stephen Warren Signed-off-by: Quentin Schulz --- added in v5 test/py/u_boot_utils.py | 6 ++ 1 file changed, 6 insertions(+) diff --git a/test/py/u_boot_utils.py b/test/py/u_boot_utils.py index bb31e57..07520ac 100644 --- a/test/py/u_boot_utils.py +++ b/test/py

Re: [U-Boot] [U-Boot, v4, 2/2] test/py: add test for whitelist of variables while importing environment

2018-06-28 Thread Quentin Schulz
Hi Tom, On Wed, Jun 27, 2018 at 11:23:38AM -0400, Tom Rini wrote: > On Wed, Jun 27, 2018 at 09:52:50AM +0200, Quentin Schulz wrote: > > Hi Tom, > > > > On Tue, Jun 26, 2018 at 10:44:59AM -0400, Tom Rini wrote: > > > On Tue, Jun 26, 2018 at 02:41:40PM +0200, Quenti

Re: [U-Boot] [U-Boot, v4, 2/2] test/py: add test for whitelist of variables while importing environment

2018-06-28 Thread Quentin Schulz
Hi Stephen, On Wed, Jun 27, 2018 at 10:07:04AM -0600, Stephen Warren wrote: > On 06/27/2018 09:23 AM, Tom Rini wrote: > > On Wed, Jun 27, 2018 at 09:52:50AM +0200, Quentin Schulz wrote: > > > Hi Tom, > > > > > > On Tue, Jun 26, 2018 at 10:44:59AM -0400, To

Re: [U-Boot] [PATCH v5 4/5] cmd: nvedit: env import can now import only variables passed as parameters

2018-06-29 Thread Quentin Schulz
Hi Wolfgang, On Fri, Jun 29, 2018 at 01:03:25PM +0200, Wolfgang Denk wrote: > Dear Quentin, > > In message > <322e6866c43f4515240ddca9456ee390b6f334c7.1530257385.git-series.quentin.sch...@bootlin.com> > you wrote: > > While the `env export` can take as parameters variables to be exported, > >

Re: [U-Boot] [PATCH 3/3] net: phy: mscc: add support for VSC8574 PHY

2018-10-30 Thread Quentin Schulz
Hi Joe, On Mon, Oct 22, 2018 at 10:25:44PM +, Joe Hershberger wrote: > On Fri, Sep 14, 2018 at 7:51 AM Quentin Schulz > wrote: > > > > The VSC8574 PHY is a 4-port PHY that is 10/100/1000BASE-T, 100BASE-FX, > > 1000BASE-X and triple-speed copper SFP capable, can comm

Re: [U-Boot] [PATCH 1/3] net: phy: mscc: add support for VSC8584 PHY

2018-10-31 Thread Quentin Schulz
Hi Joe, On Mon, Oct 22, 2018 at 09:52:38PM +, Joe Hershberger wrote: > On Fri, Sep 14, 2018 at 7:50 AM Quentin Schulz > wrote: [...] > > +#define MEDIA_OP_MODE_MASK 0x0700 > > Please use GENMASK() or BIT() for masks. > [...] > > +/* Exte

[U-Boot] [PATCH v2 0/3] add support for VSC8584 and VSC8574 Microsemi PHYs

2018-10-31 Thread Quentin Schulz
a redondant use of (0x8000 | 0x8000 | addr) in vsc8584_csr_write for vsc8574, Quentin Schulz (3): net: phy: mscc: add support for VSC8584 PHY net: phy: mscc: factorize part of config function for VSC8584 net: phy: mscc: add support for VSC8574 PHY drivers/net/phy/mscc.c | 994

[U-Boot] [PATCH v2 3/3] net: phy: mscc: add support for VSC8574 PHY

2018-10-31 Thread Quentin Schulz
modification in any register of this bank affects all PHYs of the package. Signed-off-by: Quentin Schulz --- drivers/net/phy/mscc.c | 420 + 1 file changed, 420 insertions(+) diff --git a/drivers/net/phy/mscc.c b/drivers/net/phy/mscc.c index 0861a82fc6

[U-Boot] [PATCH v2 1/3] net: phy: mscc: add support for VSC8584 PHY

2018-10-31 Thread Quentin Schulz
to not support it. Signed-off-by: Quentin Schulz --- drivers/net/phy/mscc.c | 552 + 1 file changed, 552 insertions(+) diff --git a/drivers/net/phy/mscc.c b/drivers/net/phy/mscc.c index 6f102f134e..b1d32da95c 100644 --- a/drivers/net/phy/mscc.c +++ b/drivers/net

[U-Boot] [PATCH v2 2/3] net: phy: mscc: factorize part of config function for VSC8584

2018-10-31 Thread Quentin Schulz
Part of the config is common between the VSC8584 and the VSC8574, so to prepare for the upcoming support of VSC8574, use the phy_device.priv pointer that will keep the function that holds code that is PHY-specific and that should be called during config function. Signed-off-by: Quentin Schulz

Re: [U-Boot] [PATCH 1/2] spi: pl022: Get rid of platdata ifdef's

2018-11-05 Thread Quentin Schulz
all. I also find this commit log a bit cryptic. You basically store the frequency of the clock gotten from DM instead of storing the clock itself, which is indeed a welcome simplification of the driver (and you re-order function pointers in the driver structure). > Cc: Quentin Schulz > Si

Re: [U-Boot] [PATCH v2 2/2] spi: pl022: Driver cleanup

2018-11-14 Thread Quentin Schulz
, there are two changes. IMHO, it would make sense to have two patches. Thanks, Quentin > Cc: Quentin Schulz > Signed-off-by: Jagan Teki > --- > Changes for v2: > - none > > drivers/spi/pl022_spi.c | 7 +-- > include/dm/platform_data/{pl02

Re: [U-Boot] [PATCH 1/2] spi: pl022: Get rid of platdata ifdef's

2018-11-14 Thread Quentin Schulz
Hi Jagan, On Wed, Nov 14, 2018 at 03:11:26PM +0530, Jagan Teki wrote: > On Mon, Nov 5, 2018 at 3:12 PM Quentin Schulz > wrote: > > > > Hi Jagan, > > > > On Mon, Nov 05, 2018 at 02:49:25PM +0530, Jagan Teki wrote: > > > pl022 spi driver support both OF_CO

[U-Boot] [PATCH v2 1/3] spi: add support for ARM PL022 SPI controller

2018-08-31 Thread Quentin Schulz
This adds support for the ARM PL022 SPI controller for the standard variant (0x00041022) which has a 16bit wide and 8 locations deep TX/RX FIFO. A few parts were borrowed from the Linux kernel driver. Cc: Armando Visconti Cc: Vipin Kumar Signed-off-by: Quentin Schulz --- v2: - move header

[U-Boot] [PATCH v2 2/3] arm: spear: enable SSP1, 2 and 3 clocks when SPI controller driver is built

2018-08-31 Thread Quentin Schulz
SPI controllers SSP1, 2 and 3 require to enable their respective clocks. Let's enable them only when the SPI controller driver is built. Signed-off-by: Quentin Schulz --- v2: - define and use constants for enabling SSP1/2/3 clocks when the controller is built arch/arm/cpu/arm926ejs/spear

[U-Boot] [PATCH v2 3/3] arm: spear: fix enabling of SSP2 clock

2018-08-31 Thread Quentin Schulz
The SSP2 clock is at bit 6 in the register, so the value is 0x40 unlike the current 0x70 which enables the clock of UART2, SSP1 and SSP2. Signed-off-by: Quentin Schulz --- added in v2 @Stefan: I think you'd want to test the FPGA on x600 again as it's using this constant. Having worked

[U-Boot] [PATCH] cmd: mtd: fix compilation warning for help when SYS_LONGHELP=n

2019-01-17 Thread Quentin Schulz
CONFIG_SYS_LONGHELP, since it's compiled out when calling _CMD_HELP[1] on the help text variable argument to U_BOOT_CMD. [1] https://elixir.bootlin.com/u-boot/latest/source/include/command.h#L181 Signed-off-by: Quentin Schulz --- cmd/mtd.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions

[U-Boot] Issue with memcpy when booting a fitImage on SPEAr600

2018-12-05 Thread Quentin Schulz
Hello everyone, I'm working on porting a mainline U-Boot on a custom board based on a SPEAr600. Right now, I'm facing an issue related with fitImage booting. Before starting on the fitImage issue, let me state that the zImage and the custom DTB concatenated in a uImage have the load address and

Re: [U-Boot] [PATCH v3 1/3] spi: pl022: Simplify platdata code

2018-11-25 Thread Quentin Schulz
Hi Jagan, On Sat, Nov 24, 2018 at 05:41:03PM +0530, Jagan Teki wrote: > On Thu, Nov 22, 2018 at 1:48 PM Quentin Schulz > wrote: > > > > Hi Jagan, > > > > On Thu, Nov 22, 2018 at 01:31:25PM +0530, Jagan Teki wrote: > > > On Thu, Nov 22, 2018

Re: [U-Boot] [PATCH v3 1/3] spi: pl022: Simplify platdata code

2018-11-21 Thread Quentin Schulz
p code directly in pl022_spi_ofdata_to_platdata > > Cc: Quentin Schulz > Signed-off-by: Jagan Teki > --- > Changes for v3: > - none > Changes for v2: > - Update commit message > - Use struct clk for clkdev > > drivers/spi/pl022_spi.c | 48

Re: [U-Boot] [PATCH v3 1/3] spi: pl022: Simplify platdata code

2018-11-22 Thread Quentin Schulz
Hi Jagan, On Thu, Nov 22, 2018 at 01:31:25PM +0530, Jagan Teki wrote: > On Thu, Nov 22, 2018 at 1:21 PM Quentin Schulz > wrote: > > > > Hi Jagan, > > > > On Thu, Nov 22, 2018 at 12:03:47PM +0530, Jagan Teki wrote: > > > pl022 spi driver support both OF

Re: [U-Boot] [PATCH v4 2/3] spi: pl022: Drop unnecessary include files

2018-11-27 Thread Quentin Schulz
Hi all, On Sat, Nov 24, 2018 at 05:47:22PM +0530, Jagan Teki wrote: > This patch can drop unnecessary include files in > pl022_spi driver. > > Cc: Quentin Schulz > Signed-off-by: Jagan Teki > --- > Changes for v4: > - none > Changes for v3: > - split patch f

Re: [U-Boot] [PATCH v4 3/3] dm: platform_data: spi: s/pl022_spi.h/spi_pl022.h

2018-11-27 Thread Quentin Schulz
Hi Jagan, On Sat, Nov 24, 2018 at 05:47:23PM +0530, Jagan Teki wrote: > Rename platform_data include file as spi_pl022.h from pl022_spi.h, > this is generic notation used for spi platdata include files. > > Cc: Quentin Schulz > Signed-off-by: Jagan Teki > --- > Cha

Re: [U-Boot] [PATCH v4 1/3] spi: pl022: Simplify platdata code

2018-11-27 Thread Quentin Schulz
p code directly in pl022_spi_ofdata_to_platdata > > Cc: Quentin Schulz > Signed-off-by: Jagan Teki > --- > Changes for v4: > - preserve fdtdec.h > Changes for v3: > - none > Changes for v2: > - Update commit message > - Use struct clk for clkdev

Re: [U-Boot] Issue with memcpy when booting a fitImage on SPEAr600

2019-01-07 Thread Quentin Schulz
Hi all, On Thu, Dec 06, 2018 at 11:37:14AM +0100, Thomas Petazzoni wrote: > Hello Stefan, > > On Wed, 5 Dec 2018 12:30:57 +0100, Stefan Roese wrote: > > > > It's weird to me that it's happening with this SoC because it's based on > > > ARM926ejs which is widely used I assume. Shouldn't have

Re: [U-Boot] Issue with memcpy when booting a fitImage on SPEAr600

2019-01-08 Thread Quentin Schulz
Hi Simon, On Mon, Jan 07, 2019 at 05:38:19PM -0700, Simon Glass wrote: > Hi, > > On Thu, 6 Dec 2018 at 03:37, Thomas Petazzoni > wrote: > > > > Hello Stefan, > > > > On Wed, 5 Dec 2018 12:30:57 +0100, Stefan Roese wrote: > > > > > > It's weird to me that it's happening with this SoC because

[U-Boot] [PATCH 1/3] net: phy: mscc: add support for VSC8584 PHY

2018-09-14 Thread Quentin Schulz
to not support it. Signed-off-by: Quentin Schulz --- drivers/net/phy/mscc.c | 542 + 1 file changed, 542 insertions(+) diff --git a/drivers/net/phy/mscc.c b/drivers/net/phy/mscc.c index 439f5e3c5c..0e54fe0eff 100644 --- a/drivers/net/phy/mscc.c +++ b/drivers/net

[U-Boot] [PATCH 0/3] add support for VSC8584 and VSC8574 Microsemi PHYs

2018-09-14 Thread Quentin Schulz
any modification in any register of this bank affects all PHYs of the package. Thanks, Quentin Quentin Schulz (3): net: phy: mscc: add support for VSC8584 PHY net: phy: mscc: factorize part of config function for VSC8584 net: phy: mscc: add support for VSC8574 PHY drivers/net/phy/mscc.c

[U-Boot] [PATCH 3/3] net: phy: mscc: add support for VSC8574 PHY

2018-09-14 Thread Quentin Schulz
modification in any register of this bank affects all PHYs of the package. Signed-off-by: Quentin Schulz --- drivers/net/phy/mscc.c | 413 + 1 file changed, 413 insertions(+) diff --git a/drivers/net/phy/mscc.c b/drivers/net/phy/mscc.c index b9dfcadd85

[U-Boot] [PATCH 2/3] net: phy: mscc: factorize part of config function for VSC8584

2018-09-14 Thread Quentin Schulz
Part of the config is common between the VSC8584 and the VSC8574, so to prepare for the upcoming support of VSC8574, use the phy_device.priv pointer that will keep the function that holds code that is PHY-specific and that should be called during config function. Signed-off-by: Quentin Schulz

Re: [OE-core] [yocto] Support UBI u-boot

2020-02-17 Thread Quentin Schulz
Hi JH, On Sun, Feb 09, 2020 at 08:01:15PM +1100, JH wrote: > Changed to https://source.codeaurora.org/external/imx/uboot-imx.git > branch imx_v2018.03_4.14.98_2.0.0_ga, it can now support > CONFIG_CMD_MTDPARTS ind defconfg, all MTDPARTS error gone, but then > 2018.03-r0/git/cmd/ubi.c missed

[PATCH v2] rockchip: puma/lion: update MAINTAINERS file

2022-01-03 Thread quentin . schulz
From: Quentin Schulz Philipp does not work at Theobroma Systems anymore so let's swap Philipp's address with mine. Cc: Philipp Tomsich Cc: Quentin Schulz Signed-off-by: Quentin Schulz --- v2: add commit log so that the commit can be merged @Philipp, I can update your mail address if you

Re: [PATCH] arm: dts: rk3399-puma: re-add vdd_log for uboot

2022-01-03 Thread Quentin Schulz
Hi Heiko, On 12/26/21 13:45, Heiko Stuebner wrote: The rk3399-puma board needs a 950mV vdd_log to work stable. This was already added in commit 77012e79ffc3 ("rockchip: rk3399-puma: Set VDD_LOG to 950 mV") but lost again with commit 167efc2c7a46 ("arm64: dts: rk3399: Sync v5.7-rc1 from Linux")

  1   2   3   4   5   6   7   8   9   >