On Mon, 2013-11-11 at 11:37 -0800, York Sun wrote: > On 11/11/2013 11:35 AM, Scott Wood wrote: > > On Mon, 2013-11-11 at 11:05 -0800, York Sun wrote: > >> On 11/11/2013 10:49 AM, Scott Wood wrote: > >>> My point is that the type of DDR that is present is hardware > >>> description, just as much as the type of DDR controller. "User config" > >>> means things users can choose purely in software, without a > >>> corresponding hardware change. > >>> > >> > >> This is a board-level configuration, not SoC level. Should we use > >> CONFIG_SYS_ as well? > > > > Yes. > > > >> If so, we need to come up with a new name, such as > >> CONFIG_SYS_FSL_DDR_MEM_GEN3. > > > > How about CONFIG_SYS_FSL_DDR3 for the memory, and > > CONFIG_SYS_FSL_DDR_CTRL3 for the controller? > > > > I am reluctant to change existing macro to different meaning. How about > CONFIG_SYS_FSL_DDR3_MEM, or CONFIG_SYS_FSL_MEM_DDR3?
CONFIG_SYS_FSL_DDR3 is not an existing macro. -Scott _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
