From: Stefano Babic <sbabic@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 3/8] imx: Easy enabling of SION per-pin using MUX_MODE_SION helper macro
Date: Thu, 19 Dec 2013 11:40:39 +0100 [thread overview]
Message-ID: <52B2CD27.7040003@denx.de> (raw)
In-Reply-To: <1387233845-4372-4-git-send-email-otavio@ossystems.com.br>
Hi Otavio,
On 16/12/2013 23:44, Otavio Salvador wrote:
> The macro allows easy setting in per-pin, as for example:
>
> ,----
> | imx_iomux_v3_setup_pad(MX6_PAD_NANDF_D1__GPIO_2_1 | MUX_MODE_SION);
> `----
>
> The IOMUX_CONFIG_SION allows for reading PAD value from PSR register.
>
> The following quote from the datasheet:
>
> ,----
> | ...
> | 28.4.2.2 GPIO Write Mode
> | The programming sequence for driving output signals should be as follows:
> | 1. Configure IOMUX to select GPIO mode (Via IOMUXC), also enable SION if need
> | to read loopback pad value through PSR
> | 2. Configure GPIO direction register to output (GPIO_GDIR[GDIR] set to 1b).
> | 3. Write value to data register (GPIO_DR).
> | ...
> `----
>
> This fixes the gpio_get_value to properly work when a GPIO is set for
> output and has no conflicts.
>
> Thanks for Beno?t Th?baudeau <benoit.thebaudeau@advansee.com>, Fabio
> Estevam <fabio.estevam@freescale.com> and Eric B?nard
> <eric@eukrea.com> for helping to properly trace this down.
>
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> ---
> Changes in v3: None
> Changes in v2: None
>
> arch/arm/include/asm/imx-common/iomux-v3.h | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm/include/asm/imx-common/iomux-v3.h b/arch/arm/include/asm/imx-common/iomux-v3.h
> index dc2b3ef..dec11a1 100644
> --- a/arch/arm/include/asm/imx-common/iomux-v3.h
> +++ b/arch/arm/include/asm/imx-common/iomux-v3.h
> @@ -63,6 +63,8 @@ typedef u64 iomux_v3_cfg_t;
> #define MUX_SEL_INPUT_SHIFT 59
> #define MUX_SEL_INPUT_MASK ((iomux_v3_cfg_t)0xf << MUX_SEL_INPUT_SHIFT)
>
> +#define MUX_MODE_SION ((iomux_v3_cfg_t)IOMUX_CONFIG_SION << \
> + MUX_MODE_SHIFT)
> #define MUX_PAD_CTRL(x) ((iomux_v3_cfg_t)(x) << MUX_PAD_CTRL_SHIFT)
>
> #define IOMUX_PAD(pad_ctrl_ofs, mux_ctrl_ofs, mux_mode, sel_input_ofs, \
>
Acked-by: Stefano Babic <sbabic@denx.de>
Best regards,
Stefano Babic
--
=====================================================================
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: sbabic at denx.de
=====================================================================
next prev parent reply other threads:[~2013-12-19 10:40 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-12-16 22:43 [U-Boot] [PATCH v3 0/8] Pending patches for merging Otavio Salvador
2013-12-16 22:43 ` [U-Boot] [PATCH v3 1/8] wandboard: add Future Eletronics 7" WVGA LCD extension board Otavio Salvador
2013-12-19 10:36 ` Stefano Babic
2014-01-03 17:47 ` Otavio Salvador
2013-12-16 22:43 ` [U-Boot] [PATCH v3 2/8] mx6sabresd: Add eMMC specific environment to allow U-Boot update Otavio Salvador
2014-01-03 14:30 ` Stefano Babic
2014-01-03 14:48 ` Stefano Babic
2013-12-16 22:44 ` [U-Boot] [PATCH v3 3/8] imx: Easy enabling of SION per-pin using MUX_MODE_SION helper macro Otavio Salvador
2013-12-19 10:40 ` Stefano Babic [this message]
2014-01-03 14:30 ` Stefano Babic
2013-12-16 22:44 ` [U-Boot] [PATCH v3 4/8] mx28evk: Use 512k for fdt partition to align it Otavio Salvador
2013-12-19 10:41 ` Stefano Babic
2014-01-03 14:33 ` Stefano Babic
2013-12-16 22:44 ` [U-Boot] [PATCH v3 5/8] mx28evk: Add 'nandboot' environment command Otavio Salvador
2013-12-19 10:45 ` Stefano Babic
2013-12-19 11:02 ` Fabio Estevam
2014-01-03 14:33 ` Stefano Babic
2013-12-16 22:44 ` [U-Boot] [PATCH v3 6/8] mx28evk: Extend environment to easy write of NAND system Otavio Salvador
2014-01-03 14:34 ` Stefano Babic
2013-12-16 22:44 ` [U-Boot] [PATCH v3 7/8] ARM: mx6: Change the FDT loading address to avoid overlaping Otavio Salvador
2013-12-19 10:47 ` Stefano Babic
2014-01-03 14:34 ` Stefano Babic
2013-12-16 22:44 ` [U-Boot] [PATCH v3 8/8] ARM: mx6: Allow enablement of FEC Anatop based clock for all MX6 Otavio Salvador
2013-12-19 10:50 ` Stefano Babic
2014-01-03 14:35 ` Stefano Babic
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=52B2CD27.7040003@denx.de \
--to=sbabic@denx.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox