Patchwork busybox-1.21.1/defconfig: disable rfkill

login
register
mail settings
Submitter Laszlo Papp
Date Aug. 13, 2013, 9:55 a.m.
Message ID <CAOMwXhP5bThhNkLLz8i0ddOGmv2dnwR7h=HD_jRnhO40m=VnwQ@mail.gmail.com>
Download mbox | patch
Permalink /patch/55527/
State New
Headers show

Comments

Laszlo Papp - Aug. 13, 2013, 9:55 a.m.
So, please show me how it works for this use case. Note this is note
defconfig, and has not much relevance to the kernel which is apparently
mentioned for some reason.

        return "\n".join(cnf), "\n".join(rem)



On Tue, Aug 13, 2013 at 10:52 AM, Laszlo Papp <lpapp@kde.org> wrote:

> As we have already discussed that, it is not any clear at all without
> example. Yes, the end user does not really care about the internal
> implementation of the feature ....
>
> Please provide useful examples and tutorials how to use a feature
> especially when a user (and apparently others posting here) is such
> clueless.
>
> Oh, by the way, and I have mentioned this in the bug report as well. We
> need proper documentation, examples, and tutorials; i.e. I am not ahead of
> the day when I first read this post. ;-)
>
>
> On Tue, Aug 13, 2013 at 10:48 AM, Burton, Ross <ross.burton@intel.com>wrote:
>
>> On 13 August 2013 08:32, Laszlo Papp <lpapp@kde.org> wrote:
>> > So, everyone suggested PACKAGECONFIG in here on the mailing list except
>> Khem
>> > whose reply I did not get.
>> >
>> > Yet, 4964 got closed by the "team". Could anyone please give a sane
>> > description why and what would block the end users other than fork?
>>
>> Quoting from comment 1 on 4964:
>>
>> """
>> Busybox is a rather complicated beast. PACKAGECONFIG is unfortunately
>> not sufficient to manage it's config system. It is very similar to the
>> Linux kernel config and we recently added configuration fragment
>> support to busybox from the kernel recipes:
>>
>> commit 56dc1720caa48344238d352c7b6e9b0f0d41aa54
>> Author: Chen Qi <Qi.Chen@windriver.com>
>> Date:   Tue Feb 5 14:36:40 2013 +0800
>>
>>     busybox: add config fragments
>>
>> This allows you to use the default config and include only the delta
>> in your layer's bbappend in the form of a config fragment added to the
>> SRC_URI.
>> """
>>
>> If this isn't sufficient or is buggy, then fixing this is preferred
>> (it works for the kernel, so should work for busybox).
>>
>> Ross
>>
>
>

Patch

diff --git a/meta/recipes-core/busybox/busybox.inc
b/meta/recipes-core/busybox/busybox.inc
index acd2bfb..0e84f4c 100644
--- a/meta/recipes-core/busybox/busybox.inc
+++ b/meta/recipes-core/busybox/busybox.inc
@@ -66,8 +66,6 @@  def features_to_busybox_settings(d):
        busybox_cfg('nls',  distro_features, 'CONFIG_LOCALE_SUPPORT', cnf,
rem)
        busybox_cfg('ipv4', distro_features,
'CONFIG_FEATURE_IFUPDOWN_IPV4', cnf, rem)
        busybox_cfg('ipv6', distro_features,
'CONFIG_FEATURE_IFUPDOWN_IPV6', cnf, rem)
-       busybox_cfg('wifi', distro_features, 'CONFIG_RFKILL', cnf, rem)
-       busybox_cfg('bluetooth', distro_features, 'CONFIG_RFKILL', cnf, rem)