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

DSP Registers are not saved in Context Switch #3

Open
Daimonion1980 opened this issue Mar 3, 2017 · 2 comments
Open

DSP Registers are not saved in Context Switch #3

Daimonion1980 opened this issue Mar 3, 2017 · 2 comments

Comments

@Daimonion1980
Copy link

Hey

As the dsPIC33 has the same Architecture except the additional dsp register and as you wrote your kernel is useable for dspic devices why are you not saving the dsp Registers during a context switch?

It should be inserted in File tneo/src/arch/pic24_dspic/tn_arch_pic24.S and during stack initialisation.

So that an working dsp operation with Registers ACCA*/ACCB*/DO is context save.

@dimonomid
Copy link
Owner

dimonomid commented Mar 5, 2017

Hi, thanks, that's a good point; I recall that I planned to make sure the code is correct for dsPICs, but apparently I did not get around to implement that properly.

At the moment I don't have any dsPIC device around though, so would it be possible for you to craft a PR? I'd be happy to accept it.

@Daimonion1980
Copy link
Author

I do not implement the whole kernel. At the moment i just use the context switch code from your dspic arch file to solve a special problem of my actual project.

But when i'm ready, the code is tested and microchip had a look through it i will provide you a PR.

Just give me some days....

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

No branches or pull requests

2 participants