[PATCHv3] arm64: gic-v3-its: Clear the Pending table before enabling LPIs

2021-03-04 Thread Zhiqiang Hou
From: Hou Zhiqiang The GICv3 RM requires "The first 1KB of memory for the LPI Pending tables must contain only zeros on initial allocation, and this must be visible to the Redistributors, or else the effect is UNPREDICTABLE". And as the following statement, we here clear the whole Pending

RE: [PATCHv2] arm64: gic-v3-its: Clear the Pending table before enabling LPIs

2021-03-04 Thread Z.q. Hou
Hi Laurentiu, Thanks a lot for your comments! > -Original Message- > From: Laurentiu Tudor > Sent: 2021年3月5日 13:49 > To: Z.q. Hou ; u-boot@lists.denx.de > Cc: s...@chromium.org; rayagonda.kokata...@broadcom.com; Priyanka > Jain ; Vladimir Oltean > Subject: Re: [PATCHv2] arm64:

Re: [RFC PATCH u-boot 06/12] build: support building with Link Time Optimizations

2021-03-04 Thread Bin Meng
On Wed, Mar 3, 2021 at 12:13 PM Marek Behún wrote: > > Add plumbing for building U-Boot with Link Time Optimizations (gcc > only). > > Signed-off-by: Marek Behún > --- > Kbuild | 2 ++ > Kconfig | 19 +++ > Makefile| 26

Re: [PATCH] mips: mt7628: fix the displayed DDR type of mt7628

2021-03-04 Thread Stefan Roese
On 05.03.21 04:13, Weijie Gao wrote: The MT7688KN is a multi-chip package with 8MiB DDR1 KGD. So the DDR type from bootstrap register must be ignored, and always be assumed as DDR1. This patch fixes the displayed DDR type of mt7628. Signed-off-by: Weijie Gao Reviewed-by: Stefan Roese

Re: [RFC PATCH u-boot 04/12] efi_loader: fix warning when linking with LTO

2021-03-04 Thread Bin Meng
On Wed, Mar 3, 2021 at 12:13 PM Marek Behún wrote: > > When linking with LTO, the compiler complains about type mismatch of > variables `__efi_runtime_start`, `__efi_runtime_stop`, > `__efi_runtime_rel_start` and `__efi_runtime_rel_stop`: > > include/efi_loader.h:218:21: warning: type of

Re: [PATCHv2] arm64: gic-v3-its: Clear the Pending table before enabling LPIs

2021-03-04 Thread Laurentiu Tudor
Hi Zhiqiang, On 3/5/2021 4:21 AM, Zhiqiang Hou wrote: > From: Hou Zhiqiang > > The GICv3 RM requires "The first 1KB of memory for the LPI Pending tables > must contain only zeros on initial allocation, and this must be visible > to the Redistributors, or else the effect is UNPREDICTABLE". > >

Re: [PATCH] common: board_f: Restore 85xx watchdog support

2021-03-04 Thread Chris Packham
On Fri, 5 Mar 2021, 5:09 PM Simon Glass, wrote: > Hi Chris, > > On Tue, 2 Mar 2021 at 16:00, Chris Packham > wrote: > > > > In commit 75918afa649b ("powerpc: Drop old non-generic-board code") we > > lost the call to init_85xx_watchdog() which had the effect of disabling > > support for the

Re: [PATCH v5 1/3] x86: qemu: move QFW to its own uclass

2021-03-04 Thread Asherah Connor
Hi Simon, On 21/03/04 11:03:p, Simon Glass wrote: > This looks sensible but it really is a huge patch. Can it be split up a bit? This is a rejig of v4 previously Reviewed-by: you, see https://patchwork.ozlabs.org/project/uboot/list/?series=230778=%2a. Bin Meng suggested the patch be reworked

Re: [PATCH 1/3] rtc: ds1307: Add ds1339 compatible

2021-03-04 Thread Simon Glass
On Tue, 2 Mar 2021 at 20:09, Chris Packham wrote: > > As far as u-boot is concerned the ds1339 is compatible with the other U-Boot > devices supported by the ds1307 driver. The Linux driver does expose > some additional functionality but as far as u-boot is concerned just > adding the

Re: [PATCH] common: board_f: Restore 85xx watchdog support

2021-03-04 Thread Simon Glass
Hi Chris, On Tue, 2 Mar 2021 at 16:00, Chris Packham wrote: > > In commit 75918afa649b ("powerpc: Drop old non-generic-board code") we > lost the call to init_85xx_watchdog() which had the effect of disabling > support for the watchdog on 85xx and similar SoCs (i.e. the QorIQ P > Series and T

Re: [PATCH v4 35/42] test: Rename all linker lists to have a ut_ prefix

2021-03-04 Thread Simon Glass
Hi Heinrich, On Thu, 4 Mar 2021 at 10:28, Heinrich Schuchardt wrote: > > On 3/4/21 2:51 PM, Simon Glass wrote: > > At present each test suite has its own portion of the linker_list section > > of the image, but other lists are interspersed. This makes it hard to > > enumerate all the available

Re: [PATCH] Azure/GitLab: Use buildman -T0 for binman, etc, test

2021-03-04 Thread Simon Glass
Hi Tom, Bin, On Thu, 4 Mar 2021 at 22:51, Tom Rini wrote: > > On Fri, Mar 05, 2021 at 10:05:16AM +0800, Bin Meng wrote: > > Hi Tom, > > > > On Fri, Mar 5, 2021 at 6:04 AM Tom Rini wrote: > > > > > > There are times where buildman seems to get stuck in this job when in > > > CI. Forcing

Re: [PATCH 3/5] sysinfo: Require that sysinfo_detect be called before other methods

2021-03-04 Thread Simon Glass
Hi Sean, On Mon, 1 Mar 2021 at 15:46, Sean Anderson wrote: > > This updates sysinfo documentation to document that detect() must be > called first. This allows drivers to cache information in detect() and > perform (cheaper) retrieval in the other accessors. This also modifies > the only

Re: [PATCH u-boot] regmap: fix a serious pointer casting bug

2021-03-04 Thread Simon Glass
On Wed, 3 Mar 2021 at 08:16, Marek Behún wrote: > > There is a serious bug in regmap_read() and regmap_write() functions > where an uint pointer is cast to (void *) which is then cast to (u8 *), > (u16 *), (u32 *) or (u64 *), depending on register width of the map. > > For example given a regmap

Re: [PATCH 0/4] Remove addr parameter from reset_cpu()

2021-03-04 Thread Simon Glass
Hi Harald, On Tue, 2 Mar 2021 at 06:35, Harald Seiler wrote: > > Hi, > > On Tue, 2020-12-15 at 16:47 +0100, Harald Seiler wrote: > > Hi, > > > > this is something I had on my mind for a longer time but never got > > around to actually do until now ... A while back, while working on the > >

Re: [PATCH 07/20] test: dm: Add a case to test ofnode_phy_is_fixed_link()

2021-03-04 Thread Simon Glass
On Tue, 2 Mar 2021 at 10:35, Bin Meng wrote: > > This adds a test case to test the new ofnode_phy_is_fixed_link() API. > Both the new and old DT bindings are covered. > > Signed-off-by: Bin Meng > --- > > arch/sandbox/dts/test.dts | 11 +++ > test/dm/of_extra.c| 18

Re: [PATCH v5 1/3] x86: qemu: move QFW to its own uclass

2021-03-04 Thread Simon Glass
Hi Asherah, On Mon, 1 Mar 2021 at 01:35, Asherah Connor wrote: > > We move qfw into its own uclass and split the PIO functions into a > specific driver for that uclass. The PIO driver is selected in the > qemu-x86 board config (this covers x86 and x86_64). > > A sandbox driver and test are

Re: [PATCH 06/20] of: extra: Introduce ofnode_phy_is_fixed_link() API

2021-03-04 Thread Simon Glass
+Joe Hershberger On Tue, 2 Mar 2021 at 10:35, Bin Meng wrote: > > Introduce a helper API ofnode_phy_is_fixed_link() to detect whether > the ethernet controller connects to a fixed-link pseudo-PHY device. > > Note there are two ways to describe a fixed PHY attached to an > Ethernet device: > > -

Re: [PATCH 17/20] test: dm: Add a test case for simple-bus

2021-03-04 Thread Simon Glass
On Tue, 2 Mar 2021 at 10:36, Bin Meng wrote: > > This adds a test case to verify reading of a simple-bus is > working as expected. > > Signed-off-by: Bin Meng > --- > > test/dm/Makefile | 1 + > test/dm/simple-bus.c | 33 + > 2 files changed, 34

Re: [PATCH 4/5] sysinfo: Add gpio-sysinfo driver

2021-03-04 Thread Simon Glass
Hi Sean, On Mon, 1 Mar 2021 at 16:08, Sean Anderson wrote: > > > > On 3/1/21 3:46 PM, Sean Anderson wrote: > > This uses the newly-added dm_gpio_get_values_as_int_base3 function to > > implement a sysinfo device. The revision map is stored in the device tree. > > > > Signed-off-by: Sean Anderson

Re: [PATCH 5/5] test: Add gpio-sysinfo test

2021-03-04 Thread Simon Glass
On Mon, 1 Mar 2021 at 15:46, Sean Anderson wrote: > > This adds a test for the gpio-sysinfo driver. > > Signed-off-by: Sean Anderson > --- > > arch/sandbox/dts/test.dts | 7 > test/dm/Makefile | 1 + > test/dm/sysinfo-gpio.c| 69 +++ > 3

Re: [PATCH 2/5] sysinfo: Provide some global/default IDs

2021-03-04 Thread Simon Glass
Hi Sean, On Mon, 1 Mar 2021 at 15:46, Sean Anderson wrote: > > This adds an ID for a board revision. Existing IDs have been moved above > SYSINFO_ID_END to allow for future expansion. > > Signed-off-by: Sean Anderson > --- > > drivers/sysinfo/gazerbeam.h | 10 +- >

Re: [PATCH 1/5] dm: gpio: Fix gpio_get_list_count failing with livetree

2021-03-04 Thread Simon Glass
On Mon, 1 Mar 2021 at 15:46, Sean Anderson wrote: > > of_parse_phandle_with_args (called by dev_read_phandle_with_args) does not > support getting the length of a phandle list by using the index -1. > Instead, use dev_count_phandle_with_args which supports exactly this > use-case. > > Fixes:

Re: [PATCH v2] cmd: gpt: Add option to write GPT partitions to environment variable

2021-03-04 Thread Simon Glass
On Fri, 26 Feb 2021 at 13:17, Farhan Ali wrote: > > This change would enhance the existing 'gpt read' command to allow > (optionally) writing of the read GPT partitions to an environment > variable in the UBOOT partitions layout format. This would allow users > to easily change the overall

[PATCH v2] mmc: mtk-sd: increase the minimum bus frequency

2021-03-04 Thread Weijie Gao
With a 48MHz input clock, the lowest bus frequency can be as low as 4800 / (4 * 4095) = 2930Hz. Such an extremely low frequency will cause the mmc framework take seconds to finish the initialization. Limiting the minimum bus frequency to a slightly higher value can solve the issue without any

Re: [PATCH] Azure/GitLab: Use buildman -T0 for binman, etc, test

2021-03-04 Thread Tom Rini
On Fri, Mar 05, 2021 at 10:05:16AM +0800, Bin Meng wrote: > Hi Tom, > > On Fri, Mar 5, 2021 at 6:04 AM Tom Rini wrote: > > > > There are times where buildman seems to get stuck in this job when in > > CI. Forcing single-threaded here allows us to complete and move on. > > Is this a potential

Re: [PATCH] mips: mt7628: fix ddr_type for MT7688KN

2021-03-04 Thread Weijie Gao
On Fri, 2021-02-26 at 12:23 +0100, Stefan Roese wrote: > Hi Weijie, > > On 23.02.21 08:12, Weijie Gao wrote: > > The MT7688KN is a multi-chip package with 8MiB DDR1 KGD. So the DDR type > > from bootstrap register must be ignored, and always be assumed as DDR1. > > > > This patch fixes an issue

[PATCH v2] watchdog: Add booke watchdog driver

2021-03-04 Thread Chris Packham
Add a driver for the PowerPC Book E watchdog driver that is present on a number of Freescale/NXP SoCs. Signed-off-by: Chris Packham Reviewed-by: Stefan Roese --- Changes in v2: - Remove #include of common.h - Add review from Stefan drivers/watchdog/Kconfig | 7 +++

[PATCH] mips: mt7628: fix the displayed DDR type of mt7628

2021-03-04 Thread Weijie Gao
The MT7688KN is a multi-chip package with 8MiB DDR1 KGD. So the DDR type from bootstrap register must be ignored, and always be assumed as DDR1. This patch fixes the displayed DDR type of mt7628. Signed-off-by: Weijie Gao --- arch/mips/mach-mtmips/mt7628/init.c | 3 +++ 1 file changed, 3

Re: [RFC PATCH u-boot 03/12] linker_lists: declare entries and lists externally visible

2021-03-04 Thread Bin Meng
On Wed, Mar 3, 2021 at 12:13 PM Marek Behún wrote: > > Use the `__visible` macro to declare entires and lists declared by > ll_entry_declare() and ll_entry_declare_list() externally visible, so > that when building with LTO the compiler does not optimize this data > away. > __visible is defined

Re: [RFC PATCH u-boot 02/12] sandbox: errno: avoid conflict with libc's errno

2021-03-04 Thread Bin Meng
On Wed, Mar 3, 2021 at 12:13 PM Marek Behún wrote: > > When building with LTO, the system libc's `errno` variable used in > arch/sandbox/cpu/os.c conflicts with U-Boot's `errno` (defined in > lib/errno.c) with the following error: > .../ld: errno@@GLIBC_PRIVATE: TLS definition in

Re: [PATCH] mmc: mtk-sd: increase the minimum bus frequency

2021-03-04 Thread Weijie Gao
On Fri, 2021-03-05 at 11:44 +0900, Jaehoon Chung wrote: > Dear Waijie, > > On 3/5/21 11:30 AM, Weijie Gao wrote: > > With a 48MHz input clock, the lowest bus frequency can be as low as > > 4800 / (4 * 4095) = 2930Hz. Such an extremely low frequency will cause > > the mmc framework take

Re: [PATCH] mmc: mtk-sd: increase the minimum bus frequency

2021-03-04 Thread Jaehoon Chung
Dear Waijie, On 3/5/21 11:30 AM, Weijie Gao wrote: > With a 48MHz input clock, the lowest bus frequency can be as low as > 4800 / (4 * 4095) = 2930Hz. Such an extremely low frequency will cause > the mmc framework take seconds to finish the initialization. > > Limiting the minimum bus

[PATCH] configs: mt7622: enable debug uart for mt7622_rfb_defconfig

2021-03-04 Thread Weijie Gao
Enable debug uart for mt7622_rfb_defconfig Signed-off-by: Weijie Gao --- configs/mt7622_rfb_defconfig | 3 +++ 1 file changed, 3 insertions(+) diff --git a/configs/mt7622_rfb_defconfig b/configs/mt7622_rfb_defconfig index 12f529158c..c12ce5cc0a 100644 --- a/configs/mt7622_rfb_defconfig +++

[PATCH] board: mt7629: enable compression of u-boot to reduce the size of final image

2021-03-04 Thread Weijie Gao
This patch makes use of the decompression mechanism implemented for mt7628 previously to reduce the total image size. Binman will be also removed. Signed-off-by: Weijie Gao --- Makefile| 3 +++ arch/arm/dts/mt7629-rfb-u-boot.dtsi | 18 --

[PATCH] serial: mtk: rewrite the setbrg function

2021-03-04 Thread Weijie Gao
Currently the setbrg logic of serial-mtk is messy, and should be rewritten. Also an option is added to make it possible to use highspeed-3 mode for all bauds. The new logic is: 1. If baud clock > 12MHz a) If baud <= 115200, highspeed-0 mode will be used (ns16550 compatible) b) If baud <=

[PATCH] mmc: mtk-sd: increase the minimum bus frequency

2021-03-04 Thread Weijie Gao
With a 48MHz input clock, the lowest bus frequency can be as low as 4800 / (4 * 4095) = 2930Hz. Such an extremely low frequency will cause the mmc framework take seconds to finish the initialization. Limiting the minimum bus frequency to a slightly higher value can solve the issue without any

[PATCH] dts: mt7629: enable JTAG pins by default

2021-03-04 Thread Weijie Gao
The EPHY LEDs belongs to the built-in FE switch of MT7629, which is barely used. These LED pins on reference boards are used as JTAG socket. So it's a good idea to change the default state to JTAG, and this will make it convenience for debugging. Signed-off-by: Weijie Gao ---

[PATCH] configs: mt7622: use ARMv8 Generic Timer instead of mtk_timer

2021-03-04 Thread Weijie Gao
It's better to use the generic timer which is correctly initialized by the ATF. The generic timer has higher resolution than the mtk_timer. Signed-off-by: Weijie Gao --- configs/mt7622_rfb_defconfig | 2 -- 1 file changed, 2 deletions(-) diff --git a/configs/mt7622_rfb_defconfig

[PATCH 4/4] pinctrl: mt7629: add jtag function and pin group

2021-03-04 Thread Weijie Gao
The EPHY LEDs of mt7629 can be used as JTAG. This patch adds the jtag pin group to the pinctrl driver. Signed-off-by: Weijie Gao --- drivers/pinctrl/mediatek/pinctrl-mt7629.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/pinctrl/mediatek/pinctrl-mt7629.c

[PATCH 3/4] pinctrl: mediatek: do not probe gpio driver if not enabled

2021-03-04 Thread Weijie Gao
The mtk pinctrl driver is a combination driver with support for both pinctrl and gpio. When this driver is used in SPL, gpio support may not be enabled, and this will result in a compilation error. To fix this, macros are added to make sure gpio related code will only be compiled when gpio

[PATCH 2/4] pinctrl: mediatek: add get_pin_muxing ops for mediatek pinctrl

2021-03-04 Thread Weijie Gao
From: Sam Shih This patch add get_pin_muxing support for mediatek pinctrl drivers Signed-off-by: Sam Shih --- drivers/pinctrl/mediatek/pinctrl-mtk-common.c | 14 ++ 1 file changed, 14 insertions(+) diff --git a/drivers/pinctrl/mediatek/pinctrl-mtk-common.c

[PATCH 0/4] Minor fixes for MediaTek pinctrl driver

2021-03-04 Thread Weijie Gao
This patch series provides fixes for MediaTek pinctrl driver common parts and mt7629. Sam Shih (2): pinctrl: mediatek: fix wrong assignment in mtk_get_pin_name pinctrl: mediatek: add get_pin_muxing ops for mediatek pinctrl Weijie Gao (2): pinctrl: mediatek: do not probe gpio driver if not

[PATCH 1/4] pinctrl: mediatek: fix wrong assignment in mtk_get_pin_name

2021-03-04 Thread Weijie Gao
From: Sam Shih This is a bug fix for mtk pinctrl common part. Appearently pins should be used instead of grps in mtk_get_pin_name(). Signed-off-by: Sam Shih --- drivers/pinctrl/mediatek/pinctrl-mtk-common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCHv2] arm64: gic-v3-its: Clear the Pending table before enabling LPIs

2021-03-04 Thread Zhiqiang Hou
From: Hou Zhiqiang The GICv3 RM requires "The first 1KB of memory for the LPI Pending tables must contain only zeros on initial allocation, and this must be visible to the Redistributors, or else the effect is UNPREDICTABLE". And as the following statement, we here clear the whole Pending

Re: [PATCH] Azure/GitLab: Use buildman -T0 for binman, etc, test

2021-03-04 Thread Bin Meng
Hi Tom, On Fri, Mar 5, 2021 at 6:04 AM Tom Rini wrote: > > There are times where buildman seems to get stuck in this job when in > CI. Forcing single-threaded here allows us to complete and move on. Is this a potential buildman bug? +Simon My concern is this will increase the overall CI time

Re: [PATCH] Azure/GitLab: Use buildman -T0 for binman, etc, test

2021-03-04 Thread Tom Rini
On Thu, Mar 04, 2021 at 05:03:52PM -0500, Tom Rini wrote: > There are times where buildman seems to get stuck in this job when in > CI. Forcing single-threaded here allows us to complete and move on. > > Signed-off-by: Tom Rini Applied to u-boot/master, thanks! -- Tom signature.asc

Re: [PATCH 2/2] buildman: Support single-threaded operation

2021-03-04 Thread Tom Rini
On Sat, Jan 30, 2021 at 10:17:46PM -0700, Simon Glass wrote: > At present even if only a single thread is in use, buildman still uses > threading. > > For some debugging it is helpful to do everything in the main process. > Allow -T0 to support this. > > Signed-off-by: Simon Glass Applied to

Re: Please pull u-boot-video/next-2021-03-04

2021-03-04 Thread Tom Rini
On Thu, Mar 04, 2021 at 08:33:44PM +0100, Anatolij Gustschin wrote: > Hi Tom, > > The following changes since commit c5219c4a18f2b27547ecd799914f94e48b0fa86f: > > Prepare v2021.04-rc3 (2021-03-01 22:46:41 -0500) > > are available in the Git repository at: > >

Re: [PATCH] Nokia RX-51: Enable CONFIG_CMD_BOOTD

2021-03-04 Thread Tom Rini
On Wed, Feb 24, 2021 at 01:27:29AM +0100, Pali Rohár wrote: > CONFIG_CMD_BOOTD provides 'boot' command which is required in more scripts. > > Signed-off-by: Pali Rohár Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH] fs/squashfs: Fix compilation error

2021-03-04 Thread Tom Rini
On Wed, Mar 03, 2021 at 10:52:36AM +0100, Pali Rohár wrote: > Commit 401d1c4f5d2d29c4bc4beaec95402ca23eb63295 ("common: Drop > asm/global_data.h from common header") broke compilation of squashfs > filesystem when CONFIG_CMD_SQUASHFS=y is enabled. > > Compilation is failing on error: > >

Re: reserved memory from devicetree definition

2021-03-04 Thread Sean Anderson
On 3/4/21 8:04 PM, Bin Meng wrote: On Fri, Mar 5, 2021 at 8:26 AM jorge courett wrote: Hi! Is there any way to use a reserved memory definition in the device tree to avoid that u-boot could use that SDRAM memory region? Or the only way is by compiler configuration? The use case you pointed

Re: reserved memory from devicetree definition

2021-03-04 Thread Bin Meng
On Fri, Mar 5, 2021 at 8:26 AM jorge courett wrote: > > Hi! > Is there any way to use a reserved memory definition in the device tree to > avoid that u-boot could use that SDRAM memory region? Or the only way is by > compiler configuration? The use case you pointed out is not supported by

reserved memory from devicetree definition

2021-03-04 Thread jorge courett
Hi! Is there any way to use a reserved memory definition in the device tree to avoid that u-boot could use that SDRAM memory region? Or the only way is by compiler configuration? Thanks

Re: [PATCH 3/5] sysreset: provide SBI based sysreset driver

2021-03-04 Thread Sean Anderson
On 3/4/21 12:00 PM, Heinrich Schuchardt wrote: Provide sysreset driver using the SBI system reset extension. Signed-off-by: Heinrich Schuchardt --- MAINTAINERS | 1 + arch/riscv/include/asm/sbi.h| 1 + arch/riscv/lib/sbi.c| 21 +--

Re: [PATCH v3 1/4] firmware: scmi: voltage regulator

2021-03-04 Thread Etienne Carriere
Hello Patrick, On Wed, 3 Mar 2021 at 11:09, Patrick DELAUNAY wrote: > > Hi Etienne, > > On 2/22/21 8:27 AM, Etienne Carriere wrote: > > Implement voltage regulators interfaced by the SCMI voltage domain > > protocol. The DT bindings are defined in the Linux kernel since > > SCMI voltage domain

Re: [PATCH 2/5] cmd/sbi: use constants instead of numerical values

2021-03-04 Thread Sean Anderson
On 3/4/21 12:00 PM, Heinrich Schuchardt wrote: Use constants for extension IDs. Signed-off-by: Heinrich Schuchardt --- cmd/riscv/sbi.c | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/cmd/riscv/sbi.c b/cmd/riscv/sbi.c index

Re: [PATCH 1/5] risv: add missing SBI extension definitions

2021-03-04 Thread Sean Anderson
On 3/4/21 12:00 PM, Heinrich Schuchardt wrote: Add the System Reset Extension and the Hart State Management Extension definitions. Add missing RFENCE Extension enum values. The SBI 0.1 extension constants are needed for for the sbi command. Remove an #ifdef. Cf.

Re: [PATCH 5/5] maix: enable SBI system reset for MAIX

2021-03-04 Thread Sean Anderson
On 3/4/21 12:00 PM, Heinrich Schuchardt wrote: When running in S-mode we can use the SBI system reset extension to provide the system reset. Signed-off-by: Heinrich Schuchardt --- board/sipeed/maix/maix.c| 5 + configs/sipeed_maix_smode_defconfig | 2 ++ 2 files changed, 7

Re: [PATCH 4/5] pinctrl: K210_PINCTRL depends on REGMAP and on SYSCON

2021-03-04 Thread Sean Anderson
On 3/4/21 12:00 PM, Heinrich Schuchardt wrote: Select missing Kconfig dependencies. Signed-off-by: Heinrich Schuchardt --- drivers/pinctrl/Kconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/pinctrl/Kconfig b/drivers/pinctrl/Kconfig index 77fb851114..2a859ab5d0 100644 ---

Re: [PATCH 02/20] net: phy: xilinx: Convert to use APIs which support live DT

2021-03-04 Thread Vladimir Oltean
On Thu, Mar 04, 2021 at 01:36:45PM +0800, Bin Meng wrote: > It looks that I need to rebase my series on top of yours? That would be nice, thank you. It would also create some sort of incentive to finally get my patches accepted or at least looked at, they are in the 'Needs Review/Ack' state, you

Re: [RFC PATCH u-boot 00/12] U-Boot LTO (Sandbox + ARM Nokia RX-51)

2021-03-04 Thread Marek Behun
On Thu, 4 Mar 2021 16:18:03 -0600 Adam Ford wrote: > diff --git a/arch/arm/mach-omap2/omap3/Makefile > b/arch/arm/mach-omap2/omap3/Makefile > index 91ed8ebc9f..a2cc21c6d2 100644 > --- a/arch/arm/mach-omap2/omap3/Makefile > +++ b/arch/arm/mach-omap2/omap3/Makefile > @@ -6,6 +6,8 @@ > # If

Re: [RFC PATCH u-boot 00/12] U-Boot LTO (Sandbox + ARM Nokia RX-51)

2021-03-04 Thread Adam Ford
On Thu, Mar 4, 2021 at 9:59 AM Marek Behun wrote: > > On Thu, 4 Mar 2021 09:07:33 -0600 > Adam Ford wrote: > > > Interesting. with LTO enabled, the DRAM reports 7.2 GB and hangs. > > Without LTO, U-Boot boots fine. > > > > LTO: > > OMAP3630/3730-GP ES1.2, CPU-OPP2, L3-200MHz, Max CPU Clock 1

[PATCH] Azure/GitLab: Use buildman -T0 for binman, etc, test

2021-03-04 Thread Tom Rini
There are times where buildman seems to get stuck in this job when in CI. Forcing single-threaded here allows us to complete and move on. Signed-off-by: Tom Rini --- .azure-pipelines.yml | 2 +- .gitlab-ci.yml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [RFC] devices without FDT node

2021-03-04 Thread Simon Glass
Hi Heinrich, On Thu, 4 Mar 2021 at 13:10, Heinrich Schuchardt wrote: > > Am 4. März 2021 20:26:49 MEZ schrieb Simon Glass : > >Hi Heinrich, > > > >On Thu, 4 Mar 2021 at 08:01, Heinrich Schuchardt > >wrote: > >> > >> Hello Simon, > >> > >> in U-Boot we can define drivers via U_BOOT_DRVINFO and

Re: [RFC] devices without FDT node

2021-03-04 Thread Heinrich Schuchardt
Am 4. März 2021 20:26:49 MEZ schrieb Simon Glass : >Hi Heinrich, > >On Thu, 4 Mar 2021 at 08:01, Heinrich Schuchardt >wrote: >> >> Hello Simon, >> >> in U-Boot we can define drivers via U_BOOT_DRVINFO and U_BOOT_DRIVER. >> >> My current use case is SBI system reset extension. A driver would not

Re: [PATCH] imx: Add IMX_HAB support for i.MX7ULP and i.MX8M

2021-03-04 Thread Tom Rini
On Thu, Mar 04, 2021 at 08:17:42AM -0300, Fabio Estevam wrote: > i.MX7ULP and i.MX8M may also use HAB. Add support for these > platforms. > > Signed-off-by: Fabio Estevam > --- > arch/arm/mach-imx/Kconfig| 2 +- > arch/arm/mach-imx/imx8m/Kconfig | 1 + >

Re: Re: [PATCH 07/19] video: sunxi: Remove check for ddc-i2c-bus property

2021-03-04 Thread Jernej Škrabec
Hi! Dne četrtek, 04. marec 2021 ob 02:42:21 CET je Andre Przywara napisal(a): > On Tue, 23 Feb 2021 21:46:19 +0100 > Jernej Skrabec wrote: > > > No Allwinner boards with DW-HDMI controller use separate I2C bus for > > EDID read. Remove that check. > > Apparently the Cubieboard 4 has (as the

Please pull u-boot-video/next-2021-03-04

2021-03-04 Thread Anatolij Gustschin
Hi Tom, The following changes since commit c5219c4a18f2b27547ecd799914f94e48b0fa86f: Prepare v2021.04-rc3 (2021-03-01 22:46:41 -0500) are available in the Git repository at: https://gitlab.denx.de/u-boot/custodians/u-boot-video.git tags/next-2021-03-04 for you to fetch changes up to

Re: [RFC] devices without FDT node

2021-03-04 Thread Simon Glass
Hi Heinrich, On Thu, 4 Mar 2021 at 08:01, Heinrich Schuchardt wrote: > > Hello Simon, > > in U-Boot we can define drivers via U_BOOT_DRVINFO and U_BOOT_DRIVER. > > My current use case is SBI system reset extension. A driver would not > reasonably depend on the device-tree because if the RISC-V

[PATCH] sandbox: add FAT to the list of usable env drivers

2021-03-04 Thread Heinrich Schuchardt
Add the FAT environment driver to the priority list. When testing the UEFI sub-system the EFI system partition is formatted with FAT so it is reasonable to store the environment there. Signed-off-by: Heinrich Schuchardt --- board/sandbox/sandbox.c | 1 + 1 file changed, 1 insertion(+) diff

Re: [RFC PATCH u-boot 01/12] build: use thin archives instead of incremental linking

2021-03-04 Thread Marek Behun
On Thu, 4 Mar 2021 18:57:11 +0800 Bin Meng wrote: > Hi Marek, > > On Wed, Mar 3, 2021 at 12:13 PM Marek Behún wrote: > > > > Using thin archives instead of incremental linking > > - saves disk space > > - works better with dead code elimination > > - prepares for potential LTO > > The

Re: [PATCH v4 16/16] gpio: Add a way to read 3-way strapping pins

2021-03-04 Thread Tom Rini
On Thu, Feb 04, 2021 at 09:22:09PM -0700, Simon Glass wrote: > Using the internal vs. external pull resistors it is possible to get > 27 different combinations from 3 strapping pins. Add an implementation > of this. > > This involves updating the sandbox GPIO driver to model external and >

Re: [PATCH v4 15/16] gpio: Define the log category in the uclass

2021-03-04 Thread Tom Rini
On Thu, Feb 04, 2021 at 09:22:08PM -0700, Simon Glass wrote: > This uses log_debug(), etc. but does not define the category. Fix this. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v4 14/16] gpio: sandbox: Track whether a GPIO is driven

