Re: [PATCH 2/2] ARM: don't assume 32-bit when no boards are selected

2023-06-08 Thread Ahmad Fatoum
se ignore the "depends on HAVE_ARCH_STM32MP157". > It was only my attempt to introduce an error on Kbuild file. > > Cheers, > Lior. > >> -----Original Message----- >> From: Ahmad Fatoum >> Sent: Thursday, June 8, 2023 12:18 PM >> To: Lior Weintraub ;

RE: [PATCH 2/2] ARM: don't assume 32-bit when no boards are selected

2023-06-08 Thread Lior Weintraub
ends on HAVE_ARCH_STM32MP157". It was only my attempt to introduce an error on Kbuild file. Cheers, Lior. > -Original Message- > From: Ahmad Fatoum > Sent: Thursday, June 8, 2023 12:18 PM > To: Lior Weintraub ; barebox@lists.infradead.org > Cc: Ulrich Ölmann ; Sascha Hauer > >

Re: [PATCH 2/2] ARM: don't assume 32-bit when no boards are selected

2023-06-08 Thread Ahmad Fatoum
From: Ahmad Fatoum >> Sent: Thursday, June 8, 2023 11:47 AM >> To: Lior Weintraub ; barebox@lists.infradead.org >> Cc: Ulrich Ölmann ; Sascha Hauer >> >> Subject: Re: [PATCH 2/2] ARM: don't assume 32-bit when no boards are >> selected >> >> CA

RE: [PATCH 2/2] ARM: don't assume 32-bit when no boards are selected

2023-06-08 Thread Lior Weintraub
Lior Weintraub ; barebox@lists.infradead.org > Cc: Ulrich Ölmann ; Sascha Hauer > > Subject: Re: [PATCH 2/2] ARM: don't assume 32-bit when no boards are > selected > > CAUTION: External Sender > > Hello Lior, > > On 08.06.23 10:20, Lior Weintraub wrote: > &g

Re: [PATCH 2/2] ARM: don't assume 32-bit when no boards are selected

2023-06-08 Thread Ahmad Fatoum
Hello Lior, On 08.06.23 10:20, Lior Weintraub wrote: > Hi Ahmad, > > Thanks for this fix! > Few comments: > 1. I was getting errors when I tried to apply your patch on either "master" > or "next" branch but then realized your fix was already merged to "next" :-). Ye, Sascha was fast :-) > 2. I

RE: [PATCH 2/2] ARM: don't assume 32-bit when no boards are selected

2023-06-08 Thread Lior Weintraub
Hi Ahmad, Thanks for this fix! Few comments: 1. I was getting errors when I tried to apply your patch on either "master" or "next" branch but then realized your fix was already merged to "next" :-). 2. Indeed this fix doesn't show the original error we saw but now it shows: "error: #error No boa