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

qbus device: DVK "MX" floppy controller #11840

Merged
merged 4 commits into from Feb 8, 2024
Merged

Conversation

shattered
Copy link
Contributor

No description provided.

Copy link
Member

@cuavas cuavas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I’ve mostly noticed formatting issues.

This is floppy stuff which I’m not the most qualified to review. @galibert can you review the floppy controller implementation in src/devices/bus/qbus/dvk_mx.cpp?

src/lib/formats/dvk_mx_dsk.h Outdated Show resolved Hide resolved
src/lib/formats/dvk_mx_dsk.h Outdated Show resolved Hide resolved
src/devices/bus/qbus/qbus.cpp Outdated Show resolved Hide resolved
src/devices/bus/qbus/dvk_mx.h Outdated Show resolved Hide resolved
src/devices/bus/qbus/dvk_mx.h Outdated Show resolved Hide resolved
src/devices/bus/qbus/dvk_mx.cpp Outdated Show resolved Hide resolved
src/devices/bus/qbus/dvk_mx.cpp Outdated Show resolved Hide resolved
@rb6502
Copy link
Contributor

rb6502 commented Jan 1, 2024

@shattered Are you going to clean up these issues?

flopi[i].id = i;
name[0] = '0' + i;
name[1] = 0;
floppy_connector *con = subdevice<floppy_connector>(name);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please go for a required_device_array, it's rather nicer.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure how to use it here, tbh.

Copy link
Member

@cuavas cuavas Jan 22, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You have a member like:

	required_device_array<floppy_connector, 4> m_connectors;

Initialise it like:

	m_connectors(*this, "%u", 0U)

And then you can just do like m_connectors[i]->whatever() and use it in place of literal tag string in the machine configuration function as well.


uint16_t read(offs_t offset);
void write(offs_t offset, uint16_t data);

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like that fdc can't interrupt the main cpu. If it's the case there is a somewhat better structure than predict/commit (like the wd/upd/amiga do), it's catchup on access (like diskii/iwm/swim1-2). But it's good anyway.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have another floppy code in the works, will look into that.

@cuavas cuavas merged commit b992130 into mamedev:master Feb 8, 2024
5 checks passed
Mokona pushed a commit to Mokona/mame that referenced this pull request Feb 28, 2024
stonedDiscord pushed a commit to stonedDiscord/mame that referenced this pull request Apr 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants