From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Brown Subject: Re: [PATCH 1/7] pwm: Add pwm core driver Date: Fri, 1 Oct 2010 11:00:51 -0700 Message-ID: <20101001180051.GC21157@opensource.wolfsonmicro.com> References: <4CA1BC16.3020702@metafoo.de> <4CA25841.4090702@metafoo.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org To: Arun MURTHY Cc: Jassi Brar , Trilok Soni , "linux-mips@linux-mips.org" , Lars-Peter Clausen , "linux@arm.linux.org.uk" , "kernel@pengutronix.de" , Bill Gatliff , "linux-kernel@vger.kernel.org" , Linus WALLEIJ , Marek Vasut , "kgene.kim@samsung.com" , "rpurdie@rpsys.net" , "philipp.zabel@gmail.com" , Mattias WALLIN , STEricsson_nomadik_linux , "eric.y.miao@gmail.com" , Andrew List-Id: linux-omap@vger.kernel.org On Fri, Oct 01, 2010 at 10:46:15AM +0200, Arun MURTHY wrote: > > On Fri, Oct 1, 2010 at 4:25 PM, Arun MURTHY > > > I mean not the functions but the functionality. > > > PWM is a simple device and most of its clients are controlling > > intensity > > > of backlight, leds, vibrator etc. > > > I don't think these complex functionality are required. > > oh dear ! > Here I mean why should all those function be exposed to the entire kernel, > as most of the pwm devices do not use them. While many PWM uses are very simple that doesn't mean that we'll never need to support more advanced uses. Normally we try to design APIs so that they both scale down to the simplest use cases and also up to more complex ones.