[coreboot] buildrom payload.conf patch

Myles Watson mylesgw at gmail.com
Fri May 2 16:04:42 CEST 2008


On Fri, May 2, 2008 at 7:57 AM, Myles Watson <mylesgw at gmail.com> wrote:
> This patch consolidates the payload .conf files to eliminate redundancy.  A
>  couple of them were not consolidated because it appeared that they will be
>  more specialized in the future.
>
>  follow the patch with:
>  svn rm config/payloads/etherboot.conf
>  svn rm config/payloads/memtest.conf
>  svn rm config/payloads/tint.conf
>  svn rm config/payloads/grub2.conf
>  svn rm config/payloads/gpxe.conf
>  svn rm config/payloads/coreinfo.conf
>  svn rm config/payloads/filo.conf
>
>  since they are no longer needed.
>
>  file by file changes:
>
>  Makefile:
>  Eliminated a payload copy to simplify build.
>  Added empty targets for custom and custom-clean.
>
>  packages/coreboot-v3/coreboot-v3.mk:
>  Same payload copy change as Makefile.
>
>  packages/ofw/ofw.mk:
>  Moved the SVN information here from ofw.conf because it is here for every
>  other payload.
>
>  config/payloads/ofw.conf:
>  The only thing keeping this file here is a comment about crc32sum.
>
>  config/payloads/payloads.conf:
>  Change the way PAYLOAD and PAYLOAD-y are set.  Both are needed for lab since
>  lab is the name of the payload, but isn't an actual payload.
>
>  config/payloads/lab.conf:
>  Removed redundant PAYLOAD assignment.
>
>  config/payloads/openbios.conf:
>  Include the generic and leave this file for possible fcode-utils dependency.
>
>  config/payloads/kernel.conf:
>  Removed redundant PAYLOAD and PAYLOAD-y assignment.
>
>  config/payloads/Config.in:
>  Removed extra menu for Custom Payload with only one option in it.
>
>  config/payloads/custom.conf:
>  Simplified this file since there was no CUSTOM_PAYLOAD variable set anywhere.
>  Included Makefile.generic so that lzma compression would just work.

I forgot to svn add the two new files.  New patch attached.

config/payloads/generic.conf:
The commonalities from all the other .conf files

config/payloads/libpayload-dep.conf:
A .conf file for payloads that would have been generic, but they add a
dependency on libpayload (tint, coreinfo).

>  Signed-off-by: Myles Watson <mylesgw at gmail.com>
>
>  Thanks,
>  Myles
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: consolidate_confs.diff
Type: text/x-patch
Size: 7984 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20080502/02e1fc30/attachment.diff>


More information about the coreboot mailing list