From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?utf-8?Q?Beno=C3=AEt_Th=C3=A9baudeau?= Date: Thu, 25 Apr 2013 22:32:50 +0200 (CEST) Subject: [U-Boot] [PATCH v3 1/3] iomux-v3: Place pad control definitions into common file In-Reply-To: <517991A0.5070307@denx.de> References: <5165be73.ea80ec0a.04b3.ffffb7fd@mx.google.com> <517991A0.5070307@denx.de> Message-ID: <39456465.257907.1366921970551.JavaMail.root@advansee.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Stefano, On Thursday, April 25, 2013 10:27:12 PM, Stefano Babic wrote: > On 10/04/2013 21:32, festevam at gmail.com wrote: > > From: Fabio Estevam > > > > Instead of having the same PAD control definition in each MX6 variant pin > > file, > > place it into a common location. > > > > Signed-off-by: Fabio Estevam > > --- > > Hi Fabio, > > > Changes since v2: > > - None > > Changes since v1: > > - Add missing IOMUX_CONFIG_SION definition to avoid breakage on mx6dl > > > > arch/arm/include/asm/arch-mx6/mx6dl_pins.h | 27 > > --------------------------- > > arch/arm/include/asm/arch-mx6/mx6q_pins.h | 27 > > --------------------------- > > arch/arm/include/asm/imx-common/iomux-v3.h | 27 > > ++++++++++++++++++++++++++- > > 3 files changed, 26 insertions(+), 55 deletions(-) > > > > diff --git a/arch/arm/include/asm/arch-mx6/mx6dl_pins.h > > b/arch/arm/include/asm/arch-mx6/mx6dl_pins.h > > index 9846f1b..0ed12f3 100644 > > --- a/arch/arm/include/asm/arch-mx6/mx6dl_pins.h > > +++ b/arch/arm/include/asm/arch-mx6/mx6dl_pins.h > > @@ -22,33 +22,6 @@ > > > > #include > > > > -/* Use to set PAD control */ > > -#define PAD_CTL_HYS (1 << 16) > > -#define PAD_CTL_PUS_100K_DOWN (0 << 14) > > -#define PAD_CTL_PUS_47K_UP (1 << 14) > > -#define PAD_CTL_PUS_100K_UP (2 << 14) > > -#define PAD_CTL_PUS_22K_UP (3 << 14) > > - > > -#define PAD_CTL_PUE (1 << 13) > > -#define PAD_CTL_PKE (1 << 12) > > -#define PAD_CTL_ODE (1 << 11) > > -#define PAD_CTL_SPEED_LOW (1 << 6) > > -#define PAD_CTL_SPEED_MED (2 << 6) > > -#define PAD_CTL_SPEED_HIGH (3 << 6) > > -#define PAD_CTL_DSE_DISABLE (0 << 3) > > -#define PAD_CTL_DSE_240ohm (1 << 3) > > -#define PAD_CTL_DSE_120ohm (2 << 3) > > -#define PAD_CTL_DSE_80ohm (3 << 3) > > -#define PAD_CTL_DSE_60ohm (4 << 3) > > -#define PAD_CTL_DSE_48ohm (5 << 3) > > -#define PAD_CTL_DSE_40ohm (6 << 3) > > -#define PAD_CTL_DSE_34ohm (7 << 3) > > -#define PAD_CTL_SRE_FAST (1 << 0) > > -#define PAD_CTL_SRE_SLOW (0 << 0) > > - > > I have applied this patch and I have found now that this breaks some > MX51 boards, such as efika. In fact, the same defines are set also in > /home/stefano/Projects/imx/u-boot-imx/include/asm/arch/iomux-mx51.h. > > Checking in the manual, I see that values in iomux-mx51.h are correct, > and they simply differs from the ones of MX6. We can simply add a #ifdef > CONFIG_MX51 in arch/arm/include/asm/imx-common/iomux-v3.h to fix it, but > anyway I suppose that the mx51evk does not work correctly. Can you take > a look at this issue ? I have already made a patch for that among other things. I will post it tomorrow. Best regards, Beno?t