[U-Boot] [PATCH] x86: Fix regression build issue of coreboot-x86_defconfig

Joe Hershberger joe.hershberger at gmail.com
Wed May 27 18:13:44 CEST 2015


Hi Bin,

On Wed, May 27, 2015 at 11:01 AM, Bin Meng <bmeng.cn at gmail.com> wrote:
> Hi Simon,
>
> On Wed, May 27, 2015 at 11:55 PM, Bin Meng <bmeng.cn at gmail.com> wrote:
>> Commit bd328eb "Clean all defconfigs with savedefconfig" accidentally
>> removed 'CONFIG_VENDOR_COREBOOT=y' from configs/coreboot-x86_defconfig.
>> This commit reverts the change.
>>
>> Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
>> ---
>>
>>  configs/coreboot-x86_defconfig | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/configs/coreboot-x86_defconfig b/configs/coreboot-x86_defconfig
>> index 66f94d0..799853f 100644
>> --- a/configs/coreboot-x86_defconfig
>> +++ b/configs/coreboot-x86_defconfig
>> @@ -1,4 +1,5 @@
>>  CONFIG_X86=y
>> +CONFIG_VENDOR_COREBOOT=y
>>  CONFIG_TARGET_COREBOOT=y
>>  CONFIG_OF_CONTROL=y
>>  CONFIG_DM_PCI=y
>> --
>
> Please apply this patch after commit
> http://git.denx.de/?p=u-boot/u-boot-x86.git;a=commit;h=3506805839e14a67b0971b02c7784e37b85d5fbf
> and before commit
> http://git.denx.de/?p=u-boot/u-boot-x86.git;a=commit;h=05cab1d6da2a84911fa0ec0ffa8fa038adef4dbc.
> I've verified the build with buildman on a new 'testing' branch with
> insertion of this patch.

This should be squashed as part of
http://git.denx.de/?p=u-boot/u-boot-x86.git;a=commit;h=05cab1d6da2a84911fa0ec0ffa8fa038adef4dbc

You need to remember to run savedefconfig when changing Kconfig or defconfig.

Cheers,
-Joe


More information about the U-Boot mailing list