Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gldl/yml2hdl v0.3 #24

Open
wants to merge 30 commits into
base: develop
Choose a base branch
from
Open

Gldl/yml2hdl v0.3 #24

wants to merge 30 commits into from

Conversation

taquionbcn
Copy link
Collaborator

Hi @andrewpeck @dgastler ,

In this branch are the changes I made on the regmap python scripts to make it compatible with yml2hdl v0.3.5 + gldl_patches.
It works on the l0mdt project, I maintained back-compatibility with old versions of yml2hdl and compatibility with non yml2hdl projects.

The last element I can only say because the vhdl generated makes sense, but I don't have a project for test.

I'm open to chat if you have any question.

cheers,
Guillermo

@andrewpeck
Copy link
Contributor

Hi @taquionbcn

Sorry I was away on vacation

Could you update this branch with the latest changes from devel with either a merge or rebase?

Github is allowing a merge right now since merge commits are allowed but your branch is several commits behind.

I added some CI stage to the software recently but it is not in your branch yet

After merging in the develop changes you will need to re-run the test xmls

cd tester && make

Then please review and commit the updated generated files

Thanks!

@andrewpeck
Copy link
Contributor

andrewpeck commented Apr 21, 2022

use work.{{baseName}}_Ctrl.all;
use work.{{baseName}}_Ctrl_pkg.all;

Is this change correct?

I don't think the generated packages are named _Ctrl_pkg, just _Ctrl

package MEM_TEST_CTRL is

now both template_map_with_bram and template_map are in the test suite
@andrewpeck
Copy link
Contributor

Closes #19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants