[coreboot] Re: What does mirror_payload() do and do we still need it?

[Julius Werner]

[resend with right account]

On Fri, Mar 6, 2020 at 5:50 PM Julius Werner  wrote:
> Hi,
>
> I'm trying to refactor some CBFS code (which necessarily leaks into
> the prog_loaders) and
> mirror_payload()/CONFIG_MIRROR_PAYLOAD_TO_RAM_BEFORE_LOADING is a bit
> of a complication. I'm trying to understand if this option is actually
> used by anyone. It seems to be "default n" in Kconfig and I don't see
> any mainboard/SoC select it specifically. It is not user-visible in
> menuconfig and the Chrome OS specific defconfig files don't select it
> either. In the original patch
> (https://review.coreboot.org/c/coreboot/+/5305) there seems to have
> been some discussion about which boards can even benefit from this,
> and I don't see any indication that it was ever really enabled
> anywhere.
>
> So is this actually used anywhere, by anyone? Can I just rip it out?
>
> Thanks,
> Julius