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

[Bug] hid_bootloader_cli is missing after "qmk setup" #18134

Closed
osamuaoki opened this issue Aug 22, 2022 · 3 comments
Closed

[Bug] hid_bootloader_cli is missing after "qmk setup" #18134

osamuaoki opened this issue Aug 22, 2022 · 3 comments

Comments

@osamuaoki
Copy link
Contributor

"qmk setup" doesn't support hid_bootloader_cli any more.

  • installed executable name has been changed but Makefile still call hid_bootloader_cli
  • new installed executable doesn't accept options for at90usb1286 (Lufa version still work)
  • This is probably regression due to the remote code "qmk setup" fetches.

Describe the Bug

New fresh install of qmk doesn't create hid_bootloader_cli in /usr/local/bin. I see bootloaderHID there now.

Moreover, it is not just name change. I tried it by crating symlink to make Makefile to call bootloaderHID. No luck. It complains my use of at90usb1286 (This should be accepted)

$ make VERBOSE=1 --jobs=1 cgc56:pico:flash
...
Checking file size of cgc56_pico.hex                                                                [OK]
 * The firmware size is fine - 30026/126976 (23%, 96950 bytes free)
true && 	hid_bootloader_cli -mmcu=at90usb1286 -w -v .build/cgc56_pico.hex
error opening -mmcu=at90usb1286: No such file or directory
make[1]: *** [platforms/avr/flash.mk:185: flash] Error 1
if [ -f .build/error_occurred ]; then printf "\033[31;01mMake finished with errors\n\033[0m" & exit 1; fi;
Make finished with errors
make: *** [Makefile:414: cgc56:pico:flash] Error 1
Checking file size of cgc56_pico.hex                                                                [OK]
 * The firmware size is fine - 30026/126976 (23%, 96950 bytes free)
true && 	hid_bootloader_cli -mmcu=at90usb1286 -w -v .build/cgc56_pico.hex
error opening -mmcu=at90usb1286: No such file or directory
make[1]: *** [platforms/avr/flash.mk:185: flash] Error 1
if [ -f .build/error_occurred ]; then printf "\033[31;01mMake finished with errors\n\033[0m" & exit 1; fi;
Make finished with errors
make: *** [Makefile:414: cgc56:pico:flash] Error 1

I can compile standard Lufa source in submodule to hid_bootloader_cli and it is usable with qmk if I installed it.

So some regression happened qmk version of HID bootloader host program. It is not just file name change problem.

System Information

Keyboad is not published one but essentially teensy2++ based board.
I am on Debian

@zvecr
Copy link
Member

zvecr commented Aug 22, 2022

hid_bootloader_cli has never been universally installed as part of the toolchain setup. However it is potentially installed under msys2. bootloaderHID is a completely different tool, the assumption that something has been renamed is incorrect. This would not be classified as a regression.

@sigprof
Copy link
Contributor

sigprof commented Aug 22, 2022

Keyboad is not published one but essentially teensy2++ based board.

Then you should use handwired/onekey/teensy_2pp as an example; rules.mk for that board is:

# MCU name
MCU = at90usb1286

# Bootloader selection
BOOTLOADER = halfkay

And platforms/avr/flash.mk would use either teensy-loader-cli or teensy_loader_cli (depending on which is found in $PATH) for the halfkay bootloader, not hid_bootloader_cli (although hid_bootloader_cli is a fork of teensy_loader_cli and supports those Teensy boards too). The availability of prebuilt teensy_loader_cli binaries/packages may be better than hid_bootloader_cli, so if you are using a real Teensy++ 2.0 board, you should set BOOTLOADER = halfkay.

hid_bootloader_cli would be used for BOOTLOADER = qmk-hid (if that bootloader is used, teensy_loader_cli won't work, because lufa-hid and qmk-hid use a different VID:PID pair).

@osamuaoki
Copy link
Contributor Author

osamuaoki commented Aug 24, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants