|
DB4PLE
positron Urgestein
Offline
Posts: 1278
|
|
Re:Testers wanted / Tester gesucht
« Reply #16 on: 17. July 2018, 09:23:49 »
|
|
Hi,
to be clear about the issues I see:
With new compiler: 1. Start the bootloader with a USB drive plugged in 2. Press Band- while powering on.
Result: The saving of the old firmware will fail with error 4.
Diagnosis: No clue yet.
With old compiler and correctly named bin: 1. + 2. as before
Result: Storing and writing of new firmware works without error message but the firmware does not start.
Diagnosis: My guess here is, flashing does not work as intended.
|
|
Logged
|
|
|
|
DB4PLE
positron Urgestein
Offline
Posts: 1278
|
|
Re:Testers wanted / Tester gesucht
« Reply #17 on: 17. July 2018, 09:35:57 »
|
|
Hi,
okay, I found the programming error causing the actual flashing of the firmware on the F4 and F7 to fail. I fixed that, and with the old compiler (gcc 6 2017 q2) the bootloader on the F7 works as expected.
@Friedrich: Dein Flashen auf dem F4 musste schiefgehen, den Fehler habe ich gefunden.
It seems to me that there is an interesting issue just on the F7 and new compiler (gcc 7) issue, which does not appear on the H7.
I will soon publish two sets of bootloaders, one set build with the gcc 6 and one with the gcc 7. Please test when they become ready, again mostly F4 and F7, and specifically the USB driver update.
73 Danilo
|
|
Logged
|
|
|
|
DB4PLE
positron Urgestein
Offline
Posts: 1278
|
|
Re:Testers wanted / Tester gesucht
« Reply #18 on: 17. July 2018, 11:11:52 »
|
|
GCC 6.3.1 Bootloaders
Please test if USB DRIVE Update is working for F4 and F7 (both should work).
GCC 7 bootloaders follow...
|
|
|
|
DB4PLE
positron Urgestein
Offline
Posts: 1278
|
|
Re:Testers wanted / Tester gesucht
« Reply #19 on: 17. July 2018, 11:22:03 »
|
|
GCC 7.3.1 Bootloaders
Please test USB drive function (should work for F4 and H7 and most likely does not work for F7).
As of now I have not a single clue why running a different compiler version causes a problem for the F7 but not for the essentially identical H7 processor (both more or less use the same compiler settings relevant for code generation, both are a cortex-m7 with a double precision fpu).
We'll probably find out at some point...
Please confirm my findings: gcc: F4,F7,H7 bootloaders work without any problem gcc: F4,H7 bootloaders should work, F7 does not work with USB drive (stops with error 4).
We'll keep using the gcc6 (as we have been doing for a while) as the compiler for official builds and all is almost good for now.
73 Danilo
|
|
|
|
|
OE1FHB
noch länger dabei
Offline
Posts: 222
Ich liebe dieses Forum!
|
|
Re:Testers wanted / Tester gesucht
« Reply #21 on: 17. July 2018, 14:23:56 »
|
|
Hab bl - mchf.dfu von gcc 7.3.1 geladen ohne Probleme. Nachher über USB Update von FW durchgeführt , war ohne Probleme möglich.
Vy 73 Friedrich
|
|
Logged
|
|
|
|
DB4PLE
positron Urgestein
Offline
Posts: 1278
|
|
Re:Testers wanted / Tester gesucht
« Reply #22 on: 17. July 2018, 17:08:13 »
|
|
Hallo Friedrich,
danke! Bestätigt meine Vermutung.
Grüße Danilo
|
|
Logged
|
|
|
|
DB4PLE
positron Urgestein
Offline
Posts: 1278
|
|
Re:Testers wanted / Tester gesucht
« Reply #23 on: 17. July 2018, 22:39:11 »
|
|
Hi,
I found the issue. It is not a compiler bug, so as usual the problem sat in front of the monitor... The disk read / write code requires the data to start on an memory address which can be divided by 4 (i.e. is 4 byte aligned). We did not enforce this by a compiler setting however in most cases it was aligned correctly. Only the gcc7 F7 build did not have this aligned. And with that the bootloader failed. This was pure luck that we did not have that problem earlier. Fix is easy, just tell the compiler to align the data correctly. With that done, all is working now.
Thanks to the testers.
@Andreas: The GCC 7 can and should be used for official builds I would say.
73 Danilo
|
|
Logged
|
|
|
|
|
S53DZ
schon länger dabei
Offline
Posts: 58
Ich liebe dieses Forum!
|
|
Re:Testers wanted / Tester gesucht
« Reply #25 on: 18. July 2018, 07:05:02 »
|
|
Danilo,
great job. Thanks for that. GCC 7 works fine.
PS: It is again a proof for an old saying: "All great findings happened by accident." And in spite of all the hard work. But I comfort myself that all that hard work make the accident happen.
|
|
Logged
|
|
|
|
DB4PLE
positron Urgestein
Offline
Posts: 1278
|
|
Re:Testers wanted / Tester gesucht
« Reply #26 on: 18. July 2018, 07:12:14 »
|
|
Hi Andreas,
Hi Danilo,
I have modified my building script to GCC7 now. Very good work!
vy 73 Andreas
|
|
Just as a reminder: Before you release the bootloader build with gcc 7, we should fix it also in active-devel. I'll take care of that.
73 Danilo
|
|
Logged
|
|
|
|
|
DB4PLE
positron Urgestein
Offline
Posts: 1278
|
|
Re:Testers wanted / Tester gesucht
« Reply #28 on: 18. July 2018, 08:25:35 »
|
|
Hi Andreas,
yes, Makefile does work, all the firmware and bootloaders in the zip in this thread were build using the Makefile approach on Ubuntu (using the new Windows Linux Subsystem which for this kind of application is quite nice). I also extended the .travis.yml to take care of the H7 build as well... All of that will be in the H7 pull request from my ovi40h7 branch.
However, I made the small bootloader fix go to active-devel (see my pull request). So you'll have this in right now since it is an immediate issue, not like the H7 port which can wait a few more days.
73 Danilo
|
|
Logged
|
|
|
|
|