Build Software Error

[ericwake]

Need some help with building this project. Using 'make kernel=menu', I receive the following error for the file kernel_MODplay.cpp:

error: 'class CHDMISoundBaseDevice' has no member named 'ConvertSample'

I am attempting to build this on a Windows machine using:
circle 44.3
mingw64
arm gnu toolchain aarch64-none-elf 11.2 2022.02

I have been able to successfully build and run demo projects for circle 44.3 on a Raspberry Pi 3B+ without issue.

Since I am new at this, I have probably missed something. Any help will be greatly appreciated.

UPDATE: As a heavy-handed experiment, I commented out code that used ConvertSample (error mentioned above) and was able to get the kernal8.img file created. In the 'kernel_MODplay.cpp' file, I commented out the computeSamplesAndScreenUpdate and the reference to it. In the file 'sound.h', I commented out the block for putSampleHDMI and removed references to it in the files 'kernel_sid.cpp' and 'kernal_sid8.cpp'.

[amiga23]

I managed to compile it, by commenting out the ConvertSample calls and using a newer version of circle, but the resulting binary does not work. It does not even initialize the display. I've also tried the fork of hpingel, but no success. The RAD I was able to compile and the binary works., so I guess my setup works in general.
For me it seems the code is not complete.

[ericwake]

Appreciate the replies! And no rush, but I do look forward to hearing the steps. I did look at hpingel's fork for the answer but found ConvertSample was missing there, too. The network capability looks like something I should try -- is there a particular release that works best?

After aggressively commenting the code as mentioned above, when I rename the kernal8.img file to kernal_sk64.img and copy it to the SD card, the Sidekick64 will successfully boot to menu and will load prg and crt files. The MOD player does not launch, likely because of the same, indiscriminate removal of code.

[hpingel]

@frntc has modified upstream Circle sources and he put the modified sources in here:

https://github.com/frntc/Sidekick64/tree/master/Source/Firmware/Circle/modified

You have to replace those two files in Circle. (Maybe you have to manually check if the changes are still matching newer circle versions. I have created patches of those changes that I apply in my build workflow.)

For the network kernel you need circle-stdlib.

[ericwake]

My setup of the project had already replaced the circle files with those from frenetic. Unfortunately, this does not resolve the issue of missing code for ConvertSample() method. 

My experience when using a newer version of circle, there are many more issues to deal with (e.g. need to update from '#include <circle/hdmisoundbasedevice.h>' to ''#include <circle/sound/hdmisoundbasedevice.h>', and when using frenetic's version of sysconfig.h there will be the error: 'KERNEL_STACK_SIZE' was not declared in this scope).

[hpingel]

So my issue is that I am travelling at the moment and I don't have access to my build environment at home until tonight - and then I don't have time.

Long story short: I have created a docker image that includes all the stuff to build all flavours reliably. In there I use this patch to patch the sound files.

You can give this patch a try with vanilla circle. So revert the changes to the sound files first.

I will upload the patch file here.

EDIT: The patch is a summary of the two modified files in folder "modified" and the two modified files in sibling folder "update".

So all the changes are in the repo.

EDIT2: As became clear, the attached patch fits best to Circle 45.3.x and another patch for Circle 44.3 can be found in another post further down.

sidekick_circle_vhiqsound.txt

[hpingel]

I managed to compile it, by commenting out the ConvertSample calls and using a newer version of circle, but the resulting binary does not work. It does not even initialize the display. I've also tried the fork of hpingel, but no success.

You need to specify your toolchain (gcc version, circle version, etc.) for me to comment. For example, I had issues with gcc12 and went back to gcc11. I think I last tried successfully with Circle 45.x but Circle 44.3 is the known-good standard.

I haven't compiled for nearly a year now.

[ericwake]

With the patch file you provided, I can successfully compile the project. I did need to change IEC958_HW_CHANNELS to SOUND_HW_CHANNELS and still need to actually test this on the Sidekick64. I have confidence that this is what was needed.

Not sure if I am referencing the tools correctly, but I am using a Windows machine and installed Git so that I could use Git Bash. And Git Bash includes mingw64 which uses gcc version 13.1.0 (Rev6, Built by MSYS2 project). Also installed arm gnu toolchain aarch64-none-elf 11.2 2022.02. Finally, using Circle 44.3.

@hpingel Thank you so much for taking the time to answer my questions - you really went above and beyond! I'll give you a break, but may eventually have more questions once I delve into your fork with the networking features. Thanks, again, for your help!

[hpingel]

Regarding IEC958_HW_CHANNELS / SOUND_HW_CHANNELS : My patch was for Circle 45.3.x or higher but I forgot that. Good that you were able to modify it.

For completeness sake, here is the patch for Circle 44.3.

sidekick_circle_vhiqsound_circle44.txt

[ericwake]

With my build of the kernel, I cannot get a .mod file to play, only getting a black screen. Next steps will be to try and figure out logging to get some idea of what might be happening.

[hpingel]

The following only applies if you use Circle 45.3.x or newer: add $(CIRCLEHOME)/lib/sound/libsound.a to the Makefile and include the hdmisoundbasedevice.
Compare: hpingel@08b9868

Because of this hassle I would stick to Circle 44.3 until you have everything running.

[hpingel]

If I remember correctly, I leave the sysconfig.h of Circle unchanged and override the necessary settings in Config.mk which has the same result.

I added the following lines to the Config.mk generated by configure:

DEFINE += -DKERNEL_MAX_SIZE=0x18000000
DEFINE += -DREALTIME
DEFINE += -DNO_BUSY_WAIT
DEFINE += -DNO_SD_HIGH_SPEED