Re: [U-Boot] [RESEND PATCH V2 3/3] imx: mx7: add system suspend/resume support

2018-08-29 Thread Anson Huang
Hi, all Can anyone pick up this patch set, it has been tested and suspend/resume is an important feature many people want, thanks! Anson Huang Best Regards! > -Original Message- > From: Stefan Agner > Sent: Wednesday, August 8, 2018 8:09 PM > To: Anson Huang > Cc: sba...@denx.

[U-Boot] [PATCH v4] gpio: dwapb_gpio: Add reset ctrl to driver

2018-08-29 Thread Ley Foon Tan
Add code to reset all reset signals as in gpio DT node. A reset property is an optional feature, so only print out a warning and do not fail if a reset property is not present. If a reset property is discovered, then use it to deassert, thus bringing the IP out of reset. Signed-off-by: Ley Foon T

[U-Boot] [PATCH 00/19] Marvell PXA3XX NAND driver fixes and extensions

2018-08-29 Thread kostap
From: Konstantin Porotchkin This patch series introduces the PXA3XX NAND driver changes and extensions accumulated in Marvell LSP sources. Beside the bug fixes the driver is extended to support 2K page 8-bit ECC flash devices (tested with Toshiba TC58NVG1S3ETA00 chip) and 8K page 4/8 bit ECC devi

[U-Boot] [PATCH 01/19] mtd: nand: pxa3xx_nand: Increase initial buffer size

2018-08-29 Thread kostap
From: Ofer Heifetz The initial buffer is used for the initial commands used to detect a flash device (STATUS, READID and PARAM). ONFI param page is 256 bytes, and there are three redundant copies to be read. JEDEC param page is 512 bytes, and there are also three redundant copies to be read. Hen

[U-Boot] [PATCH 04/19] mtd: nand: pxa3xx_nand: fix early spurious interrupt

2018-08-29 Thread kostap
From: Ofer Heifetz When the nand is first probe, and upon the first command start, the status bits should be cleared before the interrupts are unmasked. This commit is taken from Linux: 'commit 0b14392db2e' ("mtd: nand: pxa3xx_nand: fix early spurious interrupt") Signed-off-by: Chris Packham S

[U-Boot] [PATCH 10/19] mtd: nand: pxa3xx_nand: add support for partial chunks

2018-08-29 Thread kostap
From: Ofer Heifetz This commit is needed to properly support the 8-bits ECC configuration with 4KB pages. When pages larger than 2 KB are used on platforms using the PXA3xx NAND controller, the reading/programming operations need to be split in chunks of 2 KBs or less because the controller FIFO

[U-Boot] [PATCH 03/19] mtd: nand: pxa3xx_nand: sync pxa3xx_nand_set_sdr_timing()

2018-08-29 Thread kostap
From: Ofer Heifetz Since the pxa3xx_nand driver was added there has been a discrepancy in pxa3xx_nand_set_sdr_timing() around the setting of tWP_min and tRP_min. This brings us into line with the current Linux code. Signed-off-by: Chris Packham Signed-off-by: Ofer Heifetz Reviewed-by: Igal Lib

[U-Boot] [PATCH 02/19] mtd: nand: pxa3xx_nand: use nand_to_mtd()

2018-08-29 Thread kostap
From: Ofer Heifetz Don't store struct mtd_info in struct pxa3xx_nand_host. Instead use the one that is already part of struct nand_chip. This brings us in line with current U-boot and Linux conventions. Signed-off-by: Chris Packham Signed-off-by: Ofer Heifetz Reviewed-by: Igal Liberman Cc: St

[U-Boot] [PATCH 06/19] nand: pxa3xx: Increase READ_ID buffer and make the size static

2018-08-29 Thread kostap
From: Ofer Heifetz The read ID count should be made as large as the maximum READ_ID size, so there's no need to have dynamic size. This commit sets the hardware maximum read ID count, which should be more than enough on all cases. Also, we get rid of the read_id_bytes, and use a macro instead. T

[U-Boot] [PATCH 19/19] fix: nand: pxa3xx: Add WA for eliminating flash ready timeout

2018-08-29 Thread kostap
From: David Sniatkiwicz add delay before processing the status flags in pxa3xx_nand_irq(). Signed-off-by: David Sniatkiwicz Reviewed-by: Igal Liberman Reviewed-by: Kostya Porotchkin c: Stefan Roese Cc: Simon Glass --- drivers/mtd/nand/pxa3xx_nand.c | 3 +++ 1 file changed, 3 insertions(+)

[U-Boot] [PATCH 08/19] mtd: pxa3xx_nand: Fix initial controller configuration

2018-08-29 Thread kostap
From: Ofer Heifetz The Data Flash Control Register (NDCR) contains two types of parameters: those that are needed for device identification, and those that can only be set after device identification. Therefore, the driver can't set them all at once and instead needs to configure the first group

[U-Boot] [PATCH 05/19] mtd: nand: pxa3xx-nand: fix random command timeouts

2018-08-29 Thread kostap
From: Ofer Heifetz When 2 commands are submitted in a row, and the second is very quick, the completion of the second command might never come. This happens especially if the second command is quick, such as a status read after an erase This patch is taken from Linux: 'commit 21fc0ef9652f' ("mtd

[U-Boot] [PATCH 12/19] fix: nand: pxa3xx: Add SoC-specific enable function

2018-08-29 Thread kostap
From: Igal Liberman Add SoC-dependent function to the NAND driver for selecting NAND interface in DEVBUS MUX register. This selection is done in the BootROM if the boot device is NAND, but may be missing othervise. The NAND is selected only if it is enabled in the DT file. This patch is fixing NA

[U-Boot] [PATCH 18/19] nand: pxa3xx: Add support for 8KB page 4 and 8 bit ECC NAND

2018-08-29 Thread kostap
From: Konstantin Porotchkin Add support for NAND chips with 8KB page, 4 and 8 bit ECC (ONFI). Signed-off-by: Konstantin Porotchkin Reviewed-by: Ofer Heifetz Cc: Stefan Roese Cc: Simon Glass --- drivers/mtd/nand/pxa3xx_nand.c | 72 ++ 1 file changed, 6

[U-Boot] [PATCH 17/19] nand: pxa3xx: cosmetic: add comments to the timing layout structures

2018-08-29 Thread kostap
From: Konstantin Porotchkin Add comments with timing parameter names and some details about nand layout fileds. Remove unneeded definition. Signed-off-by: Konstantin Porotchkin Reviewed-by: Igal Liberman Cc: Stefan Roese Cc: Simon Glass --- drivers/mtd/nand/pxa3xx_nand.c | 21 ++

[U-Boot] [PATCH 11/19] a70x0: a80x0: add soc type and revision printing in boot log

2018-08-29 Thread kostap
From: zachary zhang This patch adds SoC type and revision and each module's type and revision printing for a7k and a8k boards. Signed-off-by: zachary zhang Reviewed-by: Igal Liberman Cc: Stefan Roese Cc: Simon Glass --- arch/arm/mach-mvebu/armada8k/Makefile | 1 + arch/arm/mach-mvebu/arm

[U-Boot] [PATCH 15/19] mtd: nand: pxa3xx: add support for Toshiba flash

2018-08-29 Thread kostap
From: Konstantin Porotchkin Add timings and device ID for Toshiba TC58NVG1S3HTA00 flash Signed-off-by: Konstantin Porotchkin Reviewed-by: Igal Liberman Cc: Stefan Roese Cc: Simon Glass --- drivers/mtd/nand/pxa3xx_nand.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/mtd/nand

[U-Boot] [PATCH 09/19] mtd: pxa3xx_nand: Simplify pxa3xx_nand_scan

2018-08-29 Thread kostap
From: Ofer Heifetz This commit simplifies the initial configuration performed by pxa3xx_nand_scan. No functionality change is intended. This commit is taken from Linux: 'commit 154f50fbde53' ("mtd: pxa3xx_nand: Simplify pxa3xx_nand_scan") Signed-off-by: Chris Packham Signed-off-by: Ofer Heifet

[U-Boot] [PATCH 07/19] mtd: pxa3xx_nand: Increase the initial chunk size

2018-08-29 Thread kostap
From: Ofer Heifetz The chunk size represents the size of the data chunks, which is used by the controllers that allow to split transferred data. However, the initial chunk size is used in a non-split way, during device identification. Therefore, it must be large enough for all the NAND commands

[U-Boot] [PATCH 14/19] mtd: nand: pxa3xx: add support for 2KB 8-bit flash

2018-08-29 Thread kostap
From: Victor Axelrod Add support for 2KB page 8-bit ECC strength flash layout Signed-off-by: Victor Axelrod Reviewed-by: Igal Liberman Cc: Stefan Roese Cc: Simon Glass --- drivers/mtd/nand/pxa3xx_nand.c | 27 +++ 1 file changed, 27 insertions(+) diff --git a/drivers

[U-Boot] [PATCH 13/19] mtd: nand: pxa3xx: Fix READOOB implementation

2018-08-29 Thread kostap
From: Boris Brezillon In the current driver, OOB bytes are accessed in raw mode, and when a page access is done with NDCR_SPARE_EN set and NDCR_ECC_EN cleared, the driver must read the whole spare area (64 bytes in case of a 2k page, 16 bytes for a 512 page). The driver was only reading the free

[U-Boot] [PATCH 16/19] fix: nand: Replace hardcoded page chunk size with calculated one

2018-08-29 Thread kostap
From: Konstantin Porotchkin Replace the hardcoded value of page chink with value that depends on flash page size and ECC strength. This fixes nand access errors for 2K page flashes with 8-bit ECC. Move the initial flash commannd function assignment past the ECC structures initialization for elimi

Re: [U-Boot] [PATCH V4 1/2] mmc: add HS400 support

2018-08-29 Thread Marek Vasut
On 08/29/2018 07:32 AM, Peng Fan wrote: > Hi, Hi, > Would anyone pick up this patchset? Tom, I delegated the patchset to you, > would you mind pick it up? I've been nagging Tom about this for a while , he said he will pick it for next release, but having it in this one would be IMO fine, since

[U-Boot] OSFC Tickets

2018-08-29 Thread Zaolin
Friendly reminder. 2 days until the ticket sale is closed. Buy your tickets now https://osfc.io/tickets ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH V4 1/2] mmc: add HS400 support

2018-08-29 Thread Tom Rini
On Wed, Aug 29, 2018 at 12:35:05PM +0200, Marek Vasut wrote: > On 08/29/2018 07:32 AM, Peng Fan wrote: > > Hi, > > Hi, > > > Would anyone pick up this patchset? Tom, I delegated the patchset to you, > > would you mind pick it up? > > I've been nagging Tom about this for a while , he said he w

Re: [U-Boot] [PATCH v4] gpio: dwapb_gpio: Add reset ctrl to driver

2018-08-29 Thread Marek Vasut
On 08/29/2018 10:44 AM, Ley Foon Tan wrote: > Add code to reset all reset signals as in gpio DT node. A reset property > is an optional feature, so only print out a warning and do not fail if a > reset property is not present. > > If a reset property is discovered, then use it to deassert, thus >

Re: [U-Boot] [RESEND PATCH V2 3/3] imx: mx7: add system suspend/resume support

2018-08-29 Thread Fabio Estevam
Hi Anson, On Wed, Aug 29, 2018 at 5:08 AM, Anson Huang wrote: > Hi, all > Can anyone pick up this patch set, it has been tested and > suspend/resume is an important feature many people want, thanks! Thanks for your work on this. This series looks good. Hi Stefano, Are you happy with

[U-Boot] [PATCH 4/5] cmd: mvebu: bubt: Add support for legacy Marvell SoCs

2018-08-29 Thread kostap
From: Konstantin Porotchkin Add support for image load and basic verification in bubt for legacy Marvell SoCs (A38x, A39x, ...) Signed-off-by: Konstantin Porotchkin Cc: Igal Liberman Cc: Stefan Roese --- cmd/mvebu/bubt.c | 75 +--- 1 file c

[U-Boot] [PATCH 0/5] Fixes to Marvell A38x platforms

2018-08-29 Thread kostap
From: Konstantin Porotchkin The support for legacy Marvell SoC plaform db-88f6820-gp is broken starting middle of 2017. This patch series returns functionality to this platform by fixing problem in environment handling load from SPI flash. Additionally the "bubt" command is extended to support

[U-Boot] [PATCH 5/5] defconfig: db-88f6820-gp: Add bubt command to the build

2018-08-29 Thread kostap
From: Konstantin Porotchkin Add Marvell bubt command to db-88f6820-gp defconfig Signed-off-by: Konstantin Porotchkin Cc: Igal Liberman Cc: Stefan Roese --- configs/db-88f6820-gp_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/db-88f6820-gp_defconfig b/configs/db-88f6820

[U-Boot] [PATCH 2/5] fix: mvebu: Add SPI parameters for environment setup

2018-08-29 Thread kostap
From: Konstantin Porotchkin Add definitions for CONFIG_ENV_SPI_BUS and CONFIG_ENV_SPI_CS to Armada-388-GP board configuration Signed-off-by: Konstantin Porotchkin Cc: Igal Liberman Cc: Stefan Roese --- include/configs/db-88f6820-gp.h | 6 ++ 1 file changed, 6 insertions(+) diff --git a/

[U-Boot] [PATCH 1/5] fix: env: Fix the SPI flash device setup for DM mode

2018-08-29 Thread kostap
From: Konstantin Porotchkin For some reason the spi_flash_probe_bus_cs() is called inside the setup_flash_device() with zero values in place of configurated SPI flash mode and maximum flash speed. This code causes HALT error during startup environment relocation on some platforms - namely Armada-

[U-Boot] [PATCH 3/5] fix: cmd: mvebu: Eclude mvebu commands from SPL builds

2018-08-29 Thread kostap
From: Konstantin Porotchkin Exclude mvebu commands from SPL builds Signed-off-by: Konstantin Porotchkin Cc: Igal Liberman Cc: Stefan Roese --- cmd/Makefile | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/cmd/Makefile b/cmd/Makefile index 3487c80..a61fab6 100644 --- a/cm

[U-Boot] dwc2 USB host initialization issue

2018-08-29 Thread Bogdan Harjoc
I am trying to initialize a dwc2 usb otg host using the u-boot dwc2 driver in u-boot and barebox and am trying to get past the port power-up step. The controller is activated successfully once the Linux kernel boots up and the mass storage devices are accessible, but in u-boot and barebox it seems

Re: [U-Boot] [PATCH V2 1/2] pci: Support parsing PCI controller DT subnodes

2018-08-29 Thread Marek Vasut
On 08/24/2018 08:27 PM, Marek Vasut wrote: > The PCI controller can have DT subnodes describing extra properties > of particular PCI devices, ie. a PHY attached to an EHCI controller > on a PCI bus. This patch parses those DT subnodes and assigns a node > to the PCI device instance, so that the dri

Re: [U-Boot] [PATCH 4/5] cmd: mvebu: bubt: Add support for legacy Marvell SoCs

2018-08-29 Thread Jon Nettleton
On Wed, Aug 29, 2018 at 3:36 PM wrote: > > From: Konstantin Porotchkin > > Add support for image load and basic verification in bubt > for legacy Marvell SoCs (A38x, A39x, ...) > > Signed-off-by: Konstantin Porotchkin > Cc: Igal Liberman > Cc: Stefan Roese > --- > cmd/mvebu/bubt.c | 75 > +++

Re: [U-Boot] [EXT] Re: [PATCH 4/5] cmd: mvebu: bubt: Add support for legacy Marvell SoCs

2018-08-29 Thread Kostya Porotchkin
Hi, Jon, > -Original Message- > From: Jon Nettleton > Sent: Wednesday, August 29, 2018 17:28 > To: Kostya Porotchkin > Cc: U-Boot Mailing List ; Stefan Roese > Subject: [EXT] Re: [U-Boot] [PATCH 4/5] cmd: mvebu: bubt: Add support for > legacy Marvell SoCs > > External Email > > --

Re: [U-Boot] [RESEND PATCH V2 3/3] imx: mx7: add system suspend/resume support

2018-08-29 Thread Stefano Babic
On 29/08/2018 15:31, Fabio Estevam wrote: > Hi Anson, > > On Wed, Aug 29, 2018 at 5:08 AM, Anson Huang wrote: >> Hi, all >> Can anyone pick up this patch set, it has been tested and >> suspend/resume is an important feature many people want, thanks! > > Thanks for your work on this. >

Re: [U-Boot] [PATCH V2 1/2] pci: Support parsing PCI controller DT subnodes

2018-08-29 Thread Bin Meng
+Simon Hi Marek, On Wed, Aug 29, 2018 at 10:22 PM Marek Vasut wrote: > > On 08/24/2018 08:27 PM, Marek Vasut wrote: > > The PCI controller can have DT subnodes describing extra properties > > of particular PCI devices, ie. a PHY attached to an EHCI controller > > on a PCI bus. This patch parses

Re: [U-Boot] [RESEND PATCH V2 3/3] imx: mx7: add system suspend/resume support

2018-08-29 Thread Stefan Agner
On 29.08.2018 16:43, Stefano Babic wrote: > On 29/08/2018 15:31, Fabio Estevam wrote: >> Hi Anson, >> >> On Wed, Aug 29, 2018 at 5:08 AM, Anson Huang wrote: >>> Hi, all >>> Can anyone pick up this patch set, it has been tested and >>> suspend/resume is an important feature many people wan

Re: [U-Boot] [PATCH V2 1/2] pci: Support parsing PCI controller DT subnodes

2018-08-29 Thread Marek Vasut
On 08/29/2018 05:15 PM, Bin Meng wrote: > +Simon > > Hi Marek, > > On Wed, Aug 29, 2018 at 10:22 PM Marek Vasut wrote: >> >> On 08/24/2018 08:27 PM, Marek Vasut wrote: >>> The PCI controller can have DT subnodes describing extra properties >>> of particular PCI devices, ie. a PHY attached to an

Re: [U-Boot] [U-Boot, 01/10] rockchip: enable DEBUG_UART_BOARD_INIT by default

2018-08-29 Thread Philipp Tomsich
> All Rockchip SoCs use DEBUG_UART_BOARD_INIT to init per board > UART IOMUX, enable it by default. > > Signed-off-by: Kever Yang > --- > > arch/arm/Kconfig | 2 ++ > arch/arm/mach-rockchip/Kconfig | 3 --- > 2 files changed, 2 insertions(+), 3 deletions(-) > Reviewed-by: Philip

Re: [U-Boot] [U-Boot, 04/10] rockchip: rk3188: add board_debug_uart_init()

2018-08-29 Thread Philipp Tomsich
> Use board_debug_uart_init() for UART iomux init instead of > do it in board_init_f, and move the function to soc file so > that we can find all the soc/board setting in soc file and > use a common board file. > > Signed-off-by: Kever Yang > --- > > arch/arm/mach-rockchip/rk3188-board-spl.c |

Re: [U-Boot] [U-Boot, 10/10] rockchip: rk3399: add board_debug_uart_init()

2018-08-29 Thread Philipp Tomsich
> Use board_debug_uart_init() for UART iomux init instead of > do it in board_init_f, and move the function to soc file so > that we can find all the soc/board setting in soc file and > use a common board file for all rockchip SoCs later. > > Signed-off-by: Kever Yang > --- > > arch/arm/mach-ro

Re: [U-Boot] [U-Boot, 05/10] rockchip: rk322x: move board_debug_uart_init() to rk322x.c

2018-08-29 Thread Philipp Tomsich
> Move the function to soc file so > that we can find all the soc/board setting in soc file and > use a common board file later for all rockchip SoCs. > > Signed-off-by: Kever Yang > --- > > arch/arm/mach-rockchip/rk322x-board-spl.c | 43 ++--- > arch/arm/mach-rockchip/r

Re: [U-Boot] [U-Boot,02/10] rockchip; kylin-rk3036: enabl DEBUG UART

2018-08-29 Thread Philipp Tomsich
> Enable debug uart for kylin board in defconfig. > > Signed-off-by: Kever Yang > --- > > configs/kylin-rk3036_defconfig | 4 > 1 file changed, 4 insertions(+) > Reviewed-by: Philipp Tomsich ___ U-Boot mailing list U-Boot@lists.denx.de https:/

Re: [U-Boot] [U-Boot, 03/10] rockchip: rk3036: add board_debug_uart_init()

2018-08-29 Thread Philipp Tomsich
> Use board_debug_uart_init() for UART iomux init instead of > do it in board_init_f, and move the function to soc file so > that we can find all the soc/board setting in soc file and > use a common board file. > > Signed-off-by: Kever Yang > --- > > arch/arm/mach-rockchip/rk3036-board-spl.c |

Re: [U-Boot] [U-Boot, 09/10] rockchip: rk3399: use grf structure to access reg

2018-08-29 Thread Philipp Tomsich
> Prefer to use structure to access register if we could. > > Signed-off-by: Kever Yang > --- > > arch/arm/mach-rockchip/rk3399/rk3399.c | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > Reviewed-by: Philipp Tomsich ___ U-Boot mailing l

Re: [U-Boot] [U-Boot, 06/10] rockchip: rk3288: use grf structure to access soc_con2

2018-08-29 Thread Philipp Tomsich
> Prefer to use structure to access register if we can. > > Signed-off-by: Kever Yang > --- > > arch/arm/mach-rockchip/rk3288/rk3288.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) > Reviewed-by: Philipp Tomsich ___ U-Boot mailing l

Re: [U-Boot] [U-Boot, 08/10] rockchip: rk3368: move board_debug_uart_init() to rk3368.c

2018-08-29 Thread Philipp Tomsich
> Move the function to soc file so > that we can find all the soc/board setting in soc file and > use a common board file later for all rockchip SoCs. > > Signed-off-by: Kever Yang > --- > > arch/arm/mach-rockchip/rk3368-board-spl.c | 5 - > arch/arm/mach-rockchip/rk3368-board-tpl.c | 33

Re: [U-Boot] [U-Boot, 07/10] rockchip: rk3288: add board_debug_uart_init()

2018-08-29 Thread Philipp Tomsich
> Use board_debug_uart_init() for UART iomux init instead of > do it in board_init_f, and move the function to soc file so > that we can find all the soc/board setting in soc file and > use a common board file for all rockchip SoCs later. > > Signed-off-by: Kever Yang > --- > > arch/arm/mach-ro

Re: [U-Boot] [U-Boot, 01/10] rockchip: enable DEBUG_UART_BOARD_INIT by default

2018-08-29 Thread Dr. Philipp Tomsich
Kever, Could you rebase this series to u-boot-rockchip/next and resubmit? I’d like to avoid any issues from manually fixing up git-am issues (as I happened to introduce on last set of UART changes). Thanks, Philipp. > On 29 Aug 2018, at 20:33, Philipp Tomsich > wrote: > >> All Rockchip SoCs

Re: [U-Boot] [U-Boot, 01/10] rockchip: enable DEBUG_UART_BOARD_INIT by default

2018-08-29 Thread Philipp Tomsich
> All Rockchip SoCs use DEBUG_UART_BOARD_INIT to init per board > UART IOMUX, enable it by default. > > Signed-off-by: Kever Yang > Reviewed-by: Philipp Tomsich > --- > > arch/arm/Kconfig | 2 ++ > arch/arm/mach-rockchip/Kconfig | 3 --- > 2 files changed, 2 insertions(+), 3 dele

Re: [U-Boot] [U-Boot, 04/10] rockchip: rk3188: add board_debug_uart_init()

2018-08-29 Thread Philipp Tomsich
> Use board_debug_uart_init() for UART iomux init instead of > do it in board_init_f, and move the function to soc file so > that we can find all the soc/board setting in soc file and > use a common board file. > > Signed-off-by: Kever Yang > Reviewed-by: Philipp Tomsich > --- > > arch/arm/mac

Re: [U-Boot] [U-Boot,02/10] rockchip; kylin-rk3036: enabl DEBUG UART

2018-08-29 Thread Philipp Tomsich
> Enable debug uart for kylin board in defconfig. > > Signed-off-by: Kever Yang > Reviewed-by: Philipp Tomsich > --- > > configs/kylin-rk3036_defconfig | 4 > 1 file changed, 4 insertions(+) > Acked-by: Philipp Tomsich ___ U-Boot mailing list

Re: [U-Boot] [U-Boot, 06/10] rockchip: rk3288: use grf structure to access soc_con2

2018-08-29 Thread Philipp Tomsich
> Prefer to use structure to access register if we can. > > Signed-off-by: Kever Yang > Reviewed-by: Philipp Tomsich > --- > > arch/arm/mach-rockchip/rk3288/rk3288.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) > Acked-by: Philipp Tomsich ___

Re: [U-Boot] [U-Boot, 09/10] rockchip: rk3399: use grf structure to access reg

2018-08-29 Thread Philipp Tomsich
> Prefer to use structure to access register if we could. > > Signed-off-by: Kever Yang > Reviewed-by: Philipp Tomsich > --- > > arch/arm/mach-rockchip/rk3399/rk3399.c | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > Acked-by: Philipp Tomsich ___

Re: [U-Boot] [U-Boot, 08/10] rockchip: rk3368: move board_debug_uart_init() to rk3368.c

2018-08-29 Thread Philipp Tomsich
> Move the function to soc file so > that we can find all the soc/board setting in soc file and > use a common board file later for all rockchip SoCs. > > Signed-off-by: Kever Yang > Reviewed-by: Philipp Tomsich > --- > > arch/arm/mach-rockchip/rk3368-board-spl.c | 5 - > arch/arm/mach-ro

Re: [U-Boot] [U-Boot, 05/10] rockchip: rk322x: move board_debug_uart_init() to rk322x.c

2018-08-29 Thread Philipp Tomsich
> Move the function to soc file so > that we can find all the soc/board setting in soc file and > use a common board file later for all rockchip SoCs. > > Signed-off-by: Kever Yang > Reviewed-by: Philipp Tomsich > --- > > arch/arm/mach-rockchip/rk322x-board-spl.c | 43 ++---

Re: [U-Boot] [U-Boot, 10/10] rockchip: rk3399: add board_debug_uart_init()

2018-08-29 Thread Philipp Tomsich
> Use board_debug_uart_init() for UART iomux init instead of > do it in board_init_f, and move the function to soc file so > that we can find all the soc/board setting in soc file and > use a common board file for all rockchip SoCs later. > > Signed-off-by: Kever Yang > Reviewed-by: Philipp Tomsi

Re: [U-Boot] rockchip: dts: fix unnecessary '-cells' warning

2018-08-29 Thread Philipp Tomsich
> Fix warning below: > unnecessary #address-cells/#size-cells without "ranges" or child "reg" > property > > Signed-off-by: Kever Yang > Reviewed-by: Philipp Tomsich > --- > > arch/arm/dts/rk3288-evb.dtsi | 2 -- > arch/arm/dts/rk3288-firefly.dtsi | 2 -- > arch/arm/dts/rk3288-tinker.dtsi

Re: [U-Boot] [U-Boot, 03/10] rockchip: rk3036: add board_debug_uart_init()

2018-08-29 Thread Philipp Tomsich
> Use board_debug_uart_init() for UART iomux init instead of > do it in board_init_f, and move the function to soc file so > that we can find all the soc/board setting in soc file and > use a common board file. > > Signed-off-by: Kever Yang > Reviewed-by: Philipp Tomsich > --- > > arch/arm/mac

[U-Boot] [PATCH 1/2] ARM: tegra: reserve unmapped RAM so EFI doesn't use it

2018-08-29 Thread Stephen Warren
From: Stephen Warren Tegra U-Boot ensures that board_get_usable_ram_top() never returns a value over 4GB, since some peripherals can't access such addresses. However, on systems with more than 2GB of RAM, RAM bank 1 does describe this extra RAM, so that Linux (or whatever OS) can use it, subject

[U-Boot] [PATCH 2/2] Revert "Revert "efi_loader: efi_allocate_pages is too restrictive""

2018-08-29 Thread Stephen Warren
From: Stephen Warren This reverts commit ccfc78b820e5e431c5bd73b072e7536a972e1710. Now that the underlying issue is fixed, we can revert the revert and hence restore the original EFI code. Signed-off-by: Stephen Warren --- lib/efi_loader/efi_memory.c | 2 +- 1 file changed, 1 insertion(+), 1

Re: [U-Boot] [PATCH] rockchip: add TPL_TINY_FRAMEWORK support

2018-08-29 Thread Dr. Philipp Tomsich
> On 28 Aug 2018, at 11:23, Kever Yang wrote: > > Rockchip BootRom support load firmware from storage media twice, > one for ddr sdram init code into sram and another time to load > firmware into DDR. For ddr sdram init code(which is TPL in U-Boot > project), it's OK to re-use the stack and vect

Re: [U-Boot] [PATCH 1/2] ARM: tegra: reserve unmapped RAM so EFI doesn't use it

2018-08-29 Thread Heinrich Schuchardt
On 08/29/2018 08:53 PM, Stephen Warren wrote: > From: Stephen Warren > > Tegra U-Boot ensures that board_get_usable_ram_top() never returns a value > over 4GB, since some peripherals can't access such addresses. However, on > systems with more than 2GB of RAM, RAM bank 1 does describe this extra

Re: [U-Boot] [PATCH 1/2] ARM: tegra: reserve unmapped RAM so EFI doesn't use it

2018-08-29 Thread Alexander Graf
On 29.08.18 22:52, Heinrich Schuchardt wrote: > On 08/29/2018 08:53 PM, Stephen Warren wrote: >> From: Stephen Warren >> >> Tegra U-Boot ensures that board_get_usable_ram_top() never returns a value >> over 4GB, since some peripherals can't access such addresses. However, on >> systems with more

[U-Boot] [PATCH V2 2/2] Revert "Revert "efi_loader: efi_allocate_pages is too restrictive""

2018-08-29 Thread Stephen Warren
From: Stephen Warren This reverts commit ccfc78b820e5e431c5bd73b072e7536a972e1710. Now that the underlying issue is fixed, we can revert the revert and hence restore the original EFI code. Signed-off-by: Stephen Warren --- v2: No change. --- lib/efi_loader/efi_memory.c | 2 +- 1 file changed,

[U-Boot] [PATCH V2 1/2] ARM: tegra: reserve unmapped RAM so EFI doesn't use it

2018-08-29 Thread Stephen Warren
From: Stephen Warren Tegra U-Boot ensures that board_get_usable_ram_top() never returns a value over 4GB, since some peripherals can't access such addresses. However, on systems with more than 2GB of RAM, RAM bank 1 does describe this extra RAM, so that Linux (or whatever OS) can use it, subject

Re: [U-Boot] [RFC 1/3] test/py: convert fs-test.sh to pytest

2018-08-29 Thread Heinrich Schuchardt
On 08/23/2018 09:25 AM, AKASHI Takahiro wrote: > In this commit, the same set of test cases as in test/fs/fs-test.sh > is provided using pytest framework. > Actually, fs-test.sh provides three variants:"sb" (sb command), "nonfs" > (fatxx and etc.) and "fs" (hostfs), and this patch currently support

Re: [U-Boot] [PATCH V2 1/2] ARM: tegra: reserve unmapped RAM so EFI doesn't use it

2018-08-29 Thread Heinrich Schuchardt
On 08/29/2018 11:34 PM, Stephen Warren wrote: > From: Stephen Warren > > Tegra U-Boot ensures that board_get_usable_ram_top() never returns a value > over 4GB, since some peripherals can't access such addresses. However, on > systems with more than 2GB of RAM, RAM bank 1 does describe this extra

Re: [U-Boot] [PATCH V2 1/2] pci: Support parsing PCI controller DT subnodes

2018-08-29 Thread Alexander Graf
On 29.08.18 19:07, Marek Vasut wrote: > On 08/29/2018 05:15 PM, Bin Meng wrote: >> +Simon >> >> Hi Marek, >> >> On Wed, Aug 29, 2018 at 10:22 PM Marek Vasut wrote: >>> >>> On 08/24/2018 08:27 PM, Marek Vasut wrote: The PCI controller can have DT subnodes describing extra properties of

Re: [U-Boot] [PATCH v4 02/17] reset: Return 0 if no request ops

2018-08-29 Thread Simon Glass
Hi Jagan, On 26 August 2018 at 06:38, Jagan Teki wrote: > Missing request ops from respective uclass driver > generating "synchronous abort" in Allwinner platform, > may be in arm. So return 0 if request ops is not used > for those uclass drivers. > > Cc: Simon Glass > Signed-off-by: Jagan Teki

Re: [U-Boot] [PATCH 1/1] test: fix typo in cmd_ut_category() description

2018-08-29 Thread Simon Glass
On 27 August 2018 at 14:04, Heinrich Schuchardt wrote: > > argc = 1: all tests are run > argc > 1: only argv[1] is run > > So we need argc >= 1. > > Signed-off-by: Heinrich Schuchardt > --- > include/test/suites.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Simon Glass

Re: [U-Boot] [PATCH v2 2/8] powerpc/dts: Makefile changes to clean and build dts

2018-08-29 Thread Simon Glass
On 28 August 2018 at 08:49, Jagdish Gediya wrote: > > Signed-off-by: Jagdish Gediya > --- > arch/powerpc/dts/Makefile | 12 > dts/Makefile | 2 +- > 2 files changed, 13 insertions(+), 1 deletion(-) > create mode 100644 arch/powerpc/dts/Makefile Reviewed-by: Simon Gla

Re: [U-Boot] [PATCH v2 14/15] tee: optee: support AVB trusted application

2018-08-29 Thread Simon Glass
Hi Jens, On 23 August 2018 at 04:43, Jens Wiklander wrote: > Adds configuration option OPTEE_TA_AVB and a header file describing the > interface to the AVB trusted application provided by OP-TEE. What is AVB? Can you please write it out in full? > > Tested-by: Igor Opaniuk > Reviewed-by: Igor

Re: [U-Boot] [PATCH v2 01/15] dm: fdt: scan for devices under /firmware too

2018-08-29 Thread Simon Glass
On 23 August 2018 at 04:43, Jens Wiklander wrote: > Just as /chosen may contain devices /firmware may contain devices, scan > for devices under /firmware too. > > Signed-off-by: Jens Wiklander > --- > drivers/core/root.c | 15 ++- > 1 file changed, 10 insertions(+), 5 deletions(-) R

Re: [U-Boot] [PATCH v2 03/15] cmd: avb: print error message if command fails

2018-08-29 Thread Simon Glass
On 23 August 2018 at 04:43, Jens Wiklander wrote: > Signed-off-by: Jens Wiklander > --- > cmd/avb.c | 17 + > 1 file changed, 17 insertions(+) Reviewed-by: Simon Glass ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de

Re: [U-Boot] [PATCH v2 02/15] cmd: avb read_rb: print rb_idx in hexadecimal

2018-08-29 Thread Simon Glass
On 23 August 2018 at 04:43, Jens Wiklander wrote: > Prior to this patch was do_avb_write_rb() reading supplied rb_idx as a > hexadecimal number while do_avb_read_rb() printed the read out rb_idx as > decimal number. For consistency change do_avb_read_rb() to print rb_idx > as a hexadecimal number

Re: [U-Boot] [PATCH v2 11/15] configs: sandbox: enable CONFIG_TEE (TEE uclass)

2018-08-29 Thread Simon Glass
On 23 August 2018 at 04:43, Jens Wiklander wrote: > Signed-off-by: Jens Wiklander > --- > configs/sandbox64_defconfig| 1 + > configs/sandbox_defconfig | 1 + > configs/sandbox_flattree_defconfig | 1 + > configs/sandbox_noblk_defconfig| 1 + > configs/sandbox_spl_defconfig

Re: [U-Boot] [PATCH v2 07/15] tee: add OP-TEE driver

2018-08-29 Thread Simon Glass
Hi Jens, On 23 August 2018 at 04:43, Jens Wiklander wrote: > Adds a OP-TEE driver. > > * Targets ARM and ARM64 > * Supports using any u-boot memory as shared memory U-Boot Please use this consistent. > * Probes OP-TEE version using SMCs > * Uses OPTEE message protocol version 2 to communicate

Re: [U-Boot] [PATCH 4/8] binman: Add support for powerpc 'bootpg + resetvec' entry

2018-08-29 Thread Simon Glass
Hi, On 24 August 2018 at 00:02, Jagdish Gediya wrote: > Hi Simon, > > I tried to run 'binman -T' on top of latest u-boot without any changes to > understand it. I am facing below errors, Am i missing any prerequisite? > > $./tools/binman/binman -T > FATAL ERROR: Unrecognized check name "unit_add

Re: [U-Boot] [PATCH v2 05/15] Add UCLASS_TEE for Trusted Execution Environment

2018-08-29 Thread Simon Glass
Hi Jens, On 23 August 2018 at 04:43, Jens Wiklander wrote: > Adds a uclass to interface with a TEE (Trusted Execution Environment). > > A TEE driver is a driver that interfaces with a trusted OS running in > some secure environment, for example, TrustZone on ARM cpus, or a > separate secure co-pr

Re: [U-Boot] [PATCH 2/4] efi: stub: Pass EFI system table address to U-Boot payload

2018-08-29 Thread Simon Glass
On 23 August 2018 at 09:24, Bin Meng wrote: > This updates the EFI stub codes to pass UEFI BIOS's system table > address to U-Boot payload so that U-Boot can utilize it. > > Signed-off-by: Bin Meng > --- > > include/efi.h | 10 ++ > lib/efi/efi_stub.c | 4 > 2 files changed, 1

Re: [U-Boot] [PATCH V2 2/2] pci: Update documentation to make 'compatible' string optional

2018-08-29 Thread Simon Glass
Hi Marek, On 24 August 2018 at 12:27, Marek Vasut wrote: > Reword the documentation to make it clear the compatible string is now > optional, yet still matching on it takes precedence over PCI IDs and > PCI classes. > > Signed-off-by: Marek Vasut > Cc: Simon Glass > Cc: Tom Rini > --- > V2: Ne

Re: [U-Boot] [PATCH 4/4] x86: efi: payload: Turn on acpi in the kernel command line

2018-08-29 Thread Simon Glass
On 23 August 2018 at 09:24, Bin Meng wrote: > Now that we have full Linux kernel boot support on EFI payload, > avoid pass "acpi=off" to the kernel command line. Why? > > Signed-off-by: Bin Meng > --- > > include/configs/x86-common.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Rev

Re: [U-Boot] [PATCH] x86: zimage: Remove acpi_rsdp_addr propagation to kernel boot parameters

2018-08-29 Thread Simon Glass
On 24 August 2018 at 03:18, Andy Shevchenko wrote: > On Thu, 2018-08-23 at 21:28 -0700, Bin Meng wrote: >> As of today, the proposal of adding "acpi_rsdp_addr" to the kernel >> boot protocol does not make its way to the kernel mainline. This >> creates some confusion if we leave it in the U-Boot c

Re: [U-Boot] [PATCH] pci: Support parsing PCI controller DT subnodes

2018-08-29 Thread Simon Glass
Hi Marek, On 23 August 2018 at 06:58, Marek Vasut wrote: > On 08/23/2018 12:45 PM, Simon Glass wrote: > [...] Why is pci_bus_find_devfn() failing? >>> >>> Because this function is a hack to force-bind drivers to PCI devices >>> which are described in the DT with a compatible string. This doe

Re: [U-Boot] [PATCH 1/4] x86: efi: payload: Install E820 map from EFI memory map

2018-08-29 Thread Simon Glass
On 23 August 2018 at 09:24, Bin Meng wrote: > This implements payload-specific install_e820_map() to get E820 map > from the EFI memory map descriptors. > > Signed-off-by: Bin Meng > --- > > arch/x86/cpu/efi/payload.c | 82 > ++ > 1 file changed, 82 i

Re: [U-Boot] [PATCH 3/4] x86: zimage: Support booting Linux kernel from an EFI payload

2018-08-29 Thread Simon Glass
On 23 August 2018 at 09:24, Bin Meng wrote: > At present Linux kernel loaded from U-Boot as an EFI payload does > not boot. This fills in kernel's boot params structure with the > required critical EFI information like system table address and > memory map stuff so that kernel can obtain essential

Re: [U-Boot] [PATCH v2 09/15] test: tee: test TEE uclass

2018-08-29 Thread Simon Glass
On 23 August 2018 at 04:43, Jens Wiklander wrote: > Tests the TEE uclass with a sandbox tee driver. > > Signed-off-by: Jens Wiklander > --- > drivers/tee/Kconfig | 5 +- > test/dm/Makefile| 1 + > test/dm/tee.c | 182 > 3 files changed,

Re: [U-Boot] [PATCH v2 10/15] sandbox: dt: add sandbox_tee node

2018-08-29 Thread Simon Glass
On 23 August 2018 at 04:43, Jens Wiklander wrote: > Adds a sandbox_tee node to enable the sandbox tee driver in all the > sandbox dts files. > > Signed-off-by: Jens Wiklander > --- > arch/sandbox/dts/sandbox.dts | 4 > arch/sandbox/dts/sandbox64.dts | 4 > arch/sandbox/dts/test.dts

Re: [U-Boot] [PATCH v2 08/15] Documentation: tee uclass and op-tee driver

2018-08-29 Thread Simon Glass
Hi Jens, On 23 August 2018 at 04:43, Jens Wiklander wrote: > Signed-off-by: Jens Wiklander > --- > doc/README.tee | 112 + > 1 file changed, 112 insertions(+) > create mode 100644 doc/README.tee Reviewed-by: Simon Glass Looks good, nits below.

Re: [U-Boot] [PATCH v2 12/15] arm: dt: hikey: Add optee node

2018-08-29 Thread Simon Glass
On 23 August 2018 at 04:43, Jens Wiklander wrote: > Sync with 14e21cb8f811 ("arm64: dt: hikey: Add optee node" > from Linux kernel. > > Signed-off-by: Jens Wiklander > --- > arch/arm/dts/hi6220-hikey.dts | 7 +++ > 1 file changed, 7 insertions(+) Reviewed-by: Simon Glass __

Re: [U-Boot] [PATCH] image-fdt: reserve lmb for fdt

2018-08-29 Thread Simon Glass
On 7 August 2018 at 05:44, Andy Yan wrote: > Memory region for fdt should be reserved, or they > may be allocated by other module via lmb_alloc. > Then the fdt data will be destroy. > > We found a case on a board with 64MB DRAM like bellow: > > No ethernet found. > Hit any key to stop autoboot: 0

Re: [U-Boot] [PATCH 3/3] bootcount: add DM-based backing store for bootcount

2018-08-29 Thread Simon Glass
Hi Philipp, On 23 August 2018 at 07:48, Dr. Philipp Tomsich wrote: > Simon, > >> On 23 Aug 2018, at 12:45, Simon Glass wrote: >> >> Hi Philipp, >> >> On 17 August 2018 at 06:56, Dr. Philipp Tomsich >> wrote: >>> Simon, >>> >>> On 17 Aug 2018, at 14:49, Simon Glass wrote: >>> >>> Hi Philipp, >>

Re: [U-Boot] [PATCH v2 13/15] optee: support routing of rpmb data frames to mmc

2018-08-29 Thread Simon Glass
Hi Jens, On 23 August 2018 at 04:43, Jens Wiklander wrote: > Adds support in optee supplicant to route signed (MACed) RPMB frames > from OP-TEE Secure OS to MMC and vice versa to manipulate the RPMB > partition. > > Tested-by: Igor Opaniuk > Signed-off-by: Jens Wiklander > --- > drivers/tee/op

Re: [U-Boot] [PATCH v4] gpio: dwapb_gpio: Add reset ctrl to driver

2018-08-29 Thread Ley Foon Tan
On Wed, Aug 29, 2018 at 7:57 PM Marek Vasut wrote: > > On 08/29/2018 10:44 AM, Ley Foon Tan wrote: > > Add code to reset all reset signals as in gpio DT node. A reset property > > is an optional feature, so only print out a warning and do not fail if a > > reset property is not present. > > > > If

Re: [U-Boot] [U-Boot, 01/10] rockchip: enable DEBUG_UART_BOARD_INIT by default

2018-08-29 Thread Kever Yang
Philipp, Sure, I can do it. Thanks, - Kever On 08/30/2018 02:43 AM, Dr. Philipp Tomsich wrote: > Kever, > > Could you rebase this series to u-boot-rockchip/next and resubmit? > I’d like to avoid any issues from manually fixing up git-am issues (as I > happened to introduce on last set of UART c

Re: [U-Boot] [PATCH 4/4] x86: efi: payload: Turn on acpi in the kernel command line

2018-08-29 Thread Bin Meng
Hi Simon, On Thu, Aug 30, 2018 at 8:29 AM Simon Glass wrote: > > On 23 August 2018 at 09:24, Bin Meng wrote: > > Now that we have full Linux kernel boot support on EFI payload, > > avoid pass "acpi=off" to the kernel command line. > > Why? > This is due to EFI payload does not have CONFIG_GENERA

  1   2   >