Re: [U-Boot] [PATCH] efi_loader: Fix disk dp's for pre-DM/legacy devices

2017-11-17 Thread Jonathan Gray
On Tue, Oct 10, 2017 at 04:48:01AM +1100, Jonathan Gray wrote: > On Mon, Oct 09, 2017 at 01:06:26PM -0400, Rob Clark wrote: > > On Mon, Oct 9, 2017 at 12:41 PM, Jonathan Gray wrote: > > > On Mon, Oct 09, 2017 at 12:06:24PM -0400, Rob Clark wrote: > > >> On Mon, Oct 9, 2017 at

Re: [U-Boot] Kconfig: Introduce USE_BOOTCOMMAND and migrate BOOTCOMMAND

2017-11-17 Thread Tom Rini
On Mon, Nov 06, 2017 at 06:15:11PM -0500, Tom Rini wrote: > We first introduce CONFIG_USE_BOOTCOMMAND, similar to > CONFIG_USE_BOOTARGS. We then migrate CONFIG_BOOTCOMMAND for most > CONFIG_DISTRO_DEFAULT users. In some cases platforms have a complex > scheme around this usage, and these have

Re: [U-Boot] [PATCH v2 18/18] efi_loader: helper function to add EFI object to list

2017-11-17 Thread Heinrich Schuchardt
On 11/17/2017 03:07 PM, Simon Glass wrote: On 12 November 2017 at 07:02, Heinrich Schuchardt wrote: To avoid duplicate coding provide a helper function that initializes an EFI object and adds it to the EFI object list. efi_exit() is the only place where we dereference a

Re: [U-Boot] [PATCH v2 07/18] efi_loader: efi_disk: use efi_add_protocol

2017-11-17 Thread Heinrich Schuchardt
On 11/17/2017 03:06 PM, Simon Glass wrote: On 12 November 2017 at 07:02, Heinrich Schuchardt wrote: Use efi_add_protocol to install protocols. Signed-off-by: Heinrich Schuchardt --- v2 no change --- lib/efi_loader/efi_disk.c | 39

Re: [U-Boot] mtd: cfi: Add support for status register polling

2017-11-17 Thread York Sun
On 11/17/2017 12:59 PM, York Sun wrote: > On 11/17/2017 11:04 AM, Marek Vasut wrote: >> On 11/17/2017 08:02 PM, Marek Vasut wrote: >>> On 11/17/2017 05:43 PM, York Sun wrote: On 09/12/2017 10:09 AM, Marek Vasut wrote: > The status register is optional in the AMD command sets, but it's

Re: [U-Boot] [OE-core] [PATCH] u-boot: Upgrade to 2017.11 release

2017-11-17 Thread Tom Rini
On Fri, Nov 17, 2017 at 08:19:07PM +0100, Marek Vasut wrote: > On 11/14/2017 07:32 PM, Otavio Salvador wrote: > > This upgrades the U-Boot from 2017.09 to 2017.11 release. > > > > Signed-off-by: Otavio Salvador > > --- > > > > .../u-boot/{u-boot-common_2017.09.inc =>

Re: [U-Boot] mtd: cfi: Add support for status register polling

2017-11-17 Thread York Sun
On 11/17/2017 11:04 AM, Marek Vasut wrote: > On 11/17/2017 08:02 PM, Marek Vasut wrote: >> On 11/17/2017 05:43 PM, York Sun wrote: >>> On 09/12/2017 10:09 AM, Marek Vasut wrote: The status register is optional in the AMD command sets, but it's presence can be checked by reading out CFI

Re: [U-Boot] [PATCH v2 01/18] efi_loader: efi_bootmgr: do not make hidden assignments

2017-11-17 Thread Rob Clark
On Fri, Nov 17, 2017 at 12:46 PM, Heinrich Schuchardt wrote: > On 11/17/2017 03:04 PM, Rob Clark wrote: >> >> On Sun, Nov 12, 2017 at 9:02 AM, Heinrich Schuchardt >> wrote: >>> >>> Assignments should not be made in the middle of nowhere. >>> >>>

Re: [U-Boot] [PATCH] usb: r8a66597: Add support for RZ/A series

2017-11-17 Thread Chris Brandt
Thanks for the review. On Friday, November 17, 2017, Marek Vasut wrote: > > +#ifndef RZA_USB > > This should be turned into Kconfig entry, some CONFIG_... So you want RZA_USB in Kconfig, but is it OK if it is automatically selected when an RZ/A device is selected? For example: config

Re: [U-Boot] mtd: cfi: Add support for status register polling

2017-11-17 Thread York Sun
On Nov 17, 2017, at 11:04, Marek Vasut > wrote: On 11/17/2017 08:02 PM, Marek Vasut wrote: On 11/17/2017 05:43 PM, York Sun wrote: On 09/12/2017 10:09 AM, Marek Vasut wrote: The status register is optional in the AMD command sets, but it's

Re: [U-Boot] [PATCH] u-boot: Upgrade to 2017.11 release

2017-11-17 Thread Marek Vasut
On 11/14/2017 07:32 PM, Otavio Salvador wrote: > This upgrades the U-Boot from 2017.09 to 2017.11 release. > > Signed-off-by: Otavio Salvador > --- > > .../u-boot/{u-boot-common_2017.09.inc => u-boot-common_2017.11.inc} | 2 > +- >

Re: [U-Boot] [PATCH] usb: r8a66597: Add support for RZ/A series

2017-11-17 Thread Marek Vasut
On 11/13/2017 10:21 PM, Chris Brandt wrote: > While the USB HW in the RZ/A is basically the same, there are some > differences from the original versions that were in the SH4 SoCs. > > Signed-off-by: Chris Brandt > --- > drivers/usb/host/r8a66597-hcd.c | 46 >

Re: [U-Boot] arm: config: fix default console only to specify the device

2017-11-17 Thread Tom Rini
On Sat, Oct 28, 2017 at 12:22:27AM -0400, tobet...@gmail.com wrote: > From: Dongjin Kim > > Since CONFIG_DEFAULT_CONSOLE is already started with "console=", > the console argument in CONFIG_EXTRA_ENV_SETTINGS is expanded to > "console=console=ttySAC1,115200n8" and this

Re: [U-Boot] [U-Boot, PATCHv2] dm: pci: change bus number register setting compliant with Linux

2017-11-17 Thread Tom Rini
On Fri, Oct 20, 2017 at 10:45:50AM +0800, Zhiqiang Hou wrote: > From: Minghuan Lian > > This patch is to change U-Boot PCI bus assignement compliant with Linux. > It means each PCIe controller's bus number is 0, not the current maximum > PCI bus number, when start to scan

Re: [U-Boot] nios2: 10m50: Add CPU pre-relocation in device tree

2017-11-17 Thread Tom Rini
On Thu, Jul 27, 2017 at 10:07:17PM -0700, Gan, Yau Wai wrote: > Tag CPU with dm-pre-reloc to enable driver before > relocation. > > Signed-off-by: Gan, Yau Wai > Cc: Thomas Chou > Reviewed-by: Marek Vasut Applied to u-boot/master,

Re: [U-Boot] [PATCH] usb: ehci: Fix accessors for big-endian platforms and descriptors

2017-11-17 Thread Marek Vasut
On 11/17/2017 02:26 PM, Alexey Brodkin wrote: > Commit 9000eddbae0d ("drivers/usb/ehci: Use platform-specific accessors") > broke USB 2.0 on big-endian platforms because for them writel/readl() > does automatic conversion of BE data to LE. > > Proper implementation requires to use "raw" variant

Re: [U-Boot] mtd: cfi: Add support for status register polling

2017-11-17 Thread Marek Vasut
On 11/17/2017 08:02 PM, Marek Vasut wrote: > On 11/17/2017 05:43 PM, York Sun wrote: >> On 09/12/2017 10:09 AM, Marek Vasut wrote: >>> The status register is optional in the AMD command sets, but it's >>> presence can be checked by reading out CFI table entry 0xc bit 0. >>> If the register is

Re: [U-Boot] mtd: cfi: Add support for status register polling

2017-11-17 Thread Marek Vasut
On 11/17/2017 05:43 PM, York Sun wrote: > On 09/12/2017 10:09 AM, Marek Vasut wrote: >> The status register is optional in the AMD command sets, but it's >> presence can be checked by reading out CFI table entry 0xc bit 0. >> If the register is present, prefer using it's bit 7 to determine >> if

Re: [U-Boot] [PATCH v2 06/18] efi_selftest: test EFI_DEVICE_PATH_TO_TEXT_PROTOCOL

2017-11-17 Thread Heinrich Schuchardt
On 11/17/2017 03:06 PM, Simon Glass wrote: On 12 November 2017 at 07:02, Heinrich Schuchardt wrote: Provide a test for the EFI_DEVICE_PATH_TO_TEXT_PROTOCOL protocol. Signed-off-by: Heinrich Schuchardt --- v2 no change ---

Re: [U-Boot] [PATCH v2 05/18] efi_loader: reimplement LocateDevicePath

2017-11-17 Thread Heinrich Schuchardt
On 11/17/2017 03:06 PM, Simon Glass wrote: Hi Heinrich, On 12 November 2017 at 07:02, Heinrich Schuchardt wrote: The current implementation of efi_locate_device_path does not match the UEFI specification. It completely ignores the protocol parameters. Signed-off-by:

Re: [U-Boot] [PATCH v2 01/18] efi_loader: efi_bootmgr: do not make hidden assignments

2017-11-17 Thread Heinrich Schuchardt
On 11/17/2017 03:04 PM, Rob Clark wrote: On Sun, Nov 12, 2017 at 9:02 AM, Heinrich Schuchardt wrote: Assignments should not be made in the middle of nowhere. Signed-off-by: Heinrich Schuchardt --- v2 Call efi_dp_str in debug mode only. ---

Re: [U-Boot] [PATCH] efi_loader: Exit in efi_set_bootdev() upon invalid "desc"

2017-11-17 Thread Heinrich Schuchardt
On 11/17/2017 08:47 AM, Stefan Roese wrote: When trying to load an image from a non-existent USB key, U-Boot v2017.11 crashes on my x86 platform: => load usb 0:1 0300 abc General Protection EIP: 0010:[<7b59030d>] EFLAGS: 00010286 Original EIP :[] ... This used to work in v2017.09. Testing

Re: [U-Boot] [PATCH] cmd: mac: Update description of help

2017-11-17 Thread York Sun
On 11/09/2017 02:39 AM, Prabhakar Kushwaha wrote: > mac help does not reflect correct descriptions and parameter. > > So update mac help command. > > Signed-off-by: Prabhakar Kushwaha > --- Minor change to commit message. Applied to fsl-qoriq master. Thanks. York

Re: [U-Boot] [PATCH] driver: net: fsl-mc: Use base 16 in simple_strtoul

2017-11-17 Thread York Sun
On 11/09/2017 01:26 AM, Prabhakar Kushwaha wrote: > Value provided in MC_MEM_SIZE_ENV_VAR is in hex. > > So provide 16 as base in simple_strtoul. > > Signed-off-by: Prabhakar Kushwaha > --- Minor change in commit message. Applied to fsl-qoriq master. Thanks. York

Re: [U-Boot] [PATCH 1/4] ls1043ardb: Add sd_bootcmd for distro fallback in case of sdboot

2017-11-17 Thread York Sun
On 11/09/2017 02:16 AM, Shengzhou Liu wrote: > Signed-off-by: Shengzhou Liu > --- This set is applied to fsl-qoriq master. Thanks. York ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v6 2/2] armv8: ls1088aqds: Add SD boot support for ls1088qds

