From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Wed, 06 Oct 2010 21:45:54 +0200 (CEST) Received: from mail-yw0-f49.google.com ([209.85.213.49]:56721 "EHLO mail-yw0-f49.google.com" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org with ESMTP id S1491083Ab0JFTpv (ORCPT ); Wed, 6 Oct 2010 21:45:51 +0200 Received: by ywh2 with SMTP id 2so2013301ywh.36 for ; Wed, 06 Oct 2010 12:45:44 -0700 (PDT) Received: by 10.42.171.7 with SMTP id h7mr57910icz.103.1286394344549; Wed, 06 Oct 2010 12:45:44 -0700 (PDT) Received: from localhost (c-24-18-179-55.hsd1.wa.comcast.net [24.18.179.55]) by mx.google.com with ESMTPS id d13sm1266119ibb.2.2010.10.06.12.45.41 (version=TLSv1/SSLv3 cipher=RC4-MD5); Wed, 06 Oct 2010 12:45:42 -0700 (PDT) From: Kevin Hilman To: Arun MURTHY Cc: Andrew Morton , "lars\@metafoo.de" , "kernel\@pengutronix.de" , "philipp.zabel\@gmail.com" , "robert.jarzmik\@free.fr" , "marek.vasut\@gmail.com" , "eric.y.miao\@gmail.com" , "rpurdie\@rpsys.net" , "sameo\@linux.intel.com" , "kgene.kim\@samsung.com" , "linux-omap\@vger.kernel.org" , "linux-arm-kernel\@lists.infradead.org" , "linux-kernel\@vger.kernel.org" , "linux-mips\@linux-mips.org" , STEricsson_nomadik_linux , "bgat\@billgatliff.com" Subject: Re: [PATCHv2 1/7] pwm: Add pwm core driver Organization: Deep Root Systems, LLC References: <1286280002-1636-1-git-send-email-arun.murthy@stericsson.com> <1286280002-1636-2-git-send-email-arun.murthy@stericsson.com> <20101005122225.6dda30ff.akpm@linux-foundation.org> Date: Wed, 06 Oct 2010 12:45:40 -0700 In-Reply-To: (Arun MURTHY's message of "Wed, 6 Oct 2010 06:03:28 +0200") Message-ID: <87ocb7jeaj.fsf@deeprootsystems.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.1.50 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-Path: X-Envelope-To: <"|/home/ecartis/ecartis -s linux-mips"> (uid 0) X-Orcpt: rfc822;linux-mips@linux-mips.org Original-Recipient: rfc822;linux-mips@linux-mips.org X-archive-position: 27968 X-ecartis-version: Ecartis v1.0.0 Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org X-original-sender: khilman@deeprootsystems.com Precedence: bulk X-list: linux-mips Arun MURTHY writes: [...] >> >> I suggest that you work on Kevin's comments before making any code >> changes though. > > This pwm driver also supports the Davinci pwm driver as suggested by > Kelvin. My concern isn't whether it supports davinci or not. Adapting existing drivers is the easy part. My concern is that there are now two proposals for a generic PWM framework, and I would prefer to see that those projects are aligned before anything merges. Kevin From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-yw0-f49.google.com ([209.85.213.49]:56721 "EHLO mail-yw0-f49.google.com" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org with ESMTP id S1491083Ab0JFTpv (ORCPT ); Wed, 6 Oct 2010 21:45:51 +0200 Received: by ywh2 with SMTP id 2so2013301ywh.36 for ; Wed, 06 Oct 2010 12:45:44 -0700 (PDT) From: Kevin Hilman Subject: Re: [PATCHv2 1/7] pwm: Add pwm core driver References: <1286280002-1636-1-git-send-email-arun.murthy@stericsson.com> <1286280002-1636-2-git-send-email-arun.murthy@stericsson.com> <20101005122225.6dda30ff.akpm@linux-foundation.org> Date: Wed, 06 Oct 2010 12:45:40 -0700 In-Reply-To: (Arun MURTHY's message of "Wed, 6 Oct 2010 06:03:28 +0200") Message-ID: <87ocb7jeaj.fsf@deeprootsystems.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-Path: Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org To: Arun MURTHY Cc: Andrew Morton , "lars@metafoo.de" , "kernel@pengutronix.de" , "philipp.zabel@gmail.com" , "robert.jarzmik@free.fr" , "marek.vasut@gmail.com" , "eric.y.miao@gmail.com" , "rpurdie@rpsys.net" , "sameo@linux.intel.com" , "kgene.kim@samsung.com" , "linux-omap@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "linux-mips@linux-mips.org" , STEricsson_nomadik_linux , "bgat@billgatliff.com" Message-ID: <20101006194540.Kit5QktD4AiOtXERsB3DKRaTwMvcZp5ug1FO869R30c@z> Arun MURTHY writes: [...] >> >> I suggest that you work on Kevin's comments before making any code >> changes though. > > This pwm driver also supports the Davinci pwm driver as suggested by > Kelvin. My concern isn't whether it supports davinci or not. Adapting existing drivers is the easy part. My concern is that there are now two proposals for a generic PWM framework, and I would prefer to see that those projects are aligned before anything merges. Kevin From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@deeprootsystems.com (Kevin Hilman) Date: Wed, 06 Oct 2010 12:45:40 -0700 Subject: [PATCHv2 1/7] pwm: Add pwm core driver In-Reply-To: (Arun MURTHY's message of "Wed, 6 Oct 2010 06:03:28 +0200") References: <1286280002-1636-1-git-send-email-arun.murthy@stericsson.com> <1286280002-1636-2-git-send-email-arun.murthy@stericsson.com> <20101005122225.6dda30ff.akpm@linux-foundation.org> Message-ID: <87ocb7jeaj.fsf@deeprootsystems.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Arun MURTHY writes: [...] >> >> I suggest that you work on Kevin's comments before making any code >> changes though. > > This pwm driver also supports the Davinci pwm driver as suggested by > Kelvin. My concern isn't whether it supports davinci or not. Adapting existing drivers is the easy part. My concern is that there are now two proposals for a generic PWM framework, and I would prefer to see that those projects are aligned before anything merges. Kevin