qmk_firmware icon indicating copy to clipboard operation
qmk_firmware copied to clipboard

[Keyboard] Add darkproject/kd83a_bfg_edition

Open gskygithub opened this issue 3 years ago • 14 comments

Description

Add darkproject/kd83a_bfg_edition keyboard.

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

gskygithub avatar Jun 07 '22 01:06 gskygithub

Are any of the files in the "src" folder actually needed, or are they just copy-pasted from another board/project?

drashna avatar Jun 10 '22 05:06 drashna

Are any of the files in the "src" folder actually needed, or are they just copy-pasted from another board/project?

Files in "src" folder are necessary because those files are related to flash, without those files we can not save any function in keyboard.

gskygithub avatar Jun 10 '22 05:06 gskygithub

Maybe, but it looks like the wb32_spi_master.c is a duplicate of platforms/chibios/drivers/spi_master.c, with the non-wb32 contents removed. the flash_spi.c looks to be byte for byte identical to drivers/flash/flash_spi.c

Adding these files separately introduce redundant files.

Any and all duplicate files like this should be removed.

The only unique file there appears to be the eeprom_flash.c file.

drashna avatar Jun 10 '22 06:06 drashna

Tested, only eeprom_flash.c file is saving function in keyboard.

gskygithub avatar Jun 10 '22 07:06 gskygithub

Maybe, but it looks like the wb32_spi_master.c is a duplicate of platforms/chibios/drivers/spi_master.c, with the non-wb32 contents removed. the flash_spi.c looks to be byte for byte identical to drivers/flash/flash_spi.c

Adding these files separately introduce redundant files.

Any and all duplicate files like this should be removed.

The only unique file there appears to be the eeprom_flash.c file.

Changes are done. Could you please refer to it. If anything else are needed. @drashna

gskygithub avatar Jun 13 '22 07:06 gskygithub

Is there anything else I could fix @fauxpark @zvecr ? Would be glad to help.

gskygithub avatar Jun 29 '22 01:06 gskygithub

@gksygithub gksygithub requested a review from tzarc

Items are still unaddressed.

tzarc avatar Jul 01 '22 06:07 tzarc

@gksygithub gksygithub requested a review from tzarc

Items are still unaddressed.

May I know what are those.

gskygithub avatar Jul 01 '22 06:07 gskygithub

@gksygithub gksygithub requested a review from tzarc

Items are still unaddressed.

May I know what are those.

@tzarc Could you please help me out.

gskygithub avatar Jul 01 '22 09:07 gskygithub

@fauxpark Could you please review the resolved changes and let me know If there is anything need to change. Thanks

gskygithub avatar Jul 05 '22 03:07 gskygithub

Sorry for tagging you again, @fauxpark @tzarc @zvecr. I would be really glad to help if there is something to resolve this pull request.

gskygithub avatar Jul 06 '22 05:07 gskygithub

@gksygithub gksygithub requested a review from tzarc

Items are still unaddressed.

Resolved, Could you please review. @tzarc

gskygithub avatar Jul 08 '22 07:07 gskygithub

Can anyone review this pull request please.

gskygithub avatar Jul 19 '22 01:07 gskygithub

Could someone help to review the changes please.

gskygithub avatar Jul 26 '22 10:07 gskygithub

Can someone please take a look at this pull request, Is there anything else I could fix? Would be glad to help.

gskygithub avatar Aug 03 '22 01:08 gskygithub