[coreboot] [commit] r5900 - trunk/src/lib
mylesgw at gmail.com
Fri Oct 1 18:05:14 CEST 2010
> repository service wrote:
> > lzma.c is already included unconditionally in the same file.
> > -ramstage-$(CONFIG_COMPRESSED_PAYLOAD_LZMA) += lzma.c
> Why include lzma.c unconditionally if the payload will not be
CONFIG_COMPRESSED_PAYLOAD_LZMA != CONFIG_COMPRESSED_PAYLOAD_LZMA_SUPPORTED
This config option is for adding the payload, not for all payloads.
Right now we don't have a good way to advertise lzma support in a coreboot
If build coreboot with no payload (CONFIG_COMPRESSED_PAYLOAD_LZMA = n), then
add a payload with cbfstool, how does cbfstool know that you can't handle a
In selfboot.c, the code assumes that lzma support is always present for that
I don't know how much difference it makes. You'd have to modify a few
places in the code to be able to build an image without lzma support.
More information about the coreboot