All of lore.kernel.org
 help / color / mirror / Atom feed
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
=====================================================================

  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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.