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
---
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
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,
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
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?
> 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
On 17/05/2021 17:06, Andy Shevchenko wrote:
> I would like to be helpful here and when I have time, I'll look at it
> closer if nobody beats me up to it. Currently I checked the reason why
> we skip them in my scenario:
> short test summary info
>
Commit 1ba21bb06b08 ("test: Don't unmount not (yet) mounted system")
fixes an issue in the filesystem tests where the test setup may fail
to mount an image and still attempt to unmount it. However, the commit
unintentionally breaks the test setups in two ways.
The newly created unmounted
On Thu, May 20, 2021 at 6:25 AM Marek Behún wrote:
>
> 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
Since the imx8mm beacon boards and the imx8mm venice
On 20.05.21 18:55, Andre Przywara wrote:
> 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.
>>>
>>>
On Thu, 20 May 2021 at 04:05, Rasmus Villemoes
wrote:
>
> 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.
>
>
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: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,
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:
>
> 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:
>
> 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: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
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:
>
> 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
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 04:05, Rasmus Villemoes
wrote:
>
> 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(-)
Reviewed-by: Simon Glass
On Thu, 20 May 2021 at 05:25, Marek Behún wrote:
>
> 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
> ---
>
On Thu, 20 May 2021 at 05:25, Marek Behún wrote:
>
> 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(-)
>
Reviewed-by: Simon Glass
On Thu, 20 May 2021 at 05:25, Marek Behún wrote:
>
> 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(+)
>
Reviewed-by: Simon Glass
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
On Thu, 20 May 2021 at 04:05, Rasmus Villemoes
wrote:
>
> 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
On Thu, 20 May 2021 at 05:25, Marek Behún wrote:
>
> 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.
>
>
On Thu, 20 May 2021 at 05:25, Marek Behún wrote:
>
> 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
On Thu, 20 May 2021 at 05:25, Marek Behún wrote:
>
> 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
On Thu, 20 May 2021 at 04:05, Rasmus Villemoes
wrote:
>
> 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
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
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 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
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:
>
> 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: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: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: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
> ---
>
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: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
>
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,
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,
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
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
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)
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
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
In cpu_to_be32_array() and be32_to_cpu_array() we should not compare an int
counter to a size_t parameter. Correct the type of the counter.
Signed-off-by: Heinrich Schuchardt
---
include/linux/byteorder/generic.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
Hello Lukasz,
Could you please review this patch too?
This patch fixes Fastboot behaviour,
when flashing or erasing of eMMC user area was requested.
20.05.21 18:00, Sean Anderson пише:
>
>
> On 5/20/21 10:44 AM, Oleh Kravchenko wrote:
>> Thank you, Sean!
>>
>> Could you please take a look at
Hello Lukasz,
Could you please review this patch?
This patch fixes small typos for Fastboot mmc boot2 partition.
20.05.21 17:59, Sean Anderson пише:
>
>
> On 5/14/21 5:06 PM, Oleh Kravchenko wrote:
>> The current U-Boot version has the next matches for boot partitions:
>>> mmc0boot0 to
On Thu, May 20, 2021 at 03:05:11PM +0200, Rasmus Villemoes wrote:
> Hi,
>
> When doing linux kernel work, the public-inbox archives at
> lore.kernel.org are really useful.
>
> - excellent search features
> - stable links of the form lore.kernel.org//
> - nice overview of threads
> - ability to,
On 5/20/21 4:10 PM, Giulio Benetti wrote:
[...]
@@ -165,5 +166,31 @@
clocks = <>;
status = "disabled";
};
+
+ usbphy1: usbphy@400d9000 {
+ compatible = "fsl,imxrt-usbphy";
Is there a PHY
On 5/20/21 4:10 PM, Giulio Benetti wrote:
[...]
diff --git a/arch/arm/include/asm/arch-imxrt/imx-regs.h
b/arch/arm/include/asm/arch-imxrt/imx-regs.h
index 44c95dcd11..d01e6ca2e0 100644
--- a/arch/arm/include/asm/arch-imxrt/imx-regs.h
+++ b/arch/arm/include/asm/arch-imxrt/imx-regs.h
@@ -23,4
On 5/20/21 10:44 AM, Oleh Kravchenko wrote:
Thank you, Sean!
Could you please take a look at this patch too?
https://patchwork.ozlabs.org/project/uboot/patch/20210514210620.24715-1-o...@kaa.org.ua/
You may want to CC Lukas Majewski. I believe he took fastboot patches last time
they got
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 Wed, May 19, 2021 at 4:27 PM Etienne Carriere
wrote:
>
> Remove unused OPTEE_MSG_LOGIN_* IDs and rely on the ones introduced in
> tee.h. Change optee core to treat invalid client IDs as public login.
>
> Signed-off-by: Etienne Carriere
> ---
> Changes since v1:
> - Remove ID conversion. I
On Wed, May 19, 2021 at 4:27 PM Etienne Carriere
wrote:
>
> Define identifiers for clnt_login field in struct tee_open_session_arg
> based in GlobalPlatform Device TEE IDs and on the REE_KERNEL identifier
> extension from OP-TEE OS.
>
> Signed-off-by: Etienne Carriere
> ---
> Changes since v1:
>
Thank you, Sean!
Could you please take a look at this patch too?
https://patchwork.ozlabs.org/project/uboot/patch/20210514210620.24715-1-o...@kaa.org.ua/
20.05.21 17:32, Sean Anderson пише:
>
>
> On 5/19/21 6:31 AM, Oleh Kravchenko wrote:
>> 'gpt' and 'mmc0' fastboot partitions have been
On Wed, May 19, 2021 at 4:28 PM Etienne Carriere
wrote:
>
> This change ensures both U-Boot and OP-TEE see the same content
> from shared memory when OP-TEE is invoked prior U-Boot relocation.
>
> This change is required since U-Boot may execute with data cahce off
cache
> while OP-TEE always
On 5/19/21 6:31 AM, Oleh Kravchenko wrote:
'gpt' and 'mmc0' fastboot partitions have been treated as the same device,
but it is wrong.
Fill disk_partition structure with eMMC user partition info
to properly flash data.
Signed-off-by: Oleh Kravchenko
Cc: Pantelis Antoniou
Cc: Marek Vasut
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
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
---
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
Add support for usb1 and usb2 present on i.IMXRT.
Signed-off-by: Giulio Benetti
---
arch/arm/include/asm/arch-imxrt/imx-regs.h | 4
drivers/usb/host/Kconfig | 2 +-
drivers/usb/host/ehci-mx6.c| 13 +++--
3 files changed, 12 insertions(+), 7
We need those macros to instruct drivers on how to behave for SoC specific
quirks, so let's add it as done for other i.MX SoCs.
Signed-off-by: Giulio Benetti
---
arch/arm/include/asm/arch-imx/cpu.h | 3 +++
arch/arm/include/asm/mach-imx/sys_proto.h | 4
Usb needs IMXRT1050_CLK_USBOH3 clock to be enabled, so let's add it to
clock driver.
Signed-off-by: Giulio Benetti
---
drivers/clk/imx/clk-imxrt1050.c | 2 ++
include/dt-bindings/clock/imxrt1050-clock.h | 3 ++-
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git
This patchset adds Usb Host support for imxrt1050-evk board adjusting the
compatible driver ehci-mx6. This allows to read a FAT usb key connected
to usbotg1 port of imxrt1050-evk board.
Giulio Benetti (6):
ARM: IMXRT: introduce is_imxrt*() macros and get_cpu_rev()
clk: imx: clk-imxrt1050:
Hi,
On Thu, 20 May 2021 at 07:16, Bin Meng wrote:
>
> On Thu, May 20, 2021 at 9:05 PM Rasmus Villemoes
> wrote:
> >
> > Hi,
> >
> > When doing linux kernel work, the public-inbox archives at
> > lore.kernel.org are really useful.
> >
> > - excellent search features
> > - stable links of the
On 20/05/21 09:16PM, Bin Meng wrote:
> On Thu, May 20, 2021 at 9:05 PM Rasmus Villemoes
> wrote:
> >
> > Hi,
> >
> > When doing linux kernel work, the public-inbox archives at
> > lore.kernel.org are really useful.
> >
> > - excellent search features
> > - stable links of the form
Hi Tom,
please pull the next batch of Marvell Armada related patches, mostly
the ones dropped last time because of Xilinx issue (fixed now). Here
the summary log:
- Sync Armada mvpp2 ethernet driver with Marvell version (misc
On Thu, May 20, 2021 at 9:05 PM Rasmus Villemoes
wrote:
>
> Hi,
>
> When doing linux kernel work, the public-inbox archives at
> lore.kernel.org are really useful.
>
> - excellent search features
> - stable links of the form lore.kernel.org//
> - nice overview of threads
> - ability to, with the
Hi,
When doing linux kernel work, the public-inbox archives at
lore.kernel.org are really useful.
- excellent search features
- stable links of the form lore.kernel.org//
- nice overview of threads
- ability to, with the b4 tool, download patches or whole series by
msg-id, and logic to compare
Hello,
Em qui., 20 de mai. de 2021 às 06:54, Richard Genoud <
richard.gen...@posteo.net> escreveu:
> 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
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
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
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
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
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
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
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 +
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
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
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
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
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 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 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 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
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(+)
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 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:
1 - 100 of 136 matches
Mail list logo