RE: [PATCH 00/16] usb: gadget: update

2020-10-09 Thread Peng Fan
Hi Lukasz, Are you fine with this patchset from NXP downstream? Thanks, Peng. > Subject: [PATCH 00/16] usb: gadget: update > > From: Peng Fan > > This patchset is to upstream NXP downstream gadget driver patches for > feature update. > > Jun Li (2): > usb: gadget: set correct usb_configurat

Re: [PATCH 00/16] Kconfig: Tidy up the top-level kconfig menu

2020-10-09 Thread Tom Rini
On Thu, Sep 10, 2020 at 08:21:11PM -0600, Simon Glass wrote: > At present this menu is pretty messy, with quite a few minor options shown > at the top level. This series creates a few new menus and moves things > around so that the top-level menu is cleaner. > > There is more to do, but this is a

[PATCH v2 1/1] sandbox: make SDL window resizable

2020-10-09 Thread Heinrich Schuchardt
Without resizing the SDL window showed by ./u-boot -D -l is not legible on a high resolution screen. Allow resizing the window Signed-off-by: Heinrich Schuchardt --- v2: Do not maximize the window. --- arch/sandbox/cpu/sdl.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)

Re: [PATCH 1/1] sandbox: make SDL window resizable

2020-10-09 Thread Heinrich Schuchardt
On 09.10.20 19:23, Simon Glass wrote: > Hi Heinrich, > > On Thu, 8 Oct 2020 at 03:46, Heinrich Schuchardt wrote: >> >> On 05.10.20 03:41, Simon Glass wrote: >>> Hi Heinrich, >>> >>> On Mon, 28 Sep 2020 at 19:11, Heinrich Schuchardt >>> wrote: Without resizing the SDL window showed by >

[PATCH] ARM: dts: stm32: Consistently enable internal pull-ups for SD bus

2020-10-09 Thread Marek Vasut
The default state of SD bus and clock line is logical HI. SD card IO is open-drain and pulls the bus lines LO. Always enable the SD bus pull ups to guarantee this behavior. Note that on systems with bus voltage level shifter on the SD bus, the pull ups might also be built into the level shifter, ho

Re: [Uboot-stm32] [PATCH 0/7] arm: cache: cp15: don't map reserved region with no-map property

2020-10-09 Thread Ard Biesheuvel
On Fri, 9 Oct 2020 at 19:13, Ahmad Fatoum wrote: > > Hello Patrick, > > On 10/9/20 5:52 PM, Patrick DELAUNAY wrote: > > I checked DACR behavior and CheckDomain / CheckPermission > > > > In my case the cortex A7 try to access to part of DDR / mapped cacheable > > and bufferable, protected by fire

Re: [PATCH] common, autoboot: sync functionality with Kconfig description

2020-10-09 Thread Simon Glass
Hi Heiko, On Fri, 9 Oct 2020 at 07:21, Tom Rini wrote: > > On Wed, Oct 07, 2020 at 08:06:54AM +0200, Heiko Schocher wrote: > > > add back again special case: -2 > > autoboot with no delay and no check for abort > > > > as described in Kconfig option, see common/Kconfig > > help text for option BO

Re: [PATCH v2 2/2] timer: Return count from timer_ops.get_count

2020-10-09 Thread Simon Glass
Hi Sean, On Wed, 7 Oct 2020 at 12:38, Sean Anderson wrote: > > No timer drivers return an error from get_count. Instead of possibly > returning an error, just return the count directly. > > Signed-off-by: Sean Anderson > --- > > Changes in v2: > - mchp-pit64b was added since v1, so convert it >

Re: [PATCH 4/6] README: sandbox: Change reference to sandbox details

2020-10-09 Thread Simon Glass
On Wed, 7 Oct 2020 at 22:17, Naoki Hayama wrote: > > doc/arch/index.rst is a list of links to each architecture. > As for the sandbox details, doc/arch/sandbox.rst looks better. > > Signed-off-by: Naoki Hayama > --- > > README | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by:

Re: [PATCH 1/1] sandbox: make SDL window resizable

2020-10-09 Thread Simon Glass
Hi Heinrich, On Thu, 8 Oct 2020 at 03:46, Heinrich Schuchardt wrote: > > On 05.10.20 03:41, Simon Glass wrote: > > Hi Heinrich, > > > > On Mon, 28 Sep 2020 at 19:11, Heinrich Schuchardt > > wrote: > >> > >> Without resizing the SDL window showed by > >> > >> ./u-boot -D -l > >> > >> is not

Re: [PATCH] env: sf: add support for env erase

2020-10-09 Thread Sean Anderson
On 10/9/20 12:43 PM, Harry Waschkeit wrote: > Hi Sean, > > thanks for your try and sorry for the inconvenience my beginner's mistakes > have > caused :-( > > It is definitely no good idea to use copy&paste with patch data, I should have > guessed that beforehand ... You *can* do it, you just ha

RE: [Uboot-stm32] [PATCH 0/7] arm: cache: cp15: don't map reserved region with no-map property

2020-10-09 Thread Patrick DELAUNAY
Hi Ard, > From: Ard Biesheuvel > Sent: mercredi 7 octobre 2020 15:16 > > On Wed, 7 Oct 2020 at 13:53, Ahmad Fatoum wrote: > > > > Hello, > > > > On 10/7/20 1:23 PM, Ahmad Fatoum wrote: > > > My findings[1] back then were that U-Boot did set the eXecute Never > > > bit only on OMAP, but not for

Re: [PATCH] env: sf: add support for env erase

2020-10-09 Thread Harry Waschkeit
Hi Sean, thanks for your try and sorry for the inconvenience my beginner's mistakes have caused :-( It is definitely no good idea to use copy&paste with patch data, I should have guessed that beforehand ... Anyway, concerning the complaints from checkpatch.pl: I indeed ran checkpatch.pl on my p

RE: [Uboot-stm32] [PATCH 0/7] arm: cache: cp15: don't map reserved region with no-map property

2020-10-09 Thread Patrick DELAUNAY
Hi Ahmad, > From: Ahmad Fatoum > Sent: mercredi 7 octobre 2020 13:24 > > Hello Ard, Patrick, > > On 10/7/20 12:26 PM, Ard Biesheuvel wrote: > >> The issue is solved only when the region reserved by OP-TEE is no > >> more mapped in U-Boot (mapped as DEVICE/NON-CACHEABLE wasn't > enough) > >> as

[PATCH v2 2/2] km/common: change ubicopy variable

2020-10-09 Thread Holger Brunck
Instead having a hard coded value for "cramfsaddr" after compile time, we change it to take the variable "cramfsaddr" for the ubicopy variable. This makes sure that ubicopy uses the right address, even when the value for "cramfsaddr" has changed. CC: Valentin Longchamp CC: Heiko Schocher CC: Tom

[PATCH v2 1/2] km: adapt defines and variables for new memory layout

2020-10-09 Thread Holger Brunck
Due to increasing kernel image sizes we get problems when decompressing the kernel image. To fix this we need to change the addresses where we load and where we extract the kernel. Also we need to adapt the address where to load the CRAMFS image and where to load the DTB file. While at it also harm

Re: [PATCH] env: sf: add support for env erase

2020-10-09 Thread Sean Anderson
On 10/8/20 1:27 PM, Harry Waschkeit wrote: > Command "env erase" didn't work even though CONFIG_CMD_ERASEENV was > defined, because serial flash environment routines didn't implement > erase method. > > Signed-off-by: Waschkeit, Harry > --- Hi Harry, I wanted to test out your patch, but I could

Re: [PATCH] common, autoboot: sync functionality with Kconfig description

2020-10-09 Thread Tom Rini
On Wed, Oct 07, 2020 at 08:06:54AM +0200, Heiko Schocher wrote: > add back again special case: -2 > autoboot with no delay and no check for abort > > as described in Kconfig option, see common/Kconfig > help text for option BOOTDELAY. > > Signed-off-by: Heiko Schocher Reviewed-by: Tom Rini -

Re: [PATCH 2/2] am335x_evm: Allow booting from usb-storage device

2020-10-09 Thread Tom Rini
On Sun, Oct 04, 2020 at 02:23:21PM +0300, Matwey V. Kornilov wrote: > Ping? > > пн, 24 авг. 2020 г. в 21:00, Matwey V. Kornilov : > > > Signed-off-by: Matwey V. Kornilov > > --- > > include/configs/am335x_evm.h | 7 +++ > > 1 file changed, 7 insertions(+) > > > > diff --git a/include/conf

Re: [PATCH] Define default CONFIG_PREBOOT with right config option

2020-10-09 Thread Tom Rini
On Wed, Oct 07, 2020 at 08:37:57AM +, Patrick DELAUNAY wrote: > Hi, > > > From: U-Boot On Behalf Of Peter Robinson > > Sent: mardi 29 septembre 2020 11:48 > > > > The 44758771ee commit removes CONFIG_PREBOOT but actually sets the > > USE_PREBOOT Kconfig option which isn't CONFIG_PREBOOT and

Re: [PATCH 1/1] rsa: fix retrieving public exponent on big-endian systems

2020-10-09 Thread Tom Rini
On Wed, Oct 07, 2020 at 12:17:56AM +0200, Rasmus Villemoes wrote: > On 07/10/2020 00.02, Simon Glass wrote: > > Hi Rasmus, > > > > On Tue, 6 Oct 2020 at 04:10, Rasmus Villemoes > > wrote: > >> > >> Commit fdf0819afb (rsa: fix alignment issue when getting public > >> exponent) changed the logic to

Re: [PATCH v2 5/6] qemu-arm: Drop ARCH_SUPPORT_TFABOOT

2020-10-09 Thread Tom Rini
On Wed, Sep 30, 2020 at 05:39:17PM +0100, Andre Przywara wrote: > CONFIG_ARCH_SUPPORT_TFABOOT was used on the qemu-arm64 platform to > guard a tweak to the flash bank configuration. U-Boot now reads the > current flash setup from the devicetree, so there is no need for > this option anymore. > >

Re: [PATCH v2 6/6] qemu-arm64: Enable POSITION_INDEPENDENT

2020-10-09 Thread Tom Rini
On Wed, Sep 30, 2020 at 05:39:18PM +0100, Andre Przywara wrote: > Now that PIE works when U-Boot is started from ROM, let's enable > CONFIG_POSITION_INDEPENDENT, which allows to load U-Boot also via > ARM Trusted-Firmware's fip.bin to DRAM, without tweaking the > configuration. > > To get a writa

Re: [PATCH v2 4/6] qemu-arm: Remove need to specify flash banks

2020-10-09 Thread Tom Rini
On Wed, Sep 30, 2020 at 05:39:16PM +0100, Andre Przywara wrote: > Currently we hard-code the number and initial addresses of QEMU's flash > banks, even though our code is perfectly able to gather the same > information from the DTB provided by QEMU. > This is especially annoying, since we have two

Re: [PATCH v2 3/6] arm64: PIE: Allow fixed stack pointer

2020-10-09 Thread Tom Rini
On Wed, Sep 30, 2020 at 05:39:15PM +0100, Andre Przywara wrote: > Currently selecting CONFIG_POSITION_INDEPENDENT also forces us to use an > initial stack pointer relative to the beginning of the BSS section. > This makes some sense, because this should be writable memory anyway. > > However the

Re: [PATCH] arm: Kconfig: Explain TFABOOT

2020-10-09 Thread Tom Rini
On Wed, Sep 30, 2020 at 03:45:07PM +0100, Andre Przywara wrote: > The CONFIG_TFABOOT option is more about what U-Boot DOES NOT need to do > than to support some features. > > Explain a bit more in the Kconfig help text to avoid misunderstandings. > > Signed-off-by: Andre Przywara > Reviewed-by:

Re: [PATCH v2 2/6] arm64: PIE: Skip fixups if distance is zero

2020-10-09 Thread Tom Rini
On Wed, Sep 30, 2020 at 05:39:14PM +0100, Andre Przywara wrote: > When the actual offset between link and runtime address is zero, there > is no need for patching up U-Boot early when running with > CONFIG_POSITION_INDEPENDENT. > > Skip the whole routine when the distance is 0. > > This helps wh

Re: [PATCH v2 1/1] examples: make examples/ optional

2020-10-09 Thread Tom Rini
On Wed, Sep 23, 2020 at 07:09:51PM +0200, Heinrich Schuchardt wrote: > Most users don't need the standalone API examples. Distributions like SUSE > do not supply libgcc for cross-compiling and we cannot do without on ARMv8 > for building examples/. > > Make examples selectable via symbol CONFIG_E

Re: [PATCH v2 1/6] arm64: PIE: Do not skip static relocation

2020-10-09 Thread Tom Rini
On Wed, Sep 30, 2020 at 05:39:13PM +0100, Andre Przywara wrote: > When we build an arm64 target and enable POSITION_INDEPENDENT, we were > skipping our build-time dynamic relocation fixup routine (STATIC_RELA). > > This was probably done because we didn't need it in this case, as the > PIE fixup

Re: [PATCH v6 11/12] riscv: add DT binding for BOOT button on Maix board

2020-10-09 Thread Tom Rini
On Mon, Sep 14, 2020 at 11:02:05AM -0400, Sean Anderson wrote: > From: Heinrich Schuchardt > > Add a device tree binding for the BOOT button on the Maix board. > > Signed-off-by: Heinrich Schuchardt > Reviewed-by: Sean Anderson > Signed-off-by: Sean Anderson > Reviewed-by: Rick Chen Applie

Re: [PATCH v6 10/12] riscv: Add pinmux and gpio bindings for Kendryte K210

2020-10-09 Thread Tom Rini
On Mon, Sep 14, 2020 at 11:02:04AM -0400, Sean Anderson wrote: > This patch adds the necessary device tree bindings. > > Signed-off-by: Sean Anderson > Reviewed-by: Simon Glass > Acked-by: Rick Chen Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v6 09/12] test: dm: Test for default led naming

2020-10-09 Thread Tom Rini
On Mon, Sep 14, 2020 at 11:02:03AM -0400, Sean Anderson wrote: > This modifies the existing led test to check for default led naming as > added in the previous patch. > > Signed-off-by: Sean Anderson > Reviewed-by: Simon Glass Applied to u-boot/master, thanks! -- Tom signature.asc Descript

Re: [PATCH v6 12/12] riscv: Add FPIOA and GPIO support for Kendryte K210

2020-10-09 Thread Tom Rini
On Mon, Sep 14, 2020 at 11:02:06AM -0400, Sean Anderson wrote: > This patch adds the necessary configs and docs for FPIOA and GPIO support > on the K210. > > Signed-off-by: Sean Anderson Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH] wdt: designware: fix timeout calculation due to expecting KHz

2020-10-09 Thread Tom Rini
On Thu, Sep 17, 2020 at 10:30:40AM +0100, Jack Mitchell wrote: > The timeout calculation is based on the clk being in KHz but > the clk api returns the clk value in Hz. Convert this to KHz > to calculate the correct timeout value. > > Signed-off-by: Jack Mitchell Applied to u-boot/master, thank

Re: [PATCH v6 08/12] led: gpio: Default to using node name if label is absent

2020-10-09 Thread Tom Rini
On Mon, Sep 14, 2020 at 11:02:02AM -0400, Sean Anderson wrote: > This more closely mirrors Linux's behaviour, and will make it easier to > transition to using function+color in the future. > > Signed-off-by: Sean Anderson > Reviewed-by: Simon Glass Applied to u-boot/master, thanks! -- Tom

Re: [PATCH v6 07/12] gpio: dw: Return output value when direction is out

2020-10-09 Thread Tom Rini
On Mon, Sep 14, 2020 at 11:02:01AM -0400, Sean Anderson wrote: > dm_gpio_ops.get_value can be called when the gpio is either input or > output. The current dw code always returns the input value, which is > invalid if the direction is set to out. > > Signed-off-by: Sean Anderson > Reviewed-by: L

Re: [PATCH v6 05/12] gpio: dw: Fix warnings about casting int to pointer

2020-10-09 Thread Tom Rini
On Mon, Sep 14, 2020 at 11:01:59AM -0400, Sean Anderson wrote: > Change the type of gpio_dwabp_platdata.base from fdt_addr_t to a void > pointer, since we pass it to readl. > > Signed-off-by: Sean Anderson > Reviewed-by: Ley Foon Tan > Reviewed-by: Bin Meng > Reviewed-by: Simon Glass Applied

Re: [PATCH v6 06/12] gpio: dw: Add a trailing underscore to generated name

2020-10-09 Thread Tom Rini
On Mon, Sep 14, 2020 at 11:02:00AM -0400, Sean Anderson wrote: > Previously, if there was no bank-name property, it was easy to have > confusing gpio names like "gpio1@08", instead of "gpio1@0_8". This patch > follows the example of the sifive gpio driver. > > Signed-off-by: Sean Anderson > Revi

Re: [PATCH v6 03/12] test: pinmux: Add test for pin muxing

2020-10-09 Thread Tom Rini
On Mon, Sep 14, 2020 at 11:01:57AM -0400, Sean Anderson wrote: > This extends the pinctrl-sandbox driver to support pin muxing, and adds a > test for that behaviour. The test is done in C and not python (like the > existing tests for the pinctrl uclass) because it needs to call > pinctrl_select_st

Re: [PATCH v6 01/12] pinctrl: Add pinmux property support to pinctrl-generic

2020-10-09 Thread Tom Rini
On Mon, Sep 14, 2020 at 11:01:55AM -0400, Sean Anderson wrote: > The pinmux property allows for smaller and more compact device trees, > especially when there are many pins which need to be assigned individually. > Instead of specifying an array of strings to be parsed as pins and a > function pro

Re: [PATCH v6 04/12] pinctrl: Add support for Kendryte K210 FPIOA

2020-10-09 Thread Tom Rini
On Mon, Sep 14, 2020 at 11:01:58AM -0400, Sean Anderson wrote: > The Fully-Programmable Input/Output Array (FPIOA) device controls pin > multiplexing on the K210. The FPIOA can remap any supported function to any > multifunctional IO pin. It can also perform basic GPIO functions, such as > reading

Re: [PATCH v6 02/12] pinctrl: Reformat documentation in dm/pinctrl.h

2020-10-09 Thread Tom Rini
On Mon, Sep 14, 2020 at 11:01:56AM -0400, Sean Anderson wrote: > This normalizes the documentation to conform to kernel-doc style [1]. It > also moves the documentation for pinctrl_ops inline, and adds argument and > return-value documentation. I have kept the usual function style for these > comm

Re: [PATCH 2/2] ram: add ddr4 dual x8 configuration

2020-10-09 Thread Tom Rini
On Mon, Sep 07, 2020 at 04:25:07PM +0800, Dylan Hung wrote: > the aspeed ddr sdram controller needs to know if the memory chip mounted on > the board is dual x8 die or not. Or it may get the wrong size of the > memory space. > > Signed-off-by: Dylan Hung > Reviewed-by: Ryan Chen Applied to u-b

Re: [PATCH 1/2] ram: move aspeed ram driver into drivers/ directory

2020-10-09 Thread Tom Rini
On Mon, Sep 07, 2020 at 04:25:06PM +0800, Dylan Hung wrote: > to improve the maintainability. It is more easier to modify and add > configurations of the driver in the centralized ram driver directory. > > Signed-off-by: Dylan Hung > Reviewed-by: Ryan Chen Applied to u-boot/master, thanks! -

RE: [PATCH 1/2] i2c: ocores: add i2c driver for OpenCores I2C controller

2020-10-09 Thread Pragnesh Patel
Hi Sean, >-Original Message- >From: Sean Anderson >Sent: 09 October 2020 17:58 >To: Pragnesh Patel ; u-boot@lists.denx.de >Cc: atish.pa...@wdc.com; palmerdabb...@google.com; bmeng...@gmail.com; >Paul Walmsley ( Sifive) ; anup.pa...@wdc.com; >Sagar Kadam ; r...@andestech.com; Heiko >Schoch

Re: [PATCH 1/2] i2c: ocores: add i2c driver for OpenCores I2C controller

2020-10-09 Thread Sean Anderson
On 10/9/20 1:46 AM, Pragnesh Patel wrote: > Add support for the OpenCores I2C controller IP core > (See http://www.opencores.org/projects.cgi/web/i2c/overview). > > This driver implementation is inspired from the Linux OpenCores > I2C driver available. > > Thanks to Peter Korsgaard for writing L

Re: [PATCH 0/7] arm: cache: cp15: don't map reserved region with no-map property

2020-10-09 Thread Ard Biesheuvel
On Fri, 9 Oct 2020 at 13:19, Patrick DELAUNAY wrote: > > Hi Ard, > > > From: Ard Biesheuvel > > Sent: mercredi 7 octobre 2020 12:27 > > > > On Tue, 6 Oct 2020 at 18:36, Patrick Delaunay > > wrote: > > > > > > > > > Hi, > > > > > > On STM32MP15x platform we can use OP-TEE, loaded in DDR in a regi

optee: FIT image sub-image rejected

2020-10-09 Thread Robert Delien
Hi guys, I'm trying to boot a FIT image, containing a Linux Kernel, an FDT and optee. I am using U-boot 2018.03, as it came in the NXP i.MX6UL Yocto distribution, patched it with Bryan O'Donoghue's optee patches of March 2018, and out-configured NXP's CONFIG_IMX_OPTEE. Below the DTS for my for th

RE: [PATCH 0/7] arm: cache: cp15: don't map reserved region with no-map property

2020-10-09 Thread Patrick DELAUNAY
Hi Ard, > From: Ard Biesheuvel > Sent: mercredi 7 octobre 2020 12:27 > > On Tue, 6 Oct 2020 at 18:36, Patrick Delaunay > wrote: > > > > > > Hi, > > > > On STM32MP15x platform we can use OP-TEE, loaded in DDR in a region > > protected by a firewall. This region is reserved in device with "no-map

[PATCH v2] net: lx2160a.c: Update to set ECx_PMUX precedence

2020-10-09 Thread Razvan Ionut Cirjan
As per hardware documentation, ECx_PMUX has precedence over SerDes protocol. For LX2160/LX2162 if DPMACs 17 and 18 are enabled as SGMII through SerDes protocol but ECx_PMUX configured them as RGMII, then the ports will be configured as RGMII and not SGMII. Signed-off-by: Razvan Ionut Cirjan ---

RE: [PATCH] Define default CONFIG_PREBOOT with right config option

2020-10-09 Thread Patrick DELAUNAY
Hi Simon and Tom > From: Tom Rini > Sent: mercredi 7 octobre 2020 17:45 > > On Wed, Oct 07, 2020 at 07:26:55AM -0600, Simon Glass wrote: > > Hi Patrick, > > > > On Wed, 7 Oct 2020 at 02:38, Patrick DELAUNAY > wrote: > > > > > > Hi, > > > > > > > From: U-Boot On Behalf Of Peter > > > > Robinson

RE: [PATCH] configs: stm32mp: force empty PREBOOT

2020-10-09 Thread Patrick DELAUNAY
Hi Tom, > From: Tom Rini > Sent: jeudi 8 octobre 2020 16:16 > > On Wed, Oct 07, 2020 at 10:10:20AM +0200, Patrick Delaunay wrote: > > > This patch remove the default preboot command 'usb start' for > > STMicroelectronics board. These command is added by the commit > > 324d77998ed6 ("Define defa

Re: [PATCH] ARM: dts: stm32: Do not set eth1addr if KS8851 has EEPROM

2020-10-09 Thread Patrice CHOTARD
Hi Marek On 10/8/20 3:14 PM, Marek Vasut wrote: > In case the KS8851 has external EEPROM attached to it, do not set > eth1addr at all. The network stack will read the MAC out of the > KS8851 and set eth1addr accordingly. > > Signed-off-by: Marek Vasut > Cc: Patrice Chotard > Cc: Patrick Delaunay