uboot: Make Kconfig a default build system
diff mbox series

Message ID 1508602460-4807-1-git-send-email-abrodkin@synopsys.com
State Rejected
Headers show
Series
  • uboot: Make Kconfig a default build system
Related show

Commit Message

Alexey Brodkin Oct. 21, 2017, 4:14 p.m. UTC
Given now default U-Boot version is 2017.09 and it requires
Kconfig build system it makes perfect sense to have default
build system which matches U-Boot version requirements.

Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
---
 boot/uboot/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Thomas Petazzoni Oct. 21, 2017, 4:33 p.m. UTC | #1
Hello,

On Sat, 21 Oct 2017 19:14:20 +0300, Alexey Brodkin wrote:
> Given now default U-Boot version is 2017.09 and it requires
> Kconfig build system it makes perfect sense to have default
> build system which matches U-Boot version requirements.
> 
> Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>

This will break all defconfigs of people that still use older U-Boot
versions. So I'm not sure if/when we want to do it.

Thomas
Alexey Brodkin Oct. 21, 2017, 4:44 p.m. UTC | #2
Hi Thomas,

2017-10-21 19:33 GMT+03:00 Thomas Petazzoni <
thomas.petazzoni@free-electrons.com>:

> Hello,
>
> On Sat, 21 Oct 2017 19:14:20 +0300, Alexey Brodkin wrote:
> > Given now default U-Boot version is 2017.09 and it requires
> > Kconfig build system it makes perfect sense to have default
> > build system which matches U-Boot version requirements.
> >
> > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
>
> This will break all defconfigs of people that still use older U-Boot
> versions. So I'm not sure if/when we want to do it.


Ok that makes sense.
Only 62 of defconfigs mention BR2_TARGET_UBOOT_BUILD_SYSTEM_KCONFIG and
one mention BR2_TARGET_UBOOT_BUILD_SYSTEM_LEGACY of totally 96.

So chances are indeed some of 33 other boards might fail.

-Alexey
<div dir="ltr">Hi Thomas,<div class="gmail_extra"><br><div class="gmail_quote">2017-10-21 19:33 GMT+03:00 Thomas Petazzoni <span dir="ltr">&lt;<a href="mailto:thomas.petazzoni@free-electrons.com" target="_blank">thomas.petazzoni@free-electrons.com</a>&gt;</span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello,<br>
<span class="gmail-"><br>
On Sat, 21 Oct 2017 19:14:20 +0300, Alexey Brodkin wrote:<br>
&gt; Given now default U-Boot version is 2017.09 and it requires<br>
&gt; Kconfig build system it makes perfect sense to have default<br>
&gt; build system which matches U-Boot version requirements.<br>
&gt;<br>
&gt; Signed-off-by: Alexey Brodkin &lt;<a href="mailto:abrodkin@synopsys.com">abrodkin@synopsys.com</a>&gt;<br>
<br>
</span>This will break all defconfigs of people that still use older U-Boot<br>
versions. So I&#39;m not sure if/when we want to do it.</blockquote><div><br></div><div>Ok that makes sense.</div><div>Only 62 of defconfigs mention <span style="font-size:12.8px">BR2_TARGET_UBOOT_BUILD_SYSTEM_</span><wbr style="font-size:12.8px"><span style="font-size:12.8px">KCONFIG and</span></div><div><span style="font-size:12.8px">one mention BR2_TARGET_UBOOT_BUILD_SYSTEM_LEGACY of totally 96.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">So chances are indeed some of 33 other boards might fail.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">-Alexey</span></div></div></div></div>
Thomas Petazzoni Oct. 21, 2017, 4:56 p.m. UTC | #3
Hello,

On Sat, 21 Oct 2017 19:44:35 +0300, Алексей Бродкин wrote:

> Ok that makes sense.
> Only 62 of defconfigs mention BR2_TARGET_UBOOT_BUILD_SYSTEM_KCONFIG and
> one mention BR2_TARGET_UBOOT_BUILD_SYSTEM_LEGACY of totally 96.
> 
> So chances are indeed some of 33 other boards might fail.

Well the defconfigs that are in Buildroot we could easily fix, so it's
not really the issue. The issue is more with all the defconfigs that
people/companies have locally for their own projects.

That being said, it's true at some point, it would make sense to switch
over to Kconfig as the default. The question is when to do that without
breaking things for too many users.

Thomas
Alexey Brodkin Oct. 21, 2017, 5:12 p.m. UTC | #4
Hi Thomas,

2017-10-21 19:56 GMT+03:00 Thomas Petazzoni <
thomas.petazzoni@free-electrons.com>:

> Hello,
>
> On Sat, 21 Oct 2017 19:44:35 +0300, Алексей Бродкин wrote:
>
> > Ok that makes sense.
> > Only 62 of defconfigs mention BR2_TARGET_UBOOT_BUILD_SYSTEM_KCONFIG and
> > one mention BR2_TARGET_UBOOT_BUILD_SYSTEM_LEGACY of totally 96.
> >
> > So chances are indeed some of 33 other boards might fail.
>
> Well the defconfigs that are in Buildroot we could easily fix, so it's
> not really the issue. The issue is more with all the defconfigs that
> people/companies have locally for their own projects.
>
> That being said, it's true at some point, it would make sense to switch
> over to Kconfig as the default. The question is when to do that without
> breaking things for too many users.


That's exactly the reason to upstream your stuff.
What's not in upstream it doesn't exist so why bother?

-Alexey
<div dir="ltr">Hi Thomas,<div class="gmail_extra"><br><div class="gmail_quote">2017-10-21 19:56 GMT+03:00 Thomas Petazzoni <span dir="ltr">&lt;<a href="mailto:thomas.petazzoni@free-electrons.com" target="_blank">thomas.petazzoni@free-electrons.com</a>&gt;</span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<span class=""><br>
On Sat, 21 Oct 2017 19:44:35 +0300, Алексей Бродкин wrote:<br>
<br>
&gt; Ok that makes sense.<br>
&gt; Only 62 of defconfigs mention BR2_TARGET_UBOOT_BUILD_SYSTEM_<wbr>KCONFIG and<br>
&gt; one mention BR2_TARGET_UBOOT_BUILD_SYSTEM_<wbr>LEGACY of totally 96.<br>
&gt;<br>
&gt; So chances are indeed some of 33 other boards might fail.<br>
<br>
</span>Well the defconfigs that are in Buildroot we could easily fix, so it&#39;s<br>
not really the issue. The issue is more with all the defconfigs that<br>
people/companies have locally for their own projects.<br>
<br>
That being said, it&#39;s true at some point, it would make sense to switch<br>
over to Kconfig as the default. The question is when to do that without<br>
breaking things for too many users.</blockquote><div><br></div><div>That&#39;s exactly the reason to upstream your stuff.</div><div>What&#39;s not in upstream it doesn&#39;t exist so why bother?</div><div><br></div><div>-Alexey</div></div></div></div>
Thomas Petazzoni Oct. 21, 2017, 5:18 p.m. UTC | #5
Hello,

On Sat, 21 Oct 2017 20:12:49 +0300, Алексей Бродкин wrote:

> That's exactly the reason to upstream your stuff.
> What's not in upstream it doesn't exist so why bother?

Come on, people making real products are never going to upstream their
defconfigs to Buildroot. They are highly specific, potentially
confidential. We wouldn't even accept them in Buildroot.

Thomas
Alexey Brodkin Oct. 21, 2017, 5:33 p.m. UTC | #6
Hi Thomas,

2017-10-21 20:18 GMT+03:00 Thomas Petazzoni <
thomas.petazzoni@free-electrons.com>:

> Hello,
>
> On Sat, 21 Oct 2017 20:12:49 +0300, Алексей Бродкин wrote:
>
> > That's exactly the reason to upstream your stuff.
> > What's not in upstream it doesn't exist so why bother?
>
> Come on, people making real products are never going to upstream their
> defconfigs to Buildroot. They are highly specific, potentially
> confidential. We wouldn't even accept them in Buildroot.
>

That's all understood but I'm pretty sure they don't follow upstream master
branch and it takes a second to fix that problem if it arises once people
jump onto newer BR release, right?

-Alexey
<div dir="ltr">Hi Thomas,<div class="gmail_extra"><br><div class="gmail_quote">2017-10-21 20:18 GMT+03:00 Thomas Petazzoni <span dir="ltr">&lt;<a href="mailto:thomas.petazzoni@free-electrons.com" target="_blank">thomas.petazzoni@free-electrons.com</a>&gt;</span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<span class=""><br>
On Sat, 21 Oct 2017 20:12:49 +0300, Алексей Бродкин wrote:<br>
<br>
&gt; That&#39;s exactly the reason to upstream your stuff.<br>
&gt; What&#39;s not in upstream it doesn&#39;t exist so why bother?<br>
<br>
</span>Come on, people making real products are never going to upstream their<br>
defconfigs to Buildroot. They are highly specific, potentially<br>
confidential. We wouldn&#39;t even accept them in Buildroot.<br></blockquote><div><br></div><div>That&#39;s all understood but I&#39;m pretty sure they don&#39;t follow upstream master branch and it takes a second to fix that problem if it arises once people jump onto newer BR release, right?</div><div><br></div><div>-Alexey</div></div></div></div>

Patch
diff mbox series

diff --git a/boot/uboot/Config.in b/boot/uboot/Config.in
index 8215912339ba..2d3f189a548a 100644
--- a/boot/uboot/Config.in
+++ b/boot/uboot/Config.in
@@ -6,7 +6,7 @@  config BR2_TARGET_UBOOT
 if BR2_TARGET_UBOOT
 choice
 	prompt "Build system"
-	default BR2_TARGET_UBOOT_BUILD_SYSTEM_LEGACY
+	default BR2_TARGET_UBOOT_BUILD_SYSTEM_KCONFIG
 
 config BR2_TARGET_UBOOT_BUILD_SYSTEM_LEGACY
 	bool "Legacy"