2021-03-04 Thread Tom Rini
On Thu, Feb 04, 2021 at 09:22:07PM -0700, Simon Glass wrote: > Add a new flag to keep track of whether sandbox is driving the pin, or > whether it is expecting an input signal. If it is driving, then the value > of the pin is the value being driven (0 or 1). If not driving, then we > consider the

Re: [PATCH v4 13/16] gpio: x86: Drop the deprecated methods in intel_gpio

2021-03-04 Thread Tom Rini
On Thu, Feb 04, 2021 at 09:22:06PM -0700, Simon Glass wrote: > We don't need to implement direction_input() and direction_output() > anymore. Drop them and use update_flags() instead. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP

Re: [PATCH v4 12/16] gpio: Use an 'ops' variable everywhere

2021-03-04 Thread Tom Rini
On Thu, Feb 04, 2021 at 09:22:05PM -0700, Simon Glass wrote: > Update this driver to use the common method of putting the driver > operations in an 'ops' variable install of calling gpio_get_ops() > repeatedly. Make it const since operations do not change. > > Signed-off-by: Simon Glass > >

Re: [PATCH v4 11/16] gpio: Replace direction_input() and direction_output()

2021-03-04 Thread Tom Rini
On Thu, Feb 04, 2021 at 09:22:04PM -0700, Simon Glass wrote: > The new update_flags() method is more flexible since it allows the > driver to see the full flags all at once. Use that in preference to these > two functions. Add comments to that effect. > > Signed-off-by: Simon Glass > >

Re: [PATCH v4 10/16] dm: gpio: Add a way to update flags

2021-03-04 Thread Tom Rini
On Thu, Feb 04, 2021 at 09:22:03PM -0700, Simon Glass wrote: > It is convenient to be able to adjust some of the flags for a GPIO while > leaving others alone. Add a function for this. > > Update dm_gpio_set_dir_flags() to make use of this. > > Also update dm_gpio_set_value() to use this also,

Re: [PATCH v4 09/16] gpio: sandbox: Make sandbox_gpio_set_flags() set all flags

2021-03-04 Thread Tom Rini
On Thu, Feb 04, 2021 at 09:22:02PM -0700, Simon Glass wrote: > Allow this function to see all flags, including the internal sandbox ones. > This allows the tests to fully control the behaviour of the driver. > > To make this work, move the setting of GPIOD_EXT_HIGH -to where the flags > are

Re: [PATCH v4 08/16] gpio: sandbox: Fully separate pin value from output value

2021-03-04 Thread Tom Rini
On Thu, Feb 04, 2021 at 09:22:01PM -0700, Simon Glass wrote: > At present we have the concept of a pin's external value. This is what > is used when getting the value of a pin. But we still set the > GPIOD_IS_OUT_ACTIVE flag when changing the value. This is not actually > correct, since if the

Re: [PATCH v4 42/42] doc: Explain briefly how to write new tests

2021-03-04 Thread Heinrich Schuchardt
On 04.03.21 14:51, Simon Glass wrote: > Add a second on writing tests, covering when to use Python and C, where > to put the tests, etc. Add a link to the existing Python test > documentation. > > Signed-off-by: Simon Glass > --- > > Changes in v4: > - Add correct SPDX header > > Changes in v2: >

Re: [PATCH v4 07/16] gpio: sandbox: Use a separate flag for the value

2021-03-04 Thread Tom Rini
On Thu, Feb 04, 2021 at 09:22:00PM -0700, Simon Glass wrote: > At present with the sandbox GPIO driver it is not possible to change the > value of GPIOD_IS_OUT_ACTIVE unless the GPIO is an output. This makes it > hard to test changing the flags since we need to be aware of the internal > workings

Re: [PATCH v4 06/16] gpio: sandbox: Rename GPIO dir_flags to flags

2021-03-04 Thread Tom Rini
On Thu, Feb 04, 2021 at 09:21:59PM -0700, Simon Glass wrote: > Adjust the terminology in this driver to reflect that fact that all flags > are handled, not just direction flags. > > Create a new access function to get the full GPIO state, not just the > direction flags. Drop the static

Re: [PATCH v4 04/16] gpio: Rename dm_gpio_get_dir_flags() to dm_gpio_get_flags()

2021-03-04 Thread Tom Rini
On Thu, Feb 04, 2021 at 09:21:57PM -0700, Simon Glass wrote: > This function can be used to get any flags, not just direction flags. > Rename it to avoid confusion. > > Signed-off-by: Simon Glass > Reviewed-by: Patrick Delaunay > Reviewed-by: Pratyush Yadav Applied to u-boot/next, thanks!

Re: [PATCH v4 03/16] dm: gpio: Rename get_dir_flags() method to get_flags()

2021-03-04 Thread Tom Rini
On Thu, Feb 04, 2021 at 09:21:56PM -0700, Simon Glass wrote: > It is more useful to be able to read all the flags, not just the direction > ones. In fact this is what the STM32 driver does. Update the method name > to reflect this. > > Tweak the docs a little and use 'flagsp' as the return

Re: [PATCH v4 02/16] dm: gpio: Rename set_dir_flags() method to update_flags()

2021-03-04 Thread Tom Rini
On Thu, Feb 04, 2021 at 09:21:55PM -0700, Simon Glass wrote: > The current method is a misnomer since it is also used (e.g. by stm32) to > update pull settings and open source/open drain. > > Rename it and expand the documentation to cover a few more details. > > Signed-off-by: Simon Glass >

Re: [PATCH v4 01/16] gpio: Disable functions not used with of-platdata

2021-03-04 Thread Tom Rini
On Thu, Feb 04, 2021 at 09:21:54PM -0700, Simon Glass wrote: > These functions use devicetree and cannot work with of-platdata, which has > no runtime devicetree. > > If they are used, the current linker error is confusing, since it talks > about missing functions in the bowels of driver model.

Re: Please pull u-boot-dm

2021-03-04 Thread Tom Rini
On Wed, Mar 03, 2021 at 08:27:35PM -0700, Simon Glass wrote: > Hi Tom, > > hhttps://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/6610 > > > The following changes since commit 52ba373b7825e9feab8357065155cf43dfe2f4ff: > > Merge tag 'u-boot-imx-20210303' of >

Re: [PATCH 06/19] video: sunxi: Use DW-HDMI hpd function

2021-03-04 Thread Jernej Škrabec
Dne četrtek, 04. marec 2021 ob 02:42:08 CET je Andre Przywara napisal(a): > On Tue, 23 Feb 2021 21:46:18 +0100 > > Jernej Skrabec wrote: > > It turns out that even though A64, H3 and H5 have custom PHY, standard > > hot plug detection for DW-HDMI works just fine. > > But the generic code looks

Re: [PATCH v4 39/42] doc: Move coccinelle into its own section

2021-03-04 Thread Heinrich Schuchardt
On 04.03.21 14:51, Simon Glass wrote: > This tool has nothing to do with testing. Create a new section for > 'refactoring' and move it into there. It is likely that other topics may > fall under the same heading, such as using moveconfig and search/replace > tools. Coccinelle makes static code

Re: [PATCH v4 33/42] test: log: Rename log main test file to log_ut.c

2021-03-04 Thread Heinrich Schuchardt
On 04.03.21 14:51, Simon Glass wrote: > The current name is the same as the main test runner file. Rename it to > avoid confusion. > > Signed-off-by: Simon Glass Reviewed-by: Heinrich Schuchardt

[PATCH] MAINTAINERS: Use my personal e-mail address

2021-03-04 Thread Fabio Estevam
Use my personal e-mail address for U-Boot related work. Signed-off-by: Fabio Estevam --- board/freescale/mx28evk/MAINTAINERS| 2 +- board/freescale/mx6sabreauto/MAINTAINERS | 2 +- board/freescale/mx6sabresd/MAINTAINERS | 2 +- board/freescale/mx6slevk/MAINTAINERS | 2 +-

[PATCH 1/5] risv: add missing SBI extension definitions

2021-03-04 Thread Heinrich Schuchardt
Add the System Reset Extension and the Hart State Management Extension definitions. Add missing RFENCE Extension enum values. The SBI 0.1 extension constants are needed for for the sbi command. Remove an #ifdef. Cf. https://github.com/riscv/riscv-sbi-doc/blob/master/riscv-sbi.adoc

[PATCH 2/5] cmd/sbi: use constants instead of numerical values

2021-03-04 Thread Heinrich Schuchardt
Use constants for extension IDs. Signed-off-by: Heinrich Schuchardt --- cmd/riscv/sbi.c | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/cmd/riscv/sbi.c b/cmd/riscv/sbi.c index 90c0811e14..65a2c93290 100644 --- a/cmd/riscv/sbi.c +++

[PATCH 3/5] sysreset: provide SBI based sysreset driver

2021-03-04 Thread Heinrich Schuchardt
Provide sysreset driver using the SBI system reset extension. Signed-off-by: Heinrich Schuchardt --- MAINTAINERS | 1 + arch/riscv/include/asm/sbi.h| 1 + arch/riscv/lib/sbi.c| 21 +-- drivers/sysreset/Kconfig| 11

[PATCH 4/5] pinctrl: K210_PINCTRL depends on REGMAP and on SYSCON

2021-03-04 Thread Heinrich Schuchardt
Select missing Kconfig dependencies. Signed-off-by: Heinrich Schuchardt --- drivers/pinctrl/Kconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/pinctrl/Kconfig b/drivers/pinctrl/Kconfig index 77fb851114..2a859ab5d0 100644 --- a/drivers/pinctrl/Kconfig +++

[PATCH 5/5] maix: enable SBI system reset for MAIX

2021-03-04 Thread Heinrich Schuchardt
When running in S-mode we can use the SBI system reset extension to provide the system reset. Signed-off-by: Heinrich Schuchardt --- board/sipeed/maix/maix.c| 5 + configs/sipeed_maix_smode_defconfig | 2 ++ 2 files changed, 7 insertions(+) diff --git a/board/sipeed/maix/maix.c

[PATCH 0/5] riscv: enable SBI system reset

2021-03-04 Thread Heinrich Schuchardt
The purpose of this series is to provide the UEFI ResetSystem() service at runtime on RISC-V systems. With SBI v0.3 a system reset extension is available. This allows to implement reboot and poweroff in U-Boot in a system independent way. OpenSBI already provides an implementation of the

Re: [PATCH] cmd: xtrace: Convert to bool

2021-03-04 Thread Heinrich Schuchardt
On 3/4/21 5:34 PM, Sean Anderson wrote: This variable is a boolean, not a string. Signed-off-by: Sean Anderson Reviewed-by: Heinrich Schuchardt

  1   2   3   >