CGB Support - Githubissues

deltabeard / Peanut-GB

A Game Boy (DMG) emulator single header library written in C99. Performance is prioritised over accuracy.

https://projects.deltabeard.com/peanutgb/

283 stars 40 forks source link

CGB Support #50

Open froggestspirit opened 2 years ago

froggestspirit commented 2 years ago

I added CGB support here: https://github.com/froggestspirit/Peanut-GB

I only really tested this with Pokemon Crystal, but seemed to work fine. I wasn't sure if you might want to merge it, or copy parts of it over, I know there's some formatting differences, and also the way I handled sram.

I should also note, this might only be working in the SDL2 example for now, since some changes to the C file were necessary there for rendering the palettes.

deltabeard commented 2 years ago

Thank you for your work! 😃

I would be happy if you could make a pull request, and then I can start reviewing the code. At a glance:

the formatting of code you haven't modified shouldn't be altered. So maybe we could cherry-pick 04cef564d25bcdac3471c8a6dc690b733102b154 ?
the CGB code should be within ifdefs to allow developers to disable it if their target environment has very little RAM and processing power.
~uint8_t should only be used for 8-bit integers and not for bit fields that aren't 8-bits.~ Using uint8_t for bitfields was correct.
group the CGB if statements on lines 3646 to 3652 together.
why is num_rom_banks_mask reverted to the previous version? Those three ROM size masks aren't ever used apparently.

victorbnl commented 2 years ago

The repository has been archived, I guess the author has no intention to make these changes nor to make it a pull request anymore

deltabeard commented 2 years ago

I plan on adding this feature in v2 as this will require an API change. I'll plan and put more thought into this at a later date.

kevinjones120 commented 4 months ago

can I use a st7789vw 2 inch display or can you make support for it please.

deltabeard commented 4 months ago

can I use a st7789vw 2 inch display or can you make support for it please.

That's not relevant to this issue. You can probably use that display but you have to add support for it yourself. This repository is just for the emulation core. The display driver has to be written by yourself.