public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Dave Jones <davej@redhat.com>
To: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Cc: gregkh@linuxfoundation.org, tiwai@suse.de
Subject: Re: dell_rbu: Select CONFIG_FW_LOADER_USER_HELPER explicitly
Date: Sat, 6 Jul 2013 18:14:01 -0400	[thread overview]
Message-ID: <20130706221401.GA3652@redhat.com> (raw)
In-Reply-To: <20130702192749.A2E7E660D9B@gitolite.kernel.org>

On Tue, Jul 02, 2013 at 07:27:49PM +0000, Linux Kernel wrote:
 > Gitweb:     http://git.kernel.org/linus/;a=commit;h=d05c39ea67f5786a549ac9d672d2951992b658c6
 > Commit:     d05c39ea67f5786a549ac9d672d2951992b658c6
 > Parent:     a3b2c8c7aa1ca860edcf0b0afa371d9eb2269c3c
 > Author:     Takashi Iwai <tiwai@suse.de>
 > AuthorDate: Wed May 22 18:28:37 2013 +0200
 > Committer:  Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 > CommitDate: Mon Jun 3 13:57:29 2013 -0700
 > 
 >     dell_rbu: Select CONFIG_FW_LOADER_USER_HELPER explicitly
 >     
 >     The usermode helper is mandatory for this driver.
 >     
 >     Signed-off-by: Takashi Iwai <tiwai@suse.de>
 >     Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 > ---
 >  drivers/firmware/Kconfig |    1 +
 >  1 files changed, 1 insertions(+), 0 deletions(-)
 > 
 > diff --git a/drivers/firmware/Kconfig b/drivers/firmware/Kconfig
 > index 9387630..0747872 100644
 > --- a/drivers/firmware/Kconfig
 > +++ b/drivers/firmware/Kconfig
 > @@ -64,6 +64,7 @@ config DELL_RBU
 >  	tristate "BIOS update support for DELL systems via sysfs"
 >  	depends on X86
 >  	select FW_LOADER
 > +	select FW_LOADER_USER_HELPER
 >  	help
 >  	 Say m if you want to have the option of updating the BIOS for your
 >  	 DELL system. Note you need a Dell OpenManage or Dell Update package (DUP)


This is pretty crappy. Now every distro kernel has to have that enabled,
which screws up for eg, the x86 microcode driver. (It takes 1 minute per cpu
when this is enabled).

	Dave


       reply	other threads:[~2013-07-06 22:14 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20130702192749.A2E7E660D9B@gitolite.kernel.org>
2013-07-06 22:14 ` Dave Jones [this message]
2013-07-06 22:30   ` dell_rbu: Select CONFIG_FW_LOADER_USER_HELPER explicitly Greg KH
2013-07-08  9:05     ` Takashi Iwai
2013-07-08 16:49       ` Greg KH
2013-07-08 17:52         ` Takashi Iwai
2013-07-09  3:15       ` Ming Lei
2013-07-09  5:33         ` Takashi Iwai
2013-07-09  8:43           ` Ming Lei
2013-07-09 13:32             ` Takashi Iwai
2013-07-09 14:39               ` Ming Lei
2013-07-09 14:45                 ` Takashi Iwai
2013-07-09 15:06                   ` Ming Lei
2013-07-09 15:13                     ` Takashi Iwai
2013-07-09 15:22                       ` Ming Lei
2013-07-09 15:26                         ` Takashi Iwai

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130706221401.GA3652@redhat.com \
    --to=davej@redhat.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tiwai@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox