From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michal Simek Date: Fri, 28 Feb 2014 11:17:37 +0100 Subject: [U-Boot] [PATCH v6] socfpga: Adding Scan Manager driver In-Reply-To: <1393517034-9889-1-git-send-email-clsee@altera.com> References: <1393517034-9889-1-git-send-email-clsee@altera.com> Message-ID: <53106241.8010203@monstr.eu> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 02/27/2014 05:03 PM, Chin Liang See wrote: > Scan Manager driver will be called to configure the IOCSR > scan chain. This configuration will setup the IO buffer settings > > Signed-off-by: Chin Liang See > Cc: Dinh Nguyen > Cc: Wolfgang Denk > CC: Pavel Machek > Cc: Tom Rini > Cc: Albert Aribaud > --- > Changes for v6 > - Fixed various coding style issue > Changes for v5 > - Removal of additional blank line > - Added comment for magic number > Changes for v4 > - avoid code duplication by add goto error > - include underscore to variables name > Changes for v3 > - merge the handoff file and driver into single patch > Changes for v2 > - rebase with latest v2014.01-rc1 > --- > arch/arm/cpu/armv7/socfpga/Makefile | 2 +- > arch/arm/cpu/armv7/socfpga/scan_manager.c | 211 +++++++ > arch/arm/cpu/armv7/socfpga/spl.c | 4 + > arch/arm/include/asm/arch-socfpga/scan_manager.h | 96 +++ > .../include/asm/arch-socfpga/socfpga_base_addrs.h | 1 + > board/altera/socfpga/iocsr_config.c | 657 ++++++++++++++++++++ > board/altera/socfpga/iocsr_config.h | 17 + I still have problem with content of these two files. In iocsr_config.c is ~600 lines which targets just one specific hardware design configuration. I can't see any reason why this should go to mainline and stay there. Because it brings no value. I would recommend you just to define that arrays like this const unsigned long iocsr_scan_chain0_table[]; const unsigned long iocsr_scan_chain0_table[]; ... + in header #define CONFIG_HPS_IOCSR_SCANCHAIN0_LENGTH 0 #define CONFIG_HPS_IOCSR_SCANCHAIN1_LENGTH 0 #define CONFIG_HPS_IOCSR_SCANCHAIN2_LENGTH 0 #define CONFIG_HPS_IOCSR_SCANCHAIN3_LENGTH 0 and write these 2 files by hand. Then your users will just replace them by hand for specific board/design. Thanks, Michal -- Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91 w: www.monstr.eu p: +42-0-721842854 Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/ Maintainer of Linux kernel - Xilinx Zynq ARM architecture Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 263 bytes Desc: OpenPGP digital signature URL: