* [Buildroot] [PATCH] uboot: add default for BR2_TARGET_UBOOT_PATCH
@ 2015-04-21 13:45 Frank Hunleth
2015-04-22 20:22 ` Thomas Petazzoni
0 siblings, 1 reply; 5+ messages in thread
From: Frank Hunleth @ 2015-04-21 13:45 UTC (permalink / raw)
To: buildroot
Automatically copy over the value from BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR
to make handling its deprecation easier. Thanks to Arnout for the
suggestion.
Signed-off-by: Frank Hunleth <fhunleth@troodon-software.com>
---
boot/uboot/Config.in | 1 +
1 file changed, 1 insertion(+)
diff --git a/boot/uboot/Config.in b/boot/uboot/Config.in
index 4f01499..419b183 100644
--- a/boot/uboot/Config.in
+++ b/boot/uboot/Config.in
@@ -84,6 +84,7 @@ config BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR
config BR2_TARGET_UBOOT_PATCH
string "Custom U-Boot patches"
+ default BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR
help
A space-separated list of patches to apply to U-Boot.
Each patch can be described as an URL, a local file path,
--
1.9.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] uboot: add default for BR2_TARGET_UBOOT_PATCH
2015-04-21 13:45 [Buildroot] [PATCH] uboot: add default for BR2_TARGET_UBOOT_PATCH Frank Hunleth
@ 2015-04-22 20:22 ` Thomas Petazzoni
2015-04-22 20:31 ` Frank Hunleth
0 siblings, 1 reply; 5+ messages in thread
From: Thomas Petazzoni @ 2015-04-22 20:22 UTC (permalink / raw)
To: buildroot
Dear Frank Hunleth,
On Tue, 21 Apr 2015 09:45:36 -0400, Frank Hunleth wrote:
> Automatically copy over the value from BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR
> to make handling its deprecation easier. Thanks to Arnout for the
> suggestion.
>
> Signed-off-by: Frank Hunleth <fhunleth@troodon-software.com>
> ---
> boot/uboot/Config.in | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/boot/uboot/Config.in b/boot/uboot/Config.in
> index 4f01499..419b183 100644
> --- a/boot/uboot/Config.in
> +++ b/boot/uboot/Config.in
> @@ -84,6 +84,7 @@ config BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR
>
> config BR2_TARGET_UBOOT_PATCH
> string "Custom U-Boot patches"
> + default BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR
> help
> A space-separated list of patches to apply to U-Boot.
> Each patch can be described as an URL, a local file path,
Does this actually works? I've tried removing BR2_TARGET_UBOOT_PATCH=""
from my .config, add BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR="foo bar", and
ran 'make menuconfig'. I was hoping to see "foo bar" in
BR2_TARGET_UBOOT_PATCH, but it wasn't the case. Am I missing something?
Thanks,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] uboot: add default for BR2_TARGET_UBOOT_PATCH
2015-04-22 20:22 ` Thomas Petazzoni
@ 2015-04-22 20:31 ` Frank Hunleth
2015-04-22 20:50 ` Arnout Vandecappelle
0 siblings, 1 reply; 5+ messages in thread
From: Frank Hunleth @ 2015-04-22 20:31 UTC (permalink / raw)
To: buildroot
On Wed, Apr 22, 2015 at 4:22 PM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
>
> Dear Frank Hunleth,
>
> On Tue, 21 Apr 2015 09:45:36 -0400, Frank Hunleth wrote:
> > Automatically copy over the value from BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR
> > to make handling its deprecation easier. Thanks to Arnout for the
> > suggestion.
> >
> > Signed-off-by: Frank Hunleth <fhunleth@troodon-software.com>
> > ---
> > boot/uboot/Config.in | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/boot/uboot/Config.in b/boot/uboot/Config.in
> > index 4f01499..419b183 100644
> > --- a/boot/uboot/Config.in
> > +++ b/boot/uboot/Config.in
> > @@ -84,6 +84,7 @@ config BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR
> >
> > config BR2_TARGET_UBOOT_PATCH
> > string "Custom U-Boot patches"
> > + default BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR
> > help
> > A space-separated list of patches to apply to U-Boot.
> > Each patch can be described as an URL, a local file path,
>
> Does this actually works? I've tried removing BR2_TARGET_UBOOT_PATCH=""
> from my .config, add BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR="foo bar", and
> ran 'make menuconfig'. I was hoping to see "foo bar" in
> BR2_TARGET_UBOOT_PATCH, but it wasn't the case. Am I missing something?
>
It only works if you have BR2_DEPRECATED_SINCE_2015_05 enabled. I
almost posted an email asking how useful this was. I very rarely
enable BR2_DEPRECATED. However, I justified it to myself that if
nothing else, it was documentation that the former value in
BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR could be copied verbatim.
FWIW, I did try reordering the options to see if putting
BR2_TARGET_UBOOT_PATCH first would make this work, but it didn't
change the results on my end.
Frank
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] uboot: add default for BR2_TARGET_UBOOT_PATCH
2015-04-22 20:31 ` Frank Hunleth
@ 2015-04-22 20:50 ` Arnout Vandecappelle
2015-04-24 19:02 ` Frank Hunleth
0 siblings, 1 reply; 5+ messages in thread
From: Arnout Vandecappelle @ 2015-04-22 20:50 UTC (permalink / raw)
To: buildroot
On 04/22/15 22:31, Frank Hunleth wrote:
> On Wed, Apr 22, 2015 at 4:22 PM, Thomas Petazzoni
> <thomas.petazzoni@free-electrons.com> wrote:
>>
>> Dear Frank Hunleth,
>>
>> On Tue, 21 Apr 2015 09:45:36 -0400, Frank Hunleth wrote:
>>> Automatically copy over the value from BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR
>>> to make handling its deprecation easier. Thanks to Arnout for the
>>> suggestion.
>>>
>>> Signed-off-by: Frank Hunleth <fhunleth@troodon-software.com>
>>> ---
>>> boot/uboot/Config.in | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff --git a/boot/uboot/Config.in b/boot/uboot/Config.in
>>> index 4f01499..419b183 100644
>>> --- a/boot/uboot/Config.in
>>> +++ b/boot/uboot/Config.in
>>> @@ -84,6 +84,7 @@ config BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR
>>>
>>> config BR2_TARGET_UBOOT_PATCH
>>> string "Custom U-Boot patches"
>>> + default BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR
>>> help
>>> A space-separated list of patches to apply to U-Boot.
>>> Each patch can be described as an URL, a local file path,
>>
>> Does this actually works? I've tried removing BR2_TARGET_UBOOT_PATCH=""
>> from my .config, add BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR="foo bar", and
>> ran 'make menuconfig'. I was hoping to see "foo bar" in
>> BR2_TARGET_UBOOT_PATCH, but it wasn't the case. Am I missing something?
>>
>
> It only works if you have BR2_DEPRECATED_SINCE_2015_05 enabled.
Right, it's deprecated, not legacy...
Perhaps it's better to move it to legacy instead of deprecating. Deprecating is
for when we remove a feature, but here it's more like we renamed it.
Note that string handling in legacy is a bit special, look at e.g.
BR2_PACKAGE_TZDATA_ZONELIST (renamed to BR2_TARGET_TZ_ZONELIST) for an example.
Regards,
Arnout
> I
> almost posted an email asking how useful this was. I very rarely
> enable BR2_DEPRECATED. However, I justified it to myself that if
> nothing else, it was documentation that the former value in
> BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR could be copied verbatim.
>
> FWIW, I did try reordering the options to see if putting
> BR2_TARGET_UBOOT_PATCH first would make this work, but it didn't
> change the results on my end.
>
> Frank
>
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] uboot: add default for BR2_TARGET_UBOOT_PATCH
2015-04-22 20:50 ` Arnout Vandecappelle
@ 2015-04-24 19:02 ` Frank Hunleth
0 siblings, 0 replies; 5+ messages in thread
From: Frank Hunleth @ 2015-04-24 19:02 UTC (permalink / raw)
To: buildroot
On Wed, Apr 22, 2015 at 4:50 PM, Arnout Vandecappelle <arnout@mind.be> wrote:
> On 04/22/15 22:31, Frank Hunleth wrote:
>> On Wed, Apr 22, 2015 at 4:22 PM, Thomas Petazzoni
>> <thomas.petazzoni@free-electrons.com> wrote:
>>>
>>> Dear Frank Hunleth,
>>>
>>> On Tue, 21 Apr 2015 09:45:36 -0400, Frank Hunleth wrote:
>>>> Automatically copy over the value from BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR
>>>> to make handling its deprecation easier. Thanks to Arnout for the
>>>> suggestion.
>>>>
>>>> Signed-off-by: Frank Hunleth <fhunleth@troodon-software.com>
>>>> ---
>>>> boot/uboot/Config.in | 1 +
>>>> 1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/boot/uboot/Config.in b/boot/uboot/Config.in
>>>> index 4f01499..419b183 100644
>>>> --- a/boot/uboot/Config.in
>>>> +++ b/boot/uboot/Config.in
>>>> @@ -84,6 +84,7 @@ config BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR
>>>>
>>>> config BR2_TARGET_UBOOT_PATCH
>>>> string "Custom U-Boot patches"
>>>> + default BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR
>>>> help
>>>> A space-separated list of patches to apply to U-Boot.
>>>> Each patch can be described as an URL, a local file path,
>>>
>>> Does this actually works? I've tried removing BR2_TARGET_UBOOT_PATCH=""
>>> from my .config, add BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR="foo bar", and
>>> ran 'make menuconfig'. I was hoping to see "foo bar" in
>>> BR2_TARGET_UBOOT_PATCH, but it wasn't the case. Am I missing something?
>>>
>>
>> It only works if you have BR2_DEPRECATED_SINCE_2015_05 enabled.
>
> Right, it's deprecated, not legacy...
>
> Perhaps it's better to move it to legacy instead of deprecating. Deprecating is
> for when we remove a feature, but here it's more like we renamed it.
Thanks. Changing it from deprecated to legacy is an improvement. I'll
send the new patch as soon as I finish testing it.
-Frank
>
> Note that string handling in legacy is a bit special, look at e.g.
> BR2_PACKAGE_TZDATA_ZONELIST (renamed to BR2_TARGET_TZ_ZONELIST) for an example.
>
> Regards,
> Arnout
>
>> I
>> almost posted an email asking how useful this was. I very rarely
>> enable BR2_DEPRECATED. However, I justified it to myself that if
>> nothing else, it was documentation that the former value in
>> BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR could be copied verbatim.
>>
>> FWIW, I did try reordering the options to see if putting
>> BR2_TARGET_UBOOT_PATCH first would make this work, but it didn't
>> change the results on my end.
>>
>> Frank
>>
>>
>
>
> --
> Arnout Vandecappelle arnout at mind be
> Senior Embedded Software Architect +32-16-286500
> Essensium/Mind http://www.mind.be
> G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
> LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
> GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
--
Frank Hunleth
Troodon Software LLC
Embedded Software Development
http://troodon-software.com/
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-04-24 19:02 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-04-21 13:45 [Buildroot] [PATCH] uboot: add default for BR2_TARGET_UBOOT_PATCH Frank Hunleth
2015-04-22 20:22 ` Thomas Petazzoni
2015-04-22 20:31 ` Frank Hunleth
2015-04-22 20:50 ` Arnout Vandecappelle
2015-04-24 19:02 ` Frank Hunleth
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.