From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([2001:6f8:1178:4:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.69 #1 (Red Hat Linux)) id 1Nfqsu-0002yH-M9 for barebox@lists.infradead.org; Fri, 12 Feb 2010 08:28:22 +0000 Date: Fri, 12 Feb 2010 09:28:12 +0100 From: Sascha Hauer Message-ID: <20100212082812.GT17643@pengutronix.de> References: <201001291251.16655.jbe@pengutronix.de> <201002111601.47675.jbe@pengutronix.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <201002111601.47675.jbe@pengutronix.de> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: barebox-bounces@lists.infradead.org Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH] i.MX35: Fix pad control bit positions To: Juergen Beisert Cc: barebox@lists.infradead.org Hi Juergen, On Thu, Feb 11, 2010 at 04:01:47PM +0100, Juergen Beisert wrote: > Hi Sascha, > > as discussed at FOSDEM: *ping* ;-) This is already applied. Sascha > > Juergen Beisert wrote: > > Using these macros simplify the configuration for special GPIO usage. But > > they should use correct bit positions for usage in the IOMUX_PAD() macro. > > > > Note: These are the bit positions of the i.MX35 CPU. Not checked for the > > other i.MX3x CPUs. > > > > Signed-off-by: Juergen Beisert > > > > --- > > arch/arm/mach-imx/include/mach/iomux-v3.h | 34 > > +++++++++++++++--------------- 1 file changed, 17 insertions(+), 17 > > deletions(-) > > > > Index: u-boot-v2/arch/arm/mach-imx/include/mach/iomux-v3.h > > =================================================================== > > --- u-boot-v2.orig/arch/arm/mach-imx/include/mach/iomux-v3.h > > +++ u-boot-v2/arch/arm/mach-imx/include/mach/iomux-v3.h > > @@ -68,25 +68,25 @@ struct pad_desc { > > /* > > * Use to set PAD control > > */ > > -#define PAD_CTL_DRIVE_VOLTAGE_3_3_V 0 > > -#define PAD_CTL_DRIVE_VOLTAGE_1_8_V 1 > > +#define PAD_CTL_DRIVE_VOLTAGE_3_3_V (0) > > +#define PAD_CTL_DRIVE_VOLTAGE_1_8_V (1 << 13) > > > > -#define PAD_CTL_NO_HYSTERESIS 0 > > -#define PAD_CTL_HYSTERESIS 1 > > +#define PAD_CTL_NO_HYSTERESIS (0) > > +#define PAD_CTL_HYSTERESIS (1 << 8) > > > > -#define PAD_CTL_PULL_DISABLED 0x0 > > -#define PAD_CTL_PULL_KEEPER 0xa > > -#define PAD_CTL_PULL_DOWN_100K 0xc > > -#define PAD_CTL_PULL_UP_47K 0xd > > -#define PAD_CTL_PULL_UP_100K 0xe > > -#define PAD_CTL_PULL_UP_22K 0xf > > - > > -#define PAD_CTL_OUTPUT_CMOS 0 > > -#define PAD_CTL_OUTPUT_OPEN_DRAIN 1 > > - > > -#define PAD_CTL_DRIVE_STRENGTH_NORM 0 > > -#define PAD_CTL_DRIVE_STRENGTH_HIGH 1 > > -#define PAD_CTL_DRIVE_STRENGTH_MAX 2 > > +#define PAD_CTL_PULL_DISABLED (0) > > +#define PAD_CTL_PULL_KEEPER (0xa << 4) > > +#define PAD_CTL_PULL_DOWN_100K (0xc << 4) > > +#define PAD_CTL_PULL_UP_47K (0xd << 4) > > +#define PAD_CTL_PULL_UP_100K (0xe << 4) > > +#define PAD_CTL_PULL_UP_22K (0xf << 4) > > + > > +#define PAD_CTL_OUTPUT_CMOS (0) > > +#define PAD_CTL_OUTPUT_OPEN_DRAIN (1 << 3) > > + > > +#define PAD_CTL_DRIVE_STRENGTH_NORM (0) > > +#define PAD_CTL_DRIVE_STRENGTH_HIGH (1 << 1) > > +#define PAD_CTL_DRIVE_STRENGTH_MAX (2 << 1) > > > > #define PAD_CTL_SLEW_RATE_SLOW 0 > > #define PAD_CTL_SLEW_RATE_FAST 1 > > -- > Pengutronix e.K. | Juergen Beisert | > Linux Solutions for Science and Industry | Phone: +49-8766-939 228 | > Vertretung Sued/Muenchen, Germany | Fax: +49-5121-206917-5555 | > Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de/ | > > _______________________________________________ > barebox mailing list > barebox@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/barebox > -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox