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

mos_fclose not usable if VDP in terminal mode #69

Closed
nihirash opened this issue Jun 5, 2023 · 10 comments · Fixed by #70
Closed

mos_fclose not usable if VDP in terminal mode #69

nihirash opened this issue Jun 5, 2023 · 10 comments · Fixed by #70
Labels
duplicate This issue or pull request already exists enhancement New feature or request

Comments

@nihirash
Copy link
Contributor

nihirash commented Jun 5, 2023

It tries fetch RTC data but have no possibility.

Please add flag to ignore RTC data - cause my CP/M implementation working in terminal mode and there no RTC data.

@nihirash
Copy link
Contributor Author

nihirash commented Jun 5, 2023

I've digged a bit and there's issue with rtc_update routine - it waits forever and my system is freezed cause there no packets from VDP with RTC data.

If it's possible - please make some flag for disabling RTC update routine.

@leighbb
Copy link
Contributor

leighbb commented Jun 12, 2023

I don't the history around some of the design choices. Having said that, would it not be better to switch to using the ez80 RTC for this stuff?

Ignore me - it seems there is no clock source for ez80 RTC.

@nihirash
Copy link
Contributor Author

eZ80 have no RTC buildin.

We can attach some external via I2C but honestly I think it isn't really required for Agon(even from ESP).

If this routines can be disabled - freezes won't happens.

@leighbb
Copy link
Contributor

leighbb commented Jun 13, 2023

I think there is still room for a "raw" Terminal approach, but I think it is a bit limiting. I would prefer an approach that allows you to use user defined terminal sequences to achieve the same functionality when operating in Terminal mode. There may still be a part to play for a completely "raw" terminal mode, but I think an approach that still allows you to use the full MOS functionality would be better.

I'll share a rough outline of the approach when I get a chance. I have it more or less working, but need to tidy it up.

@nihirash
Copy link
Contributor Author

This patch mainly required for CP/M. In MOS mode there no requirements for this.

@sijnstra
Copy link

Not just required to CP/M mode. I've converted my infocom interpreter to use terminal mode to have access to 80 columns and underline in the font, as well as other VT100 functionality. Now I can't save games as that required the datestamp for file creation.

@nihirash
Copy link
Contributor Author

As adhoc solution - you can grab MOS binary from cp/m chat in discord - it's vanilla MOS with my simple patch.

@sijnstra
Copy link

I ended up compiling from your fork - I have to justify buying that zilog cable somehow! Anyway, confirmed that it's working and solved my issue with creating new files while in terminal mode. This patch allowed me to confidently release my infocom interpreter, however, I am absolutely happy to change my code if a different agreed method for getting around this issue turns up.

@nihirash
Copy link
Contributor Author

You can flash it just via https://github.com/envenomator/agon-flash

If something will go wrong - you can flash Agon HAL firmware for ESP with support uploading flash via USB and when computer starts successfully rollback to Quark

@breakintoprogram breakintoprogram moved this to In Progress in AGON Quark Firmware Jul 8, 2023
@breakintoprogram
Copy link
Owner

Closing as 'Won't Do' as this is a duplicate of #70.

@github-project-automation github-project-automation bot moved this from In Progress to Done in AGON Quark Firmware Jul 8, 2023
@breakintoprogram breakintoprogram moved this from Done to Won't Do in AGON Quark Firmware Jul 8, 2023
@breakintoprogram breakintoprogram added the duplicate This issue or pull request already exists label Jul 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists enhancement New feature or request
Projects
Status: Won't Do
Development

Successfully merging a pull request may close this issue.

4 participants