[PATCH 11/51] ARM: Kconfig: Allow kprobes on Thumb-2 kernels

Tixy tixy at yxit.co.uk
Mon Jul 11 07:47:02 EDT 2011


On Mon, 2011-07-11 at 12:42 +0100, Russell King - ARM Linux wrote:
> On Mon, Jul 11, 2011 at 12:33:41PM +0100, Tixy wrote:
> > On Mon, 2011-07-11 at 15:01 +0400, Sergei Shtylyov wrote: 
> > > Hello.
> > > 
> > > On 09-07-2011 14:56, Tixy wrote:
> > > 
> > > > From: Jon Medhurst<tixy at yxit.co.uk>
> > > 
> > > > Signed-off-by: Jon Medhurst<tixy at yxit.co.uk>
> > > > ---
> > > >   arch/arm/Kconfig |    2 +-
> > > >   1 files changed, 1 insertions(+), 1 deletions(-)
> > > 
> > > > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> > > > index 9adc278..b4177c5 100644
> > > > --- a/arch/arm/Kconfig
> > > > +++ b/arch/arm/Kconfig
> > > > @@ -10,7 +10,7 @@ config ARM
> > > >   	select GENERIC_ATOMIC64 if (CPU_V6 || !CPU_32v6K || !AEABI)
> > > >   	select HAVE_OPROFILE if (HAVE_PERF_EVENTS)
> > > >   	select HAVE_ARCH_KGDB
> > > > -	select HAVE_KPROBES if (!XIP_KERNEL&&  !THUMB2_KERNEL)
> > > > +	select HAVE_KPROBES if (!XIP_KERNEL)
> > > 
> > >     Why not drop useless parens, while at it?
> > 
> > Every other 'if' clause in that file has these parenthesis so I thought
> > I would kept it consistent.
> 
> There's plenty of examples where this isn't the case, 
[...]
> We really need to stop useless additional parens for simple expressions.

OK, fair enough.




More information about the linux-arm-kernel mailing list