2017-11-17 Thread York Sun
On 11/05/2017 11:48 PM, Ashish Kumar wrote: > Signed-off-by: Ashish Kumar > Signed-off-by: Prabhakar Kushwaha > Signed-off-by: Raghav Dogra > --- > v6: > Sorting defconfig with help make savedefconfig > Applied to

Re: [U-Boot] [PATCH v2] armv8: ls1088ardb: Add distro boot support

2017-11-17 Thread York Sun
On 11/05/2017 11:49 PM, Ashish Kumar wrote: > Distro boot support gives flexibility to run distro RFS like Ubuntu's > being deployed from SD card or SATA drive. If it fails > to detect external storage, fall back to qspi/sd boot. > > Enable this by default in RDB's defconfig by selecting >

Re: [U-Boot] [PATCH v6 1/2] armv8: ls1088ardb: Add SD boot support for ls1088

2017-11-17 Thread York Sun
On 11/05/2017 11:48 PM, Ashish Kumar wrote: > Signed-off-by: Prabhakar Kushwaha > Signed-off-by: Ashish Kumar > Signed-off-by: Raghav Dogra > --- > v6: > Incorporation of review comments in README wrt to line wrap. >

Re: [U-Boot] [PATCH v4 2/2] armv8: ls1088: Enable SATA for ls1088

2017-11-17 Thread York Sun
On 11/01/2017 09:20 PM, Ashish Kumar wrote: > Signed-off-by: Ashish Kumar > Signed-off-by: Amrita Kumari > --- > v2: Rebase to top > v3: Consolidate defines in common file > v4: Protect define using CONFIG_SCSI Squashed with patch #1. Applied to

Re: [U-Boot] [PATCH v2] mtd: nand: fsl-ifc: fix support of multiple NAND devices

2017-11-17 Thread York Sun
On 10/30/2017 02:58 PM, Scott Wood wrote: > On Mon, 2017-10-30 at 18:50 +, York Sun wrote: >> On 10/20/2017 03:43 AM, Kurt Kanzenbach wrote: >>> Currently the chipselect used to identify the corresponding NAND chip is >>> stored >>> at the controller and only set during fsl_ifc_chip_init().

Re: [U-Boot] [PATCH] armv8: ls1088a: Move CONFIG_SPI_FLASH_SPANSION to defconfig

2017-11-17 Thread York Sun
On 10/29/2017 11:24 PM, Ashish Kumar wrote: > Signed-off-by: Ashish Kumar > --- Applied to fsl-qoriq master. Thanks. York ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] drivers: net: ldpaa_eth: Correct error handler for qbman_swp_acquire()

2017-11-17 Thread York Sun
On 10/26/2017 03:33 AM, Ashish Kumar wrote: > Correcting error handing for qbman_swp_acquire. The return value is zero is > an error condition since number of buffer copied is zero meaning > there are no free buffers for allocation. > > Signed-off-by: Ashish Kumar >

Re: [U-Boot] [RFC PATCH 06/10] env: Support multiple environments

2017-11-17 Thread Maxime Ripard
On Fri, Nov 17, 2017 at 03:40:51PM +0100, Lukasz Majewski wrote: > Hi Maxime, > > > Hi Lukasz, Tom, > > > > On Fri, Nov 17, 2017 at 03:00:15PM +0100, Lukasz Majewski wrote: > > > On Fri, 17 Nov 2017 08:41:05 -0500 > > > Tom Rini wrote: > > > > > > > On Fri, Nov 17, 2017

Re: [U-Boot] mtd: cfi: Add support for status register polling

2017-11-17 Thread York Sun
On 09/12/2017 10:09 AM, Marek Vasut wrote: > The status register is optional in the AMD command sets, but it's > presence can be checked by reading out CFI table entry 0xc bit 0. > If the register is present, prefer using it's bit 7 to determine > if the flash is busy over reading the flash ; this

Re: [U-Boot] mmc1 not working on Samsung snow chromebook

2017-11-17 Thread Jaehoon Chung
On 2017년 11월 17일 18:38, Guillaume Gardet wrote: > > > Le 17/11/2017 à 10:36, Lukasz Majewski a écrit : >> On Thu, 16 Nov 2017 13:29:56 +0100 >> Guillaume Gardet wrote: >> >>> I found a workaround. If I disable MMC_MODE_HS_52MHz, then it is >>> working fine. >>> >>> I

[U-Boot] Working with patchwork

2017-11-17 Thread Goldschmidt Simon
Hi, as I'm quite new to U-Boot and its patch workflow: what do I have to do to send a reviewed-by or tested-by tag for a patch? I sent it as a reply to this list but I can't see them in patchwork. Did I miss something here? Or could it be a problem in the email format used? Thanks, Simon

Re: [U-Boot] armv8: update gd after relocate

2017-11-17 Thread Tom Rini
On Fri, Nov 03, 2017 at 10:10:27AM +0800, Kever Yang wrote: > We need to update gd in assamble code after relocate, > this is a fix to: > adc421e arm: move gd handling outside of C code > > Signed-off-by: Kever Yang > Reviewed-by: Philipp Tomsich

Re: [U-Boot] [U-Boot, v2, 4/5] mmc: arm_pl180_mmci: add clock support

2017-11-17 Thread Tom Rini
On Mon, Oct 23, 2017 at 10:57:33AM +0200, patrice.chot...@st.com wrote: > From: Patrice Chotard > > Allow to get and enable MMC related clock > > Signed-off-by: Patrice Chotard Applied to u-boot/master, thanks! -- Tom signature.asc

Re: [U-Boot] [U-Boot, u-boot, 2/3] arm: meson: Add supplementary ethernet registers definitions

2017-11-17 Thread Tom Rini
On Wed, Oct 18, 2017 at 10:02:11AM +0200, Neil Armstrong wrote: > On Amlogic Meson GXL/GXM, supplementary ethernet configuration registers > were added to configure the internal RMII PHY interface. > > Signed-off-by: Neil Armstrong Applied to u-boot/master, thanks! --

Re: [U-Boot] [U-Boot, 3/5] tools: use files from scripts/dtc/libfdt where possible

2017-11-17 Thread Tom Rini
On Tue, Oct 17, 2017 at 10:30:18PM +0900, Masahiro Yamada wrote: > Prior to this commit, tools/Makefile pulls all libfdt files from > lib/libfdt. > > lib/libfdt/ and scripts/dtc/libfdt have the same copies for the > followings 6 files: > fdt.c fdt_addresses.c fdt_empty_tree.c fdt_overlay.c

Re: [U-Boot] [U-Boot, v2, 5/5] mmc: arm_pl180_mmci: add .getcd callback

2017-11-17 Thread Tom Rini
On Mon, Oct 23, 2017 at 10:57:34AM +0200, patrice.chot...@st.com wrote: > From: Patrice Chotard > > Add .getcd callback to check is MMC card is present > > Signed-off-by: Patrice Chotard Applied to u-boot/master, thanks! -- Tom

