i.MX 95 support in master U-Boot

2024-09-26 Thread Max Krummenacher
Hello Peng We are working on a i.MX 95 product. What are NXP's plan on adding i.MX 95 EVK boards into U-Boot master? Thanks for some feedback. Best regards Max

Re: [PATCH] imx: imx8: fix build when CONFIG_IMX_BOOTAUX is set

2024-08-08 Thread Max Krummenacher
Hello Peng Fan On Thu, Aug 08, 2024 at 03:16:54AM +, Peng Fan wrote: > > Subject: [PATCH] imx: imx8: fix build when CONFIG_IMX_BOOTAUX is > > set > > > > From: Max Krummenacher > > > > Use correct function name. > > > > Fixes: e8cd

Re: [PATCH v2 2/2] tools: binman: ti_board_cfg: Check for linting problems

2024-01-23 Thread Max Krummenacher
On Mon, Jan 22, 2024 at 03:11:35PM -0600, Ryan Eatmon wrote: > > > On 1/22/2024 6:59 AM, Max Krummenacher wrote: > > Hi > > > > On Fri, Jan 05, 2024 at 05:09:17PM +0530, Neha Malcom Francis wrote: > > > Use yamllint for checking whether YAML configuration

Re: [PATCH v2 2/2] tools: binman: ti_board_cfg: Check for linting problems

2024-01-22 Thread Max Krummenacher
Hi On Fri, Jan 05, 2024 at 05:09:17PM +0530, Neha Malcom Francis wrote: > Use yamllint for checking whether YAML configuration files are adhering > to default yamllint rules. If I understand this correctly this patch now runs checks to verify that yaml files which are part of the U-Boot source tr

Re: [PATCH v1 4/4] board: verdin-am62: set cpu core voltage depending on speed grade

2024-01-18 Thread Max Krummenacher
On Thu, Jan 18, 2024 at 10:50:07AM +0100, Max Krummenacher wrote: > On Wed, Jan 17, 2024 at 11:16:49AM +0100, Max Krummenacher wrote: > > From: Max Krummenacher > > > > Speed grade T requires the VDD_CORE voltage to be 0.85V if using > > the maximum core frequency. &g

Re: [PATCH v1 4/4] board: verdin-am62: set cpu core voltage depending on speed grade

2024-01-18 Thread Max Krummenacher
On Wed, Jan 17, 2024 at 11:16:49AM +0100, Max Krummenacher wrote: > From: Max Krummenacher > > Speed grade T requires the VDD_CORE voltage to be 0.85V if using > the maximum core frequency. > > Speed grades G, K, S allow the VDD_CORE voltage to be 0.75V up to the > maxim

[PATCH v1 2/4] arm: mach-k3: am62: move device identification accessor functions to header

2024-01-17 Thread Max Krummenacher
From: Max Krummenacher mach-k3/am625_fdt.c does fdt fixup depending on fields in the device identification register. Move the accessors to the device identification register as inline functions into the am62_hardware.h header, so that they can be used for other functionality. Signed-off-by: Max

[PATCH v1 4/4] board: verdin-am62: set cpu core voltage depending on speed grade

2024-01-17 Thread Max Krummenacher
From: Max Krummenacher Speed grade T requires the VDD_CORE voltage to be 0.85V if using the maximum core frequency. Speed grades G, K, S allow the VDD_CORE voltage to be 0.75V up to the maximum core frequency but allow to run at 0.85V. For efficiency in manufacturing and code maintenance we

[PATCH v1 3/4] arm: mach-k3: am62: provide more soc feature info accessors

2024-01-17 Thread Max Krummenacher
From: Max Krummenacher Add two functions, one which returns the SoC speed grade and one which returns the SoC operating temperature range. Signed-off-by: Max Krummenacher --- arch/arm/mach-k3/include/mach/am62_hardware.h | 16 1 file changed, 16 insertions(+) diff --git a

[PATCH v1 1/4] board: verdin-am62: improve comment on usb phy core voltage

2024-01-17 Thread Max Krummenacher
From: Max Krummenacher TI recommends to clear the bit independent of the used voltage. So the comment which claims to do it due to the core voltage at 0.85V is bogus. See https://e2e.ti.com/support/processors-group/processors/f/processors-forum/1252724/am625-usb-phy-core-voltage-selection-and

[PATCH v1 0/4] board: verdin-am62: set cpu core voltage depending on speed grade

2024-01-17 Thread Max Krummenacher
consumption reduction for the slower speed graded SKUs. Max Krummenacher (4): board: verdin-am62: improve comment on usb phy core voltage arm: mach-k3: am62: move device identification accessor functions to header arm: mach-k3: am62: provide more soc feature info accessors board

[PATCH] ARM: arm: colibri-imx6ull-emmc: fix emmc access

2023-01-30 Thread Max Krummenacher
From: Max Krummenacher Synchronizing the device tree with linux introduced a regression. The U-Boot specific dtsi mustn't override the alias settings for the eMMC/SD interfaces. Without this U-Boot cannot access the eMMC and boot the kernel. Fixes: c21b61bff15 ("colibri-imx

Re: [PATCH v5 1/1] u-boot-initial-env: rework make target

