BFS_LoadFile, Flash1

[eme]

N'Abend. Ich habe settingsmenu um eine Kontrasteinstellung erweitert:
http://boopfirmware.svn.sourceforge.net/viewvc/boopfirmware/boop/trunk/gui/settingsmenu.c?revision=28&view=markup#l_140

Es gibt jedoch einen unschönen Bug: Es wird nach dem Flashen ein falscher Wert geladen, deswegen ist nichts auf dem Display zu sehen.
Genau das sollte jedoch nach meinem Verständnis in Zeile 140 unterbunden werden:

Code: [Auswählen]

if(BFS_LoadFile(BFS_ID_settings, sizeof(struct settings_), (unsigned char*) &settings) != sizeof(struct settings_))

Diese Bedingung sollte True ergeben, da struct settings_ ein Byte größer ist, als die alte Struktur im Flash. Sehe ich das richtig?

Workaround: per lpctool mit Option -E oder per TestStuff "Erase Flash1" die alten Settings löschen.

[Telekatz]

Diese Bedingung sollte True ergeben, da struct settings_ ein Byte größer ist, als die alte Struktur im Flash. Sehe ich das richtig?

Ja, so hab ich mir das damals gedacht. Würde auf einem 8Bit µC wohl auch funktionieren. Aber nicht unbedingt auf einem ARM.

Die struct settings_ war vorher 12 Byte groß und ist mit mit dem Kontrastwert immernoch 12 Byte groß. Liegt an der Speicherausrichtung. Eine ungepackte struct hat hier immer eine vielfache größe von 4 und ein int liegt auf einer Adresse die restlos durch 4 teilbar ist.

Ich werd das mal etwas umprogrammieren.

[eme]

Klingt kompliziert :-)
Unabhängig davon: die Kontrasteinstellung bringt nicht so wahnsinnig viel; bei meinen Bettys sind die Graustufen bei gleichen Parametern ziemlich verschieden.
Ich werde mal ein paar Originale auslesen, ich vermute, sie bekommen bei der Herstellung einen Kalibrierwert. Vielleicht gibt's vom Displayhersteller für jedes Display gleich die optimalen Parameter mit dazu?

[eme]

Fixed durch Versionierung: http://boopfirmware.svn.sourceforge.net/viewvc/boopfirmware?view=rev&revision=31