[PATCH u-boot 00/12] config cleanups and small fixes

[Cédric Le Goater]

On 10/10/2016 08:49 AM, Joel Stanley wrote:
> On Mon, Oct 10, 2016 at 5:24 PM, Joel Stanley <joel at jms.id.au> wrote:
>> On Fri, Oct 7, 2016 at 6:24 PM, Cédric Le Goater <clg at kaod.org> wrote:
>>> On 10/06/2016 11:26 PM, Cédric Le Goater wrote:
>>>> Hello,
>>>>
>>>> Here is a set of cleanups on the configuration. It should ease the
>>>> port on v2016.11 which is a little more strict on CONFIG_ defines.
>>
>> Nice one! I had a look at all of the patches and they look good. I
>> will hold off on the last two relating to ramdisk until we see what
>> upstream say.
>>
>>>
>>> You will need this patch to start with :
>>>
>>>         https://github.com/legoater/u-boot/commit/d76a5e74286cd903c4eb96c3b5f762672121b02a
>>>
>>> forgot to send it :/
>>
>> No worries, I will include it.
> 
> I can't build the defconfig after applying your patches. I get the
> same result with your trees (05fffe5dc0ec39a11c918d1916d349708cd67cf8,
> 2d1d53622d3155193832992a11d84775e44190cf).
> 
> $ make ast_g4_nsci_defconfig
> board/aspeed/ast-g5/Kconfig:22:warning: config symbol defined without type

Yes. There is a warning. I tried to add some constants in the config file but
I am not sure this is the right way to do it.

> ***
> *** Can't find default configuration "arch/../configs/ast_g4_nsci_defconfig"!
> ***
> scripts/kconfig/Makefile:114: recipe for target 'ast_g4_nsci_defconfig' failed

hmm curious. I don't have this problem on the four platforms:

	ast2400_ncsi
	ast2400_phy
	ast2500_ncsi
	ast2500_phy

and the two branches,

	v2016.11-aspeed-openbmc
	v2016.09-aspeed-openbmc

I am using a f24 to  compile.

Cheers,

C.