[PATCH]Compress coreboot_ram also if no payload is compressed

[Patrick Georgi]

Hi,

attached patch makes coreboot_ram compression depending on
CONFIG_COMPRESS again, instead of looking at the compression
configuration for the payload

Build-tested (some coreboot_ram files were too large without compression
for a full build, and now fit again)

Signed-off-by: Patrick Georgi

[Myles Watson]

On Wed, Sep 30, 2009 at 1:41 PM, Patrick Georgi  wrote:
> Hi,
>
> attached patch makes coreboot_ram compression depending on
> CONFIG_COMPRESS again, instead of looking at the compression
> configuration for the payload

Makes sense.
> Build-tested (some coreboot_ram files were too large without compression
> for a full build, and now fit again)
>
> Signed-off-by: Patrick Georgi

Acked-by: Myles Watson

Thanks,
Myles

[PATCH]Compress coreboot_ram also if no payload is compressed

[Peter Stuge]

Myles Watson wrote:
> > attached patch makes coreboot_ram compression depending on
> > CONFIG_COMPRESS again, instead of looking at the compression
> > configuration for the payload
>
> Makes sense.

Is the uncompressor always available?

//Peter

[PATCH]Compress coreboot_ram also if no payload is compressed

[Patrick Georgi]

Am Mittwoch, den 30.09.2009, 22:23 +0200 schrieb Peter Stuge:
> Myles Watson wrote:
> > > attached patch makes coreboot_ram compression depending on
> > > CONFIG_COMPRESS again, instead of looking at the compression
> > > configuration for the payload
> >
> > Makes sense.
>
> Is the uncompressor always available?

Yes, and even if not, that's what's CONFIG_COMPRESS is for, right?
(otherwise: what would be the point of that variable?)

Patrick

[PATCH]Compress coreboot_ram also if no payload is compressed

[Peter Stuge]

Patrick Georgi wrote:
> > > > attached patch makes coreboot_ram compression depending on
> > > > CONFIG_COMPRESS again, instead of looking at the compression
> > > > configuration for the payload
> > >
> > > Makes sense.
> >
> > Is the uncompressor always available?
>
> Yes, and even if not, that's what's CONFIG_COMPRESS is for, right?

Completely right. Great!

//Peter