Re: [U-Boot] [PATCH v2 2/3] spl: add relocation support【请注意,邮件由s...@google.com代发】

2019-07-06 Thread Andy Yan
Hi Simon: Simon Glass 于2019年7月7日周日 上午1:16写道: > Hi Andy, > > On Mon, 24 Jun 2019 at 04:37, Andy Yan wrote: > > > > Hi Simon: > > > > Glad to see you online again. > > > > On 2019/6/23 上午3:10, Simon Glass wrote: > > > Hi Andy, > > > > > > On Tue, 28 May 2019 at 09:34, Andy Yan > wrote: > > >>

Re: [U-Boot] [PATCH 5/5] net: fec_mxc: Update driver to use full clks for iMX8

2019-07-06 Thread Lukasz Majewski
Hi Ye, > Add support for more clocks used by iMX8 from DTB: > ref_clock, tx_2x_clock, ahb_clock > And update get clock rate interface to support multiple fec ports. > > Signed-off-by: Ye Li > --- > drivers/net/fec_mxc.c | 47 > ++-

Re: [U-Boot] [PATCH 3/5] clk: imx8: Update imx8 clock driver

2019-07-06 Thread Lukasz Majewski
Hi Ye, > Update imx8 clock driver to support LPCG and full clocks tree for some > modules aligned with kernel. > > We classify the clock into serveral types: slice, fixed, lpcg, gpr > and mux. Is this code in sync with Linux kernel? Does the imx8 (in Linux kernel) use the Common Clock

Re: [U-Boot] [PATCH 00/52] Support i.MX8MM/N

2019-07-06 Thread Lukasz Majewski
Hi Peng, > From: Peng Fan > > This patchset is to support i.MX8MM/8MN with some update > in ddr settings and ddr driver update. > > The initial patchset to support i.MX8MM is > https://patchwork.ozlabs.org/cover/1093140/ > But in this patchset I dropped CCF, and use original CLK uclass, so >

[U-Boot] [PULL] u-boot-usb/master

2019-07-06 Thread Marek Vasut
The following changes since commit 1f83431f0053f6fb20c511c391ffc687433848cf: board: amlogic: add mailing-list to MAINTAINERS (2019-07-04 11:36:52 -0400) are available in the Git repository at: git://git.denx.de/u-boot-usb.git master for you to fetch changes up to

Re: [U-Boot] [PATCH] test/py: avb: Move AVB test to android dir

2019-07-06 Thread Sam Protsenko
Hi Simon, On Sat, Jul 6, 2019 at 7:53 PM Simon Glass wrote: > > Hi, > > On Tue, 2 Jul 2019 at 12:18, Sam Protsenko wrote: > > > > Signed-off-by: Sam Protsenko > > --- > > test/py/tests/{ => test_android}/test_avb.py | 0 > > 1 file changed, 0 insertions(+), 0 deletions(-) > > rename

[U-Boot] [PATCH v2 1/1] disk: efi: buffer overflow in part_get_info_efi()

2019-07-06 Thread Heinrich Schuchardt
In part_get_info_efi() we use the output of print_efiname() to set info->name[]. The size of info->name is PART_NAME_LEN = 32 but print_efiname() returns a string with a maximum length of PARTNAME_SZ + 1 = 37. Use snprintf() instead of sprintf() to avoid buffer overflow. Signed-off-by: Heinrich

[U-Boot] [PATCH v3 13/13] efi_loader: detach runtime in ExitBootServices()

2019-07-06 Thread Heinrich Schuchardt
Linux can be called with a command line parameter efi=novamap, cf. commit 4e46c2a95621 ("efi/arm/arm64: Allow SetVirtualAddressMap() to be omitted"). In this case SetVirtualAddressMap() is not called after ExitBootServices(). OpenBSD 32bit does not call SetVirtualAddressMap() either. Runtime

[U-Boot] [PATCH v3 12/13] efi_loader: simplify detaching

2019-07-06 Thread Heinrich Schuchardt
We do not need any array typed detach list. Let's simply update the pointers directly. Signed-off-by: Heinrich Schuchardt --- v3 new patch --- lib/efi_loader/efi_runtime.c | 40 1 file changed, 8 insertions(+), 32 deletions(-) diff --git

[U-Boot] [PATCH v3 11/13] efi_loader: decision on EFI_RT_SUPPORTED_RESET_SYSTEM

2019-07-06 Thread Heinrich Schuchardt
Move the logic determining which board supports reset at runtime to Kconfig. Signed-off-by: Heinrich Schuchardt --- v3 no change --- lib/efi_loader/Kconfig | 6 ++ lib/efi_loader/efi_runtime.c | 5 + 2 files changed, 7 insertions(+), 4 deletions(-) diff --git

[U-Boot] [PATCH v3 03/13] efi_loader: remove NULL entries from runtime detach list

2019-07-06 Thread Heinrich Schuchardt
Some entries in the system table are set to NULL in ExitBootServices(). We had them in the runtime detach list to avoid relocation of NULL. Let's instead assign the pointers dynamically in efi_initialize_system_table() to avoid the relocation entry. Signed-off-by: Heinrich Schuchardt --- v3

[U-Boot] [PATCH v3 09/13] efi_loader: bump UEFI specification number to 2.8

2019-07-06 Thread Heinrich Schuchardt
We are implementing UEFI variable RuntimeServicesSupported and set the unimplemented runtime functions return EFI_UNSUPPORTED as described in UEFI specification 2.8. So let's also advertise this specification version in our system table. Signed-off-by: Heinrich Schuchardt Reviewed-by: Alexander

[U-Boot] [PATCH v3 10/13] test/py: not all boards support UEFI runtime reset

2019-07-06 Thread Heinrich Schuchardt
As not all boards support resets at runtime do not test for it in the Python tests. Signed-off-by: Heinrich Schuchardt --- v3 new patch --- test/py/tests/test_efi_selftest.py | 4 1 file changed, 4 deletions(-) diff --git a/test/py/tests/test_efi_selftest.py

[U-Boot] [PATCH v3 08/13] efi_loader: split off detaching SetVirtualAddress()

2019-07-06 Thread Heinrich Schuchardt
The runtime services SetVirtualAddress() and ConvertPointer() become unavailable after SetVirtualAddress(). Other runtime services become unavailable after ExitBootServices. Move the update of SetVirtualAddress() and ConvertPointer() to efi_relocate_runtime_table(). Use functions with the

[U-Boot] [PATCH v3 07/13] efi_selftest: test variable services at runtime

2019-07-06 Thread Heinrich Schuchardt
Provide a unit test for the variable services at runtime. Currently we expect EFI_UNSUPPORTED to be returned as the runtime implementation is still missing. Signed-off-by: Heinrich Schuchardt --- v3 no change v2 rebase patch --- lib/efi_selftest/Makefile |

[U-Boot] [PATCH v3 00/13] efi_loader: clean up runtime detaching

2019-07-06 Thread Heinrich Schuchardt
This patch series moves the detaching of runtime services from SetVirtualAddressMap() to ExitBootServices(). The return value of unimplemented runtime services is changed to EFI_UNSUPPORTED. A unit test for variable services at runtime is implmented. In the current version it expects

[U-Boot] [PATCH v3 05/13] efi_loader: let the variable driver patch out the runtime

2019-07-06 Thread Heinrich Schuchardt
Our variable services are only provided at boottime. Therefore when leaving boottime the variable function are replaced by dummy functions returning EFI_UNSUPPORTED. Move this patching of the runtime table to the variable services implementation. Executed it in ExitBootServices(). Signed-off-by:

[U-Boot] [PATCH v3 02/13] efi_loader: initialization of variable services

2019-07-06 Thread Heinrich Schuchardt
Provide an initialization routine for variable services. Signed-off-by: Heinrich Schuchardt --- v3 no change v2 rebase patch --- include/efi_loader.h | 2 ++ lib/efi_loader/efi_setup.c| 5 + lib/efi_loader/efi_variable.c | 10 ++ 3 files changed, 17

[U-Boot] [PATCH v3 01/13] efi_loader: move efi_query_variable_info()

2019-07-06 Thread Heinrich Schuchardt
Let's keep similar things together. Move efi_query_variable_info() to lib/efi_loader/efi_variable.c Signed-off-by: Heinrich Schuchardt --- v3 no change v2 no change --- include/efi_loader.h | 5 + lib/efi_loader/efi_runtime.c | 27 ---

[U-Boot] [PATCH v3 06/13] efi_loader: unimplemented runtime services

2019-07-06 Thread Heinrich Schuchardt
Unimplemented runtime services should always return EFI_UNSUPPORTED as described in the UEFI 2.8 spec. Signed-off-by: Heinrich Schuchardt --- v3 no change v2 no change --- lib/efi_loader/efi_runtime.c | 34 ++ 1 file changed, 2 insertions(+), 32

[U-Boot] [PATCH v3 04/13] efi_loader: clean up runtime detaching

2019-07-06 Thread Heinrich Schuchardt
The detaching of the runtime will have to move to ExitBootServices() to encompass operating system that do not call SetVirtualAddressMap(). This patch changes the logic for the relocation of the pointers in the runtime table such that the relocation becomes independent of the entries in the

Re: [U-Boot] [PATCH 2/3] bouncebuf: Add DMA validation check to addr_aligned().【请注意,邮件由u-boot-boun...@lists.denx.de代发】 addr_aligned().

2019-07-06 Thread Heiko Stuebner
Hi Simon, Am Samstag, 6. Juli 2019, 19:16:29 CEST schrieb Simon Glass: > On Thu, 4 Jul 2019 at 14:43, Heiko Stübner wrote: > > Am Dienstag, 4. Juni 2019, 05:23:14 CEST schrieb Kever Yang: > > > On 05/19/2019 12:08 AM, Simon Glass wrote: > > > > On Tue, 7 May 2019 at 03:05, Christoph Muellner > >

Re: [U-Boot] [PATCH] rockchip: make_fit_atf.py: fix loadables property set error【请注意,邮件由mark.kette...@sibelius.xs4all.nl代发】

2019-07-06 Thread Christoph Müllner
On 7/6/19 5:02 PM, Kever Yang wrote: > Hi Mark, > > > On 07/05/2019 08:03 PM, Mark Kettenis wrote: >>> From: Kever Yang >>> Date: Fri, 5 Jul 2019 19:38:42 +0800 >>> >>> Hi Christoph, >>> >>> >>> On 07/05/2019 05:15 PM, Christoph Müllner wrote: On 04.07.19 11:44, Andy Yan wrote: >

Re: [U-Boot] Watchdog DM - ENOMEM(out of memory) error

2019-07-06 Thread Simon Glass
Hi, On Thu, 27 Jun 2019 at 23:23, Suniel Mahesh wrote: > > Hi Patrick, > > thanks for the pointer and I understood your point. > > CONFIG_SYS_MALLOC_LEN is defined as SZ_32M which should be enough i guess ? > > still trying to figure out what is causing that error. This is very strange,

Re: [U-Boot] [PATCH 4/4] dm: Fix parameter description of dev_read_name()

2019-07-06 Thread Simon Glass
On Fri, 5 Jul 2019 at 10:23, Bin Meng wrote: > > The comments of dev_read_name() wrongly describe "node" as its > parameter. > > Signed-off-by: Bin Meng > --- > > include/dm/read.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Reviewed-by: Simon Glass

Re: [U-Boot] [PATCH 2/4] dm: core: Call clk_set_defaults() during probe() only for a valid ofnode

2019-07-06 Thread Simon Glass
On Fri, 5 Jul 2019 at 10:23, Bin Meng wrote: > > Without a valid ofnode, it's meaningless to call clk_set_defaults() > to process various properties. > > Signed-off-by: Bin Meng > --- > > drivers/core/device.c | 14 ++ > 1 file changed, 10 insertions(+), 4 deletions(-) Reviewed-by:

Re: [U-Boot] [PATCH 1/4] dm: timer: Skip device that does not have a valid ofnode in pre_probe()

2019-07-06 Thread Simon Glass
On Fri, 5 Jul 2019 at 10:23, Bin Meng wrote: > > It is possible that a timer device has a null ofnode, hence there is > no need to further parse DT for the clock rate. > > Signed-off-by: Bin Meng > --- > > drivers/timer/timer-uclass.c | 4 > 1 file changed, 4 insertions(+) Reviewed-by:

Re: [U-Boot] [RFC 2/2] sandbox: Update dts files

2019-07-06 Thread Simon Glass
Hi Tom, On Wed, 26 Jun 2019 at 15:33, Tom Rini wrote: > > With a more modern dtc we now get lots of warnings about problems with > the sandbox dts files and then in turn the DM tests. Start addressing > these warnings by using, or not using in some cases, addresses on nodes > and update tests

Re: [U-Boot] buildman cause PC hang for long time

2019-07-06 Thread Simon Glass
Hi Peng, On Fri, 5 Jul 2019 at 03:26, Peng Fan wrote: > > > Subject: [U-Boot] buildman cause PC hang for long time > > > > Hi Simon, > > Resolved. Kconfig was wrongly used in my patch. There was a problem where if 'make oldconfig' prompted for an option it would hang, fixed by: e62a24ce27

Re: [U-Boot] [PATCH 3/4] dm: core: Set correct "status" value for a node

2019-07-06 Thread Simon Glass
On Fri, 5 Jul 2019 at 10:23, Bin Meng wrote: > > Per device tree spec, "status" property can have a value of "okay", > or "disabled", but not "disable". > > Signed-off-by: Bin Meng > --- > > drivers/core/ofnode.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Reviewed-by: Simon

Re: [U-Boot] [PATCH v2 2/3] spl: add relocation support【请注意,邮件由s...@google.com代发】

2019-07-06 Thread Simon Glass
Hi Andy, On Mon, 24 Jun 2019 at 04:37, Andy Yan wrote: > > Hi Simon: > > Glad to see you online again. > > On 2019/6/23 上午3:10, Simon Glass wrote: > > Hi Andy, > > > > On Tue, 28 May 2019 at 09:34, Andy Yan wrote: > >> Hi Simon: > >> > >> On 2019/5/23 上午3:39, Simon Glass wrote: > >> > >> Hi

Re: [U-Boot] [PATCH v1 1/3] usb: dwc3-generic: remove dm_scan_fdt_dev() from the remove() callback

2019-07-06 Thread Simon Glass
On Fri, 5 Jul 2019 at 01:34, Jean-Jacques Hiblot wrote: > > There is simply no reason to do that here. > > Signed-off-by: Jean-Jacques Hiblot > --- > > drivers/usb/dwc3/dwc3-generic.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Simon Glass

Re: [U-Boot] [EXT] Re: [PATCH 1/2] dm: i2c: Add a flag that need generate stop bit

2019-07-06 Thread Simon Glass
Hi Chuanhua, On Sun, 23 Jun 2019 at 22:48, Chuanhua Han wrote: > > + Simon Glass > > > -Original Message- > > From: Lukasz Majewski > > Sent: 2019年6月18日 16:07 > > To: Chuanhua Han > > Cc: h...@denx.de; s...@chromium.org; Biwen Li ; > > u-boot@lists.denx.de > > Subject: [EXT] Re: [PATCH

Re: [U-Boot] [EXT] Re: [PATCH v2 1/3] dm: i2c: Add a flag that needs to generate a stop bit

2019-07-06 Thread Simon Glass
Hi Chuanhua, On Sun, 23 Jun 2019 at 22:46, Chuanhua Han wrote: > > > > > -Original Message- > > From: Simon Glass > > Sent: 2019年6月23日 3:10 > > To: Chuanhua Han > > Cc: Heiko Schocher ; U-Boot Mailing List > > ; > > Biwen Li ; Lukasz Majewski > > Subject: [EXT] Re: [PATCH v2 1/3] dm:

Re: [U-Boot] [PATCH 2/3] bouncebuf: Add DMA validation check to addr_aligned().【请注意,邮件由u-boot-boun...@lists.denx.de代发】 addr_aligned().

2019-07-06 Thread Simon Glass
Hi, On Thu, 4 Jul 2019 at 14:43, Heiko Stübner wrote: > > Hi Simon, > > Am Dienstag, 4. Juni 2019, 05:23:14 CEST schrieb Kever Yang: > > On 05/19/2019 12:08 AM, Simon Glass wrote: > > > On Tue, 7 May 2019 at 03:05, Christoph Muellner > > > wrote: > > >> Currently addr_aligned() performs an

Re: [U-Boot] [PATCH] test/py: avb: Move AVB test to android dir

2019-07-06 Thread Simon Glass
Hi, On Tue, 2 Jul 2019 at 12:18, Sam Protsenko wrote: > > Signed-off-by: Sam Protsenko > --- > test/py/tests/{ => test_android}/test_avb.py | 0 > 1 file changed, 0 insertions(+), 0 deletions(-) > rename test/py/tests/{ => test_android}/test_avb.py (100%) > > diff --git

Re: [U-Boot] [PATCH v3 2/3] cmd: Add 'bcb' command to read/modify/write BCB fields

2019-07-06 Thread Eugeniu Rosca
Hi Simon, On Sat, Jun 22, 2019 at 08:09:48PM +0100, Simon Glass wrote: > On Thu, 23 May 2019 at 16:33, Eugeniu Rosca wrote: [..] > I'm going to make some general comments as I still feel that this code > is really odd. Thanks. My replies below. > > +enum bcb_cmd { > > > +

[U-Boot] Pull request: u-boot-rockchip/tags/rockchip-for-v2019.07-rc5-3

2019-07-06 Thread Kever Yang
Hi Tom, Please pull the update: - fix rockchip ATF generator script for 'loadables' property Travis: https://travis-ci.org/keveryang/u-boot/builds/554575031 Thanks, - Kever The following changes since commit ca4491f2d2af93d8b77c48c5d272d88094fc06eb: Merge tag 'rockchip-for-v2019.07-rc5-2'

Re: [U-Boot] [PATCH] rockchip: make_fit_atf.py: fix loadables property set error【请注意,邮件由mark.kette...@sibelius.xs4all.nl代发】

2019-07-06 Thread Kever Yang
Hi Mark, On 07/05/2019 08:03 PM, Mark Kettenis wrote: >> From: Kever Yang >> Date: Fri, 5 Jul 2019 19:38:42 +0800 >> >> Hi Christoph, >> >> >> On 07/05/2019 05:15 PM, Christoph Müllner wrote: >>> On 04.07.19 11:44, Andy Yan wrote: Commit b238e4b00ced ("rockchip: Cleanup of

Re: [U-Boot] [PATCH v5 6/6] env: am57xx: Implement A/B boot process

2019-07-06 Thread Sam Protsenko
Hi Eugeniu, On Fri, Jul 5, 2019 at 9:52 PM Eugeniu Rosca wrote: > > Hi Igor, > > On Fri, Jul 05, 2019 at 03:37:36PM +0300, Igor Opaniuk wrote: > [..] > > include/environment/ti/boot.h | 58 +++ > [..] > > - "name=boot,size=20M,uuid=${uuid_gpt_boot};" \ > >

Re: [U-Boot] efi_loader: test_efi_grub_net()

2019-07-06 Thread Heinrich Schuchardt
On 7/2/19 8:59 PM, Alexander Graf wrote: On 02.07.19 18:52, Heinrich Schuchardt wrote: On 7/2/19 6:16 PM, Alexander Graf wrote: On 30.06.19 17:17, Heinrich Schuchardt wrote: Hello Alex, the test/py/tests/test_efi_loader.py test for GRUB is failing for me. I get the following output: =>

[U-Boot] [RFC] travis-ci: source for grub*.efi

2019-07-06 Thread Heinrich Schuchardt
Hello Tom, currently we extract grub*.efi either from Ubuntu or from Suse packages and use these for test/py/tests/test_efi_loader.py. This implies that we do not have test files for architectures that are not supported by Ubuntu and Suse, i.e. RISC-V. Furthermore any change in the packages may

[U-Boot] [PATCH 1/1] travis-ci: add qemu-riscv64

2019-07-06 Thread Heinrich Schuchardt
Add support for testing qemu-riscv64_defconfig. Signed-off-by: Heinrich Schuchardt --- bin/travis-ci/conf.qemu-riscv64_na | 11 +++ py/travis-ci/u_boot_boardenv_qemu_riscv64_na.py | 6 ++ 2 files changed, 17 insertions(+) create mode 100644