Re: Pull request for efi-2021-10-rc4

2021-09-04 Thread Ilias Apalodimas
Hi Tom, On Sat, 4 Sept 2021 at 21:08, Tom Rini wrote: > > On Sat, Sep 04, 2021 at 08:02:49PM +0200, Heinrich Schuchardt wrote: > > > > > > Am 4. September 2021 19:39:49 MESZ schrieb Tom Rini : > > >On Sat, Sep 04, 2021 at 07:03:48PM +0200, Heinrich Schuchardt wrote: > > >> > > >> > > >> Am 4.

Re: [PATCH 01/23] mmc: Rename MMC_SUPPORT to MMC

2021-09-04 Thread Tom Rini
On Sun, Aug 08, 2021 at 12:20:09PM -0600, Simon Glass wrote: > Rename these options so that CONFIG_IS_ENABLED can be used with them. > > Signed-off-by: Simon Glass > Reviewed-by: Jaehoon Chung For the series, and with a few minor changes (see git log a48f5ff4f523..21b86803ebb1), applied to

Re: [PATCH 02/14] lmb: Use CONFIG_LMB_*_REGIONS only if they are defined

2021-09-04 Thread Tom Rini
On Sat, Sep 04, 2021 at 09:37:39PM +0200, Marek Vasut wrote: > On 9/4/21 7:01 PM, Tom Rini wrote: > > [trimming the CC list] > > > > On Sat, Sep 04, 2021 at 06:49:03PM +0200, Marek Vasut wrote: > > > On 9/4/21 6:09 PM, Tom Rini wrote: > > > > On Sat, Sep 04, 2021 at 06:05:50PM +0200, Marek Vasut

Re: [PATCH 02/14] lmb: Use CONFIG_LMB_*_REGIONS only if they are defined

2021-09-04 Thread Marek Vasut
On 9/4/21 7:01 PM, Tom Rini wrote: [trimming the CC list] On Sat, Sep 04, 2021 at 06:49:03PM +0200, Marek Vasut wrote: On 9/4/21 6:09 PM, Tom Rini wrote: On Sat, Sep 04, 2021 at 06:05:50PM +0200, Marek Vasut wrote: On 9/4/21 5:17 PM, Tom Rini wrote: On Sat, Sep 04, 2021 at 05:15:45PM +0200,

Re: Pull request for efi-2021-10-rc4

2021-09-04 Thread Tom Rini
On Sat, Sep 04, 2021 at 08:02:49PM +0200, Heinrich Schuchardt wrote: > > > Am 4. September 2021 19:39:49 MESZ schrieb Tom Rini : > >On Sat, Sep 04, 2021 at 07:03:48PM +0200, Heinrich Schuchardt wrote: > >> > >> > >> Am 4. September 2021 16:37:22 MESZ schrieb Tom Rini : > >> >On Sat, Sep 04,

Re: Pull request for efi-2021-10-rc4

2021-09-04 Thread Heinrich Schuchardt
Am 4. September 2021 19:39:49 MESZ schrieb Tom Rini : >On Sat, Sep 04, 2021 at 07:03:48PM +0200, Heinrich Schuchardt wrote: >> >> >> Am 4. September 2021 16:37:22 MESZ schrieb Tom Rini : >> >On Sat, Sep 04, 2021 at 03:08:38PM +0200, Heinrich Schuchardt wrote: >> >> >> >> >> >> Am 4.

Re: Pull request for efi-2021-10-rc4

2021-09-04 Thread Tom Rini
On Sat, Sep 04, 2021 at 07:03:48PM +0200, Heinrich Schuchardt wrote: > > > Am 4. September 2021 16:37:22 MESZ schrieb Tom Rini : > >On Sat, Sep 04, 2021 at 03:08:38PM +0200, Heinrich Schuchardt wrote: > >> > >> > >> Am 4. September 2021 15:01:11 MESZ schrieb Tom Rini : > >> >On Sat, Sep 04,

Re: Pull request for efi-2021-10-rc4

2021-09-04 Thread Heinrich Schuchardt
Am 4. September 2021 16:37:22 MESZ schrieb Tom Rini : >On Sat, Sep 04, 2021 at 03:08:38PM +0200, Heinrich Schuchardt wrote: >> >> >> Am 4. September 2021 15:01:11 MESZ schrieb Tom Rini : >> >On Sat, Sep 04, 2021 at 11:56:47AM +0200, Heinrich Schuchardt wrote: >> > >> >> Dear Tom, >> >> >> >>

Re: [PATCH 02/14] lmb: Use CONFIG_LMB_*_REGIONS only if they are defined

2021-09-04 Thread Tom Rini
[trimming the CC list] On Sat, Sep 04, 2021 at 06:49:03PM +0200, Marek Vasut wrote: > On 9/4/21 6:09 PM, Tom Rini wrote: > > On Sat, Sep 04, 2021 at 06:05:50PM +0200, Marek Vasut wrote: > > > On 9/4/21 5:17 PM, Tom Rini wrote: > > > > On Sat, Sep 04, 2021 at 05:15:45PM +0200, Marek Vasut wrote: >

Re: [PATCH 3/5] arm: Disable ATAGs support

2021-09-04 Thread Tom Rini
On Sat, Sep 04, 2021 at 06:29:50PM +0200, Stephan Gerhold wrote: > Hi Tom, > > On Mon, Aug 30, 2021 at 09:16:30AM -0400, Tom Rini wrote: > > With the exception of nokia_rx51 and icnova-a20-swac, disable ATAG > > support. A large number of platforms had enabled support but never > > supported a

Re: [PATCH 02/14] lmb: Use CONFIG_LMB_*_REGIONS only if they are defined

2021-09-04 Thread Marek Vasut
On 9/4/21 6:09 PM, Tom Rini wrote: On Sat, Sep 04, 2021 at 06:05:50PM +0200, Marek Vasut wrote: On 9/4/21 5:17 PM, Tom Rini wrote: On Sat, Sep 04, 2021 at 05:15:45PM +0200, Marek Vasut wrote: On 9/4/21 4:10 PM, Tom Rini wrote: [...] At this point, I think you should rework things to stop

Re: [PATCH 3/5] arm: Disable ATAGs support

2021-09-04 Thread Stephan Gerhold
Hi Tom, On Mon, Aug 30, 2021 at 09:16:30AM -0400, Tom Rini wrote: > With the exception of nokia_rx51 and icnova-a20-swac, disable ATAG > support. A large number of platforms had enabled support but never > supported a kernel so old as to require it. Further, some platforms are > old enough to

Re: [PATCH 02/14] lmb: Use CONFIG_LMB_*_REGIONS only if they are defined

2021-09-04 Thread Tom Rini
On Sat, Sep 04, 2021 at 06:05:50PM +0200, Marek Vasut wrote: > On 9/4/21 5:17 PM, Tom Rini wrote: > > On Sat, Sep 04, 2021 at 05:15:45PM +0200, Marek Vasut wrote: > > > On 9/4/21 4:10 PM, Tom Rini wrote: > > > [...] > > > > > > > > > > > At this point, I think you should rework things to stop

Re: [PATCH 02/14] lmb: Use CONFIG_LMB_*_REGIONS only if they are defined

2021-09-04 Thread Marek Vasut
On 9/4/21 5:17 PM, Tom Rini wrote: On Sat, Sep 04, 2021 at 05:15:45PM +0200, Marek Vasut wrote: On 9/4/21 4:10 PM, Tom Rini wrote: [...] At this point, I think you should rework things to stop making CONFIG_LMB be optional, it should be a def_bool y. I disagree, see above. The only reason

Re: [PATCH 02/14] lmb: Use CONFIG_LMB_*_REGIONS only if they are defined

2021-09-04 Thread Tom Rini
On Sat, Sep 04, 2021 at 05:15:45PM +0200, Marek Vasut wrote: > On 9/4/21 4:10 PM, Tom Rini wrote: > [...] > > > > > > > At this point, I think you should rework things to stop making > > > > > > CONFIG_LMB be optional, it should be a def_bool y. > > > > > > > > > > I disagree, see above. > > > >

Re: [PATCH 02/14] lmb: Use CONFIG_LMB_*_REGIONS only if they are defined

2021-09-04 Thread Marek Vasut
On 9/4/21 4:10 PM, Tom Rini wrote: [...] At this point, I think you should rework things to stop making CONFIG_LMB be optional, it should be a def_bool y. I disagree, see above. The only reason "tools-only_defconfig" builds a useless u-boot binary today is in CI where it would be more work

Re: [PATCH 19/23] i2c: Convert CONFIG_POWER_I2C et al to Kconfig

2021-09-04 Thread Tom Rini
On Sun, Aug 08, 2021 at 12:20:27PM -0600, Simon Glass wrote: > This converts the following to Kconfig: >CONFIG_POWER_I2C >CONFIG_POWER_LEGACY > > They are handled at the same time due to a dependency between them. > Update the Makefile rule to use legacy power only in U-Boot proper. > >

Re: [PATCH v2 1/1] riscv: show code leading to exception

2021-09-04 Thread Sean Anderson
On 9/4/21 4:36 AM, Heinrich Schuchardt wrote: To make analyzing exceptions easier output the code that leads to it. We already do the same on the ARM platform. Here is an example: => exception ebreak Unhandled exception: Breakpoint EPC: 8ff5d50e RA: 8ff5d62c

Re: [PATCH v2 1/1] riscv: show code leading to exception

2021-09-04 Thread Bin Meng
On Sat, Sep 4, 2021 at 4:37 PM Heinrich Schuchardt wrote: > > To make analyzing exceptions easier output the code that leads to it. > We already do the same on the ARM platform. > > Here is an example: > > => exception ebreak > Unhandled exception: Breakpoint > EPC: 8ff5d50e

Re: Pull request for efi-2021-10-rc4

2021-09-04 Thread Tom Rini
On Sat, Sep 04, 2021 at 03:08:38PM +0200, Heinrich Schuchardt wrote: > > > Am 4. September 2021 15:01:11 MESZ schrieb Tom Rini : > >On Sat, Sep 04, 2021 at 11:56:47AM +0200, Heinrich Schuchardt wrote: > > > >> Dear Tom, > >> > >> The following changes since commit > >>

Re: [PATCH 02/14] lmb: Use CONFIG_LMB_*_REGIONS only if they are defined

2021-09-04 Thread Tom Rini
On Sat, Sep 04, 2021 at 04:03:25PM +0200, Marek Vasut wrote: > On 8/30/21 2:01 PM, Tom Rini wrote: > > [...] > > > > > > > > > > > > > > > > > We shouldn't need this at all. LMB and > > > > > > > > > > > > > > > > LMB_USE_MAX_REGIONS are both in > > > > > > > > > > > > > > > > Kconfig and have

Re: [PATCH 02/14] lmb: Use CONFIG_LMB_*_REGIONS only if they are defined

2021-09-04 Thread Marek Vasut
On 8/30/21 2:01 PM, Tom Rini wrote: [...] We shouldn't need this at all. LMB and LMB_USE_MAX_REGIONS are both in Kconfig and have the dependencies expressed that way. However, CONFIG_LMB_MEMORY_REGIONS and CONFIG_LMB_RESERVED_REGIONS may be undefined if CONFIG_LMB and

Re: [PATCH v2 1/1] configs: qemu-riscvXX_spl_defconfig enable CMD_SBI

2021-09-04 Thread Bin Meng
On Sat, Sep 4, 2021 at 5:53 PM Heinrich Schuchardt wrote: > > Both for 64bit and 32bit at least on one board we should compile the sbi > command. Enabling it on QEMU will allow to write a test for it. > > Signed-off-by: Heinrich Schuchardt > --- > v2: > additionally enable 32bit > --- >

Re: Pull request for efi-2021-10-rc4

2021-09-04 Thread Tom Rini
On Sat, Sep 04, 2021 at 03:08:38PM +0200, Heinrich Schuchardt wrote: > > > Am 4. September 2021 15:01:11 MESZ schrieb Tom Rini : > >On Sat, Sep 04, 2021 at 11:56:47AM +0200, Heinrich Schuchardt wrote: > > > >> Dear Tom, > >> > >> The following changes since commit > >>

Re: Pull request for efi-2021-10-rc4

2021-09-04 Thread Heinrich Schuchardt
Am 4. September 2021 15:01:11 MESZ schrieb Tom Rini : >On Sat, Sep 04, 2021 at 11:56:47AM +0200, Heinrich Schuchardt wrote: > >> Dear Tom, >> >> The following changes since commit 94509b79b13e69c209199af0757afbde8d2ebd6d: >> >> btrfs: Use default subvolume as filesystem root (2021-09-01

Re: Pull request for efi-2021-10-rc4

2021-09-04 Thread Tom Rini
On Sat, Sep 04, 2021 at 11:56:47AM +0200, Heinrich Schuchardt wrote: > Dear Tom, > > The following changes since commit 94509b79b13e69c209199af0757afbde8d2ebd6d: > > btrfs: Use default subvolume as filesystem root (2021-09-01 10:11:24 > -0400) > > are available in the Git repository at: > >

Re: [PATCH] sunxi: mmc: A20: Fix MMC optimisation

2021-09-04 Thread Jaehoon Chung
Andre Przywara wrote on 9/4/21 12:49 AM: > Some SoCs (as seen on A20) seem to misreport the MMC FIFO level if the > FIFO is completely full: the level size reads as zero, but the FIFO_FULL > bit is set. We won't do a single iteration of the read loop in this > case, so will be stuck forever. >

Re: Pull request for efi-2021-10-rc4

2021-09-04 Thread Heinrich Schuchardt
Hello Tom, the doc patch was missing I have updated tag: The following changes since commit 94509b79b13e69c209199af0757afbde8d2ebd6d: btrfs: Use default subvolume as filesystem root (2021-09-01 10:11:24 -0400) are available in the Git repository at:

Pull request for efi-2021-10-rc4

2021-09-04 Thread Heinrich Schuchardt
Dear Tom, The following changes since commit 94509b79b13e69c209199af0757afbde8d2ebd6d: btrfs: Use default subvolume as filesystem root (2021-09-01 10:11:24 -0400) are available in the Git repository at: https://source.denx.de/u-boot/custodians/u-boot-efi.git tags/efi-2021-10-rc4 for you

[PATCH v2 1/1] configs: qemu-riscvXX_spl_defconfig enable CMD_SBI

2021-09-04 Thread Heinrich Schuchardt
Both for 64bit and 32bit at least on one board we should compile the sbi command. Enabling it on QEMU will allow to write a test for it. Signed-off-by: Heinrich Schuchardt --- v2: additionally enable 32bit --- configs/qemu-riscv32_spl_defconfig | 1 + configs/qemu-riscv64_spl_defconfig

[PATCH v2 1/1] riscv: show code leading to exception

2021-09-04 Thread Heinrich Schuchardt
To make analyzing exceptions easier output the code that leads to it. We already do the same on the ARM platform. Here is an example: => exception ebreak Unhandled exception: Breakpoint EPC: 8ff5d50e RA: 8ff5d62c TVAL: EPC: 8020b50e RA:

Re: [PATCH 1/2] lib: optee: remove the duplicate CONFIG_OPTEE

2021-09-04 Thread Etienne Carriere
Hello Alex and Patrick, (my apologies for my previous malformed post) On Fri, 3 Sept 2021 at 18:43, Alex G. wrote: > > Hi Patrick > > On 9/2/21 4:56 AM, Patrick Delaunay wrote: > > The configuration CONFIG_OPTEE is defined 2 times: > > 1- in lib/optee/Kconfig for support of OPTEE images loaded