Menu

#338 Broken trdos emulation

v1.2
closed-accepted
beta128 (1)
5
2016-04-24
2015-07-28
ub880d
No

I got a report from my friend, that trdos emulation is broken when it is used custom rom (for beta 128 peripheral) TR-DOS_5_05.rom (czech modification http://cygnus.speccy.cz/download/romky/trdos505.zip ). Friend said he is using this rom on real HW without problems.

Problem is, that command to access disk (e.g. CAT from trdos prompt) when diskette icon is not active (==motor off?) will return 'Disk error!' message. When diskette icon is active (green) then command will succeed (but not if it is the first command after enter to trdos prompt, which seems to be problem of displaying icon, which is green even after a minute of inactivity, maybe just wrong initial state of icon.. this patch is not fixing this icon problem).

So, to repeat this bug:

1) run fuse with command: fuse --beta128 --rom-beta128=TR-DOS_5_05.rom -m128 disk.trd
2) enter 128k editor
3) usr15616
4) CAT
5) wait for grey diskette icon
6) CAT
(Disk error! message is printed)
7) while icon is green enter another CAT
(successfull catalogue output)

I'm not a betadisk/trdos user, and also I don't know how exactly WD works, so I just used trial and test method to identify that problem appeared in revision 5232.

For the same reason, please, review the patch and change/extend it as needed.

1 Attachments

Discussion

  • ub880d

    ub880d - 2015-08-11

    This problem is present also on trdos 5.03 http://cygnus.speccy.cz/download/romky/trdos503.zip

    so maybe description of ticket should be changed.

     
  • Sergio Baldoví

    Sergio Baldoví - 2016-04-24
    • labels: --> beta128
    • summary: Broken trdos emulation when using TR-DOS_5_05.rom --> Broken trdos emulation
    • status: open --> closed-accepted
    • assigned_to: Sergio Baldoví
    • Group: future --> v1.2
     
  • Sergio Baldoví

    Sergio Baldoví - 2016-04-24

    Oops! This patch also spots the problem reported in [bugs:#331] but unexplainably we missed it. This error was not present on release 1.1.1.

    A slightly variation was committed in [r5382]. Sorry and good work!

     

    Related

    Bugs: #331
    Commit: [r5382]


Log in to post a comment.