[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <1537549106.6513.1.camel@toradex.com>
Date: Fri, 21 Sep 2018 16:58:29 +0000
From: Marcel Ziswiler <marcel.ziswiler@...adex.com>
To: "linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
"golf@...ches.com" <golf@...ches.com>,
"f.fainelli@...il.com" <f.fainelli@...il.com>,
"linux-tegra@...r.kernel.org" <linux-tegra@...r.kernel.org>
CC: "boris.brezillon@...tlin.com" <boris.brezillon@...tlin.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux@...linux.org.uk" <linux@...linux.org.uk>,
"andrew@...id.au" <andrew@...id.au>,
"joel@....id.au" <joel@....id.au>,
"herbert@...dor.apana.org.au" <herbert@...dor.apana.org.au>
Subject: Re: [RESEND PATCH 1/6] ARM: config: multi_v5: enable dynamic debug
On Wed, 2018-08-15 at 18:29 -0700, Florian Fainelli wrote:
>
> 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@...adex.com>
> > >
> > > 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.
What is the conclusion now? Is anybody gona pull this patchset in? Or
are any changes needed?
> > dynamic_debug adds a lot of additional text and a fair amount of
> > data.
> >
> >
> > > Signed-off-by: Marcel Ziswiler <marcel.ziswiler@...adex.com>
> > >
> > > ---
> > >
> > > 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@...ts.infradead.org
> > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Powered by blists - more mailing lists