From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Mon, 24 Oct 2011 09:56:50 +0200 Subject: [U-Boot] [PATCH 3/3] qong: remove unneeded IOMUX settings In-Reply-To: <1319162491-2293-3-git-send-email-festevam@gmail.com> References: <1319162491-2293-1-git-send-email-festevam@gmail.com> <1319162491-2293-3-git-send-email-festevam@gmail.com> Message-ID: <4EA51A42.8090202@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 10/21/2011 04:01 AM, Fabio Estevam wrote: > On qong board some of the USBH2 pins are set via GPR register, so don need to setup > the IOMUX for each pin individually. > > Other than that, these pins should not be configured as primary function because the primary > function selects SSI functionality. > > Let GPR register do the work and remove the unneeded IOMUX setup. > > Signed-off-by: Fabio Estevam > --- > Hi Fabio, > Stefano, > > I don't have access to the qong board to test this, but I believe this is the right thing to do here. I have not written this code and I do not know the history, but reading the manual I agree with you. The setup of the iomux is ininfluent, because the pins are already set in "Hardware mode 2" in the GPR register (GPR[11]). > > board/davedenx/qong/qong.c | 6 ------ > 1 files changed, 0 insertions(+), 6 deletions(-) > > diff --git a/board/davedenx/qong/qong.c b/board/davedenx/qong/qong.c > index de32fb5..70af593 100644 > --- a/board/davedenx/qong/qong.c > +++ b/board/davedenx/qong/qong.c > @@ -120,12 +120,6 @@ int board_early_init_f(void) > mx31_gpio_mux(IOMUX_MODE(MUX_CTL_USBH2_STP, MUX_CTL_FUNC)); > mx31_gpio_mux(IOMUX_MODE(MUX_CTL_USBH2_DATA0, MUX_CTL_FUNC)); > mx31_gpio_mux(IOMUX_MODE(MUX_CTL_USBH2_DATA1, MUX_CTL_FUNC)); > - mx31_gpio_mux(IOMUX_MODE(MUX_CTL_STXD3, MUX_CTL_FUNC)); > - mx31_gpio_mux(IOMUX_MODE(MUX_CTL_SRXD3, MUX_CTL_FUNC)); > - mx31_gpio_mux(IOMUX_MODE(MUX_CTL_SCK3, MUX_CTL_FUNC)); > - mx31_gpio_mux(IOMUX_MODE(MUX_CTL_SFS3, MUX_CTL_FUNC)); > - mx31_gpio_mux(IOMUX_MODE(MUX_CTL_STXD6, MUX_CTL_FUNC)); > - mx31_gpio_mux(IOMUX_MODE(MUX_CTL_SRXD6, MUX_CTL_FUNC)); > > #define H2_PAD_CFG (PAD_CTL_DRV_MAX | PAD_CTL_SRE_FAST | PAD_CTL_HYS_CMOS | \ > PAD_CTL_ODE_CMOS | PAD_CTL_100K_PU) Acked-by: Stefano Babic 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-0 Fax: +49-8142-66989-80 Email: office at denx.de =====================================================================