From mboxrd@z Thu Jan  1 00:00:00 1970
From: Vikram Narayanan <vikram186@gmail.com>
Date: Mon, 01 Apr 2013 22:56:23 +0530
Subject: [U-Boot] [PATCH] Add initial support for mx6slevk board
In-Reply-To: <1364677696-26770-1-git-send-email-festevam@gmail.com>
References: <1364677696-26770-1-git-send-email-festevam@gmail.com>
Message-ID: <5159C33F.8090003@gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 3/31/2013 2:38 AM, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
<snip>
> diff --git a/arch/arm/include/asm/arch-mx6/mx6sl_pins.h b/arch/arm/include/asm/arch-mx6/mx6sl_pins.h
> new file mode 100644
> index 0000000..cd974d8
> --- /dev/null
> +++ b/arch/arm/include/asm/arch-mx6/mx6sl_pins.h
> @@ -0,0 +1,1374 @@
> +/*
> + * Copyright (C) 2013 Freescale Semiconductor, Inc. All Rights Reserved.
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.
> + */
> +
> +#ifndef __ASM_ARCH_MX6_MX6SL_PINS_H__
> +#define __ASM_ARCH_MX6_MX6SL_PINS_H__
> +
> +#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)
> +

Is the above same for all the variants of i.Mx?
Can we put this into a common header file?

> +#define IOMUX_CONFIG_SION 0x10
> +#define NO_MUX_I                0
> +#define NO_PAD_I                0
> +enum {
> +	MX6_PAD_AUD_MCLK__AUDMUX_AUDIO_CLK_OUT			= IOMUX_PAD(0x02A4, 0x004C, 0, 0x0000, 0, 0),
> +	MX6_PAD_AUD_MCLK__PWM4_PWMO				= IOMUX_PAD(0x02A4, 0x004C, 1, 0x0000, 0, 0),
> +	MX6_PAD_AUD_MCLK__ECSPI3_RDY				= IOMUX_PAD(0x02A4, 0x004C, 2, 0x06B4, 0, 0),
> +	MX6_PAD_AUD_MCLK__FEC_MDC				= IOMUX_PAD(0x02A4, 0x004C, 3, 0x0000, 0, 0),

Is the mx6slevk using all the pins defined here?
If not please take off the unused IOMUX pins so that we don't add dead code.

Regards,
Vikram