Aw: [PATCH 1/1] arm: mediatek: remove unused binman config

2020-03-07 Thread Frank Wunderlich
Have tested this Patch, fixes the binman-issue on bpi-r2/bpi-r64 Tested-by: Frank Wunderlich @tom can you merge this until next rc? regards Frank

[PATCH v2 1/1] image.h: use uint32_t instead of u32 in android_image_get_dtb*

2020-03-07 Thread Heinrich Schuchardt
From: Eugeniu Rosca Replace 'u32' by 'uint32_t' in image.h, since the former may lead to build failures in U-Boot tooling (see [1]). Avoid using 'uint', since it is not a fixed-width type [2], potentially leading to a dangerous mismatch between the prototypes and definitions of the android_image

Re: [PATCH v2 1/1] image.h: use uint32_t instead of u32 in android_image_get_dtb*

2020-03-07 Thread Eugeniu Rosca
Hello Heinrich, Thank you for your perseverance. I was just about to reply in https://patchwork.ozlabs.org/patch/1239098. On Sat, Mar 07, 2020 at 10:53:44AM +0100, Heinrich Schuchardt wrote: > From: Eugeniu Rosca > > Replace 'u32' by 'uint32_t' in image.h, since the former may lead to > build f

Re: [PATCH v4 03/11] clk: actions: Add common clock driver

2020-03-07 Thread André Przywara
Hi, On 07/03/2020 06:42, Amit Singh Tomar wrote: > This patch converts S900 clock driver to something common that can > be used for other SoCs, for instance S700(few of clk registers are same). > > Signed-off-by: Amit Singh Tomar > --- > Changes since v3: > * Fixed register spelling. >

Re: [PATCH v4 03/11] clk: actions: Add common clock driver

2020-03-07 Thread Amit Tomer
Hi, > Both those include files do not exist yet. This breaks bisectability for > bubblegum_96. > So I would suggest you remove the s700 lines for now, and change the > s900 filename to match the existing one. > Then change CLK_UART to CLOCK_UART below to make it compile. > > Please check that bubb

[PATCH] mmc: tmio: sdhi: Add DMA transfer address alignment check at writing

2020-03-07 Thread Marek Vasut
From: Hiroyuki Yokoyama In R-Car Gen 3, there is a DMA controller restriction of SDHI. When the transfer exceeding the 4 kByte boundary is performed while the DRAM address is not 128 byte aligned, the bus is occupied. This patch avoids this. Signed-off-by: Hiroyuki Yokoyama Signed-off-by: Marek

[PATCH 1/5] common: bouncebuf: Permit passing custom alignment check function

2020-03-07 Thread Marek Vasut
Add extended version of the bounce_buffer_start(), which permits passing in a custom alignment checker function for the buffer. This is useful e.g. on systems with various DMA restrictions and where the checker function might be more complex than a simple CPU cache alignment check. Signed-off-by:

[PATCH 3/5] mmc: tmio: sdhi: Use bounce buffer to avoid DMA limitations

2020-03-07 Thread Marek Vasut
The R-Car SDHI DMA controller has various restrictions. To work around those restrictions without falling back to PIO, implement bounce buffer with custom alignment check function which tests for those limitations. Signed-off-by: Marek Vasut Cc: Daniel Schwierzeck Cc: Masahiro Yamada Cc: Peng F

[PATCH 2/5] ARM: rmobile: Increase malloc area size

2020-03-07 Thread Marek Vasut
Increase the malloc area size significantly to cater for bounce buffer used by the SDHI driver. Signed-off-by: Marek Vasut Cc: Daniel Schwierzeck Cc: Masahiro Yamada Cc: Peng Fan Cc: Simon Glass Cc: Tom Rini --- include/configs/rcar-gen3-common.h | 2 +- 1 file changed, 1 insertion(+), 1 de

[PATCH 4/5] mmc: Add option to adjust b_max before long read

2020-03-07 Thread Marek Vasut
Add getter function which permits adjusting the maximum number of blocks that could be read in a single sustained read transfer based on the location of the source/target buffer and length, before such transfer starts. This is mainly useful on systems which have various DMA restrictions for differ

[PATCH 5/5] mmc: tmio: sdhi: Implement get_b_max function

2020-03-07 Thread Marek Vasut
Implement get_b_max() for the Renesas R-Car SDHI controller driver, limit the b_max per hardware capabilities such that select Gen2 controllers have 16bit block transfer limit, the rest has 32bit block transfer limit and on Gen3, the block transfer limit on addresses above the 32bit boundary is set

Re: [PATCH v4 03/11] clk: actions: Add common clock driver

2020-03-07 Thread André Przywara
On 07/03/2020 15:30, Amit Tomer wrote: > Hi, > >> Both those include files do not exist yet. This breaks bisectability for >> bubblegum_96. >> So I would suggest you remove the s700 lines for now, and change the >> s900 filename to match the existing one. >> Then change CLK_UART to CLOCK_UART belo

Re: [PATCH v2 1/1] image.h: use uint32_t instead of u32 in android_image_get_dtb*

2020-03-07 Thread Heinrich Schuchardt
On 3/7/20 1:52 PM, Eugeniu Rosca wrote: Hello Heinrich, Thank you for your perseverance. I was just about to reply in https://patchwork.ozlabs.org/patch/1239098. On Sat, Mar 07, 2020 at 10:53:44AM +0100, Heinrich Schuchardt wrote: From: Eugeniu Rosca Replace 'u32' by 'uint32_t' in image.h, s

[PATCH v3 1/7] x86: fsp: Allow skipping init code when chain loading

2020-03-07 Thread Simon Glass
It is useful to be able to boot the same x86 image on a device with or without a first-stage bootloader. For example, with chromebook_coral, it is helpful for testing to be able to boot the same U-Boot (complete with FSP) on bare metal and from coreboot. It allows checking of things like CPU speed,

[PATCH v3 0/7] x86: Improve support for chain-loading U-Boot

2020-03-07 Thread Simon Glass
This little series adds a few checks into the code to allow better operation when booting a build from a previous-state loader such as coreboot. At present we have a 'coreboot' target but this runs very different code from the bare-metal targets, such as coral. There is very little in common betwe

[PATCH v3 3/7] x86: cpu: Skip init code when chain loading

2020-03-07 Thread Simon Glass
When U-Boot is not the first-stage bootloader the interrupt and cache init must be skipped, as well as init for various peripherals. Update the code to add checks for this. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: - Drop the other check in interrupt_init() which is not n

[PATCH v3 7/7] x86: Use the existing stack when chain-loading

2020-03-07 Thread Simon Glass
With chromebook_coral we normally run TPL->SPL->U-Boot. This is the 'bare metal' case. When running from coreboot we put u-boot.bin in the RW_LEGACY portion of the image, e.g. with: cbfstool image-coral.serial.bin add-flat-binary -r RW_LEGACY \ -f /tmp/b/chromebook_coral/u-boot.bin -n

[PATCH v3 5/7] board: Add a gd flag for chain loading

2020-03-07 Thread Simon Glass
When U-Boot is run from another boot loader, much of the low-level init needs to be skipped. Add a flag for this and adjust ll_boot_init() to use it. Signed-off-by: Simon Glass --- Changes in v3: - Add a new patch with a gd flag for chain loading Changes in v2: None include/asm-generic/globa

[PATCH v3 2/7] x86: apl: Skip init code when chain loading

2020-03-07 Thread Simon Glass
When U-Boot is not the first-stage bootloader the FSP-S init must be skipped. Update it to add a check. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: None arch/x86/cpu/apollolake/fsp_s.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/x86/cpu/apollolake/fsp_s.c b

[PATCH v3 6/7] x86: Add a way to detect running from coreboot

2020-03-07 Thread Simon Glass
If U-Boot is running from coreboot we need to skip low-level init. Add an way to detect this and to set the gd flag. Signed-off-by: Simon Glass --- Changes in v3: - Add new patch to detect running from coreboot Changes in v2: None arch/x86/cpu/i386/cpu.c | 15 +++ arch/x

[PATCH v3 4/7] pci: Avoid auto-config when chain loading

2020-03-07 Thread Simon Glass
When U-Boot is not the first-stage bootloader we don't want to re-configure the PCI devices, since this has already been done. Add a check to avoid this. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: - Drop patch 'dm: Avoid initing built-in devices when chain loading' drive

[PATCH] image.h: isolate android_image_* functions from tooling

2020-03-07 Thread Eugeniu Rosca
On Feb. 16, 2020, Tom reported [1] build failure of U-Boot in-tree tooling after applying https://patchwork.ozlabs.org/cover/1229663/ ("[v6,0/7] rsa: extend rsa_verify() for UEFI secure boot"). Later on, Heinrich stressed the urgency of the issue in https://patchwork.ozlabs.org/patch/1250858/#2379

Re: [PATCH v2 1/1] image.h: use uint32_t instead of u32 in android_image_get_dtb*

2020-03-07 Thread Eugeniu Rosca
Hello Heinrich, On Sat, Mar 07, 2020 at 09:31:17PM +0100, Heinrich Schuchardt wrote: > Whichever way you want to go we should finalize the topic as it stops > EFI patches from being merged. Could you please review https://patchwork.ozlabs.org/patch/1250963/ ? -- Best Regards Eugeniu Rosca

Re: [PATCH 2/4] mx6cuboxi: enable MMC iomux on board_early_init_f

2020-03-07 Thread Baruch Siach
Hi Walter, Thanks for your SPL_DM support work on this platform. One comment inline below. On Wed, Mar 04 2020, Walter Lozano wrote: > MMC iomux is done on board_mmc_init which is valid when DM_MMC is not > enabled. After enabling it, the iomux setup needs to be moved to a > valid place. > > Thi

Re: [PATCH 3/4] mx6cuboxi: customize board_boot_order to access eMMC

2020-03-07 Thread Baruch Siach
Hi Walter, On Wed, Mar 04 2020, Walter Lozano wrote: > In SPL legacy code only one MMC device is created, based on BOOT_CFG > register, which can be either SD or eMMC. In this context > board_boot_order return always MMC1 when configure to boot from > SD/eMMC. After switching to DM both SD and eMM

RE: [PATCH v4 00/14] RISC-V SiFive FU540 support SPL

2020-03-07 Thread Pragnesh Patel
Hi, >-Original Message- >From: Palmer Dabbelt >Sent: 06 March 2020 22:32 >To: Pragnesh Patel >Cc: u-boot@lists.denx.de; Atish Patra ; >bmeng...@gmail.com; Paul Walmsley ; >ja...@amarulasolutions.com; Troy Benjegerdes >; Anup Patel ; Sagar >Kadam ; Pragnesh Patel > >Subject: Re: [PATCH v4