Topic: FBAlpha Big Endian Patches

[iq_132]

Ok, pretty sure I've got it fixed. There's a crash when NVRAM is corrupted too, but that's somewhat of a non-issue atm imo.

[Twinaphex]

Noticed a bug while testing FBA libsnes on SSNES PS3 - if a savestate was made with any CPS3 game, the screen colors would be distorted heavily. This would only partly go away in the next round, but the colors would still be off.

You can see an image of what would happen here:

https://imgur.com/A5QOy

The following needs to be changed in cps3run.cpp - line 689 - change:

Code: [Select]

RamPal[(paldma_dest + i) ^ 1] = coldata;

to:

Code: [Select]

#ifdef LSB_FIRST
RamPal[(paldma_dest + i) ^ 1] = coldata;
#else
RamPal[(paldma_dest + i)] = coldata;
#endif

That fixed the issue for me at least.

[iq_132]

Noticed a bug while testing FBA libsnes on SSNES PS3 - if a savestate was made with any CPS3 game, the screen colors would be distorted heavily. This would only partly go away in the next round, but the colors would still be off.

You can see an image of what would happen here:

https://imgur.com/A5QOy

The following needs to be changed in cps3run.cpp - line 689 - change:

Code: [Select]

RamPal[(paldma_dest + i) ^ 1] = coldata;

to:

Code: [Select]

#ifdef LSB_FIRST
RamPal[(paldma_dest + i) ^ 1] = coldata;
#else
RamPal[(paldma_dest + i)] = coldata;
#endif

That fixed the issue for me at least.

Cool. ^^ Pushed this to the SVN.

[Twinaphex]

Small typo in d_psikyo4.cpp for big-endian code path -

change line 452 from:

Code: [Select]

DrvSprRAM((address) & 0x3fff] = data;

to:

Code: [Select]

DrvSprRAM[(address) & 0x3fff] = data;


[iq_132]

Cool. Fixed

[Barry Harris]

Thanks to the big-endian patches by Lantus, FBA mainline now runs on 360, Wii and PS3 through SSNES/libsnes.

Perhaps one small change can be pushed upstream to make my life a bit easier - the header file that needs to be included for a big-endian port is called 'endian.h' right now - if this could be changed to some other name it would be much appreciated. The reason for changing it is that Linux has a system header file by that name and it conflicts when I build FBA libsnes on a Linux system (I compile PS3 builds on a Linux system with the Win32 PS3 toolchain running through WINE) - so right now with every new version of FBA that is released I always have to change the name to something that won't conflict, such as 'big_endian.h'.

Thanks for pointing it out, I changed the include to burn_endian.h.

[Barry Harris]

Ok, pretty sure I've got it fixed. There's a crash when NVRAM is corrupted too, but that's somewhat of a non-issue atm imo.

gdb no longer reports the heap warnings on exit (and I got them 100% of the time before).

[iq_132]

gdb no longer reports the heap warnings on exit (and I got them 100% of the time before).

Wonderful.

[vbt]

don't forget Saturn, I'm back

[lantus]

welcome back =)

[lantus]

d_cps1 endian updates for the new stuff and some bootlegs i never bothered with in the past

[lantus]

a small endian cps1 fix for SF2MDT

[lantus]

- endian fix for System16A tiles (Shinbo set 1 and 6)
- endian fix for d_tecmo.cpp palette (rygar)

[lantus]

- endian fix for d_cybertnk

[lantus]

These Big Endian patches are from MagicSeb. Verified and tested working.

 - d_cbuster.cpp : Colorfix + No crash
 - d_simpl156.cpp : Sprites fix
 - d_supbtime.cpp : Colorfix + No crash