[coreboot] Config.lb includes
mylesgw at gmail.com
Fri Apr 17 18:25:55 CEST 2009
On Fri, Apr 17, 2009 at 10:20 AM, ron minnich <rminnich at gmail.com> wrote:
> On Fri, Apr 17, 2009 at 9:16 AM, Myles Watson <mylesgw at gmail.com> wrote:
>> I forgot to include a usage note. I tested it by adding:
>> sharedinclude "Config2.lb"
>> to src/northbridge/amd/amdk8/Config.lb
>> and it looks in
> We used to have these kinds of commands that sort of "built in" a
> path. E.g. northbridge intel/xyz had an implicit path of
> We took them out at the request of a vendor. So, just checking, is
> this really better than
> include /config/shared/Config2.lb
Not really. I guess I couldn't tell from Carl-Daniel's request what
exactly he wanted. I thought this would be enough to let him make
something work. I admit it was quick and dirty. It could be easily
changed to take a path instead of a string.
> Also, Config2? Is there a better name, since you did a good thing and
> let us use any name we want?
It was just an example to make it obvious that it wasn't hardcoded to
Thanks for the review,
More information about the coreboot