Re: [hatari-devel] DSP Bootstrap ROM

[Eero Tamminen <oak@xxxxxxxxxxxxxx>]

Hi,

On 23.9.2021 7.49, Andreas Grabher wrote:
> > Am 22.09.2021 um 22:11 schrieb Eero Tamminen <oak@xxxxxxxxxxxxxx>:
> > 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)

Are there any names for these modes in the doc
that could be used as define names?

> and mode_wait is for the timed bootstrap ROM enable/disable.
Enable/disable variables could be bools and set
with true&false.


	- Eero

> > 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>