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.

Sign up for GitHub

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

Jump to bottom

[reggen] Add regwen for each pad/pin once supported by reggen #307

Closed
msfschaffner opened this issue Oct 1, 2019 · 6 comments
Closed

[reggen] Add regwen for each pad/pin once supported by reggen #307

msfschaffner opened this issue Oct 1, 2019 · 6 comments
Assignees
@tjaychen @eunchan @msfschaffner
Labels
Component:RTL Component:Tooling Issues related to tooling, e.g. tools/scripts for doc, code generation (docgen, reggen), CSR Priority:P1 Priority: high Type:Task Tasks, to-do list.

Comments

@msfschaffner
Copy link
Contributor

The reggen currently does not properly support REGWEN masks for multireg, hence this feature is deferred until supported by reggen.
@msfschaffner msfschaffner added Component:Tooling Issues related to tooling, e.g. tools/scripts for doc, code generation (docgen, reggen), CSR Priority:P3 Priority: low Type:Task Tasks, to-do list. Component:RTL labels Oct 1, 2019
@msfschaffner msfschaffner self-assigned this Oct 1, 2019
@msfschaffner msfschaffner mentioned this issue Oct 1, 2019
Merged
@tjaychen
Copy link

tjaychen commented Oct 1, 2019

thanks!

@eunchan eunchan self-assigned this Oct 2, 2019
@eunchan
Copy link
Contributor

eunchan commented Oct 2, 2019

YATTD to me :)

@msfschaffner
Copy link
Contributor Author

Unfortunately so, yes :). Plus we should consider adding vectored resvals to multiregs.

@msfschaffner
Copy link
Contributor Author

Affected IPs (add more if you come across some that should leverage this functionality):

  • flash_ctrl
  • pinmux
  • padctrl

@tjaychen
Copy link

tjaychen commented Oct 2, 2019

@eunchan
where you think it's reasonable, i think it's perfectly fine if you want to try and break up the various features as different issues so people like me or michael (or the community at large) can take it on.

it might even make sense if you want to stub out certain definition / classes for others to implement.

@msfschaffner msfschaffner changed the title [pinmux/padctrl] Add regwen for each pad/pin once supported by reggen [reggen] Add regwen for each pad/pin once supported by reggen Oct 2, 2019
@sjgitty sjgitty added Priority:P1 Priority: high and removed Priority:P3 Priority: low labels Jul 21, 2020
@msfschaffner msfschaffner mentioned this issue Feb 11, 2021
Closed
17 tasks
@tjaychen
Copy link

This is separately tracked and also already implemented.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tjaychen tjaychen

@eunchan eunchan

@msfschaffner msfschaffner

Labels
Component:RTL Component:Tooling Issues related to tooling, e.g. tools/scripts for doc, code generation (docgen, reggen), CSR Priority:P1 Priority: high Type:Task Tasks, to-do list.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@tjaychen @eunchan @sjgitty @msfschaffner