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

lwriter.cpp: Get video output working #9049

Merged
merged 1 commit into from
Jan 1, 2022
Merged

Conversation

jrmuizel
Copy link
Contributor

No description provided.

@Tafoid
Copy link
Contributor

Tafoid commented Jan 3, 2022

All machines in lwriter.cpp:
Snapshot size changed dramatically in my tests.
Previous resolution was 1285 x 240 and now they are 9457 x 3434 in size.

@jrmuizel
Copy link
Contributor Author

jrmuizel commented Jan 3, 2022

That's expected

@jrmuizel
Copy link
Contributor Author

jrmuizel commented Jan 3, 2022

Is it possible to provide serial input as part of these tests? It would be nice to confirm that printing the test page works

@jrmuizel
Copy link
Contributor Author

jrmuizel commented Jan 3, 2022

Or actually even just testing with the dipswitches set to particular values?

@rb6502
Copy link
Contributor

rb6502 commented Jan 3, 2022

@jrmuizel The way this would normally be done is to move the LaserWriter main emulation and ROM loading etc. into a device in src/devices/machine/ and then have the existing driver instantiate that device and the terminal. Then you'd also add something to src/devices/bus/rs232/ that instantiated the LaserWriter device and connects it to incoming serial, which would allow it to talk to e.g. the Macintosh and Apple IIgs emulations.

Alternatively you could just move it wholesale to src/devices/bus/rs232 and use one of the emulated terminals (e.g. the VT-102) as a host for debugging. src/devices/bus/centronics/epson_lx810l.cpp provides an example of a working parallel dot matrix printer.

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.

3 participants