On 10/04/2013 21:32, feste...@gmail.com wrote: > From: Fabio Estevam <fabio.este...@freescale.com> > > 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 <fabio.este...@freescale.com> > ---
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 <asm/imx-common/iomux-v3.h> > > -/* 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 ? Best regards, Stefano -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de ===================================================================== _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot