Date: Sun, 27 Jan 2008 15:16:20 +0100
From: Haavard Skinnemoen <hskinnemoen@atmel.com>
To: David Brownell <david-b@pacbell.net>
Cc: Andrew Morton <akpm@linux-foundation.org>,
       Andrew Victor <linux@maxim.org.za>,
       Nicolas Ferre <nicolas.ferre@atmel.com>,
       Patrice Vilchez <patrice.vilchez@rfo.atmel.com>,
       Richard Purdie <rpurdie@rpsys.net>, linux-kernel@vger.kernel.org,
       kernel@avr32linux.org
Subject: Re: [PATCH -mm 1/2] Basic PWM driver for AVR32 and AT91
Message-ID: <20080127151620.1ea94383@siona>
In-Reply-To: <200801241253.13680.david-b@pacbell.net>
References: <1201185225-15789-1-git-send-email-hskinnemoen@atmel.com>
	<1201185225-15789-2-git-send-email-hskinnemoen@atmel.com>
	<200801241253.13680.david-b@pacbell.net>
Organization: Atmel
Mime-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8BIT
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1633
Lines: 45

On Thu, 24 Jan 2008 12:53:13 -0800
David Brownell <david-b@pacbell.net> wrote:

> On Thursday 24 January 2008, Haavard Skinnemoen wrote:
> > +config ATMEL_PWM
> > +       tristate "Atmel AT32/AT91 PWM support"
> > +       depends on (AVR32 || AT91) && EXPERIMENTAL
> 
> There's probably no need for EXPERIMENTAL except in the
> limited sense of "young driver".  :)

Yeah, I don't think EXPERIMENTAL makes sense here. Either you need the
PWM or you don't.

> There's a bug there ... it should have used ARCH_AT91 instead
> of just AT91.  Or even ARCH_AT91SAM9263 || ARCH_AT91SAM9RL.

I've turned it into ARCH_AT91. I think listing the specific chips that
have it is a bit too much; it won't break if you enable it on a chip
that doesn't have it, it will just be useless. That's what defconfigs
are for, and we still want people that test allmodconfigs on e.g.
AT91RM9200 to report any breakage.

Patch below.

Haavard

diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
index 03c0c27..23a9231 100644
--- a/drivers/misc/Kconfig
+++ b/drivers/misc/Kconfig
@@ -15,7 +15,7 @@ if MISC_DEVICES
 
 config ATMEL_PWM
 	tristate "Atmel AT32/AT91 PWM support"
-	depends on (AVR32 || AT91) && EXPERIMENTAL
+	depends on AVR32 || ARCH_AT91
 	help
 	  This option enables device driver support for the PWM channels
 	  on certain Atmel prcoessors.  Pulse Width Modulation is used for
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/