externe Konstante im Flash

A. K. schrieb:
> Auf PROGMEM Daten kann man nicht direkt zugreifen. Doku lesen.
???

Ralf schrieb:
> // const uint8_t K_AdresseZusatz PROGMEM = 48;
> // => so funktioniert's (ohne 'Const.c')

Und der erzeugte Assembler-Code ist ja korrekt. So wie man's auch direkt 
macht:

Ralf schrieb:
> 60:  20 91 26 00   lds  r18, 0x0026
> 64:  28 2b         or  r18, r24
> 66:  20 93 60 00   sts  0x0060, r18

Auf PROGMEM Daten kann man nicht direkt zugreifen.
Doku lesen => pgm_read_xxx().

A. K. schrieb:
> Kein LPM => keine korrekten Daten.
Genau. Ist mir gerade auch aufgefallen. Das hab ich völlig übersehen. 
Die Konstante wird gar nicht berücksichtigt im ASM-Code :-(
Warum dann aber Variante 2 (nicht 'extern') funktioniert, ist mir jetzt 
auch klar: die Konstante wird nicht im Flash gespeichert, sondern gleich 
an der entsprechenden Stelle in den Code gebastelt. Muss ich mal 
'pgm_read' probieren (hoffe das braucht nicht zuviel Code!)

Bleibt noch: Warum 16Bit?

1

AdressePlatine=(((~PINB) & 0b11100000) >> 5) | pgm_read_byte(&K_AdresseZusatz);

Hätt' ich auch selber drauf kommen können. Naja, taste mich noch ran an 
GCC.

Ralf schrieb:
> konnte aber dem
> Compiler nicht abgewöhnen, die Operationen (Bitverknüpfungen, Schieben)
> in 16Bit zu machen. Geht das irgendwie? Außerdem: wieso 'asr r25', wenn
> alles 'uint8_t' ist?
Ich konnt's ihm abgewöhnen :-)

1

 AdressePlatine=(((~PINB) & 0b11100000));

2

 AdressePlatine = AdressePlatine >> 5;

3

 AdressePlatine |= pgm_read_byte(&K_AdresseZusatz);

Da gibt der Compiler sich richtig Mühe!

Trotzdem: warum erst 16Bit und 'asr r25'?

Ralf schrieb:
> Trotzdem: warum erst 16Bit und 'asr r25'?

Weil die C-Definition vorschreibt, daß bei einer Rechenoperation alle 
Operanden, die kleiner als int sind, erstmal zu int erweitert werden und 
die Rechnung dann in int durchgeführt wird. Der Compiler darf diese 
Erweiterung wegoptimieren, aber nur, wenn er garantieren kann, daß in 
jedem Fall das Ergebnis dadurch nicht beeinträchtigt wird. Jetzt ist bei 
AVR aber die native Wortgröße kleiner als int, und da dieser Fall in GCC 
eine Ausnahme darstellt, sind solche Optimierungen da nicht so 
umfassend.

@Rolf Magnus
Sozusagen: Er könnte, wenn er wollte ;-)

@Karl Heinz Buchegger
Ich hatte mir extra den ASM-Code angeschaut, mit meinen Kenntnissen 
könnte ich da nichts verbessern (in Assembler hätte ich das genauso 
gemacht, außer: r30 gleich als Ziel für lpm zu verwenden, hätte ich mich 
nicht 'getraut'):

1

 AdressePlatine=(((~PINB) & 0b11100000));

2

  4a:  86 b3         in  r24, 0x16  ; 22

3

 AdressePlatine = AdressePlatine >> 5;

4

 AdressePlatine |= pgm_read_byte(&K_AdresseZusatz);

5

  4c:  e6 e2         ldi  r30, 0x26  ; 38

6

  4e:  f0 e0         ldi  r31, 0x00  ; 0

7

  50:  e4 91         lpm  r30, Z+

8

  52:  80 95         com  r24

9

  54:  82 95         swap  r24

10

  56:  86 95         lsr  r24

11

  58:  87 70         andi  r24, 0x07  ; 7

12

  5a:  e8 2b         or  r30, r24

13

  5c:  e0 93 60 00   sts  0x0060, r30

('swap + lsr' fand ich gut, dass er das im Sortiment hat :-) Bei der 
'16Bit-Variante' war das ja noch eine Schleife!)

@Karl Heinz Buchegger
Hatte gestern keine Gelegenheit mehr das zu probieren. Mit deinem 
Zweizeiler ist das ganze wieder etwas übersichtlicher.
-----
Karl Heinz Buchegger schrieb:
> durch das ~ zwingst du den Compiler, den Wert auf 16 Bit aufzublasen.

Rolf Magnus schrieb:
> Weil die C-Definition vorschreibt, daß bei einer Rechenoperation alle
> Operanden, die kleiner als int sind, erstmal zu int erweitert werden und
> die Rechnung dann in int durchgeführt wird.

Kann man sowas als Laie wissen? Dann habe ich das immer überlesen. 
Vielleicht, weil es auf dem PC nicht so wichtig ist...

Johann L. schrieb:
> Der Cast muss an der richtigen Stelle sein:
> AdressePlatine = ((((uint8_t) ~PINB) & 0b11100000) >> 5) | K_AdresseZusatz;

Ich glaube, das war die einzige Stelle, die ich nicht probiert habe. So 
ein Port hat ja nun sowas von 8Bit...
Aber: 'Wie wir jetzt wissen' ;-)
Karl Heinz Buchegger schrieb:
> durch das ~ zwingst du den Compiler, den Wert auf 16 Bit aufzublasen.

Johann L. schrieb:
> Freilich geht aus sowas:void foo2()
> {
>     AdressePlatine = (PINB >> 5) ^ K_AdresseZusatz;
> }
> indem du in K_AdresseZusatz die unteren 3 Bits setzt. Ist ja eh ne
> Konstante, und da tut das nicht weh :-)
>
Kann ich nachvollziehen :-)
Das ist natürlich mal was. Wäre ich nie drauf gekommen.