Re: [PATCH v2 1/1] riscv: reset after crash

2020-09-29 Thread Rick Chen
> From: Heinrich Schuchardt [mailto:xypron.g...@gmx.de] > Sent: Sunday, September 27, 2020 7:21 AM > To: Rick Jian-Zhi Chen(陳建志) > Cc: Simon Glass; Sean Anderson; Bin Meng; u-boot@lists.denx.de; Heinrich > Schuchardt > Subject: [PATCH v2 1/1] riscv: reset after crash > > If an exception occurs on

[PATCH] configs: lx2162aqds: Enable gpio driver in defconfig

2020-09-29 Thread Ran Wang
From: "hui.song" make lx2162aqds platform to enable gpio driver. Signed-off-by: hui.song Signed-off-by: ran.wang --- configs/lx2162aqds_tfa_SECURE_BOOT_defconfig | 1 + configs/lx2162aqds_tfa_defconfig | 1 + configs/lx2162aqds_tfa_verified_boot_defconfig | 1 + 3 files

mvebu_armada-37xx: Wrong usage of of_machine_is_compatible in u-boot master

2020-09-29 Thread Pali Rohár
Hello! In current U-Boot master branch is board_late_init() function for mvebu_armada-37xx/board.c slightly broken. There are two of_machine_is_compatible() checks for non-existent compatible string "globalscale,espressobin"

[RESEND PATCH] configs: lx2162aqds: Enable gpio driver in defconfig

2020-09-29 Thread Ran Wang
From: "hui.song" make lx2162aqds platform to enable gpio driver. Signed-off-by: Hui Song Signed-off-by: Ran Wang --- depends on: http://patchwork.ozlabs.org/project/uboot/patch/1599473527-21511-3-git-send-email-meenakshi.aggar...@nxp.com/ configs/lx2162aqds_tfa_SECURE_BOOT_defconfig | 1

[PATCH] mtd: spi-nor-ids: Add Winbond W25M512JW flash entry

2020-09-29 Thread Biju Das
Add Winbond W25M512JW flash device description. Signed-off-by: Biju Das Reviewed-by: Lad Prabhakar --- This patch series depend on [1] [1] https://patchwork.ozlabs.org/project/uboot/patch/20200917145030.29404-1-prabhakar.mahadev-lad...@bp.renesas.com/ --- drivers/mtd/spi/spi-nor-ids.c | 1 +

Re: [BUG] sandbox: './u-boot -l ' fails

2020-09-29 Thread Wolfgang Denk
Dear Heinrich, In message you wrote: > > As we always try to use the same definition in U-Boot as in glibc we > should really replace getc() by another symbol, e.g. chget() so that we > avoid confusion. We should not invent any new names when there are existing good alterntives. Why not use

Re: Pull request: u-boot-riscv/master 20200929

2020-09-29 Thread Tom Rini
On Tue, Sep 29, 2020 at 03:50:15PM +0800, ub...@andestech.com wrote: > Hi Tom, > > Please pull some riscv updates: > > - Disable CMD_IRQ for RISC-V. > - Update sipeed/maix doc > - Obtain reg of SiFive RAM via dev_read_addr_index() instead of regmap API. > - Cleans up RISC-V timer drivers and

Re: mvebu_armada-37xx: Wrong usage of of_machine_is_compatible in u-boot master

2020-09-29 Thread Andre Heider
Hi Pali, On 29/09/2020 11:52, Pali Rohár wrote: Hello! In current U-Boot master branch is board_late_init() function for mvebu_armada-37xx/board.c slightly broken. There are two of_machine_is_compatible() checks for non-existent compatible string "globalscale,espressobin" thanks for

Re: [PATCH 0/5] qemu-arm64: Allow booting via Trusted Firmware

2020-09-29 Thread Tom Rini
On Thu, Sep 24, 2020 at 01:17:10AM +0100, Andre Przywara wrote: > U-Boot on QEMU-arm64 can be used in two configurations: Loaded directly > via QEMU's -bios option, or as a non-secure payload (BL33) via > ARM Trusted Firmware-A (TF-A). > In the latter case we need to define CONFIG_TFABOOT, to

Re: [PATCH] arm64: dts: zynqmp: Fix leds subnode name for zcu100/ultra96 v1

2020-09-29 Thread Michal Simek
po 24. 8. 2020 v 12:28 odesílatel Michal Simek napsal: > > Fix the leds subnode names to match (^led-[0-9a-f]$|led). > > Similar change has been also done by commit 08dc0e5dd9aa ("arm64: dts: > meson: fix leds subnodes name"). > > The patch is fixing this warning: > avnet-ultra96-rev1.dt.yaml:

Re: [BUG] sandbox: './u-boot -l ' fails

2020-09-29 Thread Heinrich Schuchardt
Am 29. September 2020 13:12:25 MESZ schrieb Wolfgang Denk : >Dear Heinrich, > >In message you wrote: >> >> As we always try to use the same definition in U-Boot as in glibc we >> should really replace getc() by another symbol, e.g. chget() so that >we >> avoid confusion. > >We should not invent

[PATCH] spi: renesas_rpc_spi: Add compatible string for R8A774A1

2020-09-29 Thread Biju Das
Add compatible string entry for RZ/G2M (R8A774A1) SoC. Signed-off-by: Biju Das Reviewed-by: Lad Prabhakar --- drivers/spi/renesas_rpc_spi.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/spi/renesas_rpc_spi.c b/drivers/spi/renesas_rpc_spi.c index 3ea59b8fb8..a63390f1b7 100644 ---

[PATCH] arm64: zynqmp: Enable EMMC boot

2020-09-29 Thread Michal Simek
Enable EMMC boot commands to be able to change EMMC setup. Signed-off-by: Michal Simek --- configs/xilinx_versal_virt_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/xilinx_versal_virt_defconfig b/configs/xilinx_versal_virt_defconfig index 6f3ca9cb5af4..b3c615f3ce84

Re: mvebu_armada-37xx: Wrong usage of of_machine_is_compatible in u-boot master

2020-09-29 Thread Stefan Roese
On 29.09.20 14:24, Pali Rohár wrote: On Tuesday 29 September 2020 14:17:46 Andre Heider wrote: Hi Pali, On 29/09/2020 11:52, Pali Rohár wrote: Hello! In current U-Boot master branch is board_late_init() function for mvebu_armada-37xx/board.c slightly broken. There are two

Re: [PATCH v2 1/1] riscv: reset after crash

2020-09-29 Thread Heinrich Schuchardt
On 29.09.20 10:22, Rick Chen wrote: >> From: Heinrich Schuchardt [mailto:xypron.g...@gmx.de] >> Sent: Sunday, September 27, 2020 7:21 AM >> To: Rick Jian-Zhi Chen(陳建志) >> Cc: Simon Glass; Sean Anderson; Bin Meng; u-boot@lists.denx.de; Heinrich >> Schuchardt >> Subject: [PATCH v2 1/1] riscv: reset

[PATCH] clk: renesas: r8a774a1-cpg-mssr: Add R8A774A1 RPC clock

2020-09-29 Thread Biju Das
Add RPC entry into the R8A774A1 clock driver tables. Signed-off-by: Biju Das Reviewed-by: Lad Prabhakar --- drivers/clk/renesas/r8a774a1-cpg-mssr.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/clk/renesas/r8a774a1-cpg-mssr.c b/drivers/clk/renesas/r8a774a1-cpg-mssr.c index

[PATCH] arm: mvebu: Espressobin: Fix checks against machine compatible strings

2020-09-29 Thread Andre Heider
The patches changing the compatible strings to the ones used by Linux have not been merged yet, so fix the checks to use the current in-tree ones. Reported-by: Pali Rohár Signed-off-by: Andre Heider --- board/Marvell/mvebu_armada-37xx/board.c | 4 ++-- 1 file changed, 2 insertions(+), 2

Re: [PATCH 4/5] qemu: Drop ARCH_SUPPORT_TFABOOT

2020-09-29 Thread Tom Rini
On Thu, Sep 24, 2020 at 01:17:14AM +0100, Andre Przywara wrote: > CONFIG_ARCH_SUPPORT_TFABOOT seems to be a guard option to enable various > platform specific hacks, when U-Boot is run under TF-A. > Now that the QEMU port does not need to differentiate between secure > vs. non-secure anymore

Re: mvebu_armada-37xx: Wrong usage of of_machine_is_compatible in u-boot master

2020-09-29 Thread Andre Heider
On 29/09/2020 14:24, Pali Rohár wrote: On Tuesday 29 September 2020 14:17:46 Andre Heider wrote: Hi Pali, On 29/09/2020 11:52, Pali Rohár wrote: Hello! In current U-Boot master branch is board_late_init() function for mvebu_armada-37xx/board.c slightly broken. There are two

Re: [PATCH] arm: mvebu: Espressobin: Fix checks against machine compatible strings

2020-09-29 Thread Pali Rohár
On Tuesday 29 September 2020 14:34:26 Andre Heider wrote: > The patches changing the compatible strings to the ones used by Linux have > not been merged yet, so fix the checks to use the current in-tree ones. > > Reported-by: Pali Rohár > Signed-off-by: Andre Heider > --- >

[PATCH] Define default CONFIG_PREBOOT with right config option

2020-09-29 Thread Peter Robinson
The 44758771ee commit removes CONFIG_PREBOOT but actually sets the USE_PREBOOT Kconfig option which isn't CONFIG_PREBOOT and is also a bool option which means we regress because 'usb start' isn't run when expected, it should also be run for devices that have USB storage because keyboards aren't

Re: mvebu_armada-37xx: Wrong usage of of_machine_is_compatible in u-boot master

2020-09-29 Thread Pali Rohár
On Tuesday 29 September 2020 14:17:46 Andre Heider wrote: > Hi Pali, > > On 29/09/2020 11:52, Pali Rohár wrote: > > Hello! > > > > In current U-Boot master branch is board_late_init() function for > > mvebu_armada-37xx/board.c slightly broken. > > > > There are two of_machine_is_compatible()

Re: mvebu_armada-37xx: Wrong usage of of_machine_is_compatible in u-boot master

2020-09-29 Thread Stefan Roese
Hi Andre, Hi Pali, On 29.09.20 14:17, Andre Heider wrote: Hi Pali, On 29/09/2020 11:52, Pali Rohár wrote: Hello! In current U-Boot master branch is board_late_init() function for mvebu_armada-37xx/board.c slightly broken. There are two of_machine_is_compatible() checks for non-existent

Re: [PATCH] arm: mvebu: Espressobin: Fix checks against machine compatible strings

2020-09-29 Thread Andre Heider
On 29/09/2020 14:38, Pali Rohár wrote: On Tuesday 29 September 2020 14:34:26 Andre Heider wrote: The patches changing the compatible strings to the ones used by Linux have not been merged yet, so fix the checks to use the current in-tree ones. Reported-by: Pali Rohár Signed-off-by: Andre

Re: armv7R: K3: am654: U-Boot

2020-09-29 Thread Michal Simek
Hi Lokesh, On 25. 09. 20 16:17, Lokesh Vutla wrote: > Hi Michal, > > On 14/09/20 8:29 pm, Michal Simek wrote: > Hi Tom, > > On 14. 09. 20 16:47, Tom Rini wrote: On Mon, Sep 14, 2020 at 04:42:56PM +0200, Michal Simek wrote: > Hi Tom and Lokesh, > > I have been validating R5

Re: [PATCH] arm: mvebu: Espressobin: Fix checks against machine compatible strings

2020-09-29 Thread Pali Rohár
On Tuesday 29 September 2020 14:38:39 Pali Rohár wrote: > On Tuesday 29 September 2020 14:34:26 Andre Heider wrote: > > The patches changing the compatible strings to the ones used by Linux have > > not been merged yet, so fix the checks to use the current in-tree ones. > > > > Reported-by: Pali

[PATCH] rockchip: Rock960: fix up USB support

2020-09-29 Thread Peter Robinson
Fix up USB config options so keyboards and other USB devices work. --- configs/rock960-rk3399_defconfig | 9 ++--- include/configs/rock960_rk3399.h | 2 ++ 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/configs/rock960-rk3399_defconfig b/configs/rock960-rk3399_defconfig index

Re: [BUG] sandbox: './u-boot -l ' fails

2020-09-29 Thread Wolfgang Denk
Dear Heinrich, In message <0cbb3df2-632b-4dab-bd39-e2ce9f5af...@gmx.de> you wrote: > > >We should not invent any new names when there are existing good > >alterntives. > > > >Why not use getchar()? ... > please, have a look at http://patchwork.ozlabs.org/project/uboot/patch> >

[RESEND PATCH] configs: lx2162aqds: Enable gpio driver in defconfig

2020-09-29 Thread Ran Wang
From: "hui.song" make lx2162aqds platform to enable gpio driver. Signed-off-by: hui.song Signed-off-by: ran.wang --- depends on: http://patchwork.ozlabs.org/project/uboot/patch/1599473527-21511-3-git-send-email-meenakshi.aggar...@nxp.com/ configs/lx2162aqds_tfa_SECURE_BOOT_defconfig | 1

Re: [PATCH 4/5] qemu: Drop ARCH_SUPPORT_TFABOOT

2020-09-29 Thread André Przywara
On 29/09/2020 14:23, Tom Rini wrote: Hi Tom, > On Thu, Sep 24, 2020 at 01:17:14AM +0100, Andre Przywara wrote: > >> CONFIG_ARCH_SUPPORT_TFABOOT seems to be a guard option to enable various >> platform specific hacks, when U-Boot is run under TF-A. >> Now that the QEMU port does not need to

Re: [PATCH 10/10] riscv: Don't reserve AI ram in k210 dts

2020-09-29 Thread Sean Anderson
On 9/29/20 11:03 AM, Heinrich Schuchardt wrote: > Am 29. September 2020 16:18:35 MESZ schrieb Sean Anderson : >> It is no longer necessary to disallow ai ram, since it is enabled by >> the >> sram driver. >> >> Signed-off-by: Sean Anderson >> --- >> >> arch/riscv/dts/k210.dtsi | 12

Re: [PATCH 4/5] qemu: Drop ARCH_SUPPORT_TFABOOT

2020-09-29 Thread Tom Rini
On Tue, Sep 29, 2020 at 06:13:09PM +0100, André Przywara wrote: > On 29/09/2020 14:23, Tom Rini wrote: > > Hi Tom, > > > On Thu, Sep 24, 2020 at 01:17:14AM +0100, Andre Przywara wrote: > > > >> CONFIG_ARCH_SUPPORT_TFABOOT seems to be a guard option to enable various > >> platform specific

[PATCH 08/10] riscv: Probe ram in dram_init

2020-09-29 Thread Sean Anderson
If CONFIG_RAM is enabled, use the ram device to get the base/size of memory. This provides an easy way for boards/cpus to hook into the dram_init logic, without needing to provide a second SYS_CPU. Signed-off-by: Sean Anderson --- arch/riscv/cpu/generic/dram.c | 26 ++

[PATCH 10/10] riscv: Don't reserve AI ram in k210 dts

2020-09-29 Thread Sean Anderson
It is no longer necessary to disallow ai ram, since it is enabled by the sram driver. Signed-off-by: Sean Anderson --- arch/riscv/dts/k210.dtsi | 12 1 file changed, 12 deletions(-) diff --git a/arch/riscv/dts/k210.dtsi b/arch/riscv/dts/k210.dtsi index f7843985aa..7b0cd4f8f6

[PATCH 09/10] riscv: Enable AI ram on K210

2020-09-29 Thread Sean Anderson
CONFIG_SYS_SDRAM_{BASE,SIZE} are no longer necessary. The default get_effective_memsize will also work. The clock driver creates a bunch of devices, so we need to increase the pre-reloc malloc arena. Signed-off-by: Sean Anderson --- board/sipeed/maix/Kconfig | 2 ++

[PATCH 05/10] test: Add a test for fdtdec_setup_mem_size_base et al.

2020-09-29 Thread Sean Anderson
This adds a test for the various methods of extracting ram_base and ram_size from a device tree. Signed-off-by: Sean Anderson --- arch/sandbox/dts/test.dts | 12 ++ configs/sandbox64_defconfig| 2 +- configs/sandbox_defconfig | 2 +-

[PATCH 02/10] clk: Add support for the k210 clock driver pre-relocation

2020-09-29 Thread Sean Anderson
Variables which had previously been stored in .bss are moved to .data. In addition, probed needs to be reset when the clock driver is re-bound post-relocation. Signed-off-by: Sean Anderson --- drivers/clk/kendryte/clk.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-)

[PATCH 04/10] lib: fdt: Add fdtdec_setup_mem_size_base_highest

2020-09-29 Thread Sean Anderson
This is very similar to fdtdec_setup_mem_size_base_lowest, except we pick the highest ram bank, instead of the lowest. This is helpful for boards which use separate but contiguous ram banks, as it leaves the most space for loading programs. Signed-off-by: Sean Anderson --- include/fdtdec.h |

[PATCH 03/10] riscv: Enable some devices pre-relocation

2020-09-29 Thread Sean Anderson
These devices are necessary for the clock driver, which is required by the sram driver, to run pre-relocation. Signed-off-by: Sean Anderson --- arch/riscv/dts/k210.dtsi | 4 1 file changed, 4 insertions(+) diff --git a/arch/riscv/dts/k210.dtsi b/arch/riscv/dts/k210.dtsi index

[PATCH 06/10] ram: Add driver for K210 SRAM

2020-09-29 Thread Sean Anderson
This adds a driver to handle enabling the clock for the AI SRAM. This was previously done in board_init, but it needs to happen before relocation now. An alternative would be to move this to board_init_early_f, but by doing it this way we can use clk_bulk. Signed-off-by: Sean Anderson ---

[PATCH 07/10] ram: sifive: Default to y only if compiling for fu540

2020-09-29 Thread Sean Anderson
Other RISC-V targets should not have RAM_SIFIVE enabled by default. Signed-off-by: Sean Anderson --- drivers/ram/sifive/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/ram/sifive/Kconfig b/drivers/ram/sifive/Kconfig index 6aca22ab2a..b24153d971 100644 ---

[PATCH 00/10] riscv: k210: Enable use of AI ram bank

2020-09-29 Thread Sean Anderson
This ram bank was previously unusable because PLL1 was not started correctly. This series fixes that bug, and allows U-Boot to relocate into the AI ram. This provides an extra 2M of space in which to load payloads. Sean Anderson (10): clk: k210: Fix PLLs not being enabled clk: Add support

[PATCH 01/10] clk: k210: Fix PLLs not being enabled

2020-09-29 Thread Sean Anderson
After starting or setting the rate of a PLL, the enable bit must be set. This fixes a bug where the AI ram would not be accessible, because it requires PLL1 to be running. Signed-off-by: Sean Anderson --- drivers/clk/kendryte/pll.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

Re: [PATCH 10/10] riscv: Don't reserve AI ram in k210 dts

2020-09-29 Thread Heinrich Schuchardt
Am 29. September 2020 16:18:35 MESZ schrieb Sean Anderson : >It is no longer necessary to disallow ai ram, since it is enabled by >the >sram driver. > >Signed-off-by: Sean Anderson >--- > > arch/riscv/dts/k210.dtsi | 12 > 1 file changed, 12 deletions(-) > >diff --git

[RESEND PATCH 2/3] pinctrl: bcm283x: Read address from DT in ofdata_to_platdata

2020-09-29 Thread Ovidiu Panait
From: Ovidiu Panait Factor out reading IP base address to ofdata_to_platdata function, which is designed for this purpose. Also, drop the dev->priv NULL check, since this is already done by the dm core when allocating space using priv_auto_alloc_size feature. (in drivers/core/device.c ->

Re: [PATCH v3 2/3] net: phy: ca_phy: Add driver for CAxxxx SoCs

2020-09-29 Thread Tom Rini
On Tue, Sep 29, 2020 at 06:17:26PM +, Alex Nemirovsky wrote: > Tom, > Yes the intent is to allow overwrite the functionality elsewhere. I agree > that the name is too generic and should be more unique to avoid name space > conflicts. Any sort of board specific tweaking should be done via

[PATCH v2 1/1] sandbox: redefine getc()

2020-09-29 Thread Heinrich Schuchardt
The sandbox is built with the SDL2 library with invokes the X11 library which in turn calls getc(). But getc() in glibc is defined as int getc(FILE *) This does not match our definition. int getc(void) The sandbox crashes when called with parameter -l. Rename our library symbol to be

Re: [PATCH v3 2/3] net: phy: ca_phy: Add driver for CAxxxx SoCs

2020-09-29 Thread Tom Rini
On Wed, Sep 23, 2020 at 05:59:13PM -0700, Alex Nemirovsky wrote: > From: Abbie Chang > > Add phy driver support for MACs embedded inside Cortina Access SoCs > > Signed-off-by: Abbie Chang > Signed-off-by: Alex Nemirovsky > > CC: Joe Hershberger > CC: Tom Rini > CC: Aaron Tseng > > Moved

[PATCH v1] arm: socfpga: fix Gen5 enable of EMAC via FPGA

2020-09-29 Thread Ralph Siemsen
An earlier conversion from struct to defines introduced two errors, both related to setup of EMAC routed via the FPGA. One of the offsets was incorrect, and the EMAC0/EMAC1 were swapped. The effect of this was rather odd: both ports could operate at gigabit, but one of them would fail to transmit

[PATCH v2] spi: mvebu_a3700_spi: add support for cs-gpios

2020-09-29 Thread George Hilliard
The device tree has a way to specify GPIO lines as chip selects. From the binding docs: So if for example the controller has 2 CS lines, and the cs-gpios property looks like this: cs-gpios = < 0 0> <0> < 1 0> < 2 0>; Then it should be configured so that num_chipselect = 4 with

[PATCH 1/1] mmc: do not check argument of free() beforehand

2020-09-29 Thread Heinrich Schuchardt
free() checks if its argument in NULL. No need to check it twice. Signed-off-by: Heinrich Schuchardt --- drivers/mmc/sh_sdhi.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/mmc/sh_sdhi.c b/drivers/mmc/sh_sdhi.c index 315f95cce8..29f83b6554 100644 ---

Re: [PATCH v3 1/3] net: cortina_ni: Add eth support for Cortina Access CAxxxx SoCs

2020-09-29 Thread Tom Rini
On Wed, Sep 23, 2020 at 05:59:12PM -0700, Alex Nemirovsky wrote: > From: Abbie Chang > > Add Cortina Access Ethernet device driver for CA SoCs. > This driver supports both legacy and DM_ETH network models. > > Signed-off-by: Aaron Tseng > Signed-off-by: Alex Nemirovsky > Signed-off-by:

Re: [PATCH v3 1/3] net: cortina_ni: Add eth support for Cortina Access CAxxxx SoCs

2020-09-29 Thread Alex Nemirovsky
Abbie, Please have a look at Tom’s feedback to address his concerns. > On Sep 29, 2020, at 10:55 AM, Tom Rini wrote: > > On Wed, Sep 23, 2020 at 05:59:12PM -0700, Alex Nemirovsky wrote: > >> From: Abbie Chang >> >> Add Cortina Access Ethernet device driver for CA SoCs. >> This driver

Re: [PATCH v3 2/3] net: phy: ca_phy: Add driver for CAxxxx SoCs

2020-09-29 Thread Alex Nemirovsky
Tom, > On Sep 29, 2020, at 11:21 AM, Tom Rini wrote: > > On Tue, Sep 29, 2020 at 06:17:26PM +, Alex Nemirovsky wrote: > >> Tom, >> Yes the intent is to allow overwrite the functionality elsewhere. I agree >> that the name is too generic and should be more unique to avoid name space >>

RE: [PATCH] drivers: serial: probe all uart devices

2020-09-29 Thread Vabhav Sharma (OSS)
> -Original Message- > From: Stefan Roese > Sent: Tuesday, September 29, 2020 10:52 AM > To: Vabhav Sharma (OSS) ; > andre.przyw...@arm.com; u-boot@lists.denx.de; s...@chromium.org > Cc: Vabhav Sharma > Subject: Re: [PATCH] drivers: serial: probe all uart devices > > On 28.09.20

[PATCH v2] drivers: serial: probe all uart devices

2020-09-29 Thread Vabhav Sharma
From: Vabhav Sharma U-Boot DM model probe only single device at a time which is enabled and configured using device tree or platform data method. PL011 UART IP is SBSA compliant and firmware does the serial port set-up, initialization and let the kernel use UART port for sending and receiving

Re: [PATCH v3 1/3] net: cortina_ni: Add eth support for Cortina Access CAxxxx SoCs

2020-09-29 Thread Alex Nemirovsky
Moved my comment from prior response so that it’s more likely to be seen: Thanks for your patience on this one. I know we have been successful with most of the other code that has been submitted upstream. This one seems to be a challenge. Could you give us your honest constructive

Re: [PATCH v2 1/1] sandbox: redefine getc()

2020-09-29 Thread Tom Rini
On Tue, Sep 29, 2020 at 08:43:12PM +0200, Heinrich Schuchardt wrote: > The sandbox is built with the SDL2 library with invokes the X11 library > which in turn calls getc(). But getc() in glibc is defined as > > int getc(FILE *) > > This does not match our definition. > > int getc(void)

[PATCH 1/1] usb: xhci: avoid type conversion of void *

2020-09-29 Thread Heinrich Schuchardt
void * can be assigned to any pointer variable. Avoid unnecessary conversions. Signed-off-by: Heinrich Schuchardt --- drivers/usb/host/xhci-mem.c | 21 + 1 file changed, 9 insertions(+), 12 deletions(-) diff --git a/drivers/usb/host/xhci-mem.c b/drivers/usb/host/xhci-mem.c

Re: [PATCH 0/3] add "call" command

2020-09-29 Thread Tom Rini
On Sat, Sep 26, 2020 at 04:02:08PM +0200, Wolfgang Denk wrote: > Dear Simon, > > In message > you > wrote: > > > > I would like to see this also. Is the busybox version the latest? > > There might even be some tests although I can't see any. > > I have never been able to locate any other

[PATCH 1/1] clk: kendryte: no need to check argument of free()

2020-09-29 Thread Heinrich Schuchardt
free() checks if its argument is NULL. No need to check it twice. Signed-off-by: Heinrich Schuchardt --- drivers/clk/kendryte/clk.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/clk/kendryte/clk.c b/drivers/clk/kendryte/clk.c index 981b3b7699..26139fd804 100644

Re: [PATCH 1/1] clk: kendryte: no need to check argument of free()

2020-09-29 Thread Sean Anderson
On 9/29/20 3:52 PM, Heinrich Schuchardt wrote: > free() checks if its argument is NULL. No need to check it twice. > > Signed-off-by: Heinrich Schuchardt > --- > drivers/clk/kendryte/clk.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/clk/kendryte/clk.c

[RESEND PATCH 1/3] pinctrl: bcm283x: DM_FLAG_PRE_RELOC: Remove OF_CONTROL check

2020-09-29 Thread Ovidiu Panait
From: Ovidiu Panait Remove CONFIG_IS_ENABLED(OF_CONTROL) check from DM_FLAG_PRE_RELOC, since this driver only supports OF_CONTROL. drivers/pinctrl/broadcom/Kconfig: config PINCTRL_BCM283X depends on ARCH_BCM283X && PINCTRL_FULL && OF_CONTROL Cc: Matthias Brugger Signed-off-by: Ovidiu

[RESEND PATCH 3/3] pinctrl: bcm283x: Store the return value of dev_read_u32_default to int

2020-09-29 Thread Ovidiu Panait
From: Ovidiu Panait Currently, the return value of dev_read_u32_default is stored in an u32, causing the subsequent "if (function < 0)" to always be false: u32 function; ... function = dev_read_u32_default(config, "brcm,function", -1); if (function < 0) { debug("Failed reading function

Re: [PATCH v3 2/3] net: phy: ca_phy: Add driver for CAxxxx SoCs

2020-09-29 Thread Alex Nemirovsky
Tom, Yes the intent is to allow overwrite the functionality elsewhere. I agree that the name is too generic and should be more unique to avoid name space conflicts. Abbie, Please Review and update per Tom’s request. > On Sep 29, 2020, at 10:57 AM, Tom Rini wrote: > > On Wed, Sep 23, 2020

Re: [PATCH 1/1] usb: xhci: avoid type conversion of void *

2020-09-29 Thread Marek Vasut
On 9/29/20 10:03 PM, Heinrich Schuchardt wrote: > void * can be assigned to any pointer variable. Avoid unnecessary > conversions. > > Signed-off-by: Heinrich Schuchardt > --- > drivers/usb/host/xhci-mem.c | 21 + > 1 file changed, 9 insertions(+), 12 deletions(-) > > diff

Re: [PATCH v2 1/1] riscv: reset after crash

2020-09-29 Thread Rick Chen
Hi Heinrich > On 29.09.20 10:22, Rick Chen wrote: > >> From: Heinrich Schuchardt [mailto:xypron.g...@gmx.de] > >> Sent: Sunday, September 27, 2020 7:21 AM > >> To: Rick Jian-Zhi Chen(陳建志) > >> Cc: Simon Glass; Sean Anderson; Bin Meng; u-boot@lists.denx.de; Heinrich > >> Schuchardt > >> Subject:

Re: Pull request: u-boot-riscv/master 20200929

2020-09-29 Thread Rick Chen
Hi Tom > From: Tom Rini [mailto:tr...@konsulko.com] > Sent: Tuesday, September 29, 2020 7:57 PM > To: Open Source Project uboot > Cc: u-boot@lists.denx.de; Rick Jian-Zhi Chen(陳建志) > Subject: Re: Pull request: u-boot-riscv/master 20200929 > > On Tue, Sep 29, 2020 at

[PATCH 1/2] spl: Avoid printing boot device if silent console is enabled

2020-09-29 Thread Otavio Salvador
This avoid emitting message if silent console is in use, otherwise it pollutes the output and user desires a quiet console Signed-off-by: Otavio Salvador --- common/spl/spl.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/common/spl/spl.c b/common/spl/spl.c index

[PATCH 2/2] spl: Add SPL_SERIAL as requirement for SDP_USB_SDP

2020-09-29 Thread Otavio Salvador
The USB SDP protocol require the SPL serial support to allow the build to succeed. Signed-off-by: Otavio Salvador --- common/spl/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/common/spl/Kconfig b/common/spl/Kconfig index af8255a8d6a..20c5c4af3c3 100644 --- a/common/spl/Kconfig

Re: [PATCH v2] drivers: serial: probe all uart devices

2020-09-29 Thread Stefan Roese
On 29.09.20 19:26, Vabhav Sharma wrote: From: Vabhav Sharma U-Boot DM model probe only single device at a time which is enabled and configured using device tree or platform data method. PL011 UART IP is SBSA compliant and firmware does the serial port set-up, initialization and let the kernel

Re: [PATCH 1/2] spl: Avoid printing boot device if silent console is enabled

2020-09-29 Thread Stefan Roese
Hi Otavio, On 30.09.20 04:14, Otavio Salvador wrote: This avoid emitting message if silent console is in use, otherwise it pollutes the output and user desires a quiet console Signed-off-by: Otavio Salvador --- common/spl/spl.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)

Re: [PATCH 2/3] ARM: dts: Mediatek: add i2c node support for mt8512

2020-09-29 Thread Heiko Schocher
Hello mingming, Am 09.09.2020 um 08:07 schrieb mingming lee: From: Mingming Lee add i2c dts node support for mt8512 Signed-off-by: Mingming Lee --- arch/arm/dts/mt8512-bm1-emmc.dts | 12 arch/arm/dts/mt8512.dtsi | 38 +- 2 files

Re: [PATCH 1/3] i2c: mediatek: add basic driver support

2020-09-29 Thread Heiko Schocher
Hello mingming, Am 09.09.2020 um 08:07 schrieb mingming lee: From: Mingming Lee Add MediaTek I2C basic driver Signed-off-by: Mingming Lee --- drivers/i2c/Kconfig | 9 + drivers/i2c/Makefile | 1 + drivers/i2c/mt_i2c.c | 619 +++ 3

Re: [PATCH v2] spi: mvebu_a3700_spi: add support for cs-gpios

2020-09-29 Thread Stefan Roese
Hi George, thanks for the new version. One nitpicking comment below... On 29.09.20 20:34, George Hilliard wrote: The device tree has a way to specify GPIO lines as chip selects. From the binding docs: So if for example the controller has 2 CS lines, and the cs-gpios property looks

RE: [PATCH 07/10] ram: sifive: Default to y only if compiling for fu540

2020-09-29 Thread Pragnesh Patel
>-Original Message- >From: Sean Anderson >Sent: 29 September 2020 19:49 >To: u-boot@lists.denx.de >Cc: Bin Meng ; Rick Chen ; >Heinrich Schuchardt ; Sean Anderson >; Pragnesh Patel >Subject: [PATCH 07/10] ram: sifive: Default to y only if compiling for fu540 > >[External Email] Do not

Re: [PATCH 3/3] configs: mt8512: Enable I2C related configs

2020-09-29 Thread Heiko Schocher
Hello mingming, Am 09.09.2020 um 08:07 schrieb mingming lee: From: Mingming Lee Enable MTK I2C Enable I2C basic command Signed-off-by: Mingming Lee --- configs/mt8512_bm1_emmc_defconfig | 3 +++ 1 file changed, 3 insertions(+) Reviewed-by: Heiko Schocher bye, Heiko -- DENX Software

[PATCH] armv8: lx2162aqds: disable non existing pcie controllers

2020-09-29 Thread Wasim Khan
disable non existing pcie controllers on lx2162aqds Signed-off-by: Wasim Khan --- Depends on https://patchwork.ozlabs.org/project/uboot/list/?series=199900 arch/arm/dts/fsl-lx2162a-qds.dts | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git

[PATCH v2] distro_bootcmd: call EFI bootmgr even without having /EFI/boot

2020-09-29 Thread Michael Walle
Currently, the EFI bootmgr is only called if there is a EFI binary inside the path for removable media is found, i.e. /EFI/boot/. This doesn't make sense. It is the duty of the bootmgr to find out the path and name of the EFI binary to boot. It should be called even if there is no /EFI/boot

[PATCH] cmd: fat: Use do_save() for fatwrite

2020-09-29 Thread Lad Prabhakar
do_save() function defined in fs.c also supports FAT file system re-use the same for fatwrite command. Signed-off-by: Lad Prabhakar Reviewed-by: Biju Das --- cmd/fat.c | 43 +-- 1 file changed, 1 insertion(+), 42 deletions(-) diff --git a/cmd/fat.c

Pull request: u-boot-riscv/master 20200929

2020-09-29 Thread uboot
Hi Tom, Please pull some riscv updates: - Disable CMD_IRQ for RISC-V. - Update sipeed/maix doc - Obtain reg of SiFive RAM via dev_read_addr_index() instead of regmap API. - Cleans up RISC-V timer drivers and converts them to DM. - Correctly handle IPIs already pending upon prior stage bootloader

[PATCH v2 1/3] sunxi: dts: OrangePi Zero: Add SPI aliases to make bus usable with u-boot.

2020-09-29 Thread Michal Suchanek
The u-boot code relies on aliases to assign bus number. Signed-off-by: Michal Suchanek --- v2: move u-boot specific aliases to separate file --- arch/arm/dts/sun8i-h2-plus-orangepi-zero.dts | 1 + arch/arm/dts/sunxi-spi-u-boot.dtsi | 8 2 files changed, 9 insertions(+)