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

Added bending to MIDI #2740

Merged
merged 1 commit into from
Apr 19, 2018
Merged

Added bending to MIDI #2740

merged 1 commit into from
Apr 19, 2018

Conversation

adiron
Copy link
Contributor

@adiron adiron commented Apr 13, 2018

This is done via MI_BENDD and MI_BENDU. At the moment the value is
hardcoded and cannot be adjusted (future commit?) and is the max for the
midi_send_pitchbend function (up or down).

MI_BENDD and MI_BENDU both require #define MIDI_ADVANCED

MIDI pitch bend was already implemented in protocol/midi.c, I merely
added the keycodes to trigger them. :) (thanks to Jack, two years ago
in commit fb4fe52 apparently)

This is done via MI_BENDD and MI_BENDU. At the moment the value is
hardcoded and cannot be adjusted (future commit?) and is the max for the
`midi_send_pitchbend` function (up or down).

`MI_BENDD` and `MI_BENDU` both require `#define MIDI_ADVANCED`

MIDI pitch bend was already implemented in `protocol/midi.c`, I merely
added the keycodes to trigger them. :) (thanks to Jack, two years ago
in commit fb4fe52 apparently)
@jackhumbert
Copy link
Member

Nice! Thanks :)

@jackhumbert jackhumbert merged commit 3d3c093 into qmk:master Apr 19, 2018
carlpehrson pushed a commit to carlpehrson/qmk_firmware that referenced this pull request May 7, 2018
This is done via MI_BENDD and MI_BENDU. At the moment the value is
hardcoded and cannot be adjusted (future commit?) and is the max for the
`midi_send_pitchbend` function (up or down).

`MI_BENDD` and `MI_BENDU` both require `#define MIDI_ADVANCED`

MIDI pitch bend was already implemented in `protocol/midi.c`, I merely
added the keycodes to trigger them. :) (thanks to Jack, two years ago
in commit fb4fe52 apparently)
hauleth pushed a commit to hauleth/qmk_firmware that referenced this pull request Jan 24, 2019
This is done via MI_BENDD and MI_BENDU. At the moment the value is
hardcoded and cannot be adjusted (future commit?) and is the max for the
`midi_send_pitchbend` function (up or down).

`MI_BENDD` and `MI_BENDU` both require `#define MIDI_ADVANCED`

MIDI pitch bend was already implemented in `protocol/midi.c`, I merely
added the keycodes to trigger them. :) (thanks to Jack, two years ago
in commit fb4fe52 apparently)
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.

2 participants