From mboxrd@z Thu Jan 1 00:00:00 1970 From: Aneesh V Date: Wed, 18 May 2011 09:19:49 +0530 Subject: [U-Boot] [PATCH v2 16/22] omap4: calculate EMIF register values In-Reply-To: <20110517215425.7F07C1491B0A@gemini.denx.de> References: <1298893591-17636-1-git-send-email-aneesh@ti.com> <1305472900-4004-17-git-send-email-aneesh@ti.com> <20110515200523.A1A4C1491B06@gemini.denx.de> <4DD2858E.2000703@ti.com> <20110517215425.7F07C1491B0A@gemini.denx.de> Message-ID: <4DD341DD.3020706@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 Hi Wolfgang, On Wednesday 18 May 2011 03:24 AM, Wolfgang Denk wrote: > Dear Aneesh V, > > In message<4DD2858E.2000703@ti.com> you wrote: >> >> I had the same concerns too. So, I have provided a CONFIG option - >> CONFIG_SYS_EMIF_PRECALCULATED_TIMING_REGS - by which the user can >> provide the register values, so no computation will be done. > > Ah, great. Sorry, I have missed this. > >> So, if somebody is concerned about the performance they can just >> compute the values using this driver and print them out by enabling the >> debug traces, fill it up in the structs provided for it and enable >> CONFIG_SYS_EMIF_PRECALCULATED_TIMING_REGS. That should work. > > Will this also comment out the respective code, so we don't waste the > memory for it? In SPL yes, because I am using -ffunction-sections -fdata-sections along with --gc-sections, but not in U-Boot. I shall take care of this in the next revision. BTW, I was thinking that I should enable -ffunction-sections -fdata-sections in OMAP U-Boot too. What do you think? best regards, Aneesh