All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefano Babic <sbabic@denx.de>
To: Marcel Ziswiler <marcel.ziswiler@toradex.com>,
	"u-boot@lists.denx.de" <u-boot@lists.denx.de>,
	"sbabic@denx.de" <sbabic@denx.de>
Cc: "trini@konsulko.com" <trini@konsulko.com>,
	"oliver.graute@gmail.com" <oliver.graute@gmail.com>
Subject: Re: [PATCH] Revert "imx: imx8: apalis: switch to binman"
Date: Wed, 16 Nov 2022 11:11:23 +0100	[thread overview]
Message-ID: <b7c777ae-c20f-e07e-507b-0493b23a0aac@denx.de> (raw)
In-Reply-To: <364e85d983480b22221d3b8c0b4a1dc04961ebe1.camel@toradex.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
=====================================================================


  reply	other threads:[~2022-11-16 10:11 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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 [this message]
2022-11-21 17:34 ` Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=b7c777ae-c20f-e07e-507b-0493b23a0aac@denx.de \
    --to=sbabic@denx.de \
    --cc=marcel.ziswiler@toradex.com \
    --cc=oliver.graute@gmail.com \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.