From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Fri, 20 Aug 2010 16:39:18 +0200 Subject: [U-Boot] [PATCH] mx51evk: Provide a proper reset for the Ethernet PHY In-Reply-To: <46828.21571.qm@web51007.mail.re2.yahoo.com> References: <46828.21571.qm@web51007.mail.re2.yahoo.com> Message-ID: <4C6E9396.3040203@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 Fabio Estevam wrote: > Hi Stefano, > Hi Fabio, >> udelay(200); >> mxc_gpio_set(46); This must be, >> >> What do you think ? > > Yes, I agree. Do you plan to send a patch for mx51evk using this approach? > > If you do, then I can re-submit my patch after your update. I sent today a patch adding a common way to access gpios for i.MX31 and i.MX51: http://article.gmane.org/gmane.comp.boot-loaders.u-boot/83163 I have another issue. It seems to me that the lines you want to add are already in the code, only some lines later: 263 /* Enable VGEN3, VCAM, VAUDIO, VVIDEO, VSD regulators */ 264 val = VGEN3EN | VGEN3CONFIG | VCAMEN | VCAMCONFIG | 265 VVIDEOEN | VAUDIOEN | VSDEN; 266 pmic_reg_write(REG_MODE_1, val); 267 268 udelay(500); 269 270 reg = readl(GPIO2_BASE_ADDR + 0x0); 271 reg |= 0x4000; 272 writel(reg, GPIO2_BASE_ADDR + 0x0); This is exactly what you want to add, only after setting the regulators. Why do we need to set the gpio twice ? 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 =====================================================================