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

Initial port of TMK's ibmpc_usb converter code #12821

Closed
wants to merge 74 commits into from

Conversation

marfrit
Copy link
Contributor

@marfrit marfrit commented May 6, 2021

Description

I ported the ibmpc_usb converter from TMK.
hasu is combining the ps2_usb, terminal_usb and xt_usb converters in ibmpc_usb - and so should qmk.
Not entirely sure about the location of ibmpc.c and ibmpc.h, since they reside in /tmk_core/protocol/ in TMK.

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

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.
  • 0I 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).

keyboards/converter/ibm_terminal/keymaps/default/rules.mk Outdated Show resolved Hide resolved
Comment on lines +19 to +20
#ifndef CONFIG_H
#define CONFIG_H
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
#ifndef CONFIG_H
#define CONFIG_H
#pragma once

#define DEVICE_VER 0x0100
#define MANUFACTURER QMK
#define PRODUCT PS/2 Keyboard
#define DESCRIPTION convert IBM PC keyboard to USB
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
#define DESCRIPTION convert IBM PC keyboard to USB


#define IBMPC_INT_VECT INT1_vect

#endif
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
#endif

marfrit and others added 4 commits May 27, 2021 07:37
Integrate suggested changes

Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
keyboards/40percentclub/foobar/keymaps/qfoobar/config.h Outdated Show resolved Hide resolved
keyboards/40percentclub/foobar/keymaps/qfoobar/rules.mk Outdated Show resolved Hide resolved
keyboards/converter/ibm_terminal/keymaps/default/rules.mk Outdated Show resolved Hide resolved
keyboards/mitosis/keymaps/qmitosis/config.h Outdated Show resolved Hide resolved
keyboards/converter/usb_usb/keymaps/marhasu/config.h Outdated Show resolved Hide resolved
keyboards/lily58/keymaps/marfritlily/rules.mk Outdated Show resolved Hide resolved
keyboards/converter/usb_usb/keymaps/marhasu/rules.mk Outdated Show resolved Hide resolved
keyboards/converter/ibmpc_usb/rules.mk Outdated Show resolved Hide resolved
keyboards/converter/ibmpc_usb/keymaps/marfrit/rules.mk Outdated Show resolved Hide resolved
keyboards/converter/ibmpc_usb/keymaps/default/rules.mk Outdated Show resolved Hide resolved
marfrit and others added 9 commits July 23, 2021 10:01
Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
Copy link
Member

@fauxpark fauxpark left a comment

Choose a reason for hiding this comment

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

This should really be written in C.

@marfrit
Copy link
Contributor Author

marfrit commented Aug 2, 2021

This should really be written in C.

I removed the C++ stuff for the time being.

@marfrit marfrit closed this Aug 2, 2021
mangkoran referenced this pull request in marfrit/qmk_firmware Feb 23, 2022
@Muirium
Copy link

Muirium commented Aug 18, 2022

Closed? Aw. QMK really needs this.

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.

4 participants