* [PATCH] Revert "imx: imx8: apalis: switch to binman"
@ 2022-11-15 10:49 Stefano Babic
2022-11-16 8:25 ` Marcel Ziswiler
2022-11-21 17:34 ` Tom Rini
0 siblings, 2 replies; 4+ messages in thread
From: Stefano Babic @ 2022-11-15 10:49 UTC (permalink / raw)
To: u-boot; +Cc: Stefano Babic, Marcel Ziswiler
This reverts commit b8072ae848b73e89a73ba5b535324e4dc2793f37.
Signed-off-by: Stefano Babic <sbabic@denx.de>
Reported-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
---
arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi | 2 --
arch/arm/mach-imx/imx8/Kconfig | 1 -
configs/apalis-imx8_defconfig | 1 -
3 files changed, 4 deletions(-)
diff --git a/arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi b/arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi
index 8fd0e33d2b..956d724979 100644
--- a/arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi
+++ b/arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi
@@ -3,8 +3,6 @@
* Copyright 2019 Toradex AG
*/
-#include "imx8qm-u-boot.dtsi"
-
&mu {
u-boot,dm-pre-proper;
};
diff --git a/arch/arm/mach-imx/imx8/Kconfig b/arch/arm/mach-imx/imx8/Kconfig
index 91bd888308..37d12d1895 100644
--- a/arch/arm/mach-imx/imx8/Kconfig
+++ b/arch/arm/mach-imx/imx8/Kconfig
@@ -46,7 +46,6 @@ choice
config TARGET_APALIS_IMX8
bool "Support Apalis iMX8 module"
- select BINMAN
select BOARD_LATE_INIT
select IMX8QM
diff --git a/configs/apalis-imx8_defconfig b/configs/apalis-imx8_defconfig
index d0e24785ae..6226d50c8a 100644
--- a/configs/apalis-imx8_defconfig
+++ b/configs/apalis-imx8_defconfig
@@ -17,7 +17,6 @@ CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x80200000
CONFIG_REMAKE_ELF=y
CONFIG_FIT=y
-CONFIG_FIT_EXTERNAL_OFFSET=0x3000
CONFIG_FIT_VERBOSE=y
CONFIG_OF_SYSTEM_SETUP=y
CONFIG_LOG=y
--
2.34.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] Revert "imx: imx8: apalis: switch to binman"
2022-11-15 10:49 [PATCH] Revert "imx: imx8: apalis: switch to binman" Stefano Babic
@ 2022-11-16 8:25 ` Marcel Ziswiler
2022-11-16 10:11 ` Stefano Babic
2022-11-21 17:34 ` Tom Rini
1 sibling, 1 reply; 4+ messages in thread
From: Marcel Ziswiler @ 2022-11-16 8:25 UTC (permalink / raw)
To: u-boot@lists.denx.de, sbabic@denx.de
Cc: trini@konsulko.com, oliver.graute@gmail.com
On Tue, 2022-11-15 at 11:49 +0100, Stefano Babic wrote:
> This reverts commit b8072ae848b73e89a73ba5b535324e4dc2793f37.
Thanks Stefano. This way at least no completely defunct flash.bin gets generated.
As I do have further doubts about the whole topic I will continue on the discussion where the discussion is/was
happening...
> Signed-off-by: Stefano Babic <sbabic@denx.de>
> Reported-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
Acked-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> ---
> arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi | 2 --
> arch/arm/mach-imx/imx8/Kconfig | 1 -
> configs/apalis-imx8_defconfig | 1 -
> 3 files changed, 4 deletions(-)
>
> diff --git a/arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi b/arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi
> index 8fd0e33d2b..956d724979 100644
> --- a/arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi
> +++ b/arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi
> @@ -3,8 +3,6 @@
> * Copyright 2019 Toradex AG
> */
>
> -#include "imx8qm-u-boot.dtsi"
> -Marcel Ziswiler <marcel.ziswiler@toradex.com>
> &mu {
> u-boot,dm-pre-proper;
> };
> diff --git a/arch/arm/mach-imx/imx8/Kconfig b/arch/arm/mach-imx/imx8/Kconfig
> index 91bd888308..37d12d1895 100644
> --- a/arch/arm/mach-imx/imx8/Kconfig
> +++ b/arch/arm/mach-imx/imx8/Kconfig
> @@ -46,7 +46,6 @@ choice
>
> config TARGET_APALIS_IMX8
> bool "Support Apalis iMX8 module"
> - select BINMAN
> select BOARD_LATE_INIT
> select IMX8QM
>
> diff --git a/configs/apalis-imx8_defconfig b/configs/apalis-imx8_defconfig
> index d0e24785ae..6226d50c8a 100644
> --- a/configs/apalis-imx8_defconfig
> +++ b/configs/apalis-imx8_defconfig
> @@ -17,7 +17,6 @@ CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
> CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x80200000
> CONFIG_REMAKE_ELF=y
> CONFIG_FIT=y
> -CONFIG_FIT_EXTERNAL_OFFSET=0x3000
> CONFIG_FIT_VERBOSE=y
> CONFIG_OF_SYSTEM_SETUP=y
> CONFIG_LOG=y
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] Revert "imx: imx8: apalis: switch to binman"
2022-11-16 8:25 ` Marcel Ziswiler
@ 2022-11-16 10:11 ` Stefano Babic
0 siblings, 0 replies; 4+ messages in thread
From: Stefano Babic @ 2022-11-16 10:11 UTC (permalink / raw)
To: Marcel Ziswiler, u-boot@lists.denx.de, sbabic@denx.de
Cc: trini@konsulko.com, oliver.graute@gmail.com
On 16.11.22 09:25, Marcel Ziswiler wrote:
> On Tue, 2022-11-15 at 11:49 +0100, Stefano Babic wrote:
>> This reverts commit b8072ae848b73e89a73ba5b535324e4dc2793f37.
>
> Thanks Stefano. This way at least no completely defunct flash.bin gets generated.
>
Sorry for that - patch slip in, I was aware it should not be merged. I
merge at least in u-boot-imx, maybe it takes a while before sending PR
to Tom.
> As I do have further doubts about the whole topic I will continue on the discussion where the discussion is/was
> happening...
:-)
Regards,
Stefano
>
>> Signed-off-by: Stefano Babic <sbabic@denx.de>
>> Reported-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
>
> Acked-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
>
>> ---
>> arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi | 2 --
>> arch/arm/mach-imx/imx8/Kconfig | 1 -
>> configs/apalis-imx8_defconfig | 1 -
>> 3 files changed, 4 deletions(-)
>>
>> diff --git a/arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi b/arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi
>> index 8fd0e33d2b..956d724979 100644
>> --- a/arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi
>> +++ b/arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi
>> @@ -3,8 +3,6 @@
>> * Copyright 2019 Toradex AG
>> */
>>
>> -#include "imx8qm-u-boot.dtsi"
>> -Marcel Ziswiler <marcel.ziswiler@toradex.com>
>> &mu {
>> u-boot,dm-pre-proper;
>> };
>> diff --git a/arch/arm/mach-imx/imx8/Kconfig b/arch/arm/mach-imx/imx8/Kconfig
>> index 91bd888308..37d12d1895 100644
>> --- a/arch/arm/mach-imx/imx8/Kconfig
>> +++ b/arch/arm/mach-imx/imx8/Kconfig
>> @@ -46,7 +46,6 @@ choice
>>
>> config TARGET_APALIS_IMX8
>> bool "Support Apalis iMX8 module"
>> - select BINMAN
>> select BOARD_LATE_INIT
>> select IMX8QM
>>
>> diff --git a/configs/apalis-imx8_defconfig b/configs/apalis-imx8_defconfig
>> index d0e24785ae..6226d50c8a 100644
>> --- a/configs/apalis-imx8_defconfig
>> +++ b/configs/apalis-imx8_defconfig
>> @@ -17,7 +17,6 @@ CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
>> CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x80200000
>> CONFIG_REMAKE_ELF=y
>> CONFIG_FIT=y
>> -CONFIG_FIT_EXTERNAL_OFFSET=0x3000
>> CONFIG_FIT_VERBOSE=y
>> CONFIG_OF_SYSTEM_SETUP=y
>> CONFIG_LOG=y
--
=====================================================================
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, 82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de
=====================================================================
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] Revert "imx: imx8: apalis: switch to binman"
2022-11-15 10:49 [PATCH] Revert "imx: imx8: apalis: switch to binman" Stefano Babic
2022-11-16 8:25 ` Marcel Ziswiler
@ 2022-11-21 17:34 ` Tom Rini
1 sibling, 0 replies; 4+ messages in thread
From: Tom Rini @ 2022-11-21 17:34 UTC (permalink / raw)
To: Stefano Babic; +Cc: u-boot, Marcel Ziswiler
[-- Attachment #1: Type: text/plain, Size: 519 bytes --]
On Tue, Nov 15, 2022 at 11:49:40AM +0100, Stefano Babic wrote:
> This reverts commit b8072ae848b73e89a73ba5b535324e4dc2793f37.
>
> Signed-off-by: Stefano Babic <sbabic@denx.de>
> Reported-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> Acked-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
I saw you had mentioned putting this in to the -imx tree, but given that
I'm doing a -rc today I didn't want this to be missed, so I'm picking up
up directly now. Applied to u-boot/master, thanks!
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-11-21 17:35 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-11-15 10:49 [PATCH] Revert "imx: imx8: apalis: switch to binman" Stefano Babic
2022-11-16 8:25 ` Marcel Ziswiler
2022-11-16 10:11 ` Stefano Babic
2022-11-21 17:34 ` Tom Rini
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox