[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20100311104552.GD16376@n2100.arm.linux.org.uk>
Date: Thu, 11 Mar 2010 10:45:52 +0000
From: Russell King - ARM Linux <linux@....linux.org.uk>
To: Linus Walleij <linus.ml.walleij@...il.com>
Cc: Viresh KUMAR <viresh.kumar@...com>, rajeev-dlh.kumar@...com,
shiraz.hashim@...com, armando.visconti@...com,
ashish.priyadarshi@...com, linux-kernel@...r.kernel.org,
vipin.kumar@...com, amit.goel@...com, deepak.sikri@...com,
linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH 01/11] ST SPEAr: Added ARM PrimeXsys System Controller
SP810 header file
On Tue, Mar 09, 2010 at 09:14:53PM +0100, Linus Walleij wrote:
> 2010/3/3 Viresh KUMAR <viresh.kumar@...com>:
>
> > Signed-off-by: Viresh Kumar <viresh.kumar@...com>
> > ---
> > include/linux/sysctl_sp810.h | 59 ++++++++++++++++++++++++++++++++++++++++++
> > 1 files changed, 59 insertions(+), 0 deletions(-)
> > create mode 100644 include/linux/sysctl_sp810.h
> > (...)
>
> To the rest of the world this is probably a number of register definitions and
> an obscure inline function, so the entire internal kernel API does not need
> to know about it.
It'll also give people heart-attacks because they'll think it adds a new
set of deprecated binary sysctl() interfaces to the kernel.
Best drop the 'sysctl' part of the name as well.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists