After an update to the MX51 reference manual (Rev. 5), the
values of the PAD_CTL_DRV_VOT_LOW and PAD_CTL_DRV_VOT_HIGH
are now clearly wrong:

"Bit 13:
High / Low Output Voltage Range. This bit selects the output voltage mode for
SD2_CMD. 0 High output voltage mode
1 Low output voltage mode"

The values are currently negated in code - fixed.

Reported-by: David Jander <david.jan...@protonic.nl>
Signed-off-by: Stefano Babic <sba...@denx.de>
CC: Marek Vasut <marek.va...@gmail.com>
CC: David Jander <david.jan...@protonic.nl>
---
 arch/arm/include/asm/arch-mx5/iomux.h |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/include/asm/arch-mx5/iomux.h 
b/arch/arm/include/asm/arch-mx5/iomux.h
index 760371b..e3765a3 100644
--- a/arch/arm/include/asm/arch-mx5/iomux.h
+++ b/arch/arm/include/asm/arch-mx5/iomux.h
@@ -66,8 +66,8 @@ typedef enum iomux_pad_config {
        PAD_CTL_HYS_ENABLE = 0x1 << 8,  /* Hysteresis enabled */
        PAD_CTL_DDR_INPUT_CMOS = 0x0 << 9,/* DDR input CMOS */
        PAD_CTL_DDR_INPUT_DDR = 0x1 << 9,/* DDR input DDR */
-       PAD_CTL_DRV_VOT_LOW = 0x0 << 13, /* Low voltage mode */
-       PAD_CTL_DRV_VOT_HIGH = 0x1 << 13,/* High voltage mode */
+       PAD_CTL_DRV_VOT_LOW = 0x1 << 13, /* Low voltage mode */
+       PAD_CTL_DRV_VOT_HIGH = 0x0 << 13,/* High voltage mode */
 } iomux_pad_config_t;
 
 /* various IOMUX input functions */
-- 
1.7.9.5

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to