On Fri, May 21, 2021 at 12:16 PM Biwen Li wrote:
>
> From: Biwen Li
>
> IFC NOR flash base address of ls2088a is 0x85000,
beyond 4GiB?
> and env crc offset size is 0x50, so fix the macro
> CONFIG_ENV_ADDR to fix synchronous exception(access illegal address)
>
> Fixes: 59071804c1
From: Biwen Li
IFC NOR flash base address of ls2088a is 0x85000,
and env crc offset size is 0x50, so fix the macro
CONFIG_ENV_ADDR to fix synchronous exception(access illegal address)
Fixes: 59071804c1 ("configs: ls2080a: Correct ENV_ADDR value")
Signed-off-by: Biwen Li
---
Hi Alex,
On Thu, 20 May 2021 at 17:13, Alex G. wrote:
>
>
>
> On 5/20/21 12:52 PM, Simon Glass wrote:
> > Hi Alex,
> >
> > On Wed, 19 May 2021 at 20:41, Alex G. wrote:
> >>
> >>
> >>
> >> On 5/19/21 4:55 PM, Simon Glass wrote:
> >>> Hi Alex,
> >>>
> >>> On Wed, 19 May 2021 at 11:44, Alex G
On 5/20/21 12:52 PM, Simon Glass wrote:
Hi Alex,
On Wed, 19 May 2021 at 20:41, Alex G. wrote:
On 5/19/21 4:55 PM, Simon Glass wrote:
Hi Alex,
On Wed, 19 May 2021 at 11:44, Alex G wrote:
On 5/19/21 11:36 AM, Simon Glass wrote:
Hi Alexandru,
On Mon, 17 May 2021 at 10:38,
On 5/20/21 6:17 PM, Simon Glass wrote:
Hi Alex,
On Thu, 20 May 2021 at 17:13, Alex G. wrote:
On 5/20/21 12:52 PM, Simon Glass wrote:
Hi Alex,
On Wed, 19 May 2021 at 20:41, Alex G. wrote:
On 5/19/21 4:55 PM, Simon Glass wrote:
Hi Alex,
On Wed, 19 May 2021 at 11:44, Alex G
On Thu, May 20, 2021 at 3:41 PM Green Wan wrote:
>
> Add dts files for SiFive Unmatched board.
>
> Signed-off-by: Green Wan
> Reviewed-by: Rick Chen
> ---
> arch/riscv/dts/Makefile |1 +
> .../dts/fu740-hifive-unmatched-a00-ddr.dtsi | 1489 +
>
Hi Simon,
On 5/20/21 12:17 AM, Simon Glass wrote:
> Hi Michal,
>
> On Wed, 19 May 2021 at 01:16, Michal Simek wrote:
>>
>> Hi Simon,
>>
>> On 5/8/21 12:37 AM, Simon Glass wrote:
>>> U-Boot provides a verified-boot feature based around FIT, but there is
>>> no standard way of implementing it for
On Thu, May 20, 2021 at 11:27 AM Bin Meng wrote:
> Hi Green,
>
> On Thu, May 20, 2021 at 11:10 AM Green Wan wrote:
> >
> > Hi Bin,
> >
> > Thanks for the comment. Originally, I wanted to keep the binman change
> tracked separately. I'll give it a try and create the new patch if it
> doesn't.
>
This patch set is to add SiFive fu740 chip and HiFive Unmatched board
support. Patches are split into several parts:
- [PATCH v11 1/8] support for fu740 cpu
- [PATCH v11 2/8] support for fu740 clk driver
- [PATCH v11 3/8] rename and support for fu740 ram driver
- [PATCH v11 4/8] add pcie
Add SiFive fu740 cpu to support RISC-V arch
Signed-off-by: Green Wan
Reviewed-by: Bin Meng
---
arch/riscv/Kconfig| 1 +
arch/riscv/cpu/fu740/Kconfig | 37 +++
arch/riscv/cpu/fu740/Makefile | 12 +
arch/riscv/cpu/fu740/cache.c
Rename fu540_ddr.c to sifive_ddr.c and add fu740 support
Signed-off-by: Green Wan
Reviewed-by: Bin Meng
---
drivers/ram/sifive/Kconfig| 8 +-
drivers/ram/sifive/Makefile | 2 +-
.../ram/sifive/{fu540_ddr.c => sifive_ddr.c} | 89 ++-
3
Add pcie driver for SiFive fu740, the driver depends on
fu740 gpio, clk and reset driver to do init. Force running at Gen1
for better capatible enumeration.
Several devices are tested:
a) M.2 NVMe SSD
b) USB-to-PCI adapter
c) Ethernet adapter (E1000 compatible)
Signed-off-by: Green Wan
Add fu740 support. One abstract layer is added for supporting
multiple chips such as fu540 and fu740.
Signed-off-by: Green Wan
---
arch/riscv/cpu/fu540/Kconfig | 2 +-
drivers/clk/sifive/Kconfig | 8 +-
drivers/clk/sifive/Makefile | 4 +-
drivers/clk/sifive/fu540-prci.c |
Clear feature disable CSR to turn on all features of hart. The detail
is specified at section, 'SiFive Feature Disable CSR', in user manual
https://sifive.cdn.prismic.io/sifive/aee0dd4c-d156-496e-a6c4-db0cf54bbe68_sifive_U74MC_rtl_full_20G1.03.00_manual.pdf
Signed-off-by: Green Wan
Reviewed-by:
Add dts support for fu740. The HiFive Unmatched support is based on
fu740 cpu and drivers in following patch set.
Signed-off-by: Green Wan
[greentime.hu: set fu740 speed to 1.2GHz]
Signed-off-by: Greentime Hu
Reviewed-by: Bin Meng
Reviewed-by: Rick Chen
---
Add defconfig and board support for HiFive Unmatched.
Signed-off-by: Green Wan
Reviewed-by: Bin Meng
Reviewed-by: Rick Chen
---
arch/riscv/Kconfig | 4 +
board/sifive/unleashed/Kconfig | 1 +
board/sifive/unmatched/Kconfig | 51 +++
Add dts files for SiFive Unmatched board.
Signed-off-by: Green Wan
Reviewed-by: Rick Chen
---
arch/riscv/dts/Makefile |1 +
.../dts/fu740-hifive-unmatched-a00-ddr.dtsi | 1489 +
.../dts/hifive-unmatched-a00-u-boot.dtsi | 41 +
On 13/05/2021 22.02, han.xu wrote:
> On 21/05/11 07:08AM, Sean Nyekjaer wrote:
>> Caution: EXT Email
>>
>> On 11/05/2021 04.49, han.xu wrote:
>>> On 21/05/10 12:00PM, Sean Nyekjaer wrote:
Caution: EXT Email
Linux kernel defaults to use legacy bch setting, this was creating a
I removed CONFIG_IS_BUILTIN and CONFIG_IS_MODULE in commit
7d78a4547d ("linux/kconfig.h: remove unused helper macros"), but
fixdep.c still looks for those. It's harmless, but also pointless and
possibly confusing to a future reader.
Fixes: 7d78a4547d ("linux/kconfig.h: remove unused helper
The first patch ensures that vsnprintf() never returns a negative
value. Finding (indirect) callers and having them all handle that
would be a whack-a-mole game for no real gain. Even if the callers
don't directly use the return value, they might rely on vsnprintf()
always producing a
Le 17/05/2021 à 23:20, Joao Marcos Costa a écrit :
The fragmented files were not correctly read because of two issues:
- The squashfs_file_info struct has a field named 'comp', which tells if
the file's fragment is compressed or not. This field was always set to
'true' in sqfs_get_regfile_info
We have to use the *printf variant which returns the strlen() of the
resulting string, not the
how-much-would-have-been-printed-given-enough-space, in order to use
that value to inspect the last character in the string.
Signed-off-by: Rasmus Villemoes
---
common/log.c | 2 +-
1 file changed, 1
This saves some code, both in terms of #LOC and .text size, and it is
also the normal convention that foo(...) is implemented in terms of
vfoo().
Signed-off-by: Rasmus Villemoes
---
lib/vsprintf.c | 10 +-
1 file changed, 1 insertion(+), 9 deletions(-)
diff --git a/lib/vsprintf.c
U-Boot doesn't support %pS/%pF or any other kind of kallsyms-like
lookups. Remove the comment.
Signed-off-by: Rasmus Villemoes
---
lib/vsprintf.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/lib/vsprintf.c b/lib/vsprintf.c
index e3bec7489b..65d985982d 100644
--- a/lib/vsprintf.c
+++
Most callers (or callers of callers, etc.) of vsnprintf() are not
prepared for it to return a negative value.
The only case where that can currently happen is %pD, and it's IMO
more user-friendly to produce some output that clearly shows that some
"impossible" thing happened instead of having the
There's currently no user of %p[iI]6, so including ip6_addr_string()
in the image is a waste of bytes. It's easy enough to have the
compiler elide it without removing the code completely.
The closest I can find to anybody "handling" ipv6 in U-Boot currently
is in efi_net.c which does
if
Make LTO available for sandbox architecture.
Signed-off-by: Marek Behún
Reviewed-by: Bin Meng
---
arch/Kconfig | 1 +
arch/sandbox/config.mk | 4
2 files changed, 5 insertions(+)
diff --git a/arch/Kconfig b/arch/Kconfig
index 6c4b81a486..c5c03d438c 100644
--- a/arch/Kconfig
+++
Add $(CFLAGS_EFI) and remove $(CFLAGS_NON_EFI) for
efi_selftest_miniapp_exception.o.
The removal is needed when compiling with LTO - this object file needs
to be compiled without -flto.
The adding is for consistency with other miniapps.
Signed-off-by: Marek Behún
Reviewed-by: Heinrich
Document the macros __efi_runtime and __efi_runtime_data in Sphinx
style.
Signed-off-by: Marek Behún
Reviewed-by: Heinrich Schuchardt
---
include/efi_loader.h | 29 ++---
1 file changed, 26 insertions(+), 3 deletions(-)
diff --git a/include/efi_loader.h
Some toolchains are compiled so that they pass a --build-id=something
parameter to the linker implicitly.
This causes U-Boot LTO linking to fail with something like:
ld: section .note.gnu.build-id LMA ... overlaps section .text LMA ...
because U-Boot's link scripts do not currently handle
On ARM, the gd pointer is stored in registers r9 / x18. For this the
-ffixed-r9 / -ffixed-x18 flag is used when compiling, but using global
register variables causes errors when building with LTO, and these
errors are very difficult to overcome.
Richard Biener says [1]:
Note that global
When building keystone with LTO the compiler complains:
Error: selected processor does not support `smc #0' in Thumb mode
Fix this by removing -flto for the file implementing these SMC calls.
Signed-off-by: Marek Behún
---
arch/arm/mach-keystone/Makefile | 1 +
1 file changed, 1 insertion(+)
When imx28_xea_defconfig is built with LTO, the compiler complains about
the two different declarations of _start:
include/asm-generic/sections.has extern void _start(void);
arch/arm/cpu/arm926ejs/mxs/mxs.c as extern uint32_t _start;
Fix this.
Signed-off-by: Marek Behún
The Thumb instruction `ldr` is able to move high registers only from
armv7. For armv5 and armv6 we have to use `mov`.
Signed-off-by: Marek Behún
---
arch/arm/include/asm/global_data.h | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/arch/arm/include/asm/global_data.h
Build sandbox targets with LTO by default.
Signed-off-by: Marek Behún
Reviewed-by: Bin Meng
---
arch/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/Kconfig b/arch/Kconfig
index c5c03d438c..8fd39e36b4 100644
--- a/arch/Kconfig
+++ b/arch/Kconfig
@@ -125,6 +125,7 @@ config
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 /lib64/libc.so.6
section .tbss mismatches non-TLS reference in
Fix LTO build for some thumb-interwork usecases (such as for
da850evm_defconfig), where inline assmebly such as
mrc p15,0,r2,c1,c0,0
causes the compiler to fail during LTO linking with
Error: selected processor does not support `mrc p15,0,r2,c1,c0,0'
in Thumb mode
Signed-off-by:
Adam Ford says that DM3730 needs board.c compiled without LTO flags.
Signed-off-by: Marek Behún
Tested-by: Adam Ford
---
arch/arm/mach-omap2/omap3/Makefile | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/mach-omap2/omap3/Makefile
b/arch/arm/mach-omap2/omap3/Makefile
index
When building with LTO, the compiler complains about type mismatch of
function clk_bsc_enable() in file:
arch/arm/cpu/armv7/kona-common/clk-stubs.c
vs other files that define or use this function:
warning: type of ‘clk_bsc_enable’ does not match original declaration.
Change the type of this
When building highbank_defconfig with LTO, the compiler complains about
type mismatch of function ahci_link_up().
The third parameter of this function is of type u8 in
drivers/ata/ahci.c, but of type int in board/highbank/ahci.c.
There is no reason in using u8, and the code using this function
When building with LTO, the compiler complains about type mismatch of
function usb_gadget_handle_interrupts(). This function is defined
without parameters in files
arch/arm/mach-rockchip/board.c
board/samsung/common/exynos5-dt.c
but it should have one parameter, int index.
Fix this.
When building imx8mp_evk_defconfig with LTO, the compiler complains
about type mismatch of function imx_eqos_txclk_set_rate() in file
drivers/net/dwc_eth_qos.c:845:12
which contains a weak definition of this function, vs file
arch/arm/mach-imx/imx8m/clock_imx8mm.c
which contains an
When seaboard_defconfig is compiled with LTO, the compiler complains
about some instructions not being supported in ARM mode.
This is caused by arch/arm/mach-tegra/tegra20/warmboot_avp.c having
different CFLAGS declared in Makefile. This file needs to be compiled
without LTO.
Fix this by
When compiling with LTO, the compiler fails with an error saying that
`crc_table` causes a section type conflict with `efi_var_buf`.
This is because both are declared to be in the same section (via macro
`__efi_runtime_data`), but one is const while the other is not.
Put this variable into the
A lot of entries were using spaces instead of tab for alignment that's why
it is good to fix it to use the same style everywhere.
Signed-off-by: Michal Simek
---
common/spl/Kconfig | 24
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/common/spl/Kconfig
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 with 16-bit register width the code
int val =
Hello,
this is version 4 of patches adding support for LTO to U-Boot.
This series is being tested by Github/Azure CI at
https://github.com/u-boot/u-boot/pull/57
There is a problem with sandbox_clang test scenario, which I was
unable to resolve yet, or even determine correctly whether the
Use UNIQUE_ID in the __ADDRESSABLE macro.
Signed-off-by: Marek Behún
Reviewed-by: Bin Meng
---
include/linux/compiler.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/compiler.h b/include/linux/compiler.h
index 82a8a4ede9..98dd3fc4cc 100644
---
This commit does the same thing as Linux commit 33def8498fdd.
Use a more generic form for __section that requires quotes to avoid
complications with clang and gcc differences.
Remove the quote operator # from compiler_attributes.h __section macro.
Convert all unquoted __section(foo) uses to
This is how Linux does this now, see Linux commit 339f29d91acf.
Signed-off-by: Marek Behún
---
scripts/checkpatch.pl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 59a714a95f..08a827535a 100755
--- a/scripts/checkpatch.pl
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 ‘__efi_runtime_start’
does not
Improve the regular expression that matches unittest symbols in
u-boot.sym.
Currently we do not enforce no prefix in symbol string, but with the
soon to come change in linker lists declaring lists and entries with the
__ADDRESSABLE macro (because of LTO), the symbol file will contain for
every
It seems that sometimes (happening on ARM64, for example with
turris_mox_defconfig) GCC, when linking with LTO, changes the symbol
names of some functions, for example lib/string.c's memcpy() function to
memcpy.isra.0.
This is a problem however when GCC for a code such as this:
struct
When using LTO, we can throw away the --gc-sections flag, but only if
using private libgcc.
When using system's libgcc, --gc-sections is still needed, otherwise
linking will fail due to undefined references to libc's symbols.
Signed-off-by: Marek Behún
---
arch/arm/config.mk | 2 ++
1 file
When building with LTO, using -ffunction-sections/-fdata-sections is not
useful anymore.
Signed-off-by: Marek Behún
---
arch/arm/config.mk | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/arch/arm/config.mk b/arch/arm/config.mk
index e79f0104b9..f556ee3329 100644
---
Make LTO available for ARM architecture.
Signed-off-by: Marek Behún
---
arch/Kconfig | 1 +
arch/arm/lib/Makefile | 2 ++
2 files changed, 3 insertions(+)
diff --git a/arch/Kconfig b/arch/Kconfig
index 8fd39e36b4..49813a4c9b 100644
--- a/arch/Kconfig
+++ b/arch/Kconfig
@@ -33,6 +33,7
For some reason when building SPL for ARMv8 with LTO, the relocation
information is not discarded.
Discard it explicitly in the linker script.
This fixes LTO build for imx8mm_venice_defconfig.
Signed-off-by: Marek Behún
---
arch/arm/cpu/armv8/u-boot-spl.lds | 1 +
1 file changed, 1
Enable LTO for some boards that were tested by people on U-Boot Mailing
List.
Signed-off-by: Marek Behún
Tested-by: Adam Ford
Tested-by: Pali Rohár
Tested-by: Tim Harvey
---
configs/am3517_evm_defconfig | 1 +
configs/da850evm_defconfig| 1 +
Add plumbing for building U-Boot with Link Time Optimizations.
When building with LTO, $(PLATFORM_LIBS) has to be in --whole-archive /
--no-whole-archive group, otherwise some functions declared in assembly
may not be resolved and linking may fail.
Note: clang may throw away linker list symbols
Currently we use incremental linking (ld -r) to link several object
files from one directory into one built-in.o object file containing the
linked code from that directory (and its subdirectories).
Linux has, some time ago, moved to thin archives instead.
Thin archives are archives (.a) that do
Add macro __efi_runtime_rodata, for const variables with similar purpose
as those using __efi_runtime_data.
Signed-off-by: Marek Behún
Reviewed-by: Heinrich Schuchardt
---
include/efi_loader.h | 17 +
1 file changed, 17 insertions(+)
diff --git a/include/efi_loader.h
When apf27_defconfig is built with LTO, linking complains about
undefined reference to `nand_boot`. This is because it is referenced
from inline assembly. Make it visible.
Signed-off-by: Marek Behún
---
drivers/mtd/nand/raw/mxc_nand_spl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
In style of linked lists, instead of declaring symbols for boundaries
of getopt options array in the linker script, declare corresponding
sections and retrieve the boundaries via static inline functions.
Without this clang's LTO produces binary without any getopt options,
because for some reason
Indent the linking commands so that they look cosmetically better.
Signed-off-by: Marek Behún
Reviewed-by: Bin Meng
---
Makefile | 12 +++-
scripts/Makefile.spl | 17 +++--
2 files changed, 18 insertions(+), 11 deletions(-)
diff --git a/Makefile b/Makefile
Usb is now supported so add all required nodes for it in imxrt1050.
Signed-off-by: Giulio Benetti
---
arch/arm/dts/imxrt1050.dtsi | 27 +++
1 file changed, 27 insertions(+)
diff --git a/arch/arm/dts/imxrt1050.dtsi b/arch/arm/dts/imxrt1050.dtsi
index
Enable usbotg1 port node as host usb.
Signed-off-by: Giulio Benetti
---
arch/arm/dts/imxrt1050-evk.dts | 5 +
1 file changed, 5 insertions(+)
diff --git a/arch/arm/dts/imxrt1050-evk.dts b/arch/arm/dts/imxrt1050-evk.dts
index 81db1a446d..324cf7af96 100644
---
Now that usb host is supported let's enable it on this board.
Signed-off-by: Giulio Benetti
---
configs/imxrt1050-evk_defconfig | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/configs/imxrt1050-evk_defconfig b/configs/imxrt1050-evk_defconfig
index
On 5/14/21 5:06 PM, Oleh Kravchenko wrote:
The current U-Boot version has the next matches for boot partitions:
mmc0boot0 to EMMC_BOOT1
mmc0boot1 to EMMC_BOOT1 (should be EMMC_BOOT2)
This patch fixes a typo for the boot partition number.
Signed-off-by: Oleh Kravchenko
Cc: Pantelis
On 17/05/2021 16:21, Heinrich Schuchardt wrote:
> If you are asked for a sudo password, you have not install libguestfs.
>
> Please, install the missing package.
This also might have ended up not in PATH like mkfs.* weren't in mine,
does e.g. "guestmount --version" work from your shell?
On 25.03.21 08:18, Aswath Govindraju wrote:
> Use dev_seq() to read aliases node's index and pass it as device number
> for creating bulk device.
>
> Suggested-by: Grygorii Strashko
> Signed-off-by: Aswath Govindraju
> Reviewed-by: Jaehoon Chung
Since this patch merged as commit
On 20.05.21 18:32, Heinrich Schuchardt wrote:
> On 25.03.21 08:18, Aswath Govindraju wrote:
>> Use dev_seq() to read aliases node's index and pass it as device number
>> for creating bulk device.
>>
>> Suggested-by: Grygorii Strashko
>> Signed-off-by: Aswath Govindraju
>> Reviewed-by: Jaehoon
Le mer. 19 mai 2021 à 23:51, Jeremy Linton via TF-A <
t...@lists.trustedfirmware.org> a écrit :
> On 5/18/21 8:59 PM, Madhukar Pappireddy via TF-A wrote:
> > Hi,
> >
> > I tried to summarize the discussions in the previous TF-A tech forum
> regarding the proposal to adopt Hand-off Blocks (HOBs)
Le mer. 19 mai 2021 à 03:58, Madhukar Pappireddy via TF-A <
t...@lists.trustedfirmware.org> a écrit :
> Hi,
>
> I tried to summarize the discussions in the previous TF-A tech forum
> regarding the proposal to adopt Hand-off Blocks (HOBs) for passing
> information along the boot chain. I am
Hi,
My interest in boot information passing is from the perspective of a
down-boot-chain consumer. From this perspective, I have the following
preferences:
1) Whatever information passing mechanism is used (e.g. HOB or DT), we use a
common object identification scheme that may be used with
Hi,
Re Jeremy's comment:
> Using DT to pass platform info at this level is sort of crazy on an ACPI
> machine which won't have native DTs. Meaning there is an additional
> level of unnecessary indirection that needs to be converted back into a
> format which can be utilized by AML and other parts
Hi Marek,
Thank you for the patch.
On Wed, Apr 28, 2021 at 8:30 PM Marek Vasut wrote:
>
> Synchronize RZ/G2 clock tables with Linux 5.12,
> commit 9f4ad9e425a1 ("Linux 5.12") .
>
> Signed-off-by: Marek Vasut
> ---
> drivers/clk/renesas/r8a774a1-cpg-mssr.c | 14 +-
>
Hi Marek,
Thank you for the patch.
On Wed, Apr 28, 2021 at 8:30 PM Marek Vasut wrote:
>
> From: Hai Pham
>
> This supports RPCD2 clock handling. While at it, add the check point
> for RPC-IF clock RPCD2 Frequency Division Ratio, since it must be odd
> number
>
> Signed-off-by: Hai Pham
>
On Thu, 20 May 2021 18:32:38 +0200
Heinrich Schuchardt wrote:
Hi Heinrich,
> On 25.03.21 08:18, Aswath Govindraju wrote:
> > Use dev_seq() to read aliases node's index and pass it as device number
> > for creating bulk device.
> >
> > Suggested-by: Grygorii Strashko
> > Signed-off-by: Aswath
Hi Marek,
Thank you for the patch.
On Wed, Apr 28, 2021 at 8:31 PM Marek Vasut wrote:
>
> From: Hai Pham
>
> RPC clk_get_rate will return error code instead of expected clock rate.
> Fix this.
>
> Signed-off-by: Hai Pham
> Signed-off-by: Marek Vasut
> ---
>
Hi Marek,
Thank you for the patch.
On Wed, Apr 28, 2021 at 8:32 PM Marek Vasut wrote:
>
> The MODEMR register offset changed on R8A779A0, make the MODEMR offset
> configurable. Fill the offset in on all clock drivers. No functional
> change.
>
> Based off "clk: renesas: Make CPG Reset MODEMR
Hi Marek,
Thank you for the patch.
On Wed, Apr 28, 2021 at 8:32 PM Marek Vasut wrote:
>
> From: Hai Pham
>
> Base on Linux v5.10-rc2, commit 8b652aa8a1fb by Yoshihiro Shimoda
> To support other register layouts in the future, add register pointers
> of {control,status,reset,reset_clear}_regs
Hi Marek,
Thank you for the patch.
On Wed, Apr 28, 2021 at 8:33 PM Marek Vasut wrote:
>
> Most of the PLLx, MAIN, FIXED clock handlers are calling very similar
> code, which determines parent rate and then applies multiplication and
> division. The only difference is whether multiplication is
Hi Marek,
Thank you for the patch.
On Wed, Apr 28, 2021 at 8:33 PM Marek Vasut wrote:
>
> Pass struct udevice to rcar_gpio_set_direction() in preparation of
> quirk handling in rcar_gpio_set_direction(). No functional change.
>
> Signed-off-by: Marek Vasut
> ---
> drivers/gpio/gpio-rcar.c |
Hi Marek,
Thank you for the patch.
On Wed, Apr 28, 2021 at 8:33 PM Marek Vasut wrote:
>
> The V3U SoC has several unlock registers, one per register group. They
> reside at offset zero in each 0x200 bytes-sized block.
>
> To avoid adding yet another table to the PFC implementation, this
> patch
On Thu, 20 May 2021 at 05:25, Marek Behún wrote:
>
> In style of linked lists, instead of declaring symbols for boundaries
> of getopt options array in the linker script, declare corresponding
> sections and retrieve the boundaries via static inline functions.
>
> Without this clang's LTO
Hi Alex,
On Wed, 19 May 2021 at 20:41, Alex G. wrote:
>
>
>
> On 5/19/21 4:55 PM, Simon Glass wrote:
> > Hi Alex,
> >
> > On Wed, 19 May 2021 at 11:44, Alex G wrote:
> >>
> >>
> >>
> >> On 5/19/21 11:36 AM, Simon Glass wrote:
> >>> Hi Alexandru,
> >>>
> >>> On Mon, 17 May 2021 at 10:38,
On Thu, 20 May 2021 at 05:25, Marek Behún wrote:
>
> Some toolchains are compiled so that they pass a --build-id=something
> parameter to the linker implicitly.
>
> This causes U-Boot LTO linking to fail with something like:
> ld: section .note.gnu.build-id LMA ... overlaps section .text LMA
On Thu, 20 May 2021 at 05:25, Marek Behún wrote:
>
> The Thumb instruction `ldr` is able to move high registers only from
> armv7. For armv5 and armv6 we have to use `mov`.
>
> Signed-off-by: Marek Behún
> ---
> arch/arm/include/asm/global_data.h | 4 +++-
> 1 file changed, 3 insertions(+), 1
On Thu, 20 May 2021 at 05:25, Marek Behún wrote:
>
> On ARM, the gd pointer is stored in registers r9 / x18. For this the
> -ffixed-r9 / -ffixed-x18 flag is used when compiling, but using global
> register variables causes errors when building with LTO, and these
> errors are very difficult to
On Thu, 20 May 2021 at 05:25, Marek Behún wrote:
>
> It seems that sometimes (happening on ARM64, for example with
> turris_mox_defconfig) GCC, when linking with LTO, changes the symbol
> names of some functions, for example lib/string.c's memcpy() function to
> memcpy.isra.0.
>
> This is a
On Thu, 20 May 2021 at 05:25, Marek Behún wrote:
>
> Improve the regular expression that matches unittest symbols in
> u-boot.sym.
>
> Currently we do not enforce no prefix in symbol string, but with the
> soon to come change in linker lists declaring lists and entries with the
> __ADDRESSABLE
On Thu, 20 May 2021 at 05:25, Marek Behún wrote:
>
> This is how Linux does this now, see Linux commit 339f29d91acf.
>
> Signed-off-by: Marek Behún
> ---
> scripts/checkpatch.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Simon Glass
On Thu, 20 May 2021 at 05:25, Marek Behún wrote:
>
> When seaboard_defconfig is compiled with LTO, the compiler complains
> about some instructions not being supported in ARM mode.
>
> This is caused by arch/arm/mach-tegra/tegra20/warmboot_avp.c having
> different CFLAGS declared in Makefile.
On Thu, 20 May 2021 at 05:08, Michal Simek wrote:
>
> A lot of entries were using spaces instead of tab for alignment that's why
> it is good to fix it to use the same style everywhere.
>
> Signed-off-by: Michal Simek
> ---
>
> common/spl/Kconfig | 24
> 1 file changed,
> I don't see a changelog here but this is v4. Are you using patman?
Changelog is in cover letter. Unfortunately I am not using patman yet.
Marek
Am 20. Mai 2021 21:33:42 MESZ schrieb Alper Nebi Yasak
:
>On 17/05/2021 16:21, Heinrich Schuchardt wrote:
>> If you are asked for a sudo password, you have not install
>libguestfs.
>>
>> Please, install the missing package.
>
>This also might have ended up not in PATH like mkfs.* weren't in
Hi Marek,
On Thu, 20 May 2021 at 05:25, 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
Hi Rasmus,
On Thu, 20 May 2021 at 04:05, Rasmus Villemoes
wrote:
>
> Most callers (or callers of callers, etc.) of vsnprintf() are not
> prepared for it to return a negative value.
>
> The only case where that can currently happen is %pD, and it's IMO
> more user-friendly to produce some output
On Thu, 20 May 2021 at 05:25, Marek Behún wrote:
>
> Fix LTO build for some thumb-interwork usecases (such as for
> da850evm_defconfig), where inline assmebly such as
assembly
> mrc p15,0,r2,c1,c0,0
> causes the compiler to fail during LTO linking with
> Error: selected processor does not
On Thu, 20 May 2021 at 05:25, Marek Behún wrote:
>
> Add plumbing for building U-Boot with Link Time Optimizations.
>
> When building with LTO, $(PLATFORM_LIBS) has to be in --whole-archive /
> --no-whole-archive group, otherwise some functions declared in assembly
> may not be resolved and
1 - 100 of 136 matches
Mail list logo