search

MEGA65 / mega65-core

MEGA65 FPGA core
Other
240 stars 85 forks source link

PAL/NTSC default setting in config menu is being ignored #200

Closed deftmega closed 4 years ago

deftmega commented 4 years ago

In latest unstable bitstreams (sorry for no further info at this point) I have no functionality by the PAL/NTSC toggle for default video mode in config menu. However, switching via "V" key in freeze menu has been tested working.

adtbm commented 4 years ago

The Bug is still in. If you set, in MEGA65 configure (Bios) video mode to PAL and save it, MEGA65 will boot anyway, always to NTSC.

The video mode change in the freeze-menu works.

gardners commented 4 years ago

The problem is that mega65 configure is modifying sector 1 of the SD card, while the hypervisor reads from sector 1 of the system partition. Both use the hypervisor routine to do the reading and writing, so something funny is going on with that, I presume.

gardners commented 4 years ago

ok. Found problem: Util menu check happens before SYSPART can be opened. Thus syspart offset is sector $0, and the config data thus ends up in sector 0+1 of the SD card. This is actually not a bad place for it, and the easier fix is probably to just make it officially live there.

gardners commented 4 years ago

Confirmed fixed by the above commit.

It's on the 138-hdmi-audio-27mhz branch, and will get merged in when that branch gets merged in.

adtbm commented 4 years ago

Tested Bitstream: mega65r2-20200523.13-HEAD_n-0476b74+DIRTY.bit

changing in configure menu from PAL to NTSC or vice versa and saving it works now. i don't knoiw how to test issue #220

bluewaysw commented 4 years ago

Combined with #220, successfully tested and merged into stable_base branch: https://github.com/MEGA65/mega65-core/commit/0476b7475b6ddc84380ca12ec10a8fb7fea04174