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

ProDOS HDD support #7

Merged
merged 4 commits into from Mar 15, 2021
Merged

Conversation

steven-a-wilson
Copy link
Contributor

Addresses #5

@sorgelig sorgelig merged commit f1f5c27 into MiSTer-devel:master Mar 15, 2021
@alanswx
Copy link
Member

alanswx commented Feb 28, 2022

steven- I am not sure how to contact you, but I would love to discuss a few apple II things. Also, in keeping with this patch, I am noticing that it will read the same block from the HPS over and over - possibly because the hdd doesn't check the PHASE_ZERO clock - althought I tried changing that, and it still seemed to re-read the same block over and over. I am available at alan.steremberg @ gmail

@steven-a-wilson
Copy link
Contributor Author

I suspect the underlying problem is that the code that asserts hdd_read and hdd_write in hdd.vhd is only conditional on the slot select signal currently being asserted, as opposed to being conditional on an edge. I think checking PHASE_ZERO doesn't work due to the CPU being halted during the transfer (causing PHASE_ZERO to be asserted multiple times for the same CPU bus state). I'm not sure how to confirm the behavior you're describing with my current setup (my desk has been progressively rearranged in favor of work-related stuff over the past year), but I at least briefly tried a change to only assert the read/write signals on the rising edge, and the HDD support still seems to be operational, so that might be worth trying on your end if you can easily verify it.

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

Successfully merging this pull request may close these issues.

None yet

3 participants