Re: [RESEND PATCH 1/6] ARM: config: multi_v5: enable dynamic debug

From: Florian Fainelli
Date: Wed Aug 15 2018 - 21:29:55 EST




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 <marcel.ziswiler@xxxxxxxxxxx>
>>
>> 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 <marcel.ziswiler@xxxxxxxxxxx>
>>
>> ---
>>
>> 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@xxxxxxxxxxxxxxxxxxx
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>

--
Florian