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
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:
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
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
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
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".
>
>
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
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
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
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
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
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
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
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
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
> >
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
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
+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:
>
> -
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
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
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
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 +-
>
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:
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
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
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
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
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 +++
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
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
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
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
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
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
+++
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 --
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 <=
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
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
---
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
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
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
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
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
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
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
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
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
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
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:
>
>
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
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:
>
>
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
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
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
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 +--
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
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
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.
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
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
---
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
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
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
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
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
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
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 +
>
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
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
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
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
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
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
>
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
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
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
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
>
>
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
>
>
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,
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
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
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:
>
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
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
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!
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
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
>
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.
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
>
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
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
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
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 +-
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
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
+++
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
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
+++
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
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
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 - 100 of 206 matches
Mail list logo