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

e-regs.h inconsistent #13

Open
greytery opened this issue May 7, 2014 · 3 comments
Open

e-regs.h inconsistent #13

greytery opened this issue May 7, 2014 · 3 comments
Assignees

Comments

@greytery
Copy link

greytery commented May 7, 2014

Similar to issue #12
e-lib/include/e-regs.h also has a full enum list of register names, but these have not been updated/are inconsistent with the epiphany-hal-data.h version.

Should only be one version of the reg enums and names.
Is this an unnecessary/dangerous duplication?

@jeremybennett
Copy link
Contributor

I think there is similar data in e-server, and that in turn must be consistent with GDB, which in turn needs to be able to map the register numbering onto GCC/binutils DWARF representations...

Thanks for flagging this up. I'm marking it as an enhancement rather than a bug (since stuff is not actually broken at present). We need to do some design work to get this as clean as possible.

@greytery
Copy link
Author

greytery commented May 7, 2014

I had a look around and could not see e-libs using the reg names - so not actually 'broken' as you say.
As part of the sweeping up, some of the enum names are '...CFG' and not '...CONFIG' as per the ref manual. Easier to change the manual :-)

@greytery
Copy link
Author

Since this has caused further problems (see forum topic http://forums.parallella.org/viewtopic.php?f=23&t=1522#p9400) I now believe that this should be treated as a bug rather than an enhancement.

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

No branches or pull requests

3 participants