RE: [PATCH] test/py: tpm2: Skip tpm pytest based on env variable

2021-08-04 Thread T Karthik Reddy
Hello Tom Rini, > -Original Message- > From: Tom Rini > Sent: Wednesday, August 4, 2021 8:52 PM > To: Ashok Reddy Soma > Cc: u-boot@lists.denx.de; miquel.ray...@bootlin.com; git ; > T Karthik Reddy > Subject: Re: [PATCH] test/py: tpm2: Skip tpm pytest based on env variable > > On Fri,

Re: [PATCH 1/1] config: MPC8548CDS: eliminate symbol CONFIG_SYS_ID_EEPROM

2021-08-04 Thread Tom Rini
On Thu, Jul 29, 2021 at 08:27:17PM +0200, Heinrich Schuchardt wrote: > Symbol CONFIG_SYS_ID_EEPROM is defined in include/configs/MPC8548CDS.h > but never used. Remove it here and from the whitelist. > > Signed-off-by: Heinrich Schuchardt Applied to u-boot/master, thanks! -- Tom

Re: [PATCH v2] cmd:Elaborate 'blkcache' cmd HELP statement

2021-08-04 Thread Tom Rini
On Wed, Jun 30, 2021 at 11:57:45PM +0530, opensource@gmail.com wrote: > From: Adarsh Babu Kalepalli > > HELP description is provided for ‘configure’ sub-command > of ‘blkcache’. > > Signed-off-by: Adarsh Babu Kalepalli > Reviewed-by: Simon Glass Applied to u-boot/master, thanks! --

Re: [PATCH] doc: Add basic information about running CI tests

2021-08-04 Thread Tom Rini
On Tue, Jul 27, 2021 at 01:01:44PM -0400, Tom Rini wrote: > Start out by documenting general expectations on when CI is run, how > anyone can run Azure pipelines, and how GitLab CI pipelines can be run. > > Signed-off-by: Tom Rini Applied to u-boot/master, thanks! -- Tom signature.asc

Re: [PATCH] cmd:(cosmetic)Mentioned in 'chpart' command HELP text that it is for MTD devices

2021-08-04 Thread Tom Rini
On Mon, May 24, 2021 at 01:35:39PM +0530, opensource@gmail.com wrote: > From: Adarsh Babu Kalepalli > > Modified the help text of 'chpart' command ,mentioning that it is > for MTD devices. > > Signed-off-by: Adarsh Babu Kalepalli > Reviewed-by: Simon Glass Applied to u-boot/master,

Re: [PATCH] fs/squashfs: Fix some hardlinks reading the wrong inode

2021-08-04 Thread Tom Rini
On Fri, Apr 30, 2021 at 04:45:46PM +1200, Campbell Suter wrote: > In SquashFS, the contents of a directory is stored by > squashfs_directory_entry structures which contain the file's name, inode > and position within the filesystem. > > The inode number is not stored directly; instead each

Re: [PATCH] fit: Fix verification of images with external data

2021-08-04 Thread Tom Rini
On Tue, Apr 20, 2021 at 07:19:44PM +0100, John Keeping wrote: > The "-E" option to mkimage generates a FIT with external data using the > data-size and data-offset properties which must both be ignored when > verifying a signature. > > Add "data-offset" to the list of excluded properties for

Re: am335x_evm: failed to boot via USB

2021-08-04 Thread Tom Rini
On Wed, Aug 04, 2021 at 11:38:02PM +0300, Matwey V. Kornilov wrote: > Hi, > > I am trying to boot an am335x based board (BeagleBone Black) from USB > using v2021.07. I see the following issue when trying to boot via USB > (USB based network): > > > U-Boot SPL 2021.07-00013-gd0f1ad9f23 (Aug 02

Re: [PATCH 11/11] Remove including timestamp.h in version.h

2021-08-04 Thread Pali Rohár
On Wednesday 04 August 2021 18:15:55 Tom Rini wrote: > On Thu, Aug 05, 2021 at 12:14:29AM +0200, Pali Rohár wrote: > > On Wednesday 04 August 2021 18:09:14 Tom Rini wrote: > > > On Wed, Aug 04, 2021 at 05:43:41PM -0400, Sean Anderson wrote: > > > > > > > > > > > > On 8/2/21 3:21 PM, Simon Glass

Re: [PATCH 11/11] Remove including timestamp.h in version.h

2021-08-04 Thread Tom Rini
On Thu, Aug 05, 2021 at 12:14:29AM +0200, Pali Rohár wrote: > On Wednesday 04 August 2021 18:09:14 Tom Rini wrote: > > On Wed, Aug 04, 2021 at 05:43:41PM -0400, Sean Anderson wrote: > > > > > > > > > On 8/2/21 3:21 PM, Simon Glass wrote: > > > > Hi Pali, > > > > > > > > On Mon, 2 Aug 2021 at

Re: [PATCH 11/11] Remove including timestamp.h in version.h

2021-08-04 Thread Pali Rohár
On Wednesday 04 August 2021 18:09:14 Tom Rini wrote: > On Wed, Aug 04, 2021 at 05:43:41PM -0400, Sean Anderson wrote: > > > > > > On 8/2/21 3:21 PM, Simon Glass wrote: > > > Hi Pali, > > > > > > On Mon, 2 Aug 2021 at 07:20, Pali Rohár wrote: > > > > > > > > Header file version.h does not use

Re: [PATCH 11/11] Remove including timestamp.h in version.h

2021-08-04 Thread Tom Rini
On Wed, Aug 04, 2021 at 05:43:41PM -0400, Sean Anderson wrote: > > > On 8/2/21 3:21 PM, Simon Glass wrote: > > Hi Pali, > > > > On Mon, 2 Aug 2021 at 07:20, Pali Rohár wrote: > > > > > > Header file version.h does not use anything from timestamp.h. Including of > > > timestamp.h has side

Re: [PATCH] Nokia RX-51: Convert documentation to rst format

2021-08-04 Thread Pali Rohár
On Monday 02 August 2021 11:43:59 Lokesh Vutla wrote: > Hi Pali, > > On 01/08/21 4:18 pm, Pali Rohár wrote: > > On Monday 26 July 2021 12:38:05 Lokesh Vutla wrote: > >> On 22/07/21 2:55 am, Pali Rohár wrote: > >>> Signed-off-by: Pali Rohár > >>> --- > >>> board/nokia/rx51/MAINTAINERS

Re: [PATCH 11/11] Remove including timestamp.h in version.h

2021-08-04 Thread Pali Rohár
On Wednesday 04 August 2021 17:43:41 Sean Anderson wrote: > On 8/2/21 3:21 PM, Simon Glass wrote: > > Hi Pali, > > > > On Mon, 2 Aug 2021 at 07:20, Pali Rohár wrote: > > > > > > Header file version.h does not use anything from timestamp.h. Including of > > > timestamp.h has side effect which

Re: [PATCH 11/11] Remove including timestamp.h in version.h

2021-08-04 Thread Pali Rohár
On Wednesday 04 August 2021 10:40:42 Tom Rini wrote: > On Wed, Aug 04, 2021 at 08:36:27AM -0600, Simon Glass wrote: > > Hi Tom, > > > > On Wed, 4 Aug 2021 at 06:44, Tom Rini wrote: > > > > > > On Wed, Aug 04, 2021 at 07:59:21AM +0200, Wolfgang Denk wrote: > > > > > > > Dear Tom, > > > > > > > >

Re: [PATCH 11/11] Remove including timestamp.h in version.h

2021-08-04 Thread Sean Anderson
On 8/2/21 3:21 PM, Simon Glass wrote: Hi Pali, On Mon, 2 Aug 2021 at 07:20, Pali Rohár wrote: Header file version.h does not use anything from timestamp.h. Including of timestamp.h has side effect which cause recompiling object file at every make run because timestamp.h changes at every

am335x_evm: failed to boot via USB

2021-08-04 Thread Matwey V. Kornilov
Hi, I am trying to boot an am335x based board (BeagleBone Black) from USB using v2021.07. I see the following issue when trying to boot via USB (USB based network): U-Boot SPL 2021.07-00013-gd0f1ad9f23 (Aug 02 2021 - 21:29:57 +0300) Trying to boot from USB eth ## Error: flags type check failure

Re: [PATCH v3 2/3] arm: Add an __image_copy_start symbol for ARMv8

2021-08-04 Thread Scott Wood
On Sun, 2021-08-01 at 14:59 -0600, Simon Glass wrote: > This symbol is needed for binman to locate the start of the image. Add it. > > Note: the existing line to bring in the .__image_copy_start symbol does > not appear to do anything. > > Signed-off-by: Simon Glass > --- > I have copied Scott

Re: [PATCH] CI: Update to LLVM-12

2021-08-04 Thread Tom Rini
On Thu, Jul 15, 2021 at 02:24:25PM -0400, Tom Rini wrote: > The current stable release of LLVM is 12, update to that. While at it, > fix that we had not correctly upgraded to LLVM 11 previously. > > Signed-off-by: Tom Rini Updated to latest "focal" snapshot and applied to u-boot/master. --

Re: Proposal: U-Boot MTD (rewriting old story)

2021-08-04 Thread Tom Rini
On Wed, Aug 04, 2021 at 10:09:55AM -0600, Simon Glass wrote: > Hi Jagan, > > On Tue, 3 Aug 2021 at 09:01, Jagan Teki wrote: > > > > Yes, this is the old discussion and triggered again now since we had a > > discussion on the last U-Boot contributor call. > > > > I will brief the main points

[PATCH] board: ge: bx50v3: Add PCIe reset to DT

2021-08-04 Thread Sebastian Reichel
Add PCIe reset gpio to the Bx50v3 devicetree and get get rid of CONFIG_PCIE_IMX_PERST_GPIO. Signed-off-by: Sebastian Reichel --- arch/arm/dts/imx6q-bx50v3-uboot.dtsi | 10 ++ include/configs/ge_bx50v3.h | 1 - 2 files changed, 10 insertions(+), 1 deletion(-) diff --git

Re: Proposal: U-Boot MTD (rewriting old story)

2021-08-04 Thread Simon Glass
Hi Jagan, On Tue, 3 Aug 2021 at 09:01, Jagan Teki wrote: > > Yes, this is the old discussion and triggered again now since we had a > discussion on the last U-Boot contributor call. > > I will brief the main points here, as most of the details are > mentioned in previous threads. > > Here are

Re: [PATCH v2 2/3] Revert "mkeficapsule: Remove dtb related options"

2021-08-04 Thread Simon Glass
Hi Takahiro, On Mon, 2 Aug 2021 at 23:30, KASHI Takahiro wrote: > > Simon, > > On Mon, Aug 02, 2021 at 08:44:30AM -0600, Simon Glass wrote: > > This reverts commit f86caab058ff062ce72b24cd1ab9ec1253cc1352. > > Whether we choose a devicetree approach or not, we don't have > to revert this patch

Re: [PATCH] Revert "efi_capsule: Move signature from DTB to .rodata"

2021-08-04 Thread Simon Glass
Hi Takahiro, On Mon, 2 Aug 2021 at 18:21, KASHI Takahiro wrote: > > On Mon, Aug 02, 2021 at 08:44:41AM -0600, Simon Glass wrote: > > Hi, > > > > On Mon, 2 Aug 2021 at 01:15, KASHI Takahiro > > wrote: > > > > > > On Sun, Aug 01, 2021 at 08:47:15PM -0600, Simon Glass wrote: > > > > Hi Ilias, > >

Re: [PATCH] test/py: tpm2: Skip tpm pytest based on env variable

2021-08-04 Thread Tom Rini
On Fri, Jul 23, 2021 at 06:18:26AM -0600, Ashok Reddy Soma wrote: > From: T Karthik Reddy > > Tpm test cases relies on tpm device setup. Provide an environment > variable "env__tpm_device_test_skip = True" to skip the test case > if tpm device is not present. > Only needed will have to add

[PATCH] ARM: mach-at91: fix multiple cpu_reset definition when enabling SYSRESET

2021-08-04 Thread Clément Léger
When SYSRESET is enabled, cpu_reset function is also defined in sysreset-uclass.c which lead to multiple definitions of this function since reset.c is build unconditionally. Add a check in Makefile to build this file only if SYSRESET isn't enabled. SYSRESET can be enabled when building

Re: [PATCH 11/11] Remove including timestamp.h in version.h

2021-08-04 Thread Tom Rini
On Wed, Aug 04, 2021 at 08:36:27AM -0600, Simon Glass wrote: > Hi Tom, > > On Wed, 4 Aug 2021 at 06:44, Tom Rini wrote: > > > > On Wed, Aug 04, 2021 at 07:59:21AM +0200, Wolfgang Denk wrote: > > > > > Dear Tom, > > > > > > In message <20210802213100.GG9379@bill-the-cat> you wrote: > > > > > > >

[PATCH v3] mmc: Add support for enumerating MMC card in a given mode using mmc command

2021-08-04 Thread Aswath Govindraju
Add support for enumerating MMC card in a given mode using mmc rescan and mmc dev commands. The speed mode is provided as the last argument in these commands and is indicated using the index from enum bus_mode in include/mmc.h. A speed mode can be set only if it has already been enabled in the

Re: [PATCH 11/11] Remove including timestamp.h in version.h

2021-08-04 Thread Simon Glass
Hi Tom, On Wed, 4 Aug 2021 at 06:44, Tom Rini wrote: > > On Wed, Aug 04, 2021 at 07:59:21AM +0200, Wolfgang Denk wrote: > > > Dear Tom, > > > > In message <20210802213100.GG9379@bill-the-cat> you wrote: > > > > > > I think that if nothing changes the banner not changing is the right > > >

Re: [PATCH v3 07/10] watchdog: wdt-uclass.c: handle all DM watchdogs in watchdog_reset()

2021-08-04 Thread Simon Glass
Hi Rasmus, On Tue, 3 Aug 2021 at 00:48, Rasmus Villemoes wrote: > > On 02/08/2021 21.20, Simon Glass wrote: > > Hi Rasmus, > > > >> But let me one last time repeat why I think the post_probe approach is > >> the cleanest and a natural fit for DM: post_probe is (AIUI) a place > >> where a uclass

Re: [PATCH v2] mmc: Add support for enumerating MMC card in a given mode using mmc command

2021-08-04 Thread Aswath Govindraju
Hi Jaehoon, On 02/08/21 3:26 am, Jaehoon Chung wrote: > Hi Aswath, > > On 7/30/21 9:26 PM, Aswath Govindraju wrote: >> Add support for enumerating MMC card in a given mode using mmc rescan and >> mmc dev commands. The speed mode is provided as the last argument in these >> commands and is

[PATCH 3/3] include: configs: am64x_evm: Add env variables for booting to kernel using USB MSC device

2021-08-04 Thread Aswath Govindraju
Add env variables for booting to kernel from USB MSC device. The second partition in the USB MSC device needs to formatted as ext4 file system with kernel and dtb images, present in the /boot folder. Signed-off-by: Aswath Govindraju --- include/configs/am64x_evm.h | 31

[PATCH 2/3] environment: ti: mmc.h: Make the finduuid generic for usage across different device types

2021-08-04 Thread Aswath Govindraju
Make finduuid generic by making it dependent on the boot variable. For example, this can now be used for finding the uuid of partitions in usb device too. Signed-off-by: Aswath Govindraju --- include/environment/ti/mmc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 1/3] board: ti: am64x: Add support for fixing dr_mode while booting from USB

2021-08-04 Thread Aswath Govindraju
Fix the dr_mode in the U-Boot device tree blob, by reading the mode field from the USB Boot Configuration fields. The dr_mode will only be fixed when booting from USB. Signed-off-by: Aswath Govindraju --- board/ti/am64x/evm.c | 32 1 file changed, 32

[PATCH 0/3] AM64: Add support for booting to kernel from USB MSC device

2021-08-04 Thread Aswath Govindraju
The following series of patches add support for booting to kernel using USB MSC device as the boot media, in AM64 Aswath Govindraju (3): board: ti: am64x: Add support for fixing dr_mode while booting from USB environment: ti: mmc.h: Make the finduuid generic for usage across different

Re: [PATCH] board: amlogic: odroid-n2: fix fdtfile suffix for n2-plus

2021-08-04 Thread Neil Armstrong
On 04/08/2021 13:01, Christian Hewitt wrote: > The N2+ dtb is meson-g12b-odroid-n2-plus.dtb, not n2_plus, so > correct the suffix provided in the board file. Also align the > board ident string shown during boot to match. > > Fixes: 8bc780106c13 ("board: amlogic: odroid: add runtime detection of

Re: [PATCH 11/11] Remove including timestamp.h in version.h

2021-08-04 Thread Tom Rini
On Wed, Aug 04, 2021 at 07:59:21AM +0200, Wolfgang Denk wrote: > Dear Tom, > > In message <20210802213100.GG9379@bill-the-cat> you wrote: > > > > I think that if nothing changes the banner not changing is the right > > behavior. > > Hm... is it? How about "external" changes like building with

[PATCH] board: amlogic: odroid-n2: fix fdtfile suffix for n2-plus

2021-08-04 Thread Christian Hewitt
The N2+ dtb is meson-g12b-odroid-n2-plus.dtb, not n2_plus, so correct the suffix provided in the board file. Also align the board ident string shown during boot to match. Fixes: 8bc780106c13 ("board: amlogic: odroid: add runtime detection of the N2/N2+/C4/HC4 variants") Signed-off-by: Christian

Re: [PATCH 6/7] xyz-modem: Show information about finished transfer

2021-08-04 Thread Heinrich Schuchardt
On 03.08.21 16:28, Pali Rohár wrote: Show "## Start Addr" or "## Binary (...) download aborted" information like in Kermit "loadb" command. Signed-off-by: Pali Rohár --- cmd/load.c | 16 1 file changed, 16 insertions(+) diff --git a/cmd/load.c b/cmd/load.c index

Re: [PATCH 4/7] xyz-modem: Close stream after processing/sending terminate sequence

2021-08-04 Thread Heinrich Schuchardt
On 03.08.21 16:28, Pali Rohár wrote: Obviously it is not possible to send terminate sequence over stream after closing stream. xyzModem_stream_close() does not close anything; it flushes the stream. xyzModem_stream_terminate() sets xyz.at_eof = true and gives feedback to the user. So I

Re: [PATCH 1/7] xyz-modem: Fix crash after cancelling transfer

2021-08-04 Thread Heinrich Schuchardt
On 03.08.21 16:28, Pali Rohár wrote: Variable xyz.len is set to -1 on error. At the end xyzModem_stream_read() function calls memcpy() with length from variable xyz.len. If this variable is set to -1 then value passed to memcpy is casted to unsigned value, which means to copy whole address

Re: EFI from usb HDD

2021-08-04 Thread Ilias Apalodimas
Hi Michal Apologies for my late reply, I was on vacation! [...] > >> > >> When Abort happened I connected Xilinx debugger via jtag and look at cpu > >> backtrace. > > > > OK, but we are already in grub here and such a trace (in U-Boot) > > doesn't make sense. Right? > > Correct grub already

[PATCH 4/4] EFI: console: improve vidconsole unicode output

2021-08-04 Thread Artem Lapkin
If EFI_CONSOLE_UTF_SAFE is enabled and vidconsole is active, unicode characters will be replaced with "." for all console outputs. Vidconsole does not support unicode output, and your console will suffer display issues if EFI_CONSOLE_UTF_SAFE is disabled. Signed-off-by: Artem Lapkin ---

[PATCH 3/4] EFI: console: max rows and cols user limit

2021-08-04 Thread Artem Lapkin
Setup the max rows and columns limit for the EFI console output. Signed-off-by: Artem Lapkin --- lib/efi_loader/Kconfig | 12 lib/efi_loader/efi_console.c | 5 + 2 files changed, 17 insertions(+) diff --git a/lib/efi_loader/Kconfig b/lib/efi_loader/Kconfig index

[PATCH 2/4] EFI: console: query_console_size: multiplex adaptation

2021-08-04 Thread Artem Lapkin
Multiplexed adaptation of the query_console_size() function; automatically determine the minimal console area that will fit all outputs properly. Signed-off-by: Artem Lapkin --- lib/efi_loader/efi_console.c | 16 ++-- 1 file changed, 10 insertions(+), 6 deletions(-) diff --git

[PATCH 1/4] EFI: console: query_vidconsole: multiplex adaptation

2021-08-04 Thread Artem Lapkin
Fixed detection of vidconsole from within a multiplexed stdout string. As you know, a user can use a comma-separated list of devices to set stdin, stdout and stderr. For example, "setenv stdout serial,vidconsole" is a multiplexed string. Signed-off-by: Artem Lapkin ---

[PATCH 0/4] EFI: console: improves

2021-08-04 Thread Artem Lapkin
Problems with Linux Grub and U-Boot EFI console - multiplexed console output scrambled - slow refresh rate for big console sizes - vidconsole incompatible with unicode This patchset provides the following solutions: 1) Fixed detection of vidconsole from within a multiplexed stdout string. As you

Re: [PATCH 3/7] xyz-modem: Put xyzModem_stream_close debug diagnostic message into ZM_DEBUG()

2021-08-04 Thread Heinrich Schuchardt
On 03.08.21 16:28, Pali Rohár wrote: This is how all other debug / diagnostic messages are handled. This commit message is misleading. What you do is hide an output. Signed-off-by: Pali Rohár --- common/xyzModem.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH] engicam: Rename board dir, px30_core to px30

2021-08-04 Thread Jagan Teki
Engicam hardware design solutions are based on System On Modules(SoM) with various SoCs like i.MX6, i.MX8 and PX30 etc. The current directory structure uses the simplest way to understand these designs by listing each of these SoC in board/engicam/SoC and inside SoC directory has SoM file name as

[PATCH 4/4] engicam: px30: Add Engicam PX30.Core C.TOUCH 2.0 10.1" OF

2021-08-04 Thread Jagan Teki
PX30.Core is an EDIMM SOM based on Rockchip PX30 from Engicam. C.TOUCH 2.0 is a general purpose carrier board with capacitive touch interface support. 10.1" OF is a capacitive touch 10.1" Open Frame panel solutions. PX30.Core needs to mount on top of C.TOUCH 2.0 carrier with pluged 10.1" OF for

[PATCH 3/4] arm64: dts: rockchip: px30: Sync Linux-5.14-rc3 dts(i)

2021-08-04 Thread Jagan Teki
Sync the px30 devicetree files from Linux-5.14-rc3. Signed-off-by: Jagan Teki --- arch/arm/dts/px30-engicam-common.dtsi | 90 + arch/arm/dts/px30-engicam-ctouch2.dtsi| 22 +++ arch/arm/dts/px30-engicam-edimm2.2.dtsi | 59 ++

[PATCH 2/4] arm64: dts: rockchip: px30: Move dmc into -u-boot.dtsi

2021-08-04 Thread Jagan Teki
dmc node is specific to U-Boot, it is always better practice to maintain U-Boot specific nodes into -u-boot.dtsi files in order to maintain Linux dts file sync compatibility. Move the dmc into px30-u-boot.dtsi, also add dmc node explicitly in rk3326-odroid-go2-u-boot.dtsi since it is using

[PATCH 1/4] arm64: dts: rockchip: px30: Sync Linux PX30.Core files

2021-08-04 Thread Jagan Teki
Engicam PX30.Core SoM and it's associated Carrier board device tree files are different than the one it present in U-Boot. Let's Sync the same from Linux-5.14-rc3 with removal of unneeded nodes to satisfy the build. Signed-off-by: Jagan Teki --- arch/arm/dts/Makefile |

[PATCH 0/4] arm64: dts: rockchip: px30: Linux-5.14-rc3 Sync

2021-08-04 Thread Jagan Teki
Series sync Linux-5.14-rc3 devicetree files for px30 SoC. Any inputs? Jagan. Jagan Teki (4): arm64: dts: rockchip: px30: Sync Linux PX30.Core files arm64: dts: rockchip: px30: Move dmc into -u-boot.dtsi arm64: dts: rockchip: px30: Sync Linux-5.14-rc3 dts(i) engicam: px30: Add Engicam