Patchwork [1/1,Maverick] Disable CONFIG_SND_HDA_INPUT_BEEP_MODE

login
register
mail settings
Submitter Leann Ogasawara
Date May 24, 2010, 5:42 a.m.
Message ID <1274679727.24569.3440.camel@emiko>
Download mbox | patch
Permalink /patch/53374/
State Accepted
Delegated to: Leann Ogasawara
Headers show

Comments

Leann Ogasawara - May 24, 2010, 5:42 a.m.
During the UDS Maverick kernel config review session [1], we agreed to
disable CONFIG_SND_HDA_INPUT_BEEP_MODE by default.  This following patch
disables this option.  Note that this value is runtime configurable via
sysfs if people wish to enable it.

Thanks,
Leann

[1] https://wiki.ubuntu.com/KernelTeam/Specs/KernelMaverickConfigReview 

>From 99bb17ce94c81d39ac6e783fa1eaa3bd030b6180 Mon Sep 17 00:00:00 2001
From: Leann Ogasawara <leann.ogasawara@canonical.com>
Date: Sun, 23 May 2010 12:50:40 -0700
Subject: [PATCH] UBUNTU: [Config] Disable CONFIG_SND_HDA_INPUT_BEEP_MODE by default

BugLink: http://bugs.launchpad.net/bugs/582350

Set CONFIG_SND_HDA_INPUT_BEEP_MODE=0 which disables the the digital beep
interface for HD-audio by default.  Note that this value is runtime
configurable via sysfs if people wish to enable.

Signed-off-by: Leann Ogasawara <leann.ogasawara@canonical.com>
---
 debian.master/config/config.common.ports  |    2 +-
 debian.master/config/config.common.ubuntu |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
Andy Whitcroft - May 24, 2010, 8:39 a.m.
On Sun, May 23, 2010 at 10:42:07PM -0700, Leann Ogasawara wrote:
> During the UDS Maverick kernel config review session [1], we agreed to
> disable CONFIG_SND_HDA_INPUT_BEEP_MODE by default.  This following patch
> disables this option.  Note that this value is runtime configurable via
> sysfs if people wish to enable it.
> 
> Thanks,
> Leann
> 
> [1] https://wiki.ubuntu.com/KernelTeam/Specs/KernelMaverickConfigReview 
> 
> >From 99bb17ce94c81d39ac6e783fa1eaa3bd030b6180 Mon Sep 17 00:00:00 2001
> From: Leann Ogasawara <leann.ogasawara@canonical.com>
> Date: Sun, 23 May 2010 12:50:40 -0700
> Subject: [PATCH] UBUNTU: [Config] Disable CONFIG_SND_HDA_INPUT_BEEP_MODE by default
> 
> BugLink: http://bugs.launchpad.net/bugs/582350
> 
> Set CONFIG_SND_HDA_INPUT_BEEP_MODE=0 which disables the the digital beep
> interface for HD-audio by default.  Note that this value is runtime
> configurable via sysfs if people wish to enable.
> 
> Signed-off-by: Leann Ogasawara <leann.ogasawara@canonical.com>
> ---
>  debian.master/config/config.common.ports  |    2 +-
>  debian.master/config/config.common.ubuntu |    2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/debian.master/config/config.common.ports b/debian.master/config/config.common.ports
> index 0414e63..35c61e7 100644
> --- a/debian.master/config/config.common.ports
> +++ b/debian.master/config/config.common.ports
> @@ -3198,7 +3198,7 @@ CONFIG_SND_HDA_ELD=y
>  CONFIG_SND_HDA_GENERIC=y
>  CONFIG_SND_HDA_HWDEP=y
>  CONFIG_SND_HDA_INPUT_BEEP=y
> -CONFIG_SND_HDA_INPUT_BEEP_MODE=1
> +CONFIG_SND_HDA_INPUT_BEEP_MODE=0
>  CONFIG_SND_HDA_INPUT_JACK=y
>  CONFIG_SND_HDA_INTEL=m
>  CONFIG_SND_HDA_PATCH_LOADER=y
> diff --git a/debian.master/config/config.common.ubuntu b/debian.master/config/config.common.ubuntu
> index 5a070cb..3c4b7ac 100644
> --- a/debian.master/config/config.common.ubuntu
> +++ b/debian.master/config/config.common.ubuntu
> @@ -3710,7 +3710,7 @@ CONFIG_SND_HDA_ELD=y
>  CONFIG_SND_HDA_GENERIC=y
>  CONFIG_SND_HDA_HWDEP=y
>  CONFIG_SND_HDA_INPUT_BEEP=y
> -CONFIG_SND_HDA_INPUT_BEEP_MODE=1
> +CONFIG_SND_HDA_INPUT_BEEP_MODE=0
>  CONFIG_SND_HDA_INPUT_JACK=y
>  CONFIG_SND_HDA_INTEL=m
>  CONFIG_SND_HDA_PATCH_LOADER=y

Matches my memory of the session.  Slam it in.

Acked-by: Andy Whitcroft <apw@canonical.com>

-apw
Leann Ogasawara - May 24, 2010, 9:29 p.m.
Applied to Maverick master.

Thanks,
Leann

Patch

diff --git a/debian.master/config/config.common.ports b/debian.master/config/config.common.ports
index 0414e63..35c61e7 100644
--- a/debian.master/config/config.common.ports
+++ b/debian.master/config/config.common.ports
@@ -3198,7 +3198,7 @@  CONFIG_SND_HDA_ELD=y
 CONFIG_SND_HDA_GENERIC=y
 CONFIG_SND_HDA_HWDEP=y
 CONFIG_SND_HDA_INPUT_BEEP=y
-CONFIG_SND_HDA_INPUT_BEEP_MODE=1
+CONFIG_SND_HDA_INPUT_BEEP_MODE=0
 CONFIG_SND_HDA_INPUT_JACK=y
 CONFIG_SND_HDA_INTEL=m
 CONFIG_SND_HDA_PATCH_LOADER=y
diff --git a/debian.master/config/config.common.ubuntu b/debian.master/config/config.common.ubuntu
index 5a070cb..3c4b7ac 100644
--- a/debian.master/config/config.common.ubuntu
+++ b/debian.master/config/config.common.ubuntu
@@ -3710,7 +3710,7 @@  CONFIG_SND_HDA_ELD=y
 CONFIG_SND_HDA_GENERIC=y
 CONFIG_SND_HDA_HWDEP=y
 CONFIG_SND_HDA_INPUT_BEEP=y
-CONFIG_SND_HDA_INPUT_BEEP_MODE=1
+CONFIG_SND_HDA_INPUT_BEEP_MODE=0
 CONFIG_SND_HDA_INPUT_JACK=y
 CONFIG_SND_HDA_INTEL=m
 CONFIG_SND_HDA_PATCH_LOADER=y