Re: [PATCH v7 01/22] misc: add driver for the SiFive otp controller

2020-05-10 Thread Jagan Teki
On Sat, May 2, 2020 at 9:48 PM Pragnesh Patel wrote: > > Hi Jagan, > > >-Original Message- > >From: Jagan Teki > >Sent: 02 May 2020 21:21 > >To: Pragnesh Patel > >Cc: U-Boot-Denx ; Atish Patra > >; Palmer Dabbelt ; Bin > >Meng ; Paul Walmsley ; > >Troy Benjegerdes ; Anup Patel > >; Sagar

Re: [PATCH][v4 6/6] board: tbs2910: add documentation

2020-05-10 Thread Soeren Moch
On 09.05.20 19:36, Denis 'GNUtoo' Carikli wrote: > On Thu, 7 May 2020 20:12:19 +0200 > Soeren Moch wrote: >> do you plan to re-send a fixed version of this series? > I definitely do. OK. I look forward to your new series. > > I'm really sorry for the delay. No problem, it is not that urgent. I jus

Re: [PATCH v7 14/22] riscv: Add place-holder for driver compilation

2020-05-10 Thread Jagan Teki
On Sun, May 3, 2020 at 2:47 PM Pragnesh Patel wrote: > > Hi Jagan, > > >-Original Message- > >From: Jagan Teki > >Sent: 02 May 2020 21:58 > >To: Pragnesh Patel > >Cc: U-Boot-Denx ; Atish Patra > >; Palmer Dabbelt ; Bin > >Meng ; Paul Walmsley ; > >Troy Benjegerdes ; Anup Patel > >; Sagar

Re: [PATCH v8 18/21] configs: fu540: Add config options for U-Boot SPL

2020-05-10 Thread Jagan Teki
On Sat, May 9, 2020 at 8:02 PM Pragnesh Patel wrote: > > With sifive_fu540_defconfig: > > User can use FSBL or u-boot-spl.bin anyone at a time. > > For FSBL, > fsbl->fw_payload.bin (opensbi + U-Boot) > > For u-boot-spl.bin, > u-boot-spl.bin->FIT image (opensbi + U-Boot + dtb) s/U-Boot/U-Boot prop

Re: [PATCH v7 19/22] sifive: dts: fu540: Enable L2 Cache in U-Boot

2020-05-10 Thread Jagan Teki
On Sun, May 3, 2020 at 12:57 PM Pragnesh Patel wrote: > > Hi jagan, > > >-Original Message- > >From: Jagan Teki > >Sent: 02 May 2020 22:43 > >To: Pragnesh Patel > >Cc: U-Boot-Denx ; Atish Patra > >; Palmer Dabbelt ; Bin > >Meng ; Paul Walmsley ; > >Troy Benjegerdes ; Anup Patel > >; Saga

Re: [PATCH] cmd: avb: free partition buffer upon verify completion

2020-05-10 Thread Igor Opaniuk
Hi Gary, Thanks for the patch and sorry for a late reply. On Wed, May 6, 2020 at 5:06 PM Gary Bisson wrote: > > Allows to run 'avb verify' multiple times which can be useful after a > failure to be able to re-flash the partition and try again. > > Signed-off-by: Gary Bisson > --- > Hi, > > This

Re: [PATCH 5/8] efi_loader: Make the pkcs7 header parsing function an extern

2020-05-10 Thread Sughosh Ganu
On Fri, 8 May 2020 at 06:21, Akashi Takahiro wrote: > On Thu, May 07, 2020 at 04:48:30PM +0530, Sughosh Ganu wrote: > > On Thu, 7 May 2020 at 13:04, Akashi Takahiro > > > wrote: > > > > > On Thu, Apr 30, 2020 at 11:06:27PM +0530, Sughosh Ganu wrote: > > > > The pkcs7 header parsing functionality

Re: [PATCH 6/8] efi: capsule: Add support for uefi capsule authentication

2020-05-10 Thread Sughosh Ganu
On Fri, 8 May 2020 at 06:12, Akashi Takahiro wrote: > On Thu, May 07, 2020 at 05:20:35PM +0530, Sughosh Ganu wrote: > > On Thu, 7 May 2020 at 13:49, Akashi Takahiro > > > wrote: > > > > > Sughosh, > > > > > > On Thu, Apr 30, 2020 at 11:06:28PM +0530, Sughosh Ganu wrote: > > > > Add support for a

Re: [PATCH 3/8] qemu: arm64: Add support for efi firmware management protocol routines

2020-05-10 Thread Sughosh Ganu
On Sat, 9 May 2020 at 15:34, Heinrich Schuchardt wrote: > On 5/6/20 5:04 PM, Grant Likely wrote: > > > > > > On 05/05/2020 18:57, Heinrich Schuchardt wrote: > >> On 05.05.20 19:23, Grant Likely wrote: > >>> > >>> > >>> On 05/05/2020 18:04, Heinrich Schuchardt wrote: > On 05.05.20 13:15, Gran

Re: [PATCH 1/1] tools: ftdgrep: use /* fallthrough */ as needed

2020-05-10 Thread Masahiro Yamada
On Sun, May 10, 2020 at 12:12 AM Heinrich Schuchardt wrote: > > GCC recognizes /* fallthrough */ if -Wimplicit-fallthrough=3 is enabled. FYI. Linux decided to not use /* fallthrough */ any more because Clang does not recognize it. __attribute__((__fallthrough__)) is supported by both Clang and

Re: rk3399 single channel lpddr4 setup

2020-05-10 Thread Jagan Teki
Hi Jack, On Fri, Apr 24, 2020 at 1:08 AM Jack Mitchell wrote: > > On 23/04/2020 12:38, Jagan Teki wrote: > > On Tue, Apr 21, 2020 at 7:54 PM Jack Mitchell wrote: > >> > >> Has anyone worked with an rk3399 based board with a single channel RAM > >> setup? I've looked at the current code and made

Re: [PATCH 2/2] odroid-c2: enable USB host controller

2020-05-10 Thread Beniamino Galvani
On Sun, Aug 18, 2019 at 03:42:55PM +0200, Beniamino Galvani wrote: > Enable the second USB controller, which is connected to a hub with 4 > ports. The first controller is for the OTG port and is currently not > supported. > > Signed-off-by: Beniamino Galvani > --- > arch/arm/dts/meson-gxbb-odroi

Re: Pull request: u-boot-spi/master

2020-05-10 Thread Jagan Teki
On Sun, May 10, 2020 at 2:28 AM Tom Rini wrote: > > On Sat, May 09, 2020 at 09:56:09PM +0530, Jagan Teki wrote: > > > Hi Tom, > > > > Please pull this PR. > > > > Summary: > > - zap lpc32xx_ssp driver (Jagan) > > - rename of phy nodev call (Jagan) > > - iopoll with sleep_us (Jagan) > > - MX25R6435

RE: [PATCH] imx8mp_evk: Make SPL binary size smaller

2020-05-10 Thread Peng Fan
> Subject: Re: [PATCH] imx8mp_evk: Make SPL binary size smaller > > On Thu, May 7, 2020 at 8:59 AM Fabio Estevam > wrote: > > > > Commit f24dea4e1b52 ("ARM: imx8m: Fix reset in SPL on NXP iMX8MP > EVK") > > caused the u-boot-spl.bin binary size to grow by around 2000 bytes, > > which makes the bo

[PATCH V2] usb: ehci-omap: Add Support DM_USB and OF_CONTROL

2020-05-10 Thread Adam Ford
The omap3.dtsi file shows the usbhshost node with two sub-nodes for ohci and ehci. This patch file creates the usbhshost, and pulls the portX-mode information. It then locates the EHCI sub-node, and initializes the EHCI controller with the info pulled from the usbhshost node. There is still more

Re: [PATCH] imx8mp_evk: Make SPL binary size smaller

2020-05-10 Thread Marek Vasut
On 5/10/20 4:32 PM, Peng Fan wrote: >> Subject: Re: [PATCH] imx8mp_evk: Make SPL binary size smaller >> >> On Thu, May 7, 2020 at 8:59 AM Fabio Estevam >> wrote: >>> >>> Commit f24dea4e1b52 ("ARM: imx8m: Fix reset in SPL on NXP iMX8MP >> EVK") >>> caused the u-boot-spl.bin binary size to grow by a

Re: [PATCH V2] usb: ehci-omap: Add Support DM_USB and OF_CONTROL

2020-05-10 Thread Marek Vasut
On 5/10/20 4:33 PM, Adam Ford wrote: [...] > @@ -177,8 +180,13 @@ int omap_ehci_hcd_stop(void) > * Based on "drivers/usb/host/ehci-omap.c" from Linux 3.1 > * See there for additional Copyrights. > */ > +#if !CONFIG_IS_ENABLED(DM_USB) || !CONFIG_IS_ENABLED(OF_CONTROL) > + > int omap_ehci_hcd_

[PULL] u-boot-sh/master

2020-05-10 Thread Marek Vasut
The following changes since commit c5c657644bc35fd6b3d6e5517698721e90646b8d: Merge branch '2020-05-08-assorted-fixes' (2020-05-08 18:58:19 -0400) are available in the Git repository at: git://git.denx.de/u-boot-sh.git master for you to fetch changes up to 58d8c88ff81b4532f8f5884fe4138639c96

Re: [PATCH v2 00/22] common: Further reduce the size of common.h

2020-05-10 Thread Simon Glass
Hi Tom, On Fri, 8 May 2020 at 12:37, Tom Rini wrote: > > On Thu, Apr 09, 2020 at 02:14:44PM -0600, Simon Glass wrote: > > > This is yet another series to remove the contents of common.h so that > > individual files include the headers they need. > > > > With this series there are only 14 includes

Re: [PATCH v8 19/21] sifive: dts: fu540: Enable L2 Cache in U-Boot

2020-05-10 Thread Jagan Teki
On Sat, May 9, 2020 at 8:02 PM Pragnesh Patel wrote: > > Add L2 cache node to enable cache ways from U-Boot > > Signed-off-by: Pragnesh Patel > Reviewed-by: Bin Meng > --- > arch/riscv/dts/fu540-c000-u-boot.dtsi | 4 > 1 file changed, 4 insertions(+) > > diff --git a/arch/riscv/dts/fu540-c

[PATCH v6 2/2] pci: Fix typo in definition for PCI_DEV

2020-05-10 Thread Simon Glass
Fix a typo in the comment. Signed-off-by: Simon Glass --- Changes in v6: None include/pci.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/pci.h b/include/pci.h index aff56b24f9..19c9244b94 100644 --- a/include/pci.h +++ b/include/pci.h @@ -531,7 +531,7 @@ typedef

[PATCH v6 1/2] usb: Update struct usb_device to indicate speed enum

2020-05-10 Thread Simon Glass
The speed member actually uses an enum, so add this to the comment. Signed-off-by: Simon Glass --- Changes in v6: - Use the enum directly in the struct include/usb.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/usb.h b/include/usb.h index 22f6088fe6..fa9e09607e 1

Re: [PATCH v6 1/2] usb: Update struct usb_device to indicate speed enum

2020-05-10 Thread Marek Vasut
On 5/10/20 6:26 PM, Simon Glass wrote: > The speed member actually uses an enum, so add this to the comment. > > Signed-off-by: Simon Glass Reviewed-by: Marek Vasut

[PATCH] test: Use ut_asserteq_mem() where possible

2020-05-10 Thread Simon Glass
Quite a few tests still use ut_assertok(memcmp(...)) and variants. Modify them to use the macro designed for this purpose. Suggested-by: Wolfgang Wallner Signed-off-by: Simon Glass --- test/compression.c | 8 +-- test/dm/acpi.c | 6 +- test/dm/axi.c | 4 +- test/dm/dma.c |

[PATCH v3 01/22] common: Drop uuid.h from common header

2020-05-10 Thread Simon Glass
Move this uncommon header out of the common header. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: None cmd/gpt.c | 1 + cmd/nvedit_efi.c | 1 + cmd/x86/hob.c | 1 + disk/part_efi.c | 1 + fs/btrfs/btrfs.c

[PATCH v3 03/22] common: Drop flash.h from common header

2020-05-10 Thread Simon Glass
Move this uncommon header out of the common header. Fix up some style problems in flash.h while we are here. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: None arch/arm/mach-at91/arm926ejs/eflash.c | 1 + board/AndesTech/adp-ae3xx/adp-ae3xx.c | 1 + board/AndesTe

[PATCH v3 04/22] arm: Don't include common.h in header files

2020-05-10 Thread Simon Glass
It is bad practice to include common.h in other header files since it can bring in any number of superfluous definitions. It implies that some C files don't include it and thus may be missing CONFIG options that are set up by that file. The C files should include these themselves. Update some head

[PATCH v3 02/22] flash: Tidy up coding style for flash functions

2020-05-10 Thread Simon Glass
Some functions use the wrong code style and generate checkpatch errors. Fix these. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: - Split out code-style changes for flash functions into a new patch arch/arm/mach-at91/arm926ejs/eflash.c | 8 board/cobra5272/flash.c

[PATCH v3 09/22] common: Drop bootstage.h from common header

2020-05-10 Thread Simon Glass
Move this fairly uncommon header out of the common header. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: None arch/arc/lib/bootm.c| 3 ++- arch/arm/cpu/armv8/generic_timer.c | 1 + arch/arm/lib/bootm.c| 1 + arch/m68k/lib/bootm.c

[PATCH v3 06/22] part: Drop disk_partition_t typedef

2020-05-10 Thread Simon Glass
We should not be using typedefs and these make it harder to use forward declarations (to reduce header file inclusions). Drop the typedef. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: None arch/arm/mach-omap2/utils.c | 2 +- arch/x86/cpu/quark/quark.c | 1 + c

[PATCH v3 08/22] bootstage: Use BOOTSTAGE instead of BOOTSTATE

2020-05-10 Thread Simon Glass
Some of the enum members are wrong. Fix them. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: None arch/x86/lib/fsp2/fsp_meminit.c | 4 ++-- arch/x86/lib/fsp2/fsp_silicon_init.c | 4 ++-- common/board_f.c | 4 ++-- common/board_r.c

[PATCH v3 07/22] common: Drop part.h from common header

2020-05-10 Thread Simon Glass
Move this uncommon header out of the common header. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: None api/api_storage.c| 1 + arch/arm/mach-omap2/utils.c | 1 + arch/arm/mach-snapdragon/dram.c | 1 + arch/x86/include/asm/fsp/fs

[PATCH v3 16/22] common: Drop asm_offsets.h from common header

2020-05-10 Thread Simon Glass
Move this uncommon header out of the common header. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: None arch/powerpc/cpu/mpc83xx/cpu_init.c | 1 + arch/powerpc/cpu/mpc83xx/spl_minimal.c| 1 + arch/powerpc/cpu/mpc85xx/commproc.c | 1 + arch/powerpc/cpu/mpc85xx

[PATCH v3 13/22] Fix some checkpatch warnings in calls to debug()

2020-05-10 Thread Simon Glass
Fix up some incorrect code style in calls to functions in the log.h header, mostly debug(). Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: - Split out code-style changes for log functions into a new patch arch/powerpc/lib/bootm.c| 10 +- board/freesc

[PATCH v3 15/22] common: Drop asm/ptrace.h from common header

2020-05-10 Thread Simon Glass
Move this uncommon header out of the common header. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: None arch/arm/cpu/arm1136/mx35/timer.c | 1 + arch/arm/cpu/arm926ejs/mx25/timer.c | 1 + arch/arm/cpu/arm926ejs/mx27/timer.c | 1 + arch/arm/cpu/arm926ejs/spear/s

[PATCH v3 10/22] common: Drop image.h from common header

2020-05-10 Thread Simon Glass
Move this uncommon header out of the common header. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: None arch/arc/lib/bootm.c | 2 ++ arch/arm/cpu/armv8/fsl-layerscape/ls1028_ids.c | 1 + arch/arm/cpu/armv8/fsl-layerscape/mp.c | 1 + ar

[PATCH v3 17/22] common: Drop linux/bug.h from common header

2020-05-10 Thread Simon Glass
Move this uncommon header out of the common header. Signed-off-by: Simon Glass --- Changes in v3: - Update stm32mp15_optee, etc. Changes in v2: None arch/arm/mach-bcm283x/include/mach/timer.h | 1 + arch/arm/mach-imx/cmd_nandbcb.c | 1 + arch/arm/mach-tegra/ivc.c

[PATCH v3 19/22] Fix some checkpatch warnings in calls to udelay()

2020-05-10 Thread Simon Glass
Fix up some incorrect code style in calls to functions in the linux/time.h header, mostly udelay(). Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: - Split out code-style changes for time functions into a new patch arch/arm/cpu/sa1100/timer.c | 2 +- board/sbc8641

[PATCH v3 18/22] common: Drop linux/stringify.h from common header

2020-05-10 Thread Simon Glass
Move this uncommon header out of the common header. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: None arch/arm/cpu/armv7/bcm235xx/clk-core.h | 1 + arch/arm/cpu/armv7/bcm281xx/clk-core.h | 1 + board/Arcturus/ucp1020/cmd_arc.c | 1 + board/gardena/smart-g

[PATCH v3 21/22] Use __ASSEMBLY__ as the assembly macros

2020-05-10 Thread Simon Glass
Some places use __ASSEMBLER__ instead which does not work since the Makefile does not define it. Fix them. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: - Add new patch to fix occurances of __ASSEMBLER__ arch/arm/include/asm/arch-fsl-layerscape/immap_lsch3.h | 2 +- arc

[PATCH v2] test: Use ut_asserteq_mem() where possible

2020-05-10 Thread Simon Glass
Quite a few tests still use ut_assertok(memcmp(...)) and variants. Modify them to use the macro designed for this purpose. Suggested-by: Wolfgang Wallner Signed-off-by: Simon Glass --- Changes in v2: - Convert two more that were missed test/compression.c | 8 +-- test/dm/acpi.c |

Re: [PATCH] test: Use ut_asserteq_mem() where possible

2020-05-10 Thread Heinrich Schuchardt
Am May 10, 2020 5:33:06 PM UTC schrieb Simon Glass : >Quite a few tests still use ut_assertok(memcmp(...)) and variants. >Modify >them to use the macro designed for this purpose. > >Suggested-by: Wolfgang Wallner > >Signed-off-by: Simon Glass Acked-by: Heinrich Schuchardt >--- > > test/compres

Re: [PATCH V2] mkimage: fit: Do not tail-pad fitImage with external data

2020-05-10 Thread Marek Vasut
On 5/8/20 9:21 PM, Tom Rini wrote: > On Fri, May 08, 2020 at 09:00:02PM +0200, Marek Vasut wrote: >> On 5/8/20 8:47 PM, Tom Rini wrote: >>> On Fri, May 08, 2020 at 03:37:01AM +0200, Marek Vasut wrote: On 5/7/20 10:46 PM, Samuel Holland wrote: > On 5/6/20 12:02 PM, trini at konsulko.com (To

Re: Pull request: u-boot-spi/master

2020-05-10 Thread Tom Rini
On Sun, May 10, 2020 at 07:49:52PM +0530, Jagan Teki wrote: > On Sun, May 10, 2020 at 2:28 AM Tom Rini wrote: > > > > On Sat, May 09, 2020 at 09:56:09PM +0530, Jagan Teki wrote: > > > > > Hi Tom, > > > > > > Please pull this PR. > > > > > > Summary: > > > - zap lpc32xx_ssp driver (Jagan) > > > - r

[PATCH 0/2] mtd: spi-nand: Update Toshiba support

2020-05-10 Thread Robert Marko
This patch series updates the Toshiba SPI-NAND driver support to match the Linux tree. This imports 2 patches that add support for the new J series, as well as adding x4 and QE enable to previously added TC58CVG2S0HRAIJ. Yoshio Furuyama (2): mtd: spinand: toshiba: Rename function name to change

[PATCH 1/2] mtd: spinand: toshiba: Rename function name to change suffix and prefix (8Gbit)

2020-05-10 Thread Robert Marko
From: Yoshio Furuyama The suffix was changed from "G" to "J" to classify between 1st generation and 2nd generation serial NAND devices (which now belong to the Kioxia brand). As reference that's 1st generation device of 1Gbit product is "TC58CVG0S3HRAIG" 2nd generation device of 1Gbit product is

Pull request v2: u-boot-spi/master

2020-05-10 Thread Jagan Teki
Hi Tom, Summary: - zap lpc32xx_ssp driver (Jagan) - rename of phy nodev call (Jagan) - iopoll with sleep_us (Jagan) - MX25R6435F flash (Ye Li) Changes for v2: - rebase to master thanks, Jagan. The following changes since commit c5c657644bc35fd6b3d6e5517698721e90646b8d: Merge branch '2020-05

[PATCH 2/2] mtd: spinand: toshiba: Support for new Kioxia Serial NAND

2020-05-10 Thread Robert Marko
From: Yoshio Furuyama Add support for new Kioxia products. The new Kioxia products support program load x4 command, and have HOLD_D bit which is equivalent to QE bit. Signed-off-by: Yoshio Furuyama Reviewed-by: Frieder Schrempf Signed-off-by: Miquel Raynal Link: https://lore.kernel.org/linux

[PATCH v2 01/39] tbs2910: Drop the 'bd' command

2020-05-10 Thread Simon Glass
This board is very close to its limit. Drop this command before the bd refactoring, which increases the size slightly on one toolchain. Signed-off-by: Simon Glass --- Changes in v2: - Add a new patch to drop the 'bd' command for tbs2910 configs/tbs2910_defconfig | 1 + 1 file changed, 1 insert

[PATCH v2 03/39] bdinfo: riscv: Use generic bd_info

2020-05-10 Thread Simon Glass
At present riscv still uses its own private bd_info struct. Move it over to use the generic one like other archs. Signed-off-by: Simon Glass Reviewed-by: Rick Chen Reviewed-by: Bin Meng --- Changes in v2: None arch/riscv/include/asm/u-boot.h | 19 ++- 1 file changed, 2 insert

[PATCH v2 00/39] Tidy up the 'bd' command.

2020-05-10 Thread Simon Glass
The code for the 'bd' command never got the 'generic board' treatment many years ago when global_data and bd_info were converted. As a result it still has a lot of arch-specific duplication of generic code. This series aims to make as much code in this file generic as possible, so that it is easy

[PATCH v2 02/39] bdinfo: nds32: Use generic bd_info

2020-05-10 Thread Simon Glass
At present nds32 still uses its own private bd_info struct. Move it over to use the generic one like other archs. Signed-off-by: Simon Glass Reviewed-by: Rick Chen Reviewed-by: Bin Meng --- Changes in v2: None arch/nds32/include/asm/u-boot.h | 20 ++-- 1 file changed, 2 inser

[PATCH v2 05/39] bdinfo: xtensa: Create a generic do_bdinfo for xtensa

2020-05-10 Thread Simon Glass
This arch uses only the generic function. It would be nice if all the archs did the same. As a first step, create a new generic function for the 'bd' command and make xtensa use it. Signed-off-by: Simon Glass Reviewed-by: Bin Meng --- Changes in v2: None cmd/bdinfo.c | 14 ++ 1 fi

[PATCH v2 08/39] bdinfo: microblaze: Use the generic bd command

2020-05-10 Thread Simon Glass
Microblaze prints out ethernet and FDT information. This is useful to most archs, so move it into the generic code and move microblaze over to use it. Note that FDT information is shown for all boards, since they should be using device tree by now. Signed-off-by: Simon Glass Reviewed-by: Bin Men

[PATCH v2 04/39] bdinfo: m68k: Drop bd_info->bi_ipbfreq

2020-05-10 Thread Simon Glass
This field is not used anymore. Drop it. Signed-off-by: Simon Glass Reviewed-by: Bin Meng --- Changes in v2: None include/asm-generic/u-boot.h | 1 - 1 file changed, 1 deletion(-) diff --git a/include/asm-generic/u-boot.h b/include/asm-generic/u-boot.h index cc94d39069..be0014c3b3 100644 ---

[PATCH v2 06/39] bdinfo: mips: Use the generic bd command

2020-05-10 Thread Simon Glass
MIPS currently has a few extra things which are generally useful. Add them to the generic function and move MIPS over to use it. Signed-off-by: Simon Glass Reviewed-by: Bin Meng Reviewed-by: Daniel Schwierzeck --- Changes in v2: None cmd/bdinfo.c | 13 - 1 file changed, 4 inserti

[PATCH v2 07/39] bdinfo: nios2: Use the generic bd command

2020-05-10 Thread Simon Glass
Nios2 currently has some code to output SRAM information which is behind an #ifdef. No nios2 boards define this option, so the code can be removed. Move Nios2 over to use the generic function. Signed-off-by: Simon Glass Reviewed-by: Bin Meng --- Changes in v2: None cmd/bdinfo.c | 19 +---

[PATCH v2 11/39] bdinfo: sandbox: Use the generic bd command

2020-05-10 Thread Simon Glass
Sandbox has a printout of 'FB base' but this code is not used since sandbox uses driver model for everything. Move sandbox over to use the generic do_bdinfo(). Signed-off-by: Simon Glass Reviewed-by: Bin Meng --- Changes in v2: None cmd/bdinfo.c | 16 +--- 1 file changed, 1 inser

[PATCH v2 10/39] bdinfo: x86: Use the generic bd command

2020-05-10 Thread Simon Glass
This arch shows 'ethspeed' info but only the freescale drivers use it, so it can be dropped. It also calls print_bi_dram() which is safe to call from any arch since it has an #ifdef inside it. Add this to the generic do_bdinfo() and move x86 over to use it. Put it first since pytests rely on seein

[PATCH v2 12/39] bdinfo: nds32: Use the generic bd command

2020-05-10 Thread Simon Glass
This arch has none of its own info to show. Move it over to use the generic do_bdinfo(). Signed-off-by: Simon Glass Reviewed-by: Rick Chen Reviewed-by: Bin Meng --- Changes in v2: None cmd/bdinfo.c | 14 +- 1 file changed, 1 insertion(+), 13 deletions(-) diff --git a/cmd/bdinfo.

[PATCH v2 09/39] bdinfo: sh: Use the generic bd command

2020-05-10 Thread Simon Glass
This arch has no code that is not already in the generic function. Drop the arch-specific function and change sh over to use the generic one. Signed-off-by: Simon Glass Reviewed-by: Bin Meng --- Changes in v2: None cmd/bdinfo.c | 13 + 1 file changed, 1 insertion(+), 12 deletions(

[PATCH v2 13/39] bdinfo: riscv: Use the generic bd command

2020-05-10 Thread Simon Glass
This arch has none of its own info to show. Move it over to use the generic do_bdinfo(). Signed-off-by: Simon Glass Reviewed-by: Rick Chen Reviewed-by: Bin Meng --- Changes in v2: None cmd/bdinfo.c | 15 +-- 1 file changed, 1 insertion(+), 14 deletions(-) diff --git a/cmd/bdinfo

[PATCH v2 17/39] bdinfo: arc: Use the generic bd command

2020-05-10 Thread Simon Glass
There is nothing new in the arc 'bd' command beyond what is already there. Switch it over to use the generic code. Signed-off-by: Simon Glass --- Changes in v2: - Fix up inadvertent merge of PowerPC/ARM/m68k/arc patches cmd/bdinfo.c | 12 +--- 1 file changed, 1 insertion(+), 11 deletio

[PATCH v2 15/39] bdinfo: m68k: Use the generic bd command

2020-05-10 Thread Simon Glass
Unfortunately m68k has a lot of special stuff. Move it into the generic function for now, so we can have it all in one place. Signed-off-by: Simon Glass --- Changes in v2: - Fix up inadvertent merge of PowerPC/ARM/m68k/arc patches cmd/bdinfo.c | 49 +++--

[PATCH v2 14/39] bdinfo: powerpc: Use the generic bd command

2020-05-10 Thread Simon Glass
Unfortunately PowerPC has a lot of special stuff. Move it into the generic function for now, so we can have it all in one place. Signed-off-by: Simon Glass --- Changes in v2: - Fix up inadvertent merge of PowerPC/ARM/m68k/arc patches cmd/bdinfo.c | 89 --

[PATCH v2 16/39] bdinfo: arm: Use the generic bd command

2020-05-10 Thread Simon Glass
Unfortunately ARM has a lot of special stuff. Move it into the generic function for now, so we can have it all in one place. Signed-off-by: Simon Glass --- Changes in v2: - Fix up inadvertent merge of PowerPC/ARM/m68k/arc patches cmd/bdinfo.c | 102 ++---

[PATCH v2 21/39] bdinfo: Drop unnecessary inline on functions

2020-05-10 Thread Simon Glass
This serves no purpose since the compiler will inline the functions automatically. Drop use of inline in this file. Signed-off-by: Simon Glass Reviewed-by: Bin Meng --- Changes in v2: - Move __maybe_unused on print_lnum() to the early patch cmd/bdinfo.c | 14 +++--- 1 file changed, 7

[PATCH v2 22/39] bdinfo: Drop print_std_bdinfo()

2020-05-10 Thread Simon Glass
Everything in this function is standard now so it serves no purpose. Move it into the generic do_bdinfo() function. Signed-off-by: Simon Glass Reviewed-by: Bin Meng --- Changes in v2: None cmd/bdinfo.c | 15 +-- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/cmd/bd

[PATCH v2 18/39] bdinfo: Drop the option to not use the generic 'bd' command

2020-05-10 Thread Simon Glass
Now that all architectures are using the generic do_bdinfo(), drop the option to not use it. When new architectures are added, they will get at least some useful information from the generic implementation. Signed-off-by: Simon Glass Reviewed-by: Bin Meng --- Changes in v2: None cmd/bdinfo.c

[PATCH v2 20/39] bdinfo: microblaze: sh: nios2: Drop arch-specific flash info

2020-05-10 Thread Simon Glass
The differences these architectures have are minor and not worth the extra code. Use the generic version always. Tidy up the code style by removing unnecessary tabs. Signed-off-by: Simon Glass Reviewed-by: Bin Meng --- Changes in v2: None cmd/bdinfo.c | 17 +++-- 1 file changed,

[PATCH v2 24/39] bdinfo: sh: arc: Drop arch-specific print_bi_mem()

2020-05-10 Thread Simon Glass
It isn't worth having arch-specific code for such minor output differences. In fact it is better if all archs are consistent. Drop the arch-specific code in print_bi_mem() and inline it to avoid a two-line function. Signed-off-by: Simon Glass Reviewed-by: Bin Meng Acked-by: Alexey Brodkin ---

[PATCH v2 23/39] bdinfo: ppc: Drop arch-specific print_baudrate()

2020-05-10 Thread Simon Glass
This function outputs the same basic info. Since the baud rate is commonly 115200 these is often no difference. Drop the arch-specific code and inline it to avoid a one-line function. Signed-off-by: Simon Glass Reviewed-by: Bin Meng Reviewed-by: Stefan Roese --- Changes in v2: None cmd/bdinf

[PATCH v2 19/39] bdinfo: Drop unused __maybe_unused

2020-05-10 Thread Simon Glass
Some of these are not needed now. Drop them to avoid cluttering the code. Signed-off-by: Simon Glass Reviewed-by: Bin Meng --- Changes in v2: - Drop __maybe_unused on print_lnum() too cmd/bdinfo.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/cmd/bdinfo.c b/cmd/bd

[PATCH v2 27/39] bdinfo: Drop print_cpu_word_size()

2020-05-10 Thread Simon Glass
This function only has one line of code in it so inline it. Signed-off-by: Simon Glass Reviewed-by: Bin Meng --- Changes in v2: None cmd/bdinfo.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/cmd/bdinfo.c b/cmd/bdinfo.c index f22dc7a129..d0afef5b5e 100644 --- a/cmd

[PATCH v2 25/39] bdinfo: Drop print_bi_boot_params()

2020-05-10 Thread Simon Glass
This function only has one line of code in it so inline it. Signed-off-by: Simon Glass Reviewed-by: Bin Meng --- Changes in v2: None cmd/bdinfo.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/cmd/bdinfo.c b/cmd/bdinfo.c index 5ffd86e508..b65eec332d 100644 --- a/cm

[PATCH v2 29/39] bdinfo: net: ppc: Drop prints for CONFIG_HAS_ETHn

2020-05-10 Thread Simon Glass
These config options have not been migrated to Kconfig. This should be handled using driver model, iterating over the available Ethernet devices. For now, remove the code. Signed-off-by: Simon Glass Reviewed-by: Bin Meng Reviewed-by: Stefan Roese --- Changes in v2: None cmd/bdinfo.c | 15 ---

[PATCH v2 26/39] bdinfo: Drop print_bi_flash()

2020-05-10 Thread Simon Glass
This function only has three lines of code in it so inline it. This helps to put all the code in one place for future partitioning. Signed-off-by: Simon Glass --- Changes in v2: - Update commit message with an extra motivation cmd/bdinfo.c | 11 +++ 1 file changed, 3 insertions(+), 8 d

[PATCH v2 28/39] bdinfo: net: ppc: Drop bi_enet1addr and other similar info

2020-05-10 Thread Simon Glass
These values were 'old' in 2013 so it should be safe to remove them. They are never set in U-Boot anyway, so the values will always be zero. Signed-off-by: Simon Glass Reviewed-by: Bin Meng Reviewed-by: Stefan Roese --- Changes in v2: None common/board_r.c | 16 +---

[PATCH v2 30/39] bdinfo: net: Drop legacy ethernet bdinfo

2020-05-10 Thread Simon Glass
This code pre-dates driver model and the migration date is nearly upon us. Pare the print_eths() function down and enable it for driver model, since it works correctly. The IP address is already printed in print_eth_ip_addr() so we can remove that. Since this results in a one-line print_eths() fu

[PATCH v2 33/39] bdinfo: arm: Move ARM-specific info into its own file

2020-05-10 Thread Simon Glass
We don't really want to have ARM-specific code in a generic file. Create a new arch-specific function to hold it, and move it into that. Make the function weak so that any arch can implement it. Signed-off-by: Simon Glass Reviewed-by: Bin Meng --- Changes in v2: - Leave MULTI_DTB_FIT in the ge

[PATCH v2 31/39] bdinfo: net: Inline print_eth_ip_addr()

2020-05-10 Thread Simon Glass
This function only has two lines of code now, so inline it. Signed-off-by: Simon Glass Reviewed-by: Bin Meng --- Changes in v2: None cmd/bdinfo.c | 15 --- 1 file changed, 4 insertions(+), 11 deletions(-) diff --git a/cmd/bdinfo.c b/cmd/bdinfo.c index e6237daf93..e30d530411 10064

[PATCH v2 34/39] bdinfo: ppc: Move PPC-specific info into its own file

2020-05-10 Thread Simon Glass
We don't really want to have PPC-specific code in a generic file. Create a new arch-specific function to hold it, and move it into that. Make the function weak so that any arch can implement it. Signed-off-by: Simon Glass Reviewed-by: Bin Meng Reviewed-by: Stefan Roese --- Changes in v2: None

[PATCH v2 32/39] bdinfo: Export some basic printing functions

2020-05-10 Thread Simon Glass
At present the functions to print a number and a frequency are static. We want to move some of the code in here to an arch-specific file. For consistency that code should use these same functions. So export them with an appropriate name. Signed-off-by: Simon Glass Reviewed-by: Bin Meng --- Chan

[PATCH v2 35/39] bdinfo: m68k: Move m68k-specific info into its own file

2020-05-10 Thread Simon Glass
We don't really want to have m68k-specific code in a generic file. Create a new arch-specific function to hold it, and move it into that. Make the function weak so that any arch can implement it. Signed-off-by: Simon Glass Reviewed-by: Bin Meng Tested-by: Angelo Dureghello --- Changes in v2:

[PATCH v2 36/39] bdinfo: m68k: ppc: Move arch-specific code from bdinfo

2020-05-10 Thread Simon Glass
We don't have an easy way to share these three lines of code with two architectures. We also want to make it clear that this code is actually arch-specific. So just duplicate it in each arch-specific file. Signed-off-by: Simon Glass Reviewed-by: Bin Meng Reviewed-by: Stefan Roese --- Changes

[PATCH v2 38/39] bdinfo: dm: Update fb_base when using driver model

2020-05-10 Thread Simon Glass
Update this value with the address of a video device so that it shows with the 'bd' command. It would be better to obtain the address from the uclass by looking in struct video_uc_platdata for each device. We can move over to that once DM_VIDEO migration is complete. Signed-off-by: Simon Glass R

Re: [PATCH v3 1/6] clk: rk3399: Add enable/disable clks

2020-05-10 Thread Suniel Mahesh
On Sat, May 9, 2020 at 10:26 PM Jagan Teki wrote: > Yes, most of the high speed peripheral clocks > in rk3399 enabled by default. > > But it would be better to handle them via clk > enable/disable API for handling proper reset > conditions like 'usb reset' over command line. > > So, enable USB, G

[PATCH v2 37/39] bdinfo: Update the file comments

2020-05-10 Thread Simon Glass
Update the comment at the top of the file to indicate what this file does. Also drop the line at the bottom and an unnecessary header include. Signed-off-by: Simon Glass Reviewed-by: Bin Meng --- Changes in v2: None cmd/bdinfo.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) d

[PATCH v2 39/39] bdinfo: x86: vesa: Update fb_base to the correct value

2020-05-10 Thread Simon Glass
Set this value in global_data so that it is reported correctly on x86 boards. In fact, U-Boot allocates space for the frame buffer even though it is not used. Then the FSP picks the address itself (e.g. 0xb000). So the value set by U-Boot (high in memory with everything else that is relocated)

Re: [PATCH v3 2/6] clk: rk3399: Enable/Disable the PCIEPHY clk

2020-05-10 Thread Suniel Mahesh
On Sat, May 9, 2020 at 10:26 PM Jagan Teki wrote: > Enable/Disable the PCIEPHY clk for rk3399. > > CLK is clear in both enable and disable functionality. > > Signed-off-by: Jagan Teki > --- > Changes for v3: > - none > > drivers/clk/rockchip/clk_rk3399.c | 6 ++ > 1 file changed, 6 insertio

Re: [PATCH v3 3/6] pci: Add Rockchip PCIe controller driver

2020-05-10 Thread Suniel Mahesh
On Sat, May 9, 2020 at 10:26 PM Jagan Teki wrote: > Add Rockchip PCIe controller driver for rk3399 platform. > > Driver support Gen1 by operating as a Root complex. > > Thanks to Patrick for initial work. > > Signed-off-by: Patrick Wildt > Signed-off-by: Jagan Teki > Reviewed-by: Kever Yang >

Re: [PATCH v3 4/6] pci: Add Rockchip PCIe PHY controller driver

2020-05-10 Thread Suniel Mahesh
On Sat, May 9, 2020 at 10:26 PM Jagan Teki wrote: > Yes, it is possible to have a dedicated UCLASS PHY driver > for this Rockchip PCIe PHY but there are some issues on > Generic PHY framework to support the same. > > The Generic PHY framework is unable to get the PHY if > the PHY parent is of a d

Re: [PATCH v3 5/6] rockchip: Enable PCIe/M.2 on rk3399 board w/ M.2

2020-05-10 Thread Suniel Mahesh
On Sat, May 9, 2020 at 10:26 PM Jagan Teki wrote: > Enable PCIe/M.2 support on > - NanoPC-T4 > - ROC-RK3399-PC Mezzanine boards. > > Signed-off-by: Jagan Teki > --- > Changes for v3: > - none > > arch/arm/dts/rk3399-u-boot.dtsi | 1 + > configs/nanopc-t4-rk3399_defconfig| 4 ++

Re: [PATCH v3 1/6] clk: rk3399: Add enable/disable clks

2020-05-10 Thread Suniel Mahesh
On Sat, May 9, 2020 at 10:26 PM Jagan Teki wrote: > Yes, most of the high speed peripheral clocks > in rk3399 enabled by default. > > But it would be better to handle them via clk > enable/disable API for handling proper reset > conditions like 'usb reset' over command line. > > So, enable USB, G

[PATCH v2 00/35] dm: Add programmatic generation of ACPI tables (part B)

2020-05-10 Thread Simon Glass
NOTE: I have resent this as v1 to avoid confusion This is split from the original series in an attempt to get things applied in chunks. This part includes: - writing basic ACPI code for integers, strings, names, packages - writing descriptors for GPIO, I2C, interrupts, SPI - writing code to enabl

[PATCH v2 01/35] dm: core: Add an ACPI name for the root node

2020-05-10 Thread Simon Glass
This always has a fixed ACPI name so add it as a driver function. Signed-off-by: Simon Glass Reviewed-by: Wolfgang Wallner --- Changes in v2: None Changes in v1: - Capitalise ACPI_OPS_PTR drivers/core/root.c | 13 + 1 file changed, 13 insertions(+) diff --git a/drivers/core/root.

[PATCH v2 05/35] acpi: Support generation of ACPI code

2020-05-10 Thread Simon Glass
Add a new file to handle generating ACPI code programatically. This is used when information must be dynamically added to the tables, e.g. the SSDT. Initial support is just for writing simple values. Signed-off-by: Simon Glass --- Changes in v2: None Changes in v1: None include/acpi/acpigen.h

[PATCH v2 04/35] irq: Add a method to convert an interrupt to ACPI

2020-05-10 Thread Simon Glass
When generating ACPI tables we need to convert IRQs in U-Boot to the ACPI structures required by ACPI. This is a SoC-specific conversion and cannot be handled by generic code, so add a new IRQ method to do the conversion. Signed-off-by: Simon Glass --- Changes in v2: None Changes in v1: - Fix 't

[PATCH v2 03/35] acpi: Add a way to check device status

2020-05-10 Thread Simon Glass
At present U-Boot does not support the different ACPI status values, but it is best to put this logic in a central place. Add a function to get the device status. Signed-off-by: Simon Glass --- Changes in v2: None Changes in v1: None include/acpi/acpi_device.h | 28

[PATCH v2 06/35] acpi: Support generation of interrupt descriptor

2020-05-10 Thread Simon Glass
Add a function to write an interrupt descriptor to the generated ACPI code. Signed-off-by: Simon Glass --- Changes in v2: None Changes in v1: None include/acpi/acpi_device.h | 15 + lib/acpi/acpi_device.c | 118 + test/dm/acpigen.c | 31 ++

  1   2   >