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

Cleanup dongle code #25

Open
ataffanel opened this issue Mar 18, 2016 · 0 comments
Open

Cleanup dongle code #25

ataffanel opened this issue Mar 18, 2016 · 0 comments
Labels
bug enhancement prio:high

Comments

@ataffanel
Copy link
Member

Looking at the SDCC compiler manual, http://sdcc.sourceforge.net/doc/sdccman.pdf, we do a couple of bad things in the current implementation of the Crazyradio.

The main offender is the interrupt handling: it is apparently not advised to call function in the interrupt handlers but in Crazyradio the USB interrupt handling is complex and calling a lot of function.

Further work on the Crazyradio (like #22 #24 #3 #7 #4) are kind of blocked by that.

If anyone is interested about helping out, lets use this ticket to sync work.
ataffanel added a commit that referenced this issue Jul 20, 2017
@ataffanel
#25: Update gitignore
f50e00a
ataffanel added a commit that referenced this issue Jul 20, 2017
@ataffanel
#25: Move sdcc to small memory model
c2ff8e7 
Should be more efficient and produces smaller code
ataffanel added a commit that referenced this issue Jul 20, 2017
@ataffanel
#25: Initialize xdata mem with double pointer
f0f6a0f
ataffanel added a commit that referenced this issue Jul 20, 2017
@ataffanel
#25: Remove isr reg bank and add nooverlay
1c7ea0b 
 - Interrupt register bank does not help unless all function
   called by the interrupt are in the same bank, hard to
   achieve for USB interrupt
 - Add no overlay for all functions (and a bit more) called
   by the USB interrupt
ataffanel added a commit that referenced this issue Jul 20, 2017
@ataffanel
#25: Explicit signed char to compile in SDCC 3.6
9dbe293
@ataffanel ataffanel mentioned this issue Jul 26, 2017
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug enhancement prio:high
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ataffanel