[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