[U-Boot] [PATCH v3] socfpga: Adding Scan Manager driver
Pavel Machek
pavel at denx.de
Fri Feb 21 16:07:46 CET 2014
Hi!
> + /*
> + * Check if the scan chain engine is inactive and the
> + * WFIFO is empty before enabling the IO scan chain
> + */
> + if (SCAN_MGR_IO_SCAN_ENGINE_STATUS_IDLE
> + != scan_mgr_io_scan_chain_engine_is_idle(
> + MAX_WAITING_DELAY_IO_SCAN_ENGINE)) {
> + return 1;
> + }
Hmm.. function named "_is_idle" maybe should just return 0 or 1?
> + /*
> + * Check if the scan chain engine has completed the
> + * IO scan chain data shifting
> + */
> + if (SCAN_MGR_IO_SCAN_ENGINE_STATUS_IDLE
> + != scan_mgr_io_scan_chain_engine_is_idle(
> + MAX_WAITING_DELAY_IO_SCAN_ENGINE)) {
> + /* Disable IO Scan chain when error detected */
> + clrbits_le32(&scan_manager_base->en,
> + 1 << io_scan_chain_id);
> + return 1;
> + }
> + }
"goto error" would help avoid code duplication.
> +struct socfpga_scan_manager {
> + u32 stat;
> + u32 en;
> + u32 padding[2];
> + u32 fifosinglebyte;
> + u32 fifodoublebyte;
> + u32 fifoquadbyte;
> +};
some underscores should be added here.
Thanks,
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
More information about the U-Boot
mailing list