Re: [PATCH v3 00/19] efi_loader: more tightly integrate UEFI disks to driver model

Soeren Moch
2022-03-08T19:15:57Z

On 08.03.22 17:56, Simon Glass wrote:
> Hi,
>
> On Tue, 8 Mar 2022 at 09:49, Heinrich Schuchardt <xypron.glpk@gmx.de> wrote:
>>
>> On 3/8/22 12:36, AKASHI Takahiro wrote:
>>> With this patch set[1] applied, UEFI subsystem maintains a list of its
>>> disk objects dynamically at runtime based on block device's probing.
>>> (See "issues" below.)
>>>
>>> [1]https://github.com/t-akashi/u-boot/tree/efi/dm_disk
>>
>> This series together with Simon's series breaks multiple boards due to
>> size constraints:
>>
>> https://source.denx.de/u-boot/custodians/u-boot-efi/-/pipelines/11197
>>
>> Please, investigate how to work around this issue.
>
> tbs2910 - perhaps we should just drop this board? It doesn't use
> DM_SERIAL and still uses OF_EMBED

Are we again at the same point? You are breaking working boards with
(for these boards) useless additions, and all you come up with is
"remove this board". Of course without adding the board maintainer.

What a shame!

Soeren

> rcar3_salvator-x - can we put the partition changes behind a config?
> phycore-rk3288 - something going on in SPL, perhaps needs an
> additional config to disable it?
>
> Regards,
> Simon