From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Wed, 6 Feb 2013 11:47:31 -0500 Subject: [U-Boot] am33xx config_ddr api change needed to support mDDR In-Reply-To: References: Message-ID: <51128923.4000403@ti.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 02/05/2013 03:48 AM, Markus Brunner wrote: > Hi, for mDDR to work mddr_sel (bit 28) of ddrctrl->ddrioctrl has to > be set. There is no way of telling config_ddr(unsigned int pll, > unsigned int ioctrl, ...) my desired value, it won't be set atm. > > From the 1.3kB ddr_ctrl struct only 2 registers are defined, which > is fine if it's used memory mapped, however it makes it unsuitable > for passing values in the SPL. So at first glance I would extend > config_ddr() with an additional register value, because guessing > the ddr type from the values of the other parameters is, in > addition to being hackish, not guaranteed to work with every weired > ram configuration. However might it be possible, that the second > value of config_ddr() should not hold the value for > config_io_ctrl(), but the value of my desired ddrctrl->ddrioctrl? > The wiki [1] from TI says that the first should _always_ be 0x18B, > whereas the second needs to be set depending on the mem type. We > are talking about a set of registers named DDR_*_IOCTRL and a > single register called DDR_IO_CTRL, so easy to mix up. > > Ragards, Markus > > [1] > http://processors.wiki.ti.com/index.php/AM335x_EMIF_Configuration_tips Hadn't > seen that wiki page before, thanks. So a problem right now is that yes, I hadn't seen "in the wild" anything but DDR2 or DDR3 designs. I'm going to assume that on your custom hardware you are not doing the EEPROM-based board detection method that the TI boards do. We should try and lay things out so that boards where we can run-time say "we are on board X (rev Y) and must have ..." can all out the same functions for boards where we simply have to set that at build time (until we get to the some-day of using a device tree or something like that perhaps). - -- Tom -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQIcBAEBAgAGBQJREokiAAoJENk4IS6UOR1Wd90P/2hgjjH1e9E749n2PJLQhIrt r3So+ylQd+q4hzLieuwqRmi2sEtm6af+A5GPH6xoGSNiSPO+ps3mJAAaWZ04CxqC W/ZKPB3mgDkwGCIyqRAJd4EuMxKqwyTkv+NGoElRvyN3G/OAunMBq2cTMAo6s6St zo2/j8d2pbHlKQByteBN/3QWgALQ/6emqKSqhD6O5b7htinp5c8lPiRBeHNBAaRa aeTorHIA8cuSzzXrmZE7lZUlSRQxe0CyApzcXfHfdZ2QD8fAA0REAxqD3r4XR4wp 5iobESR3wl+7FEXU0hp6lI/PjCCckKbubH3s8eVwxNGRyP0BOPvVDWyVTChXaHfe jmNG10b60CLLGr4tnJSpimT26UIeE/uxCUumG8BkWQQIyPl9SYUObOlzqXSQmMIh KquiF2m9zvPNEMSLdhpKqpdt4zO55bE46PSd13FFOhiyBgpIYP/+Jk4I3RgmCg4p pXiFnK83MRhwzSeRZd9klPsN2gd8pmnLZee1EbpL4/LpOFhFGMfQl8a/BEJLiCoj GDawN4OPWUQWX6pYuhWtKpRFUiPiOWi5/EvNkfW4JYUrP+8/ycGgk14p3sUkvPIE 9kjZoZ1W2XMSqnJA8jz4oPBNPqluNadabjX7N9OB52jYSApU2qJxa9RJlD3zGrxy Daq4me3pz1dNDVUmtRVl =u++P -----END PGP SIGNATURE-----