[coreboot] v3 patch rm elfboot

Stefan Reinauer stepan at coresystems.de
Thu Feb 14 19:38:54 CET 2008

Carl-Daniel Hailfinger wrote:
> On 14.02.2008 19:15, Myles Watson wrote:
>> This patch removes elfboot and references to ELF from v3.  Since lar
>> parses ELF, there's no need to have v3 parse ELF.  The ELF files that
>> v3 could parse should always have been a subset of the ones that lar
>> could parse anyway.
>> Myles
>> Signed-off-by: Myles Watson <myles at pel.cs.byu.edu>
> Patches which remove stuff are always very welcome. A few comments below.

... as long as they do not remove functionality that is still used.

Please don't just remove this code. If you don't like to compile it in, 
create a config option to disable it. (There is such a config option 
already, so I really don't see the gain)

I say: NACK.


Because with this patch it is no longer possible to unpack a lar.

Please don't do stuff like that with levity.


coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
      Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.dehttp://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20080214/36fb081b/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 249 bytes
Desc: OpenPGP digital signature
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20080214/36fb081b/attachment.sig>

More information about the coreboot mailing list