2022-12-12 Thread Max Krummenacher
Hi Tom On Thu, Dec 8, 2022 at 9:24 PM Tom Rini wrote: > > On Mon, Nov 28, 2022 at 09:41:22AM +0100, Max Krummenacher wrote: > > > From: Max Krummenacher > > > > With LTO enabled the U-Boot initial environment is no longer stored > > in an easy accessible

[PATCH v1] test: env: add test for u-boot-initial-env creation

2022-12-09 Thread Max Krummenacher
From: Max Krummenacher Checks that `make u-boot-initial-env` creates the text file u-boot-initial-env and checks that it at least contains `board=`. Signed-off-by: Max Krummenacher --- Changes in v1: I assume every board has a board environment variable but I don't test its value. I.e

[PATCH v5 1/1] u-boot-initial-env: rework make target

2022-11-28 Thread Max Krummenacher
From: Max Krummenacher With LTO enabled the U-Boot initial environment is no longer stored in an easy accessible section in env/common.o. I.e. the section name changes from build to build, its content maybe compressed and it is annotated with additional data. Drop trying to read the initial env

[PATCH v5 0/1] Makefile: rework u-boot-initial-env target

2022-11-28 Thread Max Krummenacher
From: Max Krummenacher With CONFIG_LTO enabled the current way of extracting the configured environment no longer works, i.e. the object file content changes due to LTO. Build a host tool which prints the configured environment instead of using objcopy and friends to achive the same. The code

[PATCH v4 1/1] u-boot-initial-env: rework make target

2022-11-08 Thread Max Krummenacher
From: Max Krummenacher With LTO enabled the U-Boot initial environment is no longer stored in an easy accessible section in env/common.o. I.e. the section name changes from build to build, its content maybe compressed and it is annotated with additional data. Drop trying to read the initial env

[PATCH v4 0/1] Makefile: rework u-boot-initial-env target

2022-11-08 Thread Max Krummenacher
From: Max Krummenacher With CONFIG_LTO enabled the current way of extracting the configured environment no longer works, i.e. the object file content changes due to LTO. Build a host tool which prints the configured environment instead of using objcopy and friends to achive the same. The code

[PATCH v3 1/1] Makefile: rework u-boot-initial-env target

2022-11-04 Thread Max Krummenacher
From: Max Krummenacher With LTO enabled the U-Boot initial environment is no longer stored in an easy accessible section in env/common.o. I.e. the section name changes from build to build, its content maybe compressed and it is annotated with additional data. Drop trying to read the initial env

[PATCH v3 0/1] Makefile: rework u-boot-initial-env target

2022-11-04 Thread Max Krummenacher
From: Max Krummenacher With CONFIG_LTO enabled the current way of extracting the configured environment no longer works, i.e. the object file content changes due to LTO. Build a host tool which prints the configured environment instead of using objcopy and friends to achive the same. The code

Re: [PATCH v2 1/1] Makefile: rework u-boot-initial-env target

2022-10-31 Thread Max Krummenacher
Hi On Fri, Oct 28, 2022 at 6:40 PM Pali Rohár wrote: > > Hello! This is really much better solution! Few comments are below. > > On Friday 28 October 2022 18:18:49 Max Krummenacher wrote: > > From: Max Krummenacher > > > > With LTO enabled the U-Boot initial e

[PATCH v2 1/1] Makefile: rework u-boot-initial-env target

2022-10-28 Thread Max Krummenacher
From: Max Krummenacher With LTO enabled the U-Boot initial environment is no longer stored in an easy accessible section in env/common.o. I.e. the section name changes from build to build, its content maybe compressed and it is annotated with additional data. Drop trying to read the initial env

[PATCH v2 0/1] Makefile: rework u-boot-initial-env target

2022-10-28 Thread Max Krummenacher
From: Max Krummenacher With CONFIG_LTO enabled the current way of extracting the configured environment no longer works, i.e. the object file content changes due to LTO. Build a host tool which prints the configured environment instead of using objcopy and friends to achive the same. The code

Re: [PATCH] Makefile: fix u-boot-initial-env target if lto is enabled

2022-10-19 Thread Max Krummenacher
Hi Tom On Tue, Oct 18, 2022 at 7:57 PM Tom Rini wrote: > > On Tue, Oct 18, 2022 at 07:48:27PM +0200, Max Krummenacher wrote: > > From: Max Krummenacher > > > > With LTO enabled the U-Boot initial environment is no longer stored > > in an easy accessible section in

Re: [PATCH] Makefile: fix u-boot-initial-env target if lto is enabled

2022-10-19 Thread Max Krummenacher
; > > On Tuesday 18 October 2022 14:04:46 Tom Rini wrote: > > > > > On Tue, Oct 18, 2022 at 08:03:31PM +0200, Pali Rohár wrote: > > > > > > On Tuesday 18 October 2022 19:48:27 Max Krummenacher wrote: > > > > > > > From: Max Krummenacher &

[PATCH] Makefile: fix u-boot-initial-env target if lto is enabled

2022-10-18 Thread Max Krummenacher
From: Max Krummenacher With LTO enabled the U-Boot initial environment is no longer stored in an easy accessible section in env/common.o. I.e. the section name changes from build to build, its content maybe compressed and it is annotated with additional data. For GCC adding the option '

Re: [PATCH] imx: bootaux: respect ELF support configuration

2021-06-27 Thread Max Krummenacher
Hi Cody Am Samstag, den 26.06.2021, 22:01 -0700 schrieb Cody Gray: > When ELF support is disabled in the configuration (CONFIG_CMD_ELF), > the arch_auxiliary_core_up() function should not check whether the > specified address contains a valid ELF image. Attempting to do so > by calling the valid_e

[PATCH v1] imx: bootaux fix elf loading

2021-03-11 Thread Max Krummenacher
t;lib: elf: Move the generic elf loading/validating functions to lib") Signed-off-by: Max Krummenacher --- arch/arm/mach-imx/imx_bootaux.c | 67 - 1 file changed, 66 insertions(+), 1 deletion(-) diff --git a/arch/arm/mach-imx/imx_bootaux.c b/arch/arm/mac

Re: [PATCH v2] colibri_imx7: add readme.rst doc

2020-02-11 Thread Max Krummenacher
On Tue, 2020-02-11 at 13:34 +0200, Igor Opaniuk wrote: > From: Igor Opaniuk > > Add initial readme.rst file which provides all needed information > for obtaining a workable image ready for flashing > for both eMMC/NAND versions of Colibri iMX7. > > Signed-off-by: Igor Opaniuk > Reviewed-by: Ole

Re: [U-Boot] [PATCH v1 2/2] MAINTAINERS: change colibri_imx6/imx6ull/t30/vf maintainers

2019-11-04 Thread Max Krummenacher
On Mon, 2019-11-04 at 12:56 +0100, Igor Opaniuk wrote: > From: Igor Opaniuk > > Take over maintainership for colibri_imx6/imx6ull/t30/vf modules. > > Signed-off-by: Igor Opaniuk Acked-by: Max Krummenacher ___ U-Boot mail

Re: [U-Boot] [PATCH v1 1/2] MAINTAINERS: change apalis_imx6/colibri_imx6 maintainers

2019-11-04 Thread Max Krummenacher
On Mon, 2019-11-04 at 12:56 +0100, Igor Opaniuk wrote: > From: Igor Opaniuk > > Take over maintainership for apalis_imx6 and colibri_imx6 modules. > > Signed-off-by: Igor Opaniuk Acked-by: Max Krummenacher ___ U-Boot mail

Re: [U-Boot] [PATCH v2 4/5] imx: nandbcb: refactor update function

2019-10-23 Thread Max Krummenacher
On Mon, 2019-10-21 at 16:38 +0300, Igor Opaniuk wrote: > From: Igor Opaniuk > > Move code for writing FCB/DBBT pages to a separate function > > Signed-off-by: Igor Opaniuk Tested-by: Max Krummenacher ___ U-Boot mailing list U-Boo

Re: [U-Boot] [PATCH v2 5/5] imx: nandbcb: add support for writing BCB only

2019-10-23 Thread Max Krummenacher
> Writing 1024 bytes to 0x0: randomizing > OK > Writing 1024 bytes to 0x2: randomizing > OK > > Signed-off-by: Igor Opaniuk Tested-by: Max Krummenacher ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v2 3/5] imx: nandbcb: add support for i.MX7

2019-10-23 Thread Max Krummenacher
is_mx6()) { > + /* raw write */ > + mtd_oob_ops_t ops = { > + .datbuf = (u8 *)fcb_raw_page, > + .oobbuf = ((u8 *)fcb_raw_page) + > + mtd-&

Re: [U-Boot] [PATCH v2 2/5] nand: mxs_nand: add API for switching different BCH layouts

2019-10-23 Thread Max Krummenacher
PI for setting BCH to specific layout (and restoring it back) used by > ROM bootloader to be able to burn it in a proper way to NAND using > nandbcb command. > > Signed-off-by: Igor Opaniuk > Signed-off-by: Anti Sullin Tested-by: Max Krummenacher > --- >

Re: [U-Boot] [PATCH v2 1/5] imx: gpmi: add defines for hw randominizer

2019-10-23 Thread Max Krummenacher
ck i.MX7 APR, section > 9.6.6.3 GPMI Integrated ECC Control Register Description > (GPMI_ECCCTRLn) > > Signed-off-by: Igor Opaniuk Tested-by: Max Krummenacher > --- > > arch/arm/include/asm/mach-imx/regs-gpmi.h | 5 + > 1 file changed, 5 insertions(+) > > dif

Re: [U-Boot] [PATCH] colibri_imx6: fix broken fsl_esdhc_imx conversion

2019-09-03 Thread Max Krummenacher
amp; defined(CONFIG_SPL_BUILD) > +#if defined(CONFIG_FSL_ESDHC_IMX) && defined(CONFIG_SPL_BUILD) > /* use the following sequence: eMMC, MMC */ > struct fsl_esdhc_cfg usdhc_cfg[CONFIG_SYS_FSL_USDHC_NUM] = { > {USDHC3_BASE_ADDR}, Acked-by: Max Krummenacher Thanks! Max ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] apalis_imx6: fix broken fsl_esdhc_imx conversion

2019-09-03 Thread Max Krummenacher
#endif > > -#if defined(CONFIG_FSL_ESDHC) && defined(CONFIG_SPL_BUILD) > +#if defined(CONFIG_FSL_ESDHC_IMX) && defined(CONFIG_SPL_BUILD) > /* use the following sequence: eMMC, MMC1, SD1 */ > struct fsl_esdhc_cfg usdhc_cfg[CONFIG_SYS_FSL_USDHC_NUM] = { >

Re: [U-Boot] [PATCH] apalis_imx6: allocate specific region of memory to OP-TEE

2019-09-03 Thread Max Krummenacher
ware length */ > +#ifdef CONFIG_OPTEE_TZDRAM_SIZE > + gd->ram_size -= CONFIG_OPTEE_TZDRAM_SIZE; > +#endif > + > return 0; > } > > Acked-by: Max Krummenacher ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] apalis_imx6: add board_fit_config_name_match to support FIT in SPL

2019-09-03 Thread Max Krummenacher
t_cpu(ulong addr) > { > } I assume that you do not want to switch on CONFIG_SPL_LOAD_FIT (or OP-TEE) for the apalis_imx6_defconfig but do this rather as a preparation for a board variant. Acked-by: Max Krummenacher ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] imx: apalis_imx6: select MX6Q via Kconfig

2019-09-03 Thread Max Krummenacher
select DM > select DM_SERIAL > select DM_THERMAL Acked-by: Max Krummenacher ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 3/6] clk: imx8qm: fix usdhc2 clocks

2019-05-01 Thread Max Krummenacher
Reviewed-by: Max Krummenacher On Tue, 2019-04-30 at 12:06 +0200, Marcel Ziswiler wrote: > Trying to bring up uSDHC2 the following error message was observed: > > MMC: imx8_clk_set_rate(Invalid clk ID #60) > imx8_clk_set_rate(Invalid clk ID #60) > usdhc@5b03 - probe failed

Re: [U-Boot] [PATCH 5/6] imx8: fuse: fix fuse driver

2019-05-01 Thread Max Krummenacher
Reviewed-by: Max Krummenacher On Tue, 2019-04-30 at 12:06 +0200, Marcel Ziswiler wrote: > This fixes the i.MX 8 fuse driver to actually build for i.MX 8QM as > well. > > Signed-off-by: Marcel Ziswiler > > --- > > drivers/misc/imx8/fuse.c | 2 -- > 1 file change

Re: [U-Boot] [PATCH 2/6] arm: dts: imx8qm: add support for i2c0, i2c1, i2c2, i2c3 and i2c4

2019-05-01 Thread Max Krummenacher
Reviewed-by: Max Krummenacher On Tue, 2019-04-30 at 12:06 +0200, Marcel Ziswiler wrote: > From: Marcel Ziswiler > > Add support for i2c0, i2c1, i2c2, i2c3 and i2c4. > > Signed-off-by: Marcel Ziswiler > > --- > > arch/arm/dts/fsl-imx8qm.dtsi | 75 ++

Re: [U-Boot] [PATCH 4/6] imx8qm: fix cpu frequency reporting

2019-05-01 Thread Max Krummenacher
See review comment below. With the issue resolved: Reviewed-by: Max Krummenacher On Tue, 2019-04-30 at 12:06 +0200, Marcel Ziswiler wrote: > CPU frequency reporting failed with the following error message being > printed: > > sc_pm_get_clock_rate: resource:507 clk:2: res:3 > Cou

Re: [U-Boot] [PATCH 6/6] board: toradex: add apalis imx8qm 4gb wb it v1.0b module support

2019-05-01 Thread Max Krummenacher
See review comments below. (Marked with 'Review Max:') With the issue resolved: Reviewed-by: Max Krummenacher On Tue, 2019-04-30 at 12:06 +0200, Marcel Ziswiler wrote: > From: Marcel Ziswiler > > This commit adds initial support for the Toradex Apalis iMX8QM 4GB WB IT >

Re: [U-Boot] [PATCH 1/6] arm: dts: imx8qm: add lpuart1, lpuart2, lpuart3, lpuart4

2019-05-01 Thread Max Krummenacher
Reviewed-by: Max Krummenacher On Tue, 2019-04-30 at 12:06 +0200, Marcel Ziswiler wrote: > From: Marcel Ziswiler > > Add support for lpuart1, lpuart2, lpuart3 and lpuart4. > > Signed-off-by: Marcel Ziswiler > > --- > > arch/a

Re: [U-Boot] [PATCH] toradex: imx6: Rework PF0100 fuse programming commands to not be in SPL

2018-01-03 Thread Max Krummenacher
> On Wed, Jan 3, 2018 at 12:19 PM, Tom Rini wrote: >> The code for programming the OTP fuses on the PMIC PF0100 can only be >> used in full U-Boot, so do not build / link it into SPL. >> >> Cc: Max Krummenacher >> Cc: Stefano Babic >> Cc: Fabio Estevam &

Re: [U-Boot] [PATCH 6/6] colibri_imx6: Avoid calling setup_display() from SPL code

2017-09-24 Thread Max Krummenacher
Hi I tested this together with Anatolij's patch [1] on a Colibri iMX6. Both with the SPL and non SPL config the behaviour of the graphical output is as expected. Tested-by: Max Krummenacher Thanks Fabio. Max [1] http://patchwork.ozlabs.org/patch/8

Re: [U-Boot] [PATCH 5/6] apalis_imx6: Avoid calling setup_display() from SPL code

2017-09-24 Thread Max Krummenacher
Hi I propose to drop my patches in favor of Fabio's solution. Reviewed-by: Max Krummenacher Max Von: Anatolij Gustschin Gesendet: Samstag, 23. September 2017 21:06:57 An: Stefano Babic Cc: Fabio Estevam; u-boot@lists.denx.de; Max Krummenacher;

[U-Boot] [PATCH 1/2] configs: colibri_imx6: don't configure video in spl

2017-09-21 Thread Max Krummenacher
The functionality is not needed in the SPL. It allows to remove code conditionally in the spl case in some drivers. Signed-off-by: Max Krummenacher --- include/configs/colibri_imx6.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/configs/colibri_imx6.h b/include/configs

[U-Boot] [PATCH 2/2] configs: apalis_imx6: don't configure video in spl

2017-09-21 Thread Max Krummenacher
The functionality is not needed in the SPL. It allows to remove code conditionally in the spl case in some drivers. Signed-off-by: Max Krummenacher --- include/configs/apalis_imx6.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/configs/apalis_imx6.h b/include/configs

[U-Boot] [PATCH 0/2] configs: apalis/colibri imx6: don't configure video in spl

2017-09-21 Thread Max Krummenacher
This series remove unneeded SPL config options. This is needed to successfully compile with the following patch applied: https://lists.denx.de/pipermail/u-boot/2017-August/304010.html This series is available at http://git.toradex.com/cgit/u-boot-toradex.git/log/?h=for-next Max Krummenacher

Re: [U-Boot] [PATCH] imx6: disable clock command and print_cpuinfo code in SPL

2017-09-21 Thread Max Krummenacher
Hi On Wed, 2017-09-20 at 15:04 +0200, Stefano Babic wrote: > Hi Fabio, Anatolji, > > just to save time : this is the list of boards that are affected by this > problem. > > cgtqmx6eval > wandboard > colibri_imx6 > apalis_imx6 I will prepare a patch for these two. Max

Re: [U-Boot] [PATCH 2/2] hello_world.c: fix entry point in case of arm thumb binary

2017-08-15 Thread Max Krummenacher
Hello all Am Montag, den 14.08.2017, 17:15 -0400 schrieb Tom Rini: On Sat, Aug 12, 2017 at 11:03:46AM +0200, Max Krummenacher wrote: If compiling for thumb the U-Boot 'go' command can not jump to the entry point, as the jump will be done in the assumption that the code jumped to is

Re: [U-Boot] [PATCH 1/2] arm: use $loadaddr as the standalone entry point

2017-08-15 Thread Max Krummenacher
Hello all Am Dienstag, den 15.08.2017, 07:39 -0400 schrieb Tom Rini: > On Tue, Aug 15, 2017 at 09:32:30AM +0200, Wolfgang Denk wrote: > > > > Dear Tom, > > > > In message <20170814211300.GM20467@bill-the-cat> you wrote: > > > > > > > > > But we're talking about CONFIG_STANDALONE_LOAD_ADDR not

[U-Boot] [PATCH v2 1/2] arm: use LOADADDR as the default for STANDALONE_LOAD_ADDR

2017-08-14 Thread Max Krummenacher
Different SoCs have different RAM layouts, so using $(CONFIG_LOADADDR) instead of the constant 0xc10 for CONFIG_STANDALONE_LOAD_ADDR is probably more appropriate. Signed-off-by: Max Krummenacher --- Changes in v2: - Don't confuse loadaddr with entry point as reported by Wolfgang.

[U-Boot] [PATCH v2 2/2] hello_world.c: fix entry point in case of arm thumb binary

2017-08-14 Thread Max Krummenacher
nary needs to be compiled in the same instruction set as the U-Boot binary is compiled with. Signed-off-by: Max Krummenacher --- Changes in v2: - Don't confuse loadaddr with entry point as reported by Wolfgang. - Change the assembly magic to a C function and use __attribute__ to force arm

[U-Boot] [PATCH v2 0/2] improve hello_world standalone application for arm

2017-08-14 Thread Max Krummenacher
Changes in v2: - Don't confuse loadaddr with entry point as reported by Wolfgang. - Change the assembly magic to a C function and use __attribute__ to force arm instruction set. - Keep the entry point name hello_world as reported by Wolfgang. Max Krummenacher (2): arm: use LOADADDR as t

Re: [U-Boot] [PATCH 2/2] hello_world.c: fix entry point in case of arm thumb binary

2017-08-12 Thread Max Krummenacher
Dear Wolfgang Am Samstag, den 12.08.2017, 20:39 +0200 schrieb Wolfgang Denk: > Dear Max, > > In message <20170812090346.7887-3-max.krummenac...@toradex.com> you wrote: > > > > If compiling for thumb the U-Boot 'go' command can not jump to the entry > > point, as the jump will be done in the assu

Re: [U-Boot] [PATCH 0/2] improve hello_world standalone application for arm

2017-08-12 Thread Max Krummenacher
Dear Wolfgang Am Samstag, den 12.08.2017, 20:32 +0200 schrieb Wolfgang Denk: > Dear Max, > > In message <20170812090346.7887-1-max.krummenac...@toradex.com> you wrote: > > > > > > This series addresses > > - hardcoded entry address, use LOADADDR if available as the entry point > > instead > >

Re: [U-Boot] [PATCH 1/2] arm: use $loadaddr as the standalone entry point

2017-08-12 Thread Max Krummenacher
Dear Wolfgang Am Samstag, den 12.08.2017, 20:29 +0200 schrieb Wolfgang Denk: > Dear Max, > > In message <20170812090346.7887-2-max.krummenac...@toradex.com> you wrote: > > > > Different SoCs have different RAM layouts, so providing > > $(CONFIG_LOADADDR) instead of the constant 0xc10 for > >

[U-Boot] [PATCH 1/2] arm: use $loadaddr as the standalone entry point

2017-08-12 Thread Max Krummenacher
Different SoCs have different RAM layouts, so providing $(CONFIG_LOADADDR) instead of the constant 0xc10 for CONFIG_STANDALONE_LOAD_ADDR is probably more appropriate. Signed-off-by: Max Krummenacher --- arch/arm/config.mk| 4 doc/README.standalone | 2 +- 2 files changed, 5

[U-Boot] [PATCH 2/2] hello_world.c: fix entry point in case of arm thumb binary

2017-08-12 Thread Max Krummenacher
If compiling for thumb the U-Boot 'go' command can not jump to the entry point, as the jump will be done in the assumption that the code jumped to is using the arm instruction set. So add add a simple forwarder in arm instruction set which then jumps to the 'real' entry.

[U-Boot] [PATCH 0/2] improve hello_world standalone application for arm

2017-08-12 Thread Max Krummenacher
ent linaro compilers this does not happen so I don't propose a patch for this issue. Linaro GCC 5.4-2017.05 5.4.1 20170404 Linaro GCC 6.3-2017.05 6.3.1 20170404 Linaro GCC 7.1-2017.05 7.1.1 20170510 This series is available at http://git.toradex.com/cgit/u-boot-toradex.git/log/?h=for-next

Re: [U-Boot] [PATCH 10/17] apalis_imx6: Fix unused variable warning

2017-05-09 Thread Max Krummenacher
Hi Tom On Mon, 2017-05-08 at 22:14 -0400, Tom Rini wrote: > The variable vga_pads is never referenced, drop. > > Cc: Max Krummenacher > Signed-off-by: Tom Rini Thanks for fixing this. Acked-by: Max Krummenacher Max > --- > board/toradex/apalis_imx6/

[U-Boot] [PATCH 2/2] arm: imx: initial support for colibri imx6

2016-11-30 Thread Max Krummenacher
This adds board support for the Toradex module family Colibri iMX6. The familiy consists of a module with i.MX6 DualLite, i.MX6 Solo, both with a version for commercial and industrial temperature range. Signed-off-by: Max Krummenacher --- arch/arm/cpu/armv7/mx6/Kconfig |8

[U-Boot] [PATCH 1/2] arm: imx: initial support for apalis imx6

2016-11-30 Thread Max Krummenacher
This adds board support for the Toradex module family Apalis iMX6. The familiy consists of a module with i.MX6 Dual, i.MX6 Quad with commercial and industrial temperature range. Signed-off-by: Max Krummenacher --- arch/arm/cpu/armv7/mx6/Kconfig |8 + board/toradex

[U-Boot] [PATCH 0/2] arm: imx: add boards apalis imx6, colibri imx6

2016-11-30 Thread Max Krummenacher
is also available here: http://git.toradex.com/cgit/u-boot-toradex.git/log/?h=for-next Max Krummenacher (2): arm: imx: initial support for apalis imx6 arm: imx: initial support for colibri imx6 arch/arm/cpu/armv7/mx6/Kconfig | 16 + board/toradex/apalis_imx6

Re: [U-Boot] [PATCH v2] tools/env: fix environment alignment tests for block devices

2016-11-28 Thread Max Krummenacher
Hi Andreas 2016-11-28 10:47 GMT+01:00 Andreas Fenkart : > Hi Max, > > LGTM, see one nit below, can fixed later > > > On 11/19/2016 01:58 PM, Max Krummenacher wrote: >> >> commit 183923d3e412500bdc597d1745e2fb6f7f679ec7 enforces that the >> environment m

Re: [U-Boot] [PATCH v2] tools/env: fix environment alignment tests for block devices

2016-11-28 Thread Max Krummenacher
Hi Any news on this? The env utility is currently broken for block devices. Alternatively we could also revert commit 183923d3e412500bdc597d1745e2fb6f7f679ec7. Regards Max ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinf

[U-Boot] [PATCH v2] tools/env: fix environment alignment tests for block devices

2016-11-19 Thread Max Krummenacher
location the alignment check now errors out for perfectly legal settings. Fix this by defaulting to the standard blocksize of 0x200 for environments stored in a block device. That keeps the fw_env.config files for block devices working even with that new check. Signed-off-by: Max Krummenacher

Re: [U-Boot] [PATCH] tools/env: fix environment alignment tests for block devices

2016-11-19 Thread Max Krummenacher
Hi Andreas Am Freitag, den 18.11.2016, 14:26 +0100 schrieb Andreas Fenkart: > 2016-11-18 11:38 GMT+01:00 Max Krummenacher : > > commit 183923d3e412500bdc597d1745e2fb6f7f679ec7 enforces that the > > environment must start at an erase block boundary. > > > > F

[U-Boot] [PATCH] tools/env: fix environment alignment tests for block devices

2016-11-18 Thread Max Krummenacher
location the alignment check now errors out for perfectly legal settings. Fix this by defaulting to the standard blocksize of 0x200 for environments stored in a block device. That keeps the fw_env.config files for block devices working even with that new check. Signed-off-by: Max Krummenacher

Re: [U-Boot] [PATCH] arm: mx6: specify SPL padding

2016-11-15 Thread Max Krummenacher
Hi Stefan On Mon, 2016-11-14 at 18:06 -0800, Stefan Agner wrote: > From: Stefan Agner > > When building SPL with eMMC boot support, specify standard padding > for payload to match the load sector during boot. This allows to > use the u-boot-with-spl.imx build target to generate a direcly > flash

[U-Boot] [PATCH v2 2/2] imx: make ipu's di configurable

2016-11-01 Thread Max Krummenacher
The ipu has two display interfaces. Make the used one a parameter in struct display_info_t instead of using unconditionally DI0. DI0 is the default setting. Signed-off-by: Max Krummenacher --- Changes in v2: - none arch/arm/imx-common/video.c | 2 +- arch/arm/include/asm/imx

[U-Boot] [PATCH v2 1/2] spl: mmc: fix switch statement

2016-11-01 Thread Max Krummenacher
If CONFIG_SPL_LIBCOMMON_SUPPORT is not defined there is a lone case statement at the end of the switch leading to a compile error. Remove the offending case statement. | common/spl/spl_mmc.c:339:7: error: label at end of compound statement Signed-off-by: Max Krummenacher Reviewed-by: Tom Rini

Re: [U-Boot] [PATCH v3 0/7] toradex: config block handling

2016-10-20 Thread Max Krummenacher
ck() > - also reword commit message accordingly > - use checkboard() rather than checkboard_fallback() > - drop CUSTOM_BOARDINFO > - drop patch 1 'colibri_imx7/vf: move to custom > checkboard_fallback()' >   in favour of Stefan's proposed solution Whole series: Acked-

Re: [U-Boot] [PATCH 3/3] mx6: synchronize SPL to u-boot offset

2016-10-17 Thread Max Krummenacher
2016-10-17 14:45 GMT+02:00 Marek Vasut : > On 10/17/2016 02:28 PM, Max Krummenacher wrote: >> Hi Marek >> >>>>>> #if defined(CONFIG_SPL_MMC_SUPPORT) >>>>>> #define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR138 /* >>>>>> offs

Re: [U-Boot] [PATCH 3/3] mx6: synchronize SPL to u-boot offset

2016-10-17 Thread Max Krummenacher
Hi Marek #if defined(CONFIG_SPL_MMC_SUPPORT) #define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR138 /* offset 69KB */ +#define CONFIG_SPL_PAD_TO 0x11000 /* offset 68KB */ >>> >>> Does this mess up boards which can boot from both SD and other boot >>> med

Re: [U-Boot] [PATCH 1/3] spl: mmc: fix switch statement

2016-10-15 Thread Max Krummenacher
Hi Marek Am Samstag, den 15.10.2016, 19:29 +0200 schrieb Marek Vasut: > On 10/15/2016 07:10 PM, Max Krummenacher wrote: > > If CONFIG_SPL_LIBCOMMON_SUPPORT is not defined there is a lone case > > statement > > at the end of the switch leading to a compile error. > >

Re: [U-Boot] [PATCH 3/3] mx6: synchronize SPL to u-boot offset

2016-10-15 Thread Max Krummenacher
Hi Marak > > --- a/include/configs/imx6_spl.h > > +++ b/include/configs/imx6_spl.h > > @@ -23,6 +23,8 @@ > > *which consists of a 4K header in front of us that contains > > the IVT, DCD > > *and some padding thus 'our' max size is really 0x00908000 - > > 0x00918000 > > *or 64KB >

[U-Boot] [PATCH 3/3] mx6: synchronize SPL to u-boot offset

2016-10-15 Thread Max Krummenacher
Synchronize CONFIG_SPL_PAD_TO (used at build time for u-boot-with-spl.imx) with CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR (used by SPL to read u-boot image). Signed-off-by: Max Krummenacher --- include/configs/imx6_spl.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/include/configs

[U-Boot] [PATCH 2/3] imx: make ipu's di configurable

2016-10-15 Thread Max Krummenacher
The ipu has two display interfaces. Make the used one a parameter in struct display_info_t instead of using unconditionally DI0. DI0 is the default setting. Signed-off-by: Max Krummenacher --- arch/arm/imx-common/video.c | 2 +- arch/arm/include/asm/imx-common/video.h | 1 + 2

[U-Boot] [PATCH 1/3] spl: mmc: fix switch statement

2016-10-15 Thread Max Krummenacher
If CONFIG_SPL_LIBCOMMON_SUPPORT is not defined there is a lone case statement at the end of the switch leading to a compile error. Remove the offending case statement. | common/spl/spl_mmc.c:339:7: error: label at end of compound statement Signed-off-by: Max Krummenacher --- common/spl

[U-Boot] CONFIG_USE_ARCH_MEMSET/MEMCPY and SPL not working

2016-10-15 Thread Max Krummenacher
Hi I'm moving an i.MX6 based system to use SPL. With CONFIG_USE_ARCH_MEMSET/MEMCPY enabled the SPL build fails with undefined reference to `memset' undefined reference to `memcpy' The following RFC Patch should have made it in mainline by way of u -boot-arm, however it didn't. http://lists.denx.d

Re: [U-Boot] [PATCH] mx7_common: initialize generic timer on all CPU's

2016-07-26 Thread Max Krummenacher
fine CONFIG_SYSCOUNTER_TIMER > #define CONFIG_SC_TIMER_CLK 800 /* 8Mhz */ > +#define CONFIG_TIMER_CLK_FREQ CONFIG_SC_TIMER_CLK > #define CONFIG_SYS_FSL_CLK > > #define CONFIG_SYS_BOOTM_LEN 0x100 > -- > 2.9.0 Thanks & Regards, Max Krummenacher _

[U-Boot] [PATCH v3 2/2] nand: extend nand torture

2016-06-13 Thread Max Krummenacher
ize 0x2 (block size 0x2) Passed: 1, failed: 0 ==> nand torture 100 4 NAND torture: device 0 offset 0x100 size 0x4 (block size 0x2) Passed: 2, failed: 0 Signed-off-by: Max Krummenacher --- Changes in v3: - findings from Benoît: - aligned offset and endoffset t

[U-Boot] [PATCH v3 1/2] nand: nand torture: follow sync with linux v4.6

2016-06-13 Thread Max Krummenacher
follow parameter name change (nand to mtd) to fix compiler error. Signed-off-by: Max Krummenacher --- Changes in v3: - none Changes in v2: - Patch v1 1/1 went into master, but Scott's patch series syncing with kernel v4.6 introduced an additional compile time error. drivers/mtd

Re: [U-Boot] [PATCH v2 1/2] nand: nand torture: follow sync with linux v4.6

2016-06-09 Thread Max Krummenacher
Am Donnerstag, den 09.06.2016, 12:10 -0500 schrieb Scott Wood: > On Thu, 2016-06-09 at 10:35 +0200, Max Krummenacher wrote: > > Hi Scott > > > > 2016-06-09 1:47 GMT+02:00 Scott Wood : > > > On 06/07/2016 06:47 AM, Max Krummenacher wrote: > > > > follow

Re: [U-Boot] [PATCH v2 2/2] nand: extend nand torture

2016-06-09 Thread Max Krummenacher
Hi Benoît, 2016-06-09 1:41 GMT+02:00 Benoît Thébaudeau : > Hi Max, > > On Tue, Jun 7, 2016 at 1:46 PM, Max Krummenacher wrote: >> diff --git a/cmd/nand.c b/cmd/nand.c >> index 583a18f..8ade5e2 100644 >> --- a/cmd/nand.c >> +++ b/cmd/nand.c ... >> +

Re: [U-Boot] [PATCH v2 1/2] nand: nand torture: follow sync with linux v4.6

2016-06-09 Thread Max Krummenacher
Hi Scott 2016-06-09 1:47 GMT+02:00 Scott Wood : > On 06/07/2016 06:47 AM, Max Krummenacher wrote: >> follow parameter name change (nand to mtd) to fix compiler error. >> >> Signed-off-by: Max Krummenacher >> >> --- >> >> Changes in v2: >> - Patc

[U-Boot] [PATCH v2 2/2] nand: extend nand torture

2016-06-07 Thread Max Krummenacher
ize 0x2 (nand block size 0x2) passed 1, failed 0 ==> nand torture 100 4 NAND torture: device 0 offset 0x100 size 0x4 (nand block size 0x2) passed 2, failed 0 Signed-off-by: Max Krummenacher --- Changes in v2: - findings from Benoît: - change interface to be offse

[U-Boot] [PATCH v2 1/2] nand: nand torture: follow sync with linux v4.6

2016-06-07 Thread Max Krummenacher
follow parameter name change (nand to mtd) to fix compiler error. Signed-off-by: Max Krummenacher --- Changes in v2: - Patch v1 1/1 went into master, but Scott's patch series syncing with kernel v4.6 introduced an additional compile time error. drivers/mtd/nand/nand_util.c | 2 +- 1

Re: [U-Boot] [PATCH 2/2] nand: extend nand torture

2016-06-07 Thread Max Krummenacher
Hi Benoît, Thank you for your review. I wanted to wait for Scott's patchseries to make it into master to allow for potential needed changes. 2016-05-31 22:21 GMT+02:00 Benoît Thébaudeau : ... >> Extend this by allowing for a second parameter specifying the byte offset >> to the last block to be

[U-Boot] [PATCH 1/2] nand: fix nand torture to use changed mtd api

2016-05-31 Thread Max Krummenacher
d-off-by: Max Krummenacher --- drivers/mtd/nand/nand_util.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/mtd/nand/nand_util.c b/drivers/mtd/nand/nand_util.c index 71285b6..c957c69 100644 --- a/drivers/mtd/nand/nand_util.c +++ b/drivers/mtd/nand/nand_u

[U-Boot] [PATCH 2/2] nand: extend nand torture

2016-05-31 Thread Max Krummenacher
set 0x100 size 0x2 passed 1, failed 0 ==> nand torture 100 104 NAND torture: device 0 offset 0x100 size 0x2 passed 2, failed 0 Signed-off-by: Max Krummenacher --- cmd/nand.c | 25 + 1 file changed, 21 insertions(+), 4 deletions(-) diff --git