Re: [U-Boot] [U-Boot, 5/5] lib: libfdt: wrap scripts/dtc/libfdt/* where possible

2017-11-17 Thread Tom Rini
On Tue, Oct 17, 2017 at 10:30:20PM +0900, Masahiro Yamada wrote: > lib/libfdt/ and scripts/dtc/libfdt have the same copies for the > followings 6 files: > fdt.c fdt_addresses.c fdt_empty_tree.c fdt_overlay.c fdt_strerr.c > fdt_sw.c > > Make them a wrapper of scripts/dtc/libfdt/*. This is

Re: [U-Boot] clk: clk_stm32f7: fix PLL clock division factor

2017-11-17 Thread Tom Rini
On Thu, Oct 26, 2017 at 01:23:19PM +0200, patrice.chot...@st.com wrote: > From: Patrice Chotard > > Fix clock division factor initialization for RCC_PLLCFGR > registers. > > PLLR bits (bit 31-28) in RCC_PLLCFGR must not be cleared, > it's a forbidden value. So update

Re: [U-Boot] [U-Boot, 2/2] configs: dragonboard410c: Save environment data on eMMC

2017-11-17 Thread Tom Rini
On Mon, Nov 06, 2017 at 02:16:38PM +0100, Jorge Ramirez-Ortiz wrote: > Save the environment data at the end of the boot partition on emmc > > Signed-off-by: Jorge Ramirez-Ortiz Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [U-Boot] [U-Boot, v2, 3/4] pylibfdt: move pylibfdt to scripts/dtc/pylibfdt and refactor makefile

2017-11-17 Thread Tom Rini
On Tue, Oct 17, 2017 at 01:42:43PM +0900, Masahiro Yamada wrote: > The pylibfdt is used by dtoc (and, indirectly by binman), but there > is no reason why it must be generated in the tools/ directory. > > Recently, U-Boot switched over to the bundled DTC, and the directory > structure under

Re: [U-Boot] board: ti: dra71x-evm: Hook LDO1 of LP8733 to EN_PIN

2017-11-17 Thread Tom Rini
On Thu, Oct 12, 2017 at 10:18:45AM +0530, Keerthy wrote: > All regulators are hooked to EN_Pin at reset so that EN Pin controls > their state. Hook the LDO1 regulator to EN pin which at reset is not > hooked. This applies only to LP8733. > > Signed-off-by: Keerthy >

Re: [U-Boot] test/py: add timestamps to log

2017-11-17 Thread Tom Rini
On Fri, Oct 27, 2017 at 11:04:08AM -0600, Stephen Warren wrote: > From: Stephen Warren > > It can be useful to record how long tests take; this can help debug slow > running test systems or track changes in performance over time. Enhance > the test system to record

Re: [U-Boot] omap2: nand: Make NAND_OMAP_GPMC_PREFETCH default

2017-11-17 Thread Tom Rini
On Fri, Oct 20, 2017 at 04:55:51PM -0400, Tom Rini wrote: > This option provides better performance and should really always be > enabled. Make this be default y. > > Signed-off-by: Tom Rini > Acked-by: Jeroen Hofstee Applied to u-boot/master,

Re: [U-Boot] [U-Boot, 2/5] libfdt: change libfdt_internal.h to a wrapper of scripts/dtc/libfdt/*

2017-11-17 Thread Tom Rini
On Tue, Oct 17, 2017 at 10:30:17PM +0900, Masahiro Yamada wrote: > Fortunately, U-Boot did not modify libfdt_internal.h locally. > > Change it to a wrapper of scripts/dtc/libfdt/fdt.h, which will be > periodically synced with the upstream DTC (or kernel). > > Signed-off-by: Masahiro Yamada

Re: [U-Boot] [U-Boot, u-boot, 1/3] ARM: dts: Synchronize Amlogic from Linux Mainline 4.13.5

2017-11-17 Thread Tom Rini
On Thu, Oct 12, 2017 at 03:50:30PM +0200, Neil Armstrong wrote: > Synchronize the Amlogic ARM64 dts from mainline Linux 4.13.5 > > In the preparation of the support of the Amlogic P212 board, > import the corresponding meson-gxl-s905x-p212.dts file. > > Signed-off-by: Neil Armstrong

Re: [U-Boot] [U-Boot, v2, 2/5] mmc: arm_pl180_mmci: adapt driver to DM usage

2017-11-17 Thread Tom Rini
On Mon, Oct 23, 2017 at 10:57:31AM +0200, patrice.chot...@st.com wrote: > From: Patrice Chotard > > Convert this driver to driver model. > This driver is also used by VEXPRESS platforms which doesn't > use driver model. > > Tested on STM32F746 and STM32F769 platforms. >

Re: [U-Boot] [U-Boot, v2, 1/4] scripts/dtc: add fdt_overlay.c and fdt_addresses.c to sync script

2017-11-17 Thread Tom Rini
On Tue, Oct 17, 2017 at 01:42:41PM +0900, Masahiro Yamada wrote: > From: Rob Herring > > libfdt has gained some new files. We need to include them in the > kernel's copy. > > Reported-by: Kyle Yan > Signed-off-by: Rob Herring >

Re: [U-Boot] [U-Boot, u-boot, 1/3] net: phy: Add Amlogic Meson GXL Internal PHY support

2017-11-17 Thread Tom Rini
On Wed, Oct 18, 2017 at 10:02:10AM +0200, Neil Armstrong wrote: > The Amlogic Meson GXL/GXM families embeds an internal RMII Ethernet PHY. > > The PHY acts as a generic PHY but needs a slight configuration right > before it's configuration. > > Signed-off-by: Neil Armstrong

Re: [U-Boot] [U-Boot,u-boot,2/3] pinctrl: meson: Add GXL Support

2017-11-17 Thread Tom Rini
On Thu, Oct 12, 2017 at 03:50:31PM +0200, Neil Armstrong wrote: > Add the Amlogic Meson GXL pinctrl support based on the GXBB driver and > the synchronized DTS from Linux 4.13.5 > > Signed-off-by: Neil Armstrong > Reviewed-by: Beniamino Galvani

Re: [U-Boot] [PATCH v2 3/5] tpm: add tpm_get_random()

2017-11-17 Thread Simon Glass
Hi. On 17 November 2017 at 08:41, wrote: > > On 3 October 2017 at 09:55, André Draszik wrote: > > Please add a commit message. Also where is this used? Is it dead code? > > Acked-by: Simon Glass > I applied this with changes: - added a

Re: [U-Boot] serial: nulldev: Implement "pending" function to fix tstc return "true"

2017-11-17 Thread Tom Rini
On Thu, Nov 02, 2017 at 11:39:51PM -0700, Wilson Lee wrote: > In U-boot, serial_tstc was use to determine is there have a character in > serial console that pending for read. If there is no "pending" function > implemented in serial driver, the serial-uclass will return "true(1)" > to indicate

Re: [U-Boot] [U-Boot, v2, 4/4] pylibfdt: compile pylibfdt only when dtoc/binman is necessary

2017-11-17 Thread Tom Rini
On Tue, Oct 17, 2017 at 01:42:44PM +0900, Masahiro Yamada wrote: > Currently, pylibfdt is always compiled if swig is installed on your > machine. It is really annoying because most of targets (excepts > x86, sunxi, rockchip) do not use dtoc or binman. > > "checkbinman" and "checkdtoc" are

Re: [U-Boot] [U-Boot, 1/5] libfdt: change fdt.h to a wrapper of scripts/dtc/libfdt/*

2017-11-17 Thread Tom Rini
On Tue, Oct 17, 2017 at 10:30:16PM +0900, Masahiro Yamada wrote: > Fortunately, U-Boot did not modify fdt.h locally. > > Change it to a wrapper of scripts/dtc/libfdt/fdt.h, which will be > periodically synced with the upstream DTC (or kernel). > > Signed-off-by: Masahiro Yamada

Re: [U-Boot] [U-Boot, 1/2] env: Save environment at the end of an MMC partition

2017-11-17 Thread Tom Rini
On Mon, Nov 06, 2017 at 02:16:37PM +0100, Jorge Ramirez-Ortiz wrote: > Allow the platform to define a partition by name at the end of which > the environment data will be located. > > Signed-off-by: Jorge Ramirez-Ortiz Applied to u-boot/master, thanks! -- Tom

Re: [U-Boot] [U-Boot, v2, 2/4] scripts/dtc: Update to upstream version v1.4.5-3-gb1a60033c110

2017-11-17 Thread Tom Rini
On Tue, Oct 17, 2017 at 01:42:42PM +0900, Masahiro Yamada wrote: > This adds the following commits from upstream: > > b1a6003 tests: Add a test for overlays syntactic sugar > 737b2df overlay: Add syntactic sugar version of overlays > 497432f checks: Use proper format modifier for size_t >

Re: [U-Boot] [U-Boot, u-boot, 3/3] arm: add initial support for Amlogic P212 based on Meson GXL family

2017-11-17 Thread Tom Rini
On Thu, Oct 12, 2017 at 03:50:32PM +0200, Neil Armstrong wrote: > This adds platform code for the Amlogic P212 reference board based on a > Meson GXL (S905X) SoC with the Meson GXL configuration. > > This initial submission only supports UART and MMC/SDCard, support for the > internal Ethernet

Re: [U-Boot] [PATCH] sandbox: Add clrbits/setbits macros

2017-11-17 Thread sjg
We borrow the macros for these functions from ARM and remove references to '__raw_'. Signed-off-by: Tom Rini --- arch/sandbox/include/asm/io.h | 57 +++ 1 file changed, 57 insertions(+) Applied to u-boot-dm thanks!

Re: [U-Boot] [U-Boot, v2, 3/5] mmc: arm_pl180_mmci: add bus_width DT property support

2017-11-17 Thread Tom Rini
On Mon, Oct 23, 2017 at 10:57:32AM +0200, patrice.chot...@st.com wrote: > From: Patrice Chotard > > Allow to get "bus-width" property from device tree > > Signed-off-by: Patrice Chotard Applied to u-boot/master, thanks! -- Tom signature.asc

Re: [U-Boot] [U-Boot, v2, 1/5] mmc: arm_pl180_mmci: update arm_pl180_mmci_init() prototype

2017-11-17 Thread Tom Rini
On Mon, Oct 23, 2017 at 10:57:30AM +0200, patrice.chot...@st.com wrote: > From: Patrice Chotard > > Update arm_pl180_mmci_init() prototype by adding struct mmc** > param. This is needed before converting this driver to driver model > in order to use arm_pl180_mmci_init()

Re: [U-Boot] [U-Boot, u-boot, 3/3] arm: amlogic: p212: Add support for Ethernet with Internal PHY

2017-11-17 Thread Tom Rini
On Wed, Oct 18, 2017 at 10:02:12AM +0200, Neil Armstrong wrote: > This patch adds support for the Internal RMII Ethernet PHY on the > Amlogic P212 Reference Board. > > Signed-off-by: Neil Armstrong Applied to u-boot/master, thanks! -- Tom signature.asc Description:

Re: [U-Boot] [PATCH v2 2/5] tpm: fix reading of permanent flags

2017-11-17 Thread sjg
On 3 October 2017 at 09:55, André Draszik wrote: > From: André Draszik > > The offset of the permanent flags structure is in a different > place in the response compared to what the code is doing, > which gives us a completely useless result. > > Fix by

Re: [U-Boot] [PATCH v2 1/5] SPL: fix printing of image name

2017-11-17 Thread sjg
Hi Andre, On 1 November 2017 at 03:09, André Draszik wrote: > > Hi, > > > On Tue, 2017-10-03 at 16:55 +0100, André Draszik wrote: > > From: André Draszik > > > > The maximum length of the name of the image is > > obviously not sizeof(), which is just the >

Re: [U-Boot] omap3: omap3_logic: Move pinmuxing to header file

2017-11-17 Thread Tom Rini
On Mon, Nov 06, 2017 at 07:57:30PM -0600, Adam Ford wrote: > To keep the board file smaller and clean, let's move the pinmux to the header > file. > > Signed-off-by: Adam Ford > > diff --git a/board/logicpd/omap3som/omap3logic.c > b/board/logicpd/omap3som/omap3logic.c >

Re: [U-Boot] [U-Boot,4/5] linux/types.h: add typedef of uintptr_t

2017-11-17 Thread Tom Rini
On Tue, Oct 17, 2017 at 10:30:19PM +0900, Masahiro Yamada wrote: > Add this typedef in the same place as in Linux. This is necessary > to refactor libfdt inclusion. > > U-Boot also defines it in include/compiler.h. Of course it should > not do that, but I do not want to open a can of worms. >

Re: [U-Boot] am33xx: Add a function to query MPU voltage in uV

2017-11-17 Thread Tom Rini
On Wed, Oct 11, 2017 at 06:42:23PM +0200, Felix Brack wrote: > For the DM TPS65910 driver I'm working on, querying the MPU voltage > should return a value in uV. This value can then be used by the > regulator's standard function set_value to set the MPU voltage. > > Signed-off-by: Felix Brack

Re: [U-Boot] arm64: support running at addr other than linked to

2017-11-17 Thread Tom Rini
On Thu, Nov 02, 2017 at 06:11:27PM -0600, Stephen Warren wrote: > From: Stephen Warren > > This is required in the case where U-Boot is typically loaded and run at > a particular address, but for some reason the RAM at that location is not > available, e.g. due to memory

Re: [U-Boot] [PATCH] cmd/fdt.c align data buffer to avoid unaligned word access

2017-11-17 Thread sjg
On 28 September 2017 at 03:29, Bernhard Messerklinger wrote: > Since the compiler is free to place a char array to any address in > memory (in this case the stack), also to a non word aligned address the > function "fdt_prop_parse" runs into troubles upon

Re: [U-Boot] [PATCH] dm: core: Correct address cast in dev_read_addr_ptr()

2017-11-17 Thread sjg
On Sun, Oct 01, 2017 at 12:31:21AM +0200, Dr. Philipp Tomsich wrote: > > > On 30 Sep 2017, at 22:32, Tom Rini wrote: > > > > On Sat, Sep 30, 2017 at 06:08:02PM +0200, Dr. Philipp Tomsich wrote: > >> Simon, > >> > >>> On 28 Sep 2017, at 18:14, Dr. Philipp Tomsich > >>>

Re: [U-Boot] [PATCH v2 5/5] tpm: add more missing va_end()

2017-11-17 Thread sjg
On 3 October 2017 at 09:55, André Draszik wrote: > From: André Draszik > > While commit 36d35345b1f6 ("tpm: add missing va_end") added > some missing calls to va_end(), it missed a few places. > > Signed-off-by: André Draszik > --- >

Re: [U-Boot] [PATCH v2 3/5] tpm: add tpm_get_random()

2017-11-17 Thread sjg
On 3 October 2017 at 09:55, André Draszik wrote: Please add a commit message. Also where is this used? Is it dead code? Acked-by: Simon Glass > From: André Draszik > > Signed-off-by: André Draszik > --- >

Re: [U-Boot] [PATCH] dm: core: fix member name in ofnode_union documentation

2017-11-17 Thread sjg
On 9 November 2017 at 04:44, Baruch Siach wrote: > Fixes: 4984de2baaa ("dm: core: Add ofnode to represent device tree nodes") > Cc: Simon Glass > Signed-off-by: Baruch Siach > --- > include/dm/ofnode.h | 2 +- > 1 file changed, 1

Re: [U-Boot] [PATCH 4/5] tpm: add more useful NV storage permission flags

2017-11-17 Thread sjg
On 3 October 2017 at 09:52, André Draszik wrote: > From: André Draszik > > TPM_NV_PER_PPREAD: physical presence needed for reading > TPM_NV_PER_WRITEDEFINE: persistent write lock by writing size 0 > TPM_NV_PER_WRITEALL: write in one go > > Signed-off-by:

Re: [U-Boot] [RFC PATCH] usb: ehci: do not invalidate a NULL buffer

2017-11-17 Thread Marek Vasut
On 11/17/2017 03:28 PM, Dirk Behme wrote: > Its a valid use case to call ehci_submit_async() with a NULL buffer > with length 0. E.g. from usb_set_configuration(). > > As invalidate_dcache_range() isn't able to judge if the address > NULL is valid or not (depending on the SoC hardware

Re: [U-Boot] [RFC PATCH 06/10] env: Support multiple environments

2017-11-17 Thread Lukasz Majewski
Hi Maxime, > Hi Lukasz, Tom, > > On Fri, Nov 17, 2017 at 03:00:15PM +0100, Lukasz Majewski wrote: > > On Fri, 17 Nov 2017 08:41:05 -0500 > > Tom Rini wrote: > > > > > On Fri, Nov 17, 2017 at 10:24:37AM +0100, Lukasz Majewski wrote: > > > > Hi Maxime, > > > > > > >

[U-Boot] [RFC PATCH] usb: ehci: do not invalidate a NULL buffer

2017-11-17 Thread Dirk Behme
Its a valid use case to call ehci_submit_async() with a NULL buffer with length 0. E.g. from usb_set_configuration(). As invalidate_dcache_range() isn't able to judge if the address NULL is valid or not (depending on the SoC hardware configuration it might be valid) do the check in

Re: [U-Boot] [PATCH 3/4] sunxi: Add default partition scheme

2017-11-17 Thread Alexander Graf
On 17.11.17 14:04, Maxime Ripard wrote: > On Fri, Nov 17, 2017 at 12:21:49PM +, Andre Przywara wrote: >> Hi, >> >> On 17/11/17 08:27, Maxime Ripard wrote: >>> >>> I guess I also have a side question here. How do the installers deal >>> with the ESP partition? Would they create a new

Re: [U-Boot] [PATCH v4 12/18] efi_selftest: allow to select a single test for exexution

2017-11-17 Thread Simon Glass
On 18 October 2017 at 10:13, Heinrich Schuchardt wrote: > Environment variable efi_selftest is passed as load options > to the selftest application. It is used to select a single > test to be executed. > > The load options are an UTF8 string. Yet I decided to keep > the name

Re: [U-Boot] [PATCH v2 16/18] test/py: check return code of helloworld

2017-11-17 Thread Simon Glass
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote: > Check that helloworld.efi returns EFI_SUCCESS. > > Signed-off-by: Heinrich Schuchardt > --- > v2 > new patch > --- > test/py/tests/test_efi_loader.py | 2 ++ > 1 file changed, 2

Re: [U-Boot] [PATCH v2 17/18] efi_loader: pass handle of loaded image

2017-11-17 Thread Simon Glass
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote: > The handle of a loaded image is the value of the handle > member of the loaded image info object and not the > address of the loaded image info. > > Signed-off-by: Heinrich Schuchardt > --- > v2 >

Re: [U-Boot] [RFC PATCH 06/10] env: Support multiple environments

2017-11-17 Thread Maxime Ripard
Hi Lukasz, Tom, On Fri, Nov 17, 2017 at 03:00:15PM +0100, Lukasz Majewski wrote: > On Fri, 17 Nov 2017 08:41:05 -0500 > Tom Rini wrote: > > > On Fri, Nov 17, 2017 at 10:24:37AM +0100, Lukasz Majewski wrote: > > > Hi Maxime, > > > > > > > Now that we have everything in

Re: [U-Boot] [PATCH v2 13/18] efi_selftest: compile without special compiler flags

2017-11-17 Thread Simon Glass
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote: > As the selftest is not compiled as an EFI binary we do not > need special compiler flags. > > This avoids the checkarmreloc error on vexpress_ca15_tc2. > > Signed-off-by: Heinrich Schuchardt > ---

Re: [U-Boot] [PATCH v2 15/18] efi_loader: output load options in helloworld

2017-11-17 Thread Simon Glass
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote: > We need to test if we pass a valid image handle when loading > and EFI application. This cannot be done in efi_selftest as > it is not loaded as an image. > > So let's enhance helloworld to write the load options to

Re: [U-Boot] [PATCH v4 12/18] efi_selftest: correctly cleanup after selftest

2017-11-17 Thread Simon Glass
On 18 October 2017 at 10:13, Heinrich Schuchardt wrote: > After executing bootefi selftest > * restore GD > * unlink the load image handle > * return 0 or 1 and not a truncated efi_status_t. > > Signed-off-by: Heinrich Schuchardt > --- > v4 > no

Re: [U-Boot] [PATCH v2 14/18] efi_selftest: add missing line feed

2017-11-17 Thread Simon Glass
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote: > Add a missing line feed for an error message. > > Signed-off-by: Heinrich Schuchardt > --- > v2 > new patch > --- > lib/efi_selftest/efi_selftest.c | 2 +- > 1 file changed, 1

Re: [U-Boot] [PATCH v2 12/18] efi_loader: manage protocols in a linked list

2017-11-17 Thread Simon Glass
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote: > Signed-off-by: Heinrich Schuchardt > --- > v2 > no change > --- > include/efi_loader.h | 6 ++- > lib/efi_loader/efi_boottime.c | 107 >

Re: [U-Boot] [PATCH 1/1] Makefile: add coccicheck target

2017-11-17 Thread Simon Glass
On 10 November 2017 at 11:54, Heinrich Schuchardt wrote: > Coccinelle is a program for static code analysis. > For details on Coccinelle see > > http://coccinelle.lip6.fr/ > > Add scripts/coccicheck and Documentation/dev-tools/coccinelle.rst > copied from Linux kernel

Re: [U-Boot] [PATCH 4/5] tpm: add more useful NV storage permission flags

2017-11-17 Thread Simon Glass
On 3 October 2017 at 09:52, André Draszik wrote: > From: André Draszik > > TPM_NV_PER_PPREAD: physical presence needed for reading > TPM_NV_PER_WRITEDEFINE: persistent write lock by writing size 0 > TPM_NV_PER_WRITEALL: write in one go > > Signed-off-by:

Re: [U-Boot] [PATCH v2 08/18] efi_loader: efi_net: use efi_add_protocol

2017-11-17 Thread Simon Glass
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote: > Use efi_add_protocol to add protocols. > > Signed-off-by: Heinrich Schuchardt > --- > v2 > no change > --- > lib/efi_loader/efi_net.c | 34 -- > 1 file

Re: [U-Boot] [PATCH v2 11/18] efi_loader: simplify find_obj

2017-11-17 Thread Simon Glass
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote: > Use function efi_search_protocol. Use efi_search_protocol(). > > Signed-off-by: Heinrich Schuchardt > --- > v2 > no change > --- > lib/efi_loader/efi_device_path.c | 43 >

Re: [U-Boot] [PATCH v2 06/18] efi_selftest: test EFI_DEVICE_PATH_TO_TEXT_PROTOCOL

2017-11-17 Thread Simon Glass
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote: > Provide a test for the EFI_DEVICE_PATH_TO_TEXT_PROTOCOL protocol. > > Signed-off-by: Heinrich Schuchardt > --- > v2 > no change > --- > lib/efi_selftest/Makefile | 3 + >

Re: [U-Boot] [PATCH v2 07/18] efi_loader: efi_disk: use efi_add_protocol

2017-11-17 Thread Simon Glass
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote: > Use efi_add_protocol to install protocols. > > Signed-off-by: Heinrich Schuchardt > --- > v2 > no change > --- > lib/efi_loader/efi_disk.c | 39 +++ > 1

Re: [U-Boot] [PATCH v2 02/18] efi_loader: size of media device path node represenation

2017-11-17 Thread Simon Glass
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote: > In the format specifier we want to specify the maximum width > in case an ending \0 is missing. > > So slen must be used as precision and not as field width. > > Signed-off-by: Heinrich Schuchardt

Re: [U-Boot] [PATCH v2 7/9] efi_loader: consistently use efi_uintn_t in boot services

2017-11-17 Thread Simon Glass
On 6 November 2017 at 13:17, Heinrich Schuchardt wrote: > Consistenly use efi_uintn_t wherever the UEFI spec uses > UINTN in boot services interfaces. > > Signed-off-by: Heinrich Schuchardt > --- > v2 > no change > --- > include/efi_api.h

Re: [U-Boot] [PATCH 1/1] x86: don't compare pointers to 0

2017-11-17 Thread Simon Glass
On 10 November 2017 at 13:06, Heinrich Schuchardt wrote: > x86_vendor_name is defined as > > static const char *const x86_vendor_name[] > > So its elements should not be compared to 0. > > Remove superfluous paranthesis. > > Problem identified with Coccinelle. > >

Re: [U-Boot] [PATCH v2 18/18] efi_loader: helper function to add EFI object to list

2017-11-17 Thread Simon Glass
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote: > To avoid duplicate coding provide a helper function that > initializes an EFI object and adds it to the EFI object > list. > > efi_exit() is the only place where we dereference a handle de-reference? > to obtain a

  1   2   >