Fix DI return code checks

[Leseratte10]

The old code retried DI_Read calls until the return value was not negative. However, that never happens. DI_Read and most other DI functions have 1 as the success case, and positive numbers larger than 1 for errors. This means that without this PR, reads could sometimes fail (due to disc issues or whatever), and instead of noticing and retrying, Brainslug would just assume that the data was read correctly.

[Leseratte10]

Addition: That also means that the di_has_partition variable in di.c is probably invalid / wrong, as it also doesn't correctly check the return code. That's not yet fixed with this PR, though.

[suprstarrd]

If you're still actively working on this PR...

brainslug-wii/src/apploader/apploader_get_ios.c

Lines 115 to 118 in 88a4069

	while ((ret = DI_DiscInserted()) != 1) {
	if (ret < 0) {
	continue;
	}

Is this meant to still be ret < 0?