Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754415AbbEZN0j (ORCPT ); Tue, 26 May 2015 09:26:39 -0400 Received: from [134.134.136.65] ([134.134.136.65]:22899 "EHLO mga03.intel.com" rhost-flags-FAIL-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1754340AbbEZN0N convert rfc822-to-8bit (ORCPT ); Tue, 26 May 2015 09:26:13 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.13,498,1427785200"; d="scan'208";a="498320576" From: "Jie, Yang" To: Takashi Iwai CC: "broonie@kernel.org" , "alsa-devel@alsa-project.org" , "linux-kernel@vger.kernel.org" , "Girdwood, Liam R" , "Zhang, Vivian" Subject: RE: [PATCH 1/2] ALSA: Kconfig: add config menu for reduced memory footprint Thread-Topic: [PATCH 1/2] ALSA: Kconfig: add config menu for reduced memory footprint Thread-Index: AQHQl6M59m2GFbJVB0awbOPxgAfsfJ2Nkh2AgACGY5A= Date: Tue, 26 May 2015 12:32:43 +0000 Message-ID: References: <1432638265-16646-1-git-send-email-yang.jie@intel.com> <1432638265-16646-2-git-send-email-yang.jie@intel.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2558 Lines: 85 > -----Original Message----- > From: Takashi Iwai [mailto:tiwai@suse.de] > Sent: Tuesday, May 26, 2015 7:05 PM > To: Jie, Yang > Cc: broonie@kernel.org; alsa-devel@alsa-project.org; linux- > kernel@vger.kernel.org; Girdwood, Liam R; Zhang, Vivian > Subject: Re: [PATCH 1/2] ALSA: Kconfig: add config menu for reduced > memory footprint > > At Tue, 26 May 2015 19:04:24 +0800, > Jie Yang wrote: > > > > For some embedded devices, we need reduce code size and data footprint > > as much as possible, e.g. disabling procfs, hw/sw params refinement, > > mmap, dpcm, dapm, compressed API... > > > > Here add root config menu for those configuration, and disable procfs > > once reduced memory footprint is selected. > > > > Signed-off-by: Jie Yang > > --- > > sound/Kconfig | 16 ++++++++++++++++ > > 1 file changed, 16 insertions(+) > > > > diff --git a/sound/Kconfig b/sound/Kconfig index 5a240e0..6760dfa > > 100644 > > --- a/sound/Kconfig > > +++ b/sound/Kconfig > > @@ -68,6 +68,22 @@ menuconfig SND > > > > if SND > > > > +menuconfig SND_REDUCED_MEMORY_FOOTPRINT > > + bool "Reduced Memory Footprint Support" > > + default n > > + help > > + Say 'Y' to enable Reduced Memory Footprint Support, which may > > + reduce code size and data footprint as much as possible. > > I'd like to move this into sound/core. > Also, the kconfig name appears a bit too lengthy to me. > > > +config SND_PROC_FS > > + bool "Sound Proc FS Support" > > + depends on PROC_FS && !SND_REDUCED_MEMORY_FOOTPRINT > > This should be > bool "Sound Proc FS Support" > if !SND_REDUCED_MEMORY_FOOTPRINT > depends on PROC_FS > > In this way, this prompt would appear only when > SND_REDUCED_MEMORY_FOOTPRINT is set, i.e. only for experts. Takashi, I tested and found same for these two way, the prompt will appear only when PROC_FS=y and SND_REDUCED_MEMORY_FOOTPRINT is *not* set. BTW, seems here we should use without '!'? bool "Sound Proc FS Support" if SND_REDUCED_MEMORY_FOOTPRINT depends on PROC_FS ~Keyon > > > thanks, > > Takashi > > > + default y > > + help > > + Say 'N' to disable Sound proc FS, which may reduce code size about > > + 9KB on x86_64 platform. > > + If unsure say Y. > > + > > source "sound/core/Kconfig" > > > > source "sound/drivers/Kconfig" > > -- > > 1.9.1 > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/