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

WOZ: All even disk i/o read accesses will read the data latch #672

Closed
tomcw opened this issue Jul 30, 2019 · 2 comments
Closed

WOZ: All even disk i/o read accesses will read the data latch #672

tomcw opened this issue Jul 30, 2019 · 2 comments
Labels
Milestone

Comments

@tomcw
Copy link
Contributor

tomcw commented Jul 30, 2019

From #662, @peterferrie said:

At $8D2E, LDA $C08E,X is returning the wrong nibble (#$AD instead of #$FF, and that #$FF should be about to be dropped).

OK, I'm only updating the latch for $C0EC. So this #$AD is from the previous $C0EC latch read, ie. when it read D5AAAD. But, yes, strictly speaking all latch reads (ie. $C0En even addresses) should update the latch. I'll fix that.

@tomcw tomcw added the bug label Jul 30, 2019
tomcw added a commit that referenced this issue Jul 30, 2019
. Even disk i/o read accesses will update the data latch
. All disk i/o write accesses will update the data latch
@Archange427
Copy link

Not strictly related to this WOZ reading problem, but I remember that some groups of the french underground cracking scene in the 80s loved to use "exotic" addresses for their reading routine (like $C0E0 instead of $C0EC).
When I was doing disk preservation, I had to patch a lot of these 'fresh' .DSK to work on AppleWin.
Could you also fix this issue for DSK ?
I can send you some DSK for tests if you want!

@tomcw
Copy link
Contributor Author

tomcw commented Sep 7, 2019

I converted some of your .dsk's to .woz, and was able to test this - so I'll close this one.
And the specific fix for .dsk/nib is being tracked in another issue.

@tomcw tomcw closed this as completed Sep 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants