MEGA65 / mega65-core

MEGA65 FPGA core
Other
240 stars 85 forks source link

HYPPO-DOS: refactor d81attach / d81detach calls #628

Closed lydon42 closed 4 hours ago

lydon42 commented 1 year ago

There is currently a systematic problem with the image attach/detach functions.

Currently we have a d81attach0 and d81attach1 function call, to attach an image to either drive 0 or 1. Then there is a d81detach which will detach both images at once. Both calls will update the internal process descriptor to reflect the image currently attached.

There is no call to just detach one image, which in my eyes is missing. Because of that BASIC MOUNT will just use pokes to unmount the image on one drive, which is then not reflected in the display of the FREEZER. In the same way it is not actually detaching the image, but it is attaching the internal drive. For this there is also no call. And so there is also no update to process descriptor.

The proposal is to:

Continuing from #620 (split because bug and enhancement are separate).

EDIT: This can now be found in branch 760-hyppo-dos13 (see #760)

lydon42 commented 8 months ago

removed from batch 3 milestone, this should go into the after batch 3 hyppo dos branch, which also implements writing and other stuff. Is not needed for the factory core, can easily go into standard core.

lydon42 commented 23 hours ago

This has now changed to become ONE call dos_attach which will handle all attaching and detaching.

lydon42 commented 4 hours ago

The multi purpose dos_attach is working well with BASIC and a modified Midnight Mega. It can be currently found in 760-hyppo-dos13 branch. Closing this, integration with the rest will happen in 760.