xf::remap() function optimization (URAM utilization)#5
Open
alekrop wants to merge 81 commits into
Open
Conversation
…in c-sim and co-sim. II=1.
Updated Remap, Minmax, StereoBM, MST, Delay and Dense-non-pyramidal o…
…ter on the board.
…thesis of URAM reading (checked via board implementation).
… and become incompatible with kernel)
…lication with kernel.
…oh rates is done.
…OLL for cleanup loop with fully partioned arrays.
…ples to call top level function of Gaussian filter from xfopencv library
added 3 commits
May 15, 2018 02:21
# Conflicts: # .gitignore # include/imgproc/xf_remap.hpp
…s bm_state member as separate integers. To reduce number of kernel parameter only "flexible" parameter (which are not calculating from template) are passing. The XOCC flow can't work with structures/class as kernel parameter. Unfortunately if try to pass structure as array of simple type (char/int/etc) then impossible copy element of this array to members of the kernel local instantiation of structure - compilation passing, sw emulation passing, but hw emulation and FPGA run fail due to XOCC issue.
…tion and FPGA (kernel suspend)
…ly done (no co-sim and other checks).
…sted fix of possible padding problem.
…etics) are rolled-back.
… template parameters are removed.
…nction is roughly done (no co-sim and further checks).
…in auMedianBlur() function by restructuring loops.
… in auMedianBlur() to exclude deadlocks in co-sim.
…kariound to exclude X values in co-sim.
…e xFPyrDownGaussianBlur() to decrease its pipeline depth.
…PyrDownGaussianBlur() function.
… every partition equal to URAM to overcome cascading timing issue.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
URAM usage is enabled by template parameter USE_URAM, which is false by default. Functionality is bit-exact with BRAM implementaion: checks pass C-sim, RTL co-sim and zcu104_rv board runs for NN and Bi-linear filtering modes on fractional coordinates.