Problem mit pgm_read_byte

[rhood (Gast)]

Hallo!
Ich habe hier eine 5*7 Fonttabelle und eine 16*12 Font Tabelle
und diese haben folgende struktur

const uint8_t __Font5x7[] PROGMEM = {
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,// (space)

...
tFont Font5x7 = {(PGM_P)__Font5x7, 7, 8, 7, 1, 0};

const uint8_t __Font12x16[] PROGMEM = {
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, // "(space)"
  0x7C, 0x00, 0xFF, 0x33, 0xFF, 0x33, 0x7C, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, // "!"

...
tFont Font12x16 = {(PGM_P)__Font12x16, 11, 16, 22, 2, 0};

diese müssen über eine funktion putstr auf dem display DOGS102 zu sehen 
sein, nun für die kleiensc hriftart funktioniert es, aber weshalb auch 
immer gehts beim großen font nicht

hier die vorgegebene font struktur:

typedef struct
{
  PGM_P font;
  unsigned char Width;
  unsigned char Height;
  unsigned char CharSize;
  unsigned char Lines;
  unsigned char Prop;
} tFont;

void DOGS102_PutStr(unsigned char PosX, unsigned char Line, char * pdata, tFont font)
{
  unsigned int Len = 0;
  unsigned char Data = 0;
  unsigned char l;
  unsigned char FontIndex;
  unsigned char ZeroLine;

  while (*pdata)
  {
    unsigned char w;
    FontIndex = 0;
    for (w = 0; w < font.Width; w++)
    {
      ZeroLine = 0;
      for (l = 0; l < font.Lines; l++)
      {
        Data = pgm_read_byte(&font.font[(*pdata-Font_AsciiOffset)*(font.CharSize)+(w+1)*(l+1)-1]);// PROBLEM LIEGT HIER
        FontIndex++;
        ZeroLine |= Data;
        LCDCache[(DOGS102_PIXELX * (Line + l)) + PosX + Len] = Data;
      }
      Len++;
      if (ZeroLine == 0 && w > 1 && font.Prop == 0)
      break;
      
      // falls ein Zeichen genau font.Width Bytes benoetigt, dann am Ende noch
      // eine Leerspalte einfuegen
      if (w == font.Width - 1)
      {
        Data = 0x00;
        
        for (l = 0; l < font.Lines; l++)
        LCDCache[(DOGS102_PIXELX * Line) + PosX + Len] = Data;
        Len++;
      }
    }
    pdata++;
  }
  
  // write all lines needed for the font
  for (l = 0; l < font.Lines; l++)
  {
    WriteCache(PosX, Line + l, Len);
  }
}

format_ascii_font ist ascii wert vom space

kann mir vllt jemand da weiterhelfen?

[Karl H. (kbuchegg)]

Der Teil hier

   .... (w+1)*(l+1)-1

ist unlogisch.

Da Arrays in C beginnend mit 0 indiziert werden und das ganze im Grunde 
ein 2-dimensionales Array abbildet, wäre

   .... w * font.Lines + l

wesentlich logischer
Eventuell auch

   .... w * l * font.width

je nachdem in welcher Reihenfolge die Bytes in den Font Pixeldaten 
angeordnet sind.

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

rhood schrieb:
> aber weshalb auch immer gehts beim großen font nicht
WAS "geht" denn "nicht"?
Wie äussert sich der Fehler?

[Masl (Gast)]

Lass das pgm_read Gedöns und nimm __flash. Das ergibt einfacheren und 
übersichtlicheren Code.

[Karl H. (kbuchegg)]

Masl schrieb:
> Lass das pgm_read Gedöns und nimm __flash. Das ergibt einfacheren und
> übersichtlicheren Code.

Ist in dem Fall ziemlich egal.
Ob er mit __flash falsch in den Speicher indiziert oder ob er die 
Speicheradresse für pgm_read_byte falsch ausrechnet, schenkt sich 
nichts. Es ist in der gleichen Weise falsch.

[Karl H. (kbuchegg)]

Karl Heinz schrieb:

Die Variante hier

> ein 2-dimensionales Array abbildet, wäre
>

>    .... w * font.Lines + l
>

ist aufgrund

>   0x7C, 0x00, 0xFF, 0x33, 0xFF, 0x33, 0x7C, 0x00,
>   0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   0x00, 0x00, 0x00, 0x00, 0x00, 0x00, // "!"

die richtige. Die beiden Bytes, die senkrecht aneinander gelegt das 
Zeichen ergeben, folgen offenbar unmittelbar hintereinander.

d.h das ist

O = Byte oben
U = Byte unten

                 O   U   O   U   O   U   O   U ...
                7C  00  FF  33  FF  33  7C  00 ...

Pixelspalte      1   1   2   2   3   3   4   4 ....

Um also an die Bytes der x-ten Pixelspalte zu kommen, muss man die 
Spaltennummer mit der Anzahl der Bytes pro Spalte multiplizeren. Die 
einzelnen Bytes jeder Spalte folgen dann direkt hintereinander.
Font.Lines ist genau diese Anzahl an Bytes
w ist die Nummer der jeweils auszugebenden Pixelspalte.

Daher

    for (w = 0; w < font.Width; w++)
    {
      for (l = 0; l < font.Lines; l++)
      {
        gib das Byte mit den Pixeln von
          Pixeldaten_des_Zeichen[ w * Font.Lines + l ]
        aus.
      }
    }