[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4B99CEFE.2080505@st.com>
Date: Fri, 12 Mar 2010 10:49:58 +0530
From: Viresh KUMAR <viresh.kumar@...com>
To: Russell King - ARM Linux <linux@....linux.org.uk>
Cc: Linus Walleij <linus.ml.walleij@...il.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 3/11/2010 4:15 PM, Russell King - ARM Linux wrote:
> 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.
>
OK. will be removed.
--
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