On 07/10/2016 16:30, Akshay Bhat wrote: > Hi Stefano > > On Fri, Oct 7, 2016 at 6:25 AM, Stefano Babic <sba...@denx.de> wrote: >> >> But not pushed: in fact, this canoot be built anymore. You add a new >> CONFIG_ (but do you really need it ?), and this is not allowed anymore. >> It generates a build error because all CONFIG_ should be set via Kconfig. >> >> arm: + ge_b850v3 >> +Error: You must add new CONFIG options using Kconfig >> +The following new ad-hoc CONFIG options were detected: >> +CONFIG_BOOTARGS_EXTRA >> + >> >> I think it os ok if you rename it and let it as local #define in your >> configuration file. > > Thanks for the feedback, it makes sense to rename it as local #define. > Have submitted v2 patch with the fix. >
Thanks to send the new version: I will apply it directly and it can be part of my PR (I want to send it today to Tom). Best regards, Stefano -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de ===================================================================== _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
