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

[Keyboard] Fix function references IDB 60 #10810

Merged
merged 3 commits into from
Oct 31, 2020
Merged

Conversation

drashna
Copy link
Member

@drashna drashna commented Oct 31, 2020

Description

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout/userspace (addition or update)
  • Documentation

Issues Fixed or Closed by This PR

  • compiler issue on gcc 8+

Checklist

  • My code follows the code style of this project: C, Python
  • I have read the PR Checklist document and have made the appropriate changes.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

@drashna drashna added the bug label Oct 31, 2020
@drashna drashna requested a review from a team October 31, 2020 05:29
keyboards/idb/idb_60/idb_60.h Outdated Show resolved Hide resolved
@drashna drashna requested a review from a team October 31, 2020 11:39
@drashna
Copy link
Member Author

drashna commented Oct 31, 2020

@itspngu Just a heads up on the change

Copy link
Contributor

@itspngu itspngu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey! Thanks, I actually meant to look into the reports of this not compiling anymore, but never got around to it. Since you're already on this, apparently these got mixed up somewhere down the line, so the LEDs are driven in inverse (they're on when they're meant to be off and vice versa).

keyboards/idb/idb_60/idb_60.c Outdated Show resolved Hide resolved
keyboards/idb/idb_60/idb_60.c Outdated Show resolved Hide resolved
keyboards/idb/idb_60/idb_60.c Outdated Show resolved Hide resolved
keyboards/idb/idb_60/idb_60.c Outdated Show resolved Hide resolved
@drashna
Copy link
Member Author

drashna commented Oct 31, 2020

Not a problem, at all! Mostly wanted to make sure it compiled, but more than willing to add some other fixes, too!

@itspngu
Copy link
Contributor

itspngu commented Oct 31, 2020

Not a problem, at all! Mostly wanted to make sure it compiled, but more than willing to add some other fixes, too!

Awesome, thanks!

@fauxpark fauxpark merged commit a3f1def into qmk:master Oct 31, 2020
@drashna drashna deleted the fix/isb_60 branch October 31, 2020 12:20
morrijr pushed a commit to morrijr/qmk_firmware that referenced this pull request Nov 1, 2020
oscarcarlsson pushed a commit to oscarcarlsson/qmk_firmware that referenced this pull request Nov 2, 2020
ringmaster pushed a commit to ringmaster/qmk_firmware that referenced this pull request Nov 5, 2020
tacahiroy pushed a commit to tacahiroy/qmk_firmware that referenced this pull request Nov 8, 2020
ea2809 pushed a commit to ea2809/qmk_firmware that referenced this pull request Jan 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants