Re: [PATCH v5 00/10] Add support for extension boards detection and DT overlays application

2021-05-13 Thread Thomas Petazzoni
real hardware wit CMD_FDT) and > am otherwise world build testing it again now. Excellent, thanks. Once again, do not hesitate to let us know if there's any remaining issue to investigate and fix, or any concern about the overall design/approach. Best regards, Thomas Petazzoni --

Re: [PATCH v5 00/10] Add support for extension boards detection and DT overlays application

2021-05-12 Thread Thomas Petazzoni
at is needed to get this series considered for merging. Thanks a lot! Thomas Petazzoni -- Thomas Petazzoni, co-owner and CEO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com

Re: Linux mvneta driver unable to read MAC address from HW

2020-10-12 Thread Thomas Petazzoni
of all unused clocks at the end of the boot. When the driver is built-in, there is a driver adding a reference to the clock before all unused clocks are disabled. When the driver is compiled as a module, this does not happen. So indeed, the correct solution here is to have U-Boot pass the MAC address

Re: [fs/squashfs PATCH v3 2/2] avoid 64-bit divisions on 32-bit

2020-10-08 Thread Thomas Petazzoni
.cur_dev->blksz); > > - return DIV_ROUND_UP(table_size + *offset, ctxt.cur_dev->blksz); > + return (table_size + *offset + ctxt.cur_dev->blksz - 1) >> > ctxt.cur_dev->log2blksz; Why are you not using DIV_ROUND_UP_ULL() here ? Thanks, Thomas Petazzoni -- Thomas Petazzoni, CTO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com

Re: [RFC PATCH v2 1/1] Fix missing __udivdi3 in SquashFS implementation.

2020-10-01 Thread Thomas Petazzoni
We could use DIV_ROUND_UP_ULL() I guess. > else > - blk_list_size = file_size / blk_size; > + blk_list_size = file_size > LOG2(blk_size); Very bad mistake here: > should be >>. Best regards, Thomas -- Thomas Petazzoni, CTO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com

Re: [PATCH 1/1] fs/squashfs: parameter check sqfs_read_metablock()

2020-09-23 Thread Thomas Petazzoni
On Wed, 23 Sep 2020 21:13:51 +0200 Heinrich Schuchardt wrote: > We should check if the incoming parameter file_mapping is not NULL instead > of checking after adding an offset. > > Reported-by: Coverity CID 307210 > Signed-off-by: Heinrich Schuchardt Acked-by: Thomas Petazzo

[PATCH] MAINTAINERS: add myself as reviewer for SquashFS

2020-09-14 Thread Thomas Petazzoni
As I have followed the development of the SquashFS support in U-Boot as part of Joao Marcos work, it makes sense to get Cc'ed on contributions/bug reports related to the squashfs support. Signed-off-by: Thomas Petazzoni --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --

Re: [PATCH v4 1/5] fs/squashfs: Add init and clean-up functions to decompression

2020-08-18 Thread Thomas Petazzoni
ret = uncompress(dest, dest_len, source, src_len); These two changes are not related to this patch. Thomas -- Thomas Petazzoni, CTO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com

Re: [PATCH v3 2/4] fs/squashfs: add support for ZSTD decompression

2020-08-18 Thread Thomas Petazzoni
nt to the decompressor function, so that sqfs_ctxt doesn't need to be a global variable. Best regards, Thomas -- Thomas Petazzoni, CTO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com

Re: [PATCH v3 1/4] fs/squashfs: Add init. and clean-up functions to decompression

2020-08-18 Thread Thomas Petazzoni
rom that file. If you keep the "static" keyword, you can keep the variable named "ctxt" and avoid a lengthy patch. > + ret = sqfs_decompressor_init(&sqfs_ctxt); See you're passing a reference to the squashfs context here, so this variable does not need to be

Re: [PATCH 2/2] fs/squashfs: add support for ZSTD decompression

2020-08-11 Thread Thomas Petazzoni
eanup,close} function. Best regards, Thomas -- Thomas Petazzoni, CTO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com

Re: [PATCH 0/3] squashfs fixes

2020-07-16 Thread Thomas Petazzoni
iling U-Boot for ARM for example, as I know you're also testing this on an ARM platform. Thomas -- Thomas Petazzoni, CTO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com

[PATCH] fs/squashfs/sqfs.c: use sqfs_read_sblk() in sqfs_probe()

2020-07-15 Thread Thomas Petazzoni
sqfs_probe() currently duplicates what sqfs_read_sblk() is doing, so use the latter in the former to avoid some small code duplication. Signed-off-by: Thomas Petazzoni --- fs/squashfs/sqfs.c | 14 -- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/fs/squashfs/sqfs.c b

[PATCH 3/3] fs/squashfs: do not use CMD_RET_* defines in the filesystem code

2020-07-15 Thread Thomas Petazzoni
sqfs_get_regfile_info() and sqfs_get_lregfile_info(). - The sqfs_size() function is modified to return "ret" at the end, so that the error code is properly propagated. Signed-off-by: Thomas Petazzoni --- fs/squashfs/sqfs.c | 61 +++--- 1 file c

[PATCH 2/3] fs/squashfs: use "struct disk_partition" instead of "disk_partition_t"

2020-07-15 Thread Thomas Petazzoni
disk_partition_t was dropped in 0528979fa7ab ("part: Drop disk_partition_t typedef"). Signed-off-by: Thomas Petazzoni --- fs/squashfs/sqfs.c | 5 +++-- include/squashfs.h | 4 +++- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/fs/squashfs/sqfs.c b/fs/squashfs/sq

[PATCH 0/3] squashfs fixes

2020-07-15 Thread Thomas Petazzoni
’ 207 | struct squashfs_directory_index index[0]; | ^ fs/squashfs/sqfs_inode.c:31:29: note: defined here ‘ldir’ 31 | struct squashfs_ldir_inode ldir; | ^~~~ Could you investigate this ? Thanks, Thomas Thomas

[PATCH 1/3] cmd/sqfs.c, include/fs.h: fix build failures, don't expose do_sqfs_*() functions globally

2020-07-15 Thread Thomas Petazzoni
ly expose do_sqfs_ls() and do_sqfs_load(), they are only used within cmd/sqfs.c. This allows to drop the do_sqfs_load() prototype from include/fs.h. Signed-off-by: Thomas Petazzoni --- cmd/sqfs.c | 5 +++-- include/fs.h | 12 2 files changed, 3 insertions(+), 14 deletions(-) diff

Re: [PATCH 3/4] fs/squashfs: add sources for zlib decompression

2020-07-15 Thread Thomas Petazzoni
unzip.c already has a function called zunzip(), which seems to do pretty much the same thing: decompress from a memory buffer into another memory buffer. Are you sure you cannot use this existing zunzip() function ? zunzip() is already used by UBIFS. Thomas -- Thomas Petazzoni, CTO, Bootlin Emb

Re: [PATCH 1/4] fs/squashfs: new filesystem

2020-07-10 Thread Thomas Petazzoni
SPL or only in U-Boot proper. It was not clear in your initial e-mail that this was the issue you were pointing. Thomas -- Thomas Petazzoni, CTO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com

Re: [PATCH 1/4] fs/squashfs: new filesystem

2020-07-10 Thread Thomas Petazzoni
ystem, then the SPL needs FAT filesystem support to load U-Boot proper, and U-Boot proper needs FAT filesystem support to load the Linux kernel image. Am I missing something here ? Best regards, Thomas -- Thomas Petazzoni, CTO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com

[U-Boot] [PATCH] tools/Makefile: build host tools with -std=gnu99

2019-03-30 Thread Thomas Petazzoni
ding the host tools with gcc 4.7. Signed-off-by: Thomas Petazzoni --- tools/Makefile | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/Makefile b/tools/Makefile index 081383d7a7..d377d85f74 100644 --- a/tools/Makefile +++ b/tools/Makefile @@ -254,7 +254,8 @@ HOST_EXTRA

Re: [U-Boot] [PATCH] lib/crc16: use non-C99 loop style

2019-02-13 Thread Thomas Petazzoni
ebian system encountered this build failure. Buildroot is used in lots of "enterprise" contexts, where enterprise often means "the IT forces the poor developers to use antique Linux systems for their job", and we're trying to make the life of those poor developers slightly

[U-Boot] [PATCH] lib/crc16: use non-C99 loop style

2019-02-13 Thread Thomas Petazzoni
coding style used in the rest of U-Boot allows to fix this build issue. Signed-off-by: Thomas Petazzoni --- lib/crc16.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/lib/crc16.c b/lib/crc16.c index f46ba727c9..89d2cff131 100644 --- a/lib/crc16.c +++ b/lib/crc16.c

Re: [U-Boot] Issue with memcpy when booting a fitImage on SPEAr600

2018-12-06 Thread Thomas Petazzoni
ent memmove() when the areas are overlapping ? Perhaps Simon Glass, who did the change to use memcpy() in cb0eae8cf8aaca76910dee4c7eb536d0814d1bd2 could comment on that ? Best regards, Thomas -- Thomas Petazzoni, CTO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v3 25/28] cmd: nand/sf: isolate legacy code

2018-12-05 Thread Thomas Petazzoni
condition will be true. Best regards, Thomas -- Thomas Petazzoni, CTO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v13 5/7] cmd: mtd: add 'mtd' command

2018-10-08 Thread Thomas Petazzoni
if (mtdparts && old_mtdparts && mtdids && old_mtdids && !strcmp(mtdparts, old_mtdparts) && !strcmp(mtdids, old_mtdids))) return 0; Best regards, Thomas -- Thomas Petazzoni, CTO, Bootlin Embedded Linux and Ke

Re: [U-Boot] Tools build failure when libfdt-devel is installed

2018-03-04 Thread Thomas Petazzoni
clude paths of tools are so cryptic. Oh yes it is cryptic! Thomas -- Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering http://bootlin.com ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

[U-Boot] Tools build failure when libfdt-devel is installed

2018-03-03 Thread Thomas Petazzoni
nodeoffset, ^~~ make[1]: *** [scripts/Makefile.host:116: tools/aisimage.o] Error 1 make: *** [Makefile:1506: tools-only] Error 2 -- Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering http://b

Re: [U-Boot] [PATCH] serial: serial_sh: take into account port->clk_mode for SH7785/7786

2017-09-24 Thread Thomas Petazzoni
: [U-Boot] Initializing global_data on SuperH before board_init_f() ? Date: Tue, 15 Aug 2017 23:07:30 +0200 Thanks! Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com ___ U-Boot mailing li

Re: [U-Boot] undefined reference to `environment'

2017-09-13 Thread Thomas Petazzoni
been able to push out. How did you end up solving this problem? As I said, I don't think https://patchwork.ozlabs.org/patch/806263/ is the correct solution. Best regards, Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com

Re: [U-Boot] [PATCH] serial: serial_sh: take into account port->clk_mode for SH7785/7786

2017-09-13 Thread Thomas Petazzoni
Hello, On Mon, 28 Aug 2017 14:16:38 +0200, Thomas Petazzoni wrote: > In the SH7785/SH7786 case, the SCSCR value is harcoded to be 0x3a, > which means bits CKE1/CKE0 have the value 10b. This tells the SCIF > that the "External clock/SCIF_SCK pin functions as clock input". >

Re: [U-Boot] I/O accessors on SuperH and endianness

2017-09-13 Thread Thomas Petazzoni
Hello, Has anyone any comments/suggestions on the below questions? How is this problem solved on other architectures? Thanks, Thomas On Mon, 28 Aug 2017 14:32:49 +0200, Thomas Petazzoni wrote: > Hello, > > As you've noticed, I'm porting U-Boot to a SH4 board running &g

Re: [U-Boot] undefined reference to `environment'

2017-08-29 Thread Thomas Petazzoni
ded.o in their linker script, plus one Freescale platform. Again, changing extra- to obj- in env/Makefile also fixes the issue. Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com ___ U-Boot

Re: [U-Boot] undefined reference to `environment'

2017-08-29 Thread Thomas Petazzoni
It is added to extra-y by: extra-$(CONFIG_ENV_IS_IN_FLASH) += embedded.o I dumped extra-y right after this line, and it does contain embedded.o. However, it does not get added to the link stage. Why is this an extra-$() rather than an obj-$() ? Best regards, Thomas -- Thomas Petazzoni, CTO,

Re: [U-Boot] undefined reference to `environment'

2017-08-29 Thread Thomas Petazzoni
> Thomas, could you please take a look? Sure, I'll have a look. I guess I should be able to reproduce by building ms7722se_defconfig ? Best regards, Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com

[U-Boot] I/O accessors on SuperH and endianness

2017-08-28 Thread Thomas Petazzoni
sh_eth_reg_addr(eth, enum_index)); } -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH] serial: serial_sh: take into account port->clk_mode for SH7785/7786

2017-08-28 Thread Thomas Petazzoni
de = EXT_CLK to indicate such a setting. In order for the serial_sh driver to work on a SH7786 platform that does not use SCIF_SCK as a clock input, we have to adjust the SCSCR_INIT value, to have CKE1/CKE0 set to 00b. This is similar to what is done for other SH platforms. Signed-off-by: T

Re: [U-Boot] [PATCH 0/8] Clean up CONFIG_SYS_{CBSIZE, PBSIZE, MAXARGS, BARGSIZE} definitions

2017-08-24 Thread Thomas Petazzoni
move this to > Kconfig so it's easy to use a default setting, and only boards with > special needs have to adjust this in their defconfig files. OK. So let's review this first patch series. Once we agree on it and it gets merged, I can work on the second step of moving those co

Re: [U-Boot] [PATCH 0/8] Clean up CONFIG_SYS_{CBSIZE, PBSIZE, MAXARGS, BARGSIZE} definitions

2017-08-24 Thread Thomas Petazzoni
same board, most people will find the default value to be OK, some people will need a larger number of command arguments or a bigger Linux kernel command line. Let me know what your preference is. Thanks, Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel

[U-Boot] [PATCH 6/8] include/configs: drop default definitions of CONFIG_SYS_MAXARGS

2017-08-23 Thread Thomas Petazzoni
Now that include/config_fallbacks.h define a sane fallback for CONFIG_SYS_MAXARGS, we can drop the definition of this constant in all configurations that were using the default value. Signed-off-by: Thomas Petazzoni --- include/configs/10m50_devboard.h | 1 - include/configs

[U-Boot] [PATCH 8/8] include/configs: remove numerous CONFIG_SYS_BARGSIZE definitions

2017-08-23 Thread Thomas Petazzoni
CONFIG_SYS_BARGSIZE == 512 (common.image.c) is suitable, as it is equal or larger. Signed-off-by: Thomas Petazzoni --- include/configs/10m50_devboard.h | 1 - include/configs/3c120_devboard.h | 1 - include/configs/B4860QDS.h | 1 - include/configs/BSC9132QDS.h | 2

[U-Boot] [PATCH 7/8] include/configs: remove default values of CONFIG_SYS_BARGSIZE

2017-08-23 Thread Thomas Petazzoni
CONFIG_SYS_BARGSIZE is already defined to 512 in common/image.c when not defined. Therefore, there is no point in having board configuration files define it to 512. Signed-off-by: Thomas Petazzoni --- include/configs/MigoR.h| 1 - include/configs/ap325rxa.h | 2 -- include

[U-Boot] [PATCH 5/8] include/config_fallbacks.h: add default for CONFIG_SYS_MAXARGS

2017-08-23 Thread Thomas Petazzoni
remove the default definition from 216 platform .h files, leaving only 56 platforms with non-default values (15, 24, 32, 48, 64, 96, 128 or 256). Signed-off-by: Thomas Petazzoni --- include/config_fallbacks.h | 4 1 file changed, 4 insertions(+) diff --git a/include/config_fallbacks.h b

[U-Boot] [PATCH 3/8] include/config_fallbacks.h: change fallback for CONFIG_SYS_PBSIZE

2017-08-23 Thread Thomas Petazzoni
forms. Signed-off-by: Thomas Petazzoni --- include/config_fallbacks.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/config_fallbacks.h b/include/config_fallbacks.h index f9461f0..09fcee4 100644 --- a/include/config_fallbacks.h +++ b/include/config_fallbacks.h @@ -81,7

[U-Boot] [PATCH 2/8] include/configs: remove CONFIG_SYS_CBSIZE when the default value is used

2017-08-23 Thread Thomas Petazzoni
Now that include/config_fallbacks.h define a sane fallback for CONFIG_SYS_CBSIZE, we can drop the definition of this constant in all configurations that were using the default value. Signed-off-by: Thomas Petazzoni --- include/configs/10m50_devboard.h| 1 - include/configs/3c120_devboard.h

[U-Boot] [PATCH 1/8] include/config_fallbacks.h: add default for CONFIG_SYS_CBSIZE

2017-08-23 Thread Thomas Petazzoni
provide a sane fallback value in config_fallbacks.h, so that platforms can rely on it instead of having to explicitly define it. We use 1024 when KGDB is enabled, and 256 otherwise, which is what the majority of the platforms are doing. Signed-off-by: Thomas Petazzoni --- include/config_fallb

[U-Boot] [PATCH 0/8] Clean up CONFIG_SYS_{CBSIZE, PBSIZE, MAXARGS, BARGSIZE} definitions

2017-08-23 Thread Thomas Petazzoni
-Boot commands. Perhaps we want to remove those options entirely from the .h files, and simply have a Kconfig option instead? Also, let me know if I have missed some existing work in this area. Thanks for your feedback! Thomas Petazzoni Thomas Petazzoni (8): include/config_fallbacks.h: add

Re: [U-Boot] [PATCH 1/2] board/spear/common: move code to arch/arm/cpu/arm926ejs/spear/

2017-08-19 Thread Thomas Petazzoni
needs to be done in terms of cleanup/Kconfig migration, I can try to find some time to look into that. Thanks! Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] Initializing global_data on SuperH before board_init_f() ?

2017-08-19 Thread Thomas Petazzoni
.l ._reloc_dst, r4 > +4: mov.l r1, @-r4 > + cmp/hs r4, r13 > + bf 4b > + This indeed should solve the problem, though I'm not able to test right now, as I don't have access to the board and JTAG right now. I'll get back to you when I've been able to test. Thanks, Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] Initializing global_data on SuperH before board_init_f() ?

2017-08-16 Thread Thomas Petazzoni
ce for the global data on the stack - Call board_init_f_init_reserve() to zero initialize it I guess the SH code can be adapted to use this logic, I'll have a look into that. Could take a while though, since I'm not fluent in SH assembly. Thanks for the hint! Thomas -- Thomas Petazzoni, CTO,

[U-Boot] [PATCH] common/board_f.c: remove CONFIG_SYS_GENERIC_GLOBAL_DATA

2017-08-15 Thread Thomas Petazzoni
CONFIG_SYS_GENERIC_GLOBAL_DATA is no longer used by any board or platform, so support for it can be dropped. Signed-off-by: Thomas Petazzoni --- README | 6 -- common/board_f.c | 27 --- scripts/config_whitelist.txt | 1 - 3 files

[U-Boot] Initializing global_data on SuperH before board_init_f() ?

2017-08-15 Thread Thomas Petazzoni
27;ve currently worked-around the problem by adding a memset() to zero of the global_data at the beginning of board_init_f(), but I'd prefer to find an upstreamable fix. Thanks! Thomas Petazzoni -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-ele

Re: [U-Boot] [PATCH 1/3] arch/sh: don't bring common/env_embedded.o into the link

2017-08-15 Thread Thomas Petazzoni
Hello, On Thu, 3 Aug 2017 08:20:20 +0900, Nobuhiro Iwamatsu wrote: > Hi, > > 2017-07-29 6:46 GMT+09:00 Thomas Petazzoni > : > > The linker script for SuperH brings the .ppcenv and .ppcenvr section > > of common/env_embedded.o into the .text section. However, the .ppcenv

[U-Boot] [PATCH 2/2] arch/arm/cpu/arm926ejs/spear/cpu.c: guard do_switch_ecc() with CONFIG_NAND_FSMC

2017-08-15 Thread Thomas Petazzoni
do_switch_ecc() calls fsmc_nand_switch_ecc(), which is a direct function call into drivers/mtd/nand/fsmc_nand.c. However, this function is not guarded by CONFIG_NAND_FSMC, which results to a build failure if CONFIG_NAND_FSMC is disabled. Signed-off-by: Thomas Petazzoni --- arch/arm/cpu

[U-Boot] [PATCH 1/2] board/spear/common: move code to arch/arm/cpu/arm926ejs/spear/

2017-08-15 Thread Thomas Petazzoni
oves it to arch/arm/cpu/arm926ejs/spear/, with the rest of the SPEAr related code. Signed-off-by: Thomas Petazzoni --- arch/arm/cpu/arm926ejs/spear/Makefile | 2 ++ .../arm/cpu/arm926ejs/spear}/spr_lowlevel_init.S | 0 .../common => arch/arm/cpu/arm926ejs/spea

[U-Boot] [PATCH 3/3] common/env_embedded.c: rename PPCENV/PPCTEXT macros

2017-07-28 Thread Thomas Petazzoni
Signed-off-by: Thomas Petazzoni --- common/env_embedded.c | 10 -- include/env_default.h | 2 +- 2 files changed, 5 insertions(+), 7 deletions(-) diff --git a/common/env_embedded.c b/common/env_embedded.c index 4532589..43694db 100644 --- a/common/env_embedded.c +++ b/common/env_

[U-Boot] [PATCH 2/3] common/env_embedded.c: drop support for CONFIG_SYS_USE_PPCENV

2017-07-28 Thread Thomas Petazzoni
cases: - We're building the host tool tools/envcrc, in which case the environment is place with no special section attribute (so it depends up in .data) - We're building U-Boot itself, in which case the environnement is placed in the .text section. Signed-off-by: Thomas

[U-Boot] [PATCH 1/3] arch/sh: don't bring common/env_embedded.o into the link

2017-07-28 Thread Thomas Petazzoni
sections from common/env_embedded.o into the .text section. Signed-off-by: Thomas Petazzoni --- arch/sh/cpu/u-boot.lds | 4 1 file changed, 4 deletions(-) diff --git a/arch/sh/cpu/u-boot.lds b/arch/sh/cpu/u-boot.lds index f185b4d..2f2bdb5 100644 --- a/arch/sh/cpu/u-boot.lds +++ b/arch/sh/cpu

[U-Boot] [PATCH 0/3] env_embedded related fixes/improvements

2017-07-28 Thread Thomas Petazzoni
hanks, Thomas Thomas Petazzoni (3): arch/sh: don't bring common/env_embedded.o into the link common/env_embedded.c: drop support for CONFIG_SYS_USE_PPCENV common/env_embedded.c: rename PPCENV/PPCTEXT macros arch/sh/cpu/u-boot.lds | 4 common/env_embedded.c | 23 +++---

[U-Boot] [PATCH] arch/sh: allow building in big-endian mode

2017-07-28 Thread Thomas Petazzoni
-Boot for SuperH in big endian mode. To allow this, the only change needed is to adjust the OUTPUT_FORMAT() in the linker script. Signed-off-by: Thomas Petazzoni --- arch/sh/cpu/u-boot.lds | 5 + 1 file changed, 5 insertions(+) diff --git a/arch/sh/cpu/u-boot.lds b/arch/sh/cpu/u-boot.lds index

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-12 Thread Thomas Petazzoni
ins there if I can't get something else spun up in > a chroot soon :) Gcc 7.x toolchains are high on the priority list for toolchains.free-electrons.com :) Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com

Re: [U-Boot] [PATCH v1 15/41] net: mvpp2: introduce PPv2.2 HW descriptors and adapt accessors

