From mboxrd@z Thu Jan 1 00:00:00 1970 From: Amit Virdi Date: Tue, 27 Mar 2012 11:29:53 +0530 Subject: [U-Boot] [PATCH 08/25] SPEAr: Add configuration options for spear3xx and spear6xx boards In-Reply-To: <201203261430.48891.sr@denx.de> References: <1331121854-20494-1-git-send-email-amit.virdi@st.com> <201203071454.08691.sr@denx.de> <4F705C9C.1010606@st.com> <201203261430.48891.sr@denx.de> Message-ID: <4F715759.9070305@st.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Stefan, >>> >>> Can't you just remove this config.mk file completely? >>> CONFIG_SYS_TEXT_BASE can be moved to the config header instead. What >>> else is this file needed for? >> >> Could you please explain which config header you are referring to here? >> Do you mean include/configs/spear..... ? > > Yes. > >> For SPEAr310 and SPEAr320, config.mk is needed to make image (by >> specifying FLASH=PNOR during compile time) that can access CFI compliant >> PNOR flash. > > In the example above (spear300), only CONFIG_SYS_TEXT_BASE is left in > config.mk. This can be moved to the config header (include/configs/spear-xxx- > h). > > You should be able to move the other remaining options (PNOR...) to > boards.cfg. Or is this not possible? > It is possible and it adds to the clarity, thanks for this nice suggestion. I'll remove all the config.mk files from the board/spear/spearxx/ directories and add targets in boards.cfg for making u-boot build with environment variables placed in PNOR flash. Best Regards Amit Virdi