From mboxrd@z Thu Jan 1 00:00:00 1970 From: Florian Fainelli Subject: Re: [RESEND PATCH 1/6] ARM: config: multi_v5: enable dynamic debug Date: Wed, 15 Aug 2018 18:29:46 -0700 Message-ID: References: <20180814140612.12554-1-marcel@ziswiler.com> <9461e62248cc7c1b5585004ff8cf7268b823e29c.camel@perches.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <9461e62248cc7c1b5585004ff8cf7268b823e29c.camel@perches.com> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: Joe Perches , Marcel Ziswiler , linux-arm-kernel@lists.infradead.org, linux-tegra@vger.kernel.org Cc: Herbert Xu , Boris Brezillon , Marcel Ziswiler , linux-kernel@vger.kernel.org, Russell King , Andrew Jeffery , Joel Stanley List-Id: linux-tegra@vger.kernel.org On 08/14/2018 07:16 AM, Joe Perches wrote: > On Tue, 2018-08-14 at 16:06 +0200, Marcel Ziswiler wrote: >> From: Marcel Ziswiler >> >> Dynamic debug is really helpful. > > It's useful, but perhaps not appropriate. > > Most arm configs are for small resource constrained systems. Yes, but multi_v*_defconfig, by definition are not, so this really seems acceptable to me here. > dynamic_debug adds a lot of additional text and a fair amount of data. > > >> Signed-off-by: Marcel Ziswiler >> >> --- >> >> arch/arm/configs/multi_v5_defconfig | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/arch/arm/configs/multi_v5_defconfig b/arch/arm/configs/multi_v5_defconfig >> index 318b76fa26d1..b302ac68d587 100644 >> --- a/arch/arm/configs/multi_v5_defconfig >> +++ b/arch/arm/configs/multi_v5_defconfig >> @@ -275,6 +275,7 @@ CONFIG_NLS_CODEPAGE_850=y >> CONFIG_NLS_ISO8859_1=y >> CONFIG_NLS_ISO8859_2=y >> CONFIG_NLS_UTF8=y >> +CONFIG_DYNAMIC_DEBUG=y >> CONFIG_DEBUG_INFO=y >> CONFIG_DEBUG_FS=y >> CONFIG_MAGIC_SYSRQ=y > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > -- Florian