From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thierry Reding Date: Wed, 20 Aug 2014 08:38:41 +0200 Subject: [U-Boot] [PATCH 19/23] ARM: tegra: Enable PCIe on Beaver In-Reply-To: References: <1408346196-30419-1-git-send-email-thierry.reding@gmail.com> <1408346196-30419-20-git-send-email-thierry.reding@gmail.com> Message-ID: <20140820063840.GG13793@ulmo> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Tue, Aug 19, 2014 at 01:48:55PM +0000, Marcel Ziswiler wrote: > > + /* TPS659110: LDO2_REG = 1.05V, ACTIVE */ > > + data[0] = 0x15; > > + addr = 0x31; > > + > > + err = i2c_read(PMU_I2C_ADDRESS, addr, 1, data, 1); > > You sure about that read? > > Me wondering why it did not work on my shiny Apalis T30. > > > + if (err) { > > + debug("failed to set AVDD supply\n"); > > + return err; > > + } > > Yeah, that one just did not wana set anything (;-p). Hehe... this should probably be i2c_write() instead. Perhaps this is on by default on Beaver and Cardhu but not on the particular revision that Apalis T30 uses? Can you test whether things start working if you turn that into a write instead? Thierry -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 819 bytes Desc: not available URL: