Sansa E200V2 Bootloader is broken in master

[amachronic]

So it makes my changes in unicode.c unnecessary, doesn't it?

Yes.

Also backtraces are not present in current bootloader release so it's not something we removed from released functionality.

OK, if it's not a regression that's fine with me.

Everything I've done is merged so let me know when you want to merge your changes. I also merged the multiboot v1 patches.

[bahus]

build them on top of his patchsets he isn't going to get upset

Not sure what you meant. But I've already cherry picked amachronic's patches to my local branch and reported results:

Also we don't make new bootloaders often so could you test some of these patches with your builds?

It works for me and adds around 100 bytes to thumb  size (119676 without, 119759 with those patches)

[bahus]

Everything I've done is merged so let me know when you want to merge your changes. I also merged the multiboot v1 patches.

I tested bootloader on top of the latest changes -  it works. So I think it's ready to be merged.

[Bilgus]

Out of curiosity I tested this no thumb build without `usb_storage_disconnect` fix. Still doesn't boot. So it doesn't look like some alignment issue to me. `usb_storage_disconnect` is the cause...

agreed -- thatd be our smoking gun