| Re: [hatari-devel] DSP Bootstrap ROM |
[ Thread Index |
Date Index
| More lists.tuxfamily.org/hatari-devel Archives
]
- To: hatari-devel@xxxxxxxxxxxxxxxxxxx
- Subject: Re: [hatari-devel] DSP Bootstrap ROM
- From: Andreas Grabher <andreas_g86@xxxxxxxxxx>
- Date: Thu, 23 Sep 2021 06:49:24 +0200
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=icloud.com; s=1a1hai; t=1632372568; bh=G8fmAbHRkx+5USE6WS+/AdB2ZlFvZxwHrycitf8Q94U=; h=From:Content-Type:Mime-Version:Date:Subject:Message-Id:To; b=Oj4e+ADTqB//xl139W8NUQTT6V/2n0DDzl+xC9zwNQtohdTbTgC62kDXI1/H3bMKF DBHf3XsPJDsT31GqyAbRtPAHd81lNBNKXBEoCFUKU+NKk2LMQ4X+hcUKbQSlBadlOD s3VjMqQvQi6HLNVAoRugwPb8QgJbjbL8xyX4QD8IO5rXFMLKL6338tpSMvmlK/ujha WdVw6/zkmKecnewoz84oNIvsncBIAFzNcZEqf5Xv6mbGXANFpYi0LIurSmg3ABVNV/ 5M460jut1uAQO5Q6BL37aWrhytBrw8KWNw/6NPFad7wBVSRdG0IEZNBq9YJxsJP+4S QvxsLCDh5ptLQ==
> Am 22.09.2021 um 22:11 schrieb Eero Tamminen <oak@xxxxxxxxxxxxxx>:
>
> Hi,
>
> Few comments...
>
> * Code calls Statusbar_SetDspLed(), but that's missing
Sorry, that is an oversight. It is specific to Previous and indicates that the DSP is active.
>
> * Add defines for magic (mode / mode_wait) constants
I do not understand. Mode is a number between 0 and 3 (you find it in the datasheet of the 56001) and mode_wait is for the timed bootstrap ROM enable/disable.
>
> Why BOOTSTRAP_ROM ifdef is needed i.e. what downside it has?
At least with Previous the extension causes some loss of efficiency because the waiting loop (waiting for program to be supplied by the host) is permanently running.
>
>
> - Eero
>
>> On 22.9.2021 22.43, Andreas Grabher wrote:
>> Appended is an improved version of the patch. This works for me in all cases.
>>> Am 20.09.2021 um 21:20 schrieb Andreas Grabher <andreas_g86@xxxxxxxxxx>:
>>>
>>> Hello all,
>>>
>>> I am considering to implement DSP bootstrap ROM into Previous (NeXT Computer emulator) to make it more realistic. I already have a patch (appended). It seems to have a minor bug, maybe some oversight. But it already works for the most part. Are you interested in this for Hatari?
>>>
>>> Best wishes,
>>>
>>> Andreas
>>>
>>>
>>> <dsp_patch.diff>
>
>