2017-03-22 Thread Thomas Petazzoni
those braces before submitting the patches to the kernel, so they are not here in the code merged upstream: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git/tree/drivers/net/ethernet/marvell/mvpp2.c#n1219 Stefan: I think you used the latest branch I pushed on Github, but I

Re: [U-Boot] [PATCH 07/10] sunxi: Convert CONFIG_RGMII to Kconfig

2017-02-27 Thread Thomas Petazzoni
O_ACTIVE_HIGH>; }; So it seems weird to encode it in the defconfig if the information is already available in the Device Tree. In addition, the option names are very generic: CONFIG_SATAPWR, CONFIG_MACPWR, even though they are completely sunxi specific. So if those options really nee

Re: [U-Boot] [PATCH v2] Make FIT support really optional

2016-06-07 Thread Thomas Petazzoni
; not possible to disable FIT in the Buildroot package. My patch aims > to fix that defect. Ah, indeed, I forgot that the CONFIG_FIT option already existed. In this case, I agree that it should either exist and work, or not exist. Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux

Re: [U-Boot] [PATCH v2] Make FIT support really optional

2016-06-07 Thread Thomas Petazzoni
rt in the U-Boot, but only ensures that the relevant runtime dependencies (i.e DTC) are enabled. Of course, if the U-Boot developers want to make FIT support in the bootloader itself an optional feature, why not, but it's clearly not a requirement from our side. Best regards, Thomas

Re: [U-Boot] [PATCH] ARM: Add Support for the VInCo platform

2015-12-16 Thread Thomas Petazzoni
ld put before common header */ "should put" ? > +#define CONFIG_SYS_NO_FLASH > + > +#ifdef CONFIG_SYS_TEXT_BASE > +#undef CONFIG_SYS_TEXT_BASE > +#endif Why here? Nothing has been included so far, I guess this is going to be defined by the next line, no? > +#include &

[U-Boot] [PATCHv2] tools: use pkg-config when available to get SSL flags

2015-05-16 Thread Thomas Petazzoni
the previous behavior of hardcoding -lssl -lcrypto if pkg-config is not available or fails with an error. Signed-off-by: Thomas Petazzoni --- Changes since v1: - Use both libssl and libcrypto when calling pkg-config, because some symbols of libcrypto are used directly by u-boot. This wasn&#

[U-Boot] [PATCH] tools: use pkg-config when available to get SSL flags

2015-05-12 Thread Thomas Petazzoni
the previous behavior of hardcoding -lssl -lcrypto if pkg-config is not available or fails with an error. Signed-off-by: Thomas Petazzoni --- tools/Makefile | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/Makefile b/tools/Makefile index 4bbb153..e1bcd59 100644 --- a

Re: [U-Boot] [PATCH v2 10/13] arm: mvebu: Add dynamic SoC detection to mbus driver

2015-04-14 Thread Thomas Petazzoni
coherency. But are you actually using HW I/O coherency in U-Boot for Armada XP ? It would probably be simpler to not use HW I/O coherency at all in U-Boot, be it for AXP or A38x. Best regards, Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering ht

Re: [U-Boot] u-boot summit / miniconf this year?

2015-04-07 Thread Thomas Petazzoni
ounced during the final speech of Tim Bird at ELC 2015, but I don't remember the exact dates. Best regards, Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ___ U-Boot mailing

Re: [U-Boot] [PATCH] fix: tools: kwbimage.c: Initialize headersz to suppress warning

2014-11-21 Thread Thomas Petazzoni
case, we exit the tool, so the memcpy() is never reached. Maybe gcc doesn't realize we're getting out of the function in the default case. But oh well, if it fixes a warning :-) Best regards, Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] tools/env: change stripping strategy to allow no-stripping

2014-08-27 Thread Thomas Petazzoni
rk fine if stripping doesn't work, as it leaves the final binary untouched. Signed-off-by: Thomas Petazzoni Cc: Masahiro Yamada Cc: Sonic Zhang --- An improved solution would be to be able to override STRIP completely by setting it to /bin/true, but the main Makefile enforces STRIP = $(CROS

Re: [U-Boot] [PATCH V2] arm: omap: i2c: don't zero cnt in i2c_write

2013-12-02 Thread Thomas Petazzoni
stop bit is sent before > actual data transmission takes place. > > Therefore, remove all writes that zero the cnt register. > > Cc: Heiko Schocher > Cc: Thomas Petazzoni > Cc: Tom Rini > Cc: Lubomir Popov > Cc: Enric Balletbo Serra > Signed-off-by: Nikita K

Re: [U-Boot] OMAP3 i2c issues on IGEP, u-boot 2013.10

2013-11-27 Thread Thomas Petazzoni
Since I've absolutely no idea of the background for the problem, would you mind submitting a proper patch with a good explanation? I will be happy to test it, of course. Thanks! Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-elect

Re: [U-Boot] OMAP3 i2c issues on IGEP, u-boot 2013.10

2013-11-27 Thread Thomas Petazzoni
Check if pads/pull-ups of bus 0 are properly configured i2c_read: error waiting for addr ACK (status=0x0) TWL4030:USB:Read[0xfd] Error 1 [... more of the same stuff indefinitely ...] Thanks! Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering htt

Re: [U-Boot] OMAP3 i2c issues on IGEP, u-boot 2013.10

2013-11-27 Thread Thomas Petazzoni
.org/patch/294593/ Thanks for this patch. Unfortunately, I've applied it, and it doesn't fix the problem for me. I still have those I2C issues (did 3 boots of the IGEP boards, two of the boot failed with an endless stream of "i2c_read (addr phase): pads on bus 0 probably not confi

Re: [U-Boot] OMAP3 i2c issues on IGEP, u-boot 2013.10

2013-11-27 Thread Thomas Petazzoni
Dear Enric Balletbo Serra, On Wed, 27 Nov 2013 14:56:15 +0100, Enric Balletbo Serra wrote: > 2013/11/27 Thomas Petazzoni : > > Hello, > > > > We've recently updated from u-boot 2013.04 to u-boot 2013.10 on our > > IGEP boards (OMAP3 based, U-Boot shows "OM

[U-Boot] OMAP3 i2c issues on IGEP, u-boot 2013.10

2013-11-27 Thread Thomas Petazzoni
s out that reverting this commit actually fixes the problem. No more error messages, no more hang at boot. The commit message says that it was tested on OMAP4, OMAP5 and AM335x, but apparently OMAP3 isn't working all that well with this commit. Best regard

Re: [U-Boot] 2013 Plumber's CFP: Fastboot

2013-05-14 Thread Thomas Petazzoni
with 'fastboot', which is the Android-specific protocol to flash a system through USB (see http://en.wikipedia.org/wiki/Android_software_development#Fastboot). Besides this minor comment, I'm looking forward for the notes of this discussion on this very interesting topic. Best

Re: [U-Boot] u-boot boot process is broken, how do i recover?

2013-04-07 Thread Thomas Petazzoni
send a series of magic bytes to the UART. This is what kwboot does before sending the binary using the Xmodem protocol. Best regards, Thomas -- Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. ht

Re: [U-Boot] u-boot boot process is broken, how do i recover?

2013-04-04 Thread Thomas Petazzoni
NetgearDUOV3-V1008 On Marvell hardware, you can send a different U-Boot using the serial port, even if the on-flash U-Boot is broken. Have a look at tools/kwboot.c in the mainline U-Boot sources. It uses the serial port to send using the Xmodem protocol, a new U-Boot payload to the Kirkwood SoC.

Re: [U-Boot] [NEXT PATCH v1 5/7] SPL: Added MLO for mx35 SOC to SPL Makefile

2012-09-10 Thread Thomas Petazzoni
ename, in order to make it clear that Freescale SPL cannot be used in the same way as the TI SPL. Naming it MLO will certainly confuse users having previous experience with TI stuff: it will lead them to believe that creating a FAT filesystem and putting the MLO file in it will be the necessary s

Re: [U-Boot] [PATCH] mx6q: mx6qsabrelite: add GPIO_0__CCM_CLKO and GPIO_3__CCM_CLKO2 pin mux

2012-08-01 Thread Thomas Petazzoni
pinctrl-0 = <&pinctrl_some_config_name>; And then in the imx6q.dtsi, the pinctrl_some_config_name pinctrl configuration needs to be defined. But I guess this starts to be a bit off-topic for the U-Boot list. What about raising the issue with i.MX 6 kernel maintainers on LAKML instead? B

Re: [U-Boot] [PATCH] mx6q: mx6qsabrelite: add GPIO_0__CCM_CLKO and GPIO_3__CCM_CLKO2 pin mux

2012-08-01 Thread Thomas Petazzoni
ng set the right pinmux configuration. Best regards, Thomas -- Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH u-boot-arm/next v2 1/1] OMAP3: igep00x0: add SPL support for IGEP-based boards

2012-07-18 Thread Thomas Petazzoni
MCFG_165(512 << 20); > + *ctrla = MICRON_V_ACTIMA_165; > + *ctrlb = MICRON_V_ACTIMB_165; > + *rfr_ctrl = SDP_3430_SDRC_RFR_CTRL_165MHz; I thought the NAND version of the IGEPv2 was capable of using the 200 Mhz timings. At least, from the limited testing I had done, it seemed to work.

Re: [U-Boot] u-boot on c6x CPU ( texas instruments c6000 )

2012-06-19 Thread Thomas Petazzoni
architecture, and has its own instruction set. Thomas -- Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com ___ U-Boot mailing list U-Boot@lists.

Re: [U-Boot] "make menuconfig" does not work

2012-04-27 Thread Thomas Petazzoni
lexibility than Yocto. Best regards, Thomas -- Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] "make menuconfig" does not work

2012-04-27 Thread Thomas Petazzoni
even have support for a few first stage bootloaders: at the moment AT91Bootstrap, first stage for LPC32xx, X-Loader, and we are interested in adding more when needed. Regards, Thomas -- Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, tra

Re: [U-Boot] [PATCHv2 02/10] mkenvimage: correct and clarify comments and error messages

2011-12-20 Thread Thomas Petazzoni
> + if (filesize + 1 > envsize) { > + fprintf(stderr, "The input file is larger than the environment > partition size\n"); > +>>>>>>> mkenvimage: correct and clarify comments and error messages Seems like your forgot to resolve some

Re: [U-Boot] [PATCHv10] new tool mkenvimage: generates an env image from an arbitrary config file

2011-10-19 Thread Thomas Petazzoni
nal tool > > Signed-off-by: David Wagner > Acked-by: Mike Frysinger > Tested-by: Thomas Petazzoni Unless there are further comments, would it be possible to get this merged for the upcoming U-Boot, or is the merge window already closed? Thanks! Thomas -- Thomas Petazzoni, Free

Re: [U-Boot] [PATCHv6] new tool mkenvimage: generates an env image from an arbitrary config file

2011-09-26 Thread Thomas Petazzoni
Hello, Le Fri, 2 Sep 2011 10:48:03 +0200, David Wagner a écrit : > +"-s -o \n" ^^ typo Thomas -- Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-

Re: [U-Boot] [PATCH] new tool mkenvimage: generates an env image from an arbitrary config file

2011-08-05 Thread Thomas Petazzoni
= fopen(bin_filename, "w"); > + if (fwrite(dataptr, 1, datasize, bin_file) != datasize) > + fprintf(stderr, "fwrite() failed: %s\n", strerror(errno)); Missing exit with error here. > + ret = fclose(bin_file); > + > +out: > + if (txt_filename) > + free(txt_filename); > + if (bin_filename) > + free(bin_filename); > + return ret; No need to do useless clean up. Regards, Thomas -- Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/2] atmel: Update support of board AT91SAM9M10G45-EK to new style

2011-08-05 Thread Thomas Petazzoni
0G45-EK, and this will require a small modification in AT91Bootstrap. It is a bit amazing to see that the size of U-Boot has increased from 154912 bytes in version 1.3.4 to 233252 bytes in the current Git (and the latter has no support for Dataflash, while the former does). Regards, Thomas -- T

Re: [U-Boot] [PATCH 2/2] atmel: Update support of board AT91SAM9M10G45-EK to new style

2011-08-04 Thread Thomas Petazzoni
n this platform. Unfortunately, even with this fix in place, U-Boot still doesn't show any sign of life on this platform... Regards, Thomas -- Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-el

[U-Boot] [PATCH] atmel: Update support of board AT91SAM9M10G45-EK to new style

2011-08-04 Thread Thomas Petazzoni
Based on earlier work by Alex Waterman . Signed-off-by: Thomas Petazzoni --- MAKEALL |1 - Makefile| 21 --- board/atmel/at91sam9m10g45ek/at91sam9m10g45ek.c | 115 -- board/atmel/at91sam9m10g45ek

Re: [U-Boot] [PATCH 2/2] atmel: Update support of board AT91SAM9M10G45-EK to new style

2011-08-04 Thread Thomas Petazzoni
f, then the "U-Boot 2011.06- (Aug 04 2011 ...)" message isn't displayed on the debug serial port. Thomas -- Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com

Re: [U-Boot] [PATCH 1/2] atmel: update at91sam9m10g45 SoC support to new style

2011-08-04 Thread Thomas Petazzoni
us issues you mentioned. Best regards, Thomas -- Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com ___ U-Boot mailing list U-Boot@lists.den

[U-Boot] [PATCH 2/2] atmel: Update support of board AT91SAM9M10G45-EK to new style

2011-08-04 Thread Thomas Petazzoni
Based on earlier work by Alex Waterman . Signed-off-by: Thomas Petazzoni --- Makefile| 21 --- board/atmel/at91sam9m10g45ek/at91sam9m10g45ek.c | 114 +++-- board/atmel/at91sam9m10g45ek/led.c |6 +- boards.cfg

[U-Boot] [PATCH 1/2] atmel: update at91sam9m10g45 SoC support to new style

2011-08-04 Thread Thomas Petazzoni
Based on earlier work by Alex Waterman . Signed-off-by: Thomas Petazzoni --- .../cpu/arm926ejs/at91/at91sam9m10g45_devices.c| 114 ++-- arch/arm/include/asm/arch-at91/at91sam9g45.h | 12 ++ .../arm/include/asm/arch-at91/at91sam9g45_matrix.h | 194 +++- 3

[U-Boot] (no subject)

2011-08-04 Thread Thomas Petazzoni
since v1: - Fixed multiple style issues - Fixed the DBGU support - Support pull-ups as done for at91sam9260 Best regards, Thomas Petazzoni -- Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electron

  1   2   >