3
\$\begingroup\$

I'm writing the firmware for a simple testing device that would need to store a very large lookup table (I need 13500 entries with values 0..100 decimal). The target PIC MCU is PIC16F1829 (8K words of program memory), and the compiler is XC8.

Obviously, this won't work:

static const uint8_t table[13500] = { 0x01, 0x23, ... };

as it will be stored as a sequence of RETLW instructions, each taking one program word, and we only have ~8K of these.

However, I can pack two 7-bit values into one 14-bit word, so if I had the capability of doing

static const uint14_t table[6750] = { 0x1181, ... };

I can imagine that reading table[0] should translate to using the flash memory read functionality to read the progmem word there. I can do the unpacking afterwards.

The actual setup/work code on the MCU would be quite minimal, about 300 words, so I'm within bounds.

Is there a way to define such arrays cleanly (using C constructs only), or one has to resort to assembly, as discussed here?

Edit:

The intended use of this lookup table is to replay a specific waveform using filtered PWM. The setup/work code is very minimal, it only initializes the PWM module and writes the waveform to the PWM period register at regular intervals. I don't think the unpacking would add a lot of code, it's just a few shifts.

Edit #2:

Since some of answers/comments suggest or assume properties of the data and suggest not storing it in full, but using interpolation, computation or compression:

The data comes from measured physical quantity which was recorded via a separate device. Reproducing it verbatim is important (so interpolation isn't recommended). It is also reasonably random, since gzip only achieves 35% compression when the data is stored in binary, in bytes. Since each byte only uses 101 different values instead of 255, by Huffman encoding alone you'd expect ~17% compression. Also the cost of adding the inflate algorithm for this PIC is probably more than the compression saves...

Edit #3:

The RETLW trick the XC8 compiler uses seems to not be a common knowledge, so let's explain why I'm talking about RETLW all the time and dismissing answers because of it.

The 8-bit PIC micros are examples of a Harvard architecture, with separated code and data address spaces. So if you wanted an initialized global array like

char mytable[5] = { 1, 2, 3, 4, 5 };

storing these 5 bytes is not as simple as putting them somewhere in the flash inside the PIC and later mapping them into the data segment, You just cannot say "these 5 bytes are part of the data segment". Since this code is valid C, XC8 has to implement it by running some initialization code that literally does something akin to mytable[0] = 1; mytable[1] = 2; .... This is very expensive in terms of code space required to do this initialization and is to be avoided by any seasoned PIC dev.

However, you can forego some of this penalty by making your table const:

const char mytable[5] = { 1, 2, 3, 4, 5 };

Since the compiler knows you cannot change those values, you might get away with not actually keeping them in the data address space. They may reside in the code address space only. So if you wanted mytable[i], the compiler internally replaces it with a function call (something like fetch_mytable(i)), and implements this function using an array of instructions in the code address space, something like this

... prologue: address computation, jump to address ...
retlw 1
retlw 2
retlw 3
retlw 4
retlw 5

This is the body of the function fetch_mytable(i), and in the prologue you do a computed goto based on i, and the respective RETLW instruction is jumped to; it stores the desired value into the W register and returns from the function.

This method is simple and quick, and is a first class citizen in the XC8 compiler environment - it does this kind of trick quietly under the hood for static/global const arrays.

My gripe with this method is that it takes 5 program words (70 bits = 5*14) to store just 40 bits worth of data (5*8). Each RETLW instruction "wastes" 6 bits for the RETLW opcode. Each PIC instruction is 14 bits, the program memory is organized into words of 14 bits, but this is not exposed in C.

Another approach for implementing a lookup table is similar, but instead of the RETLW array, up to 14 bit values can be stored in the program memory directly. And instead of doing a computed goto, use the program memory readout mechanism (EEADR/EECON1 et al) to read the program word of a particular address. This approach is slower, as it requires more boilerplate readout code, but doesn't waste any bits, and for a large array it really makes a difference.

So my question is whether this second approach can also be accessed from C, as a first class citizen, just like the other method. I've tried making an array of uint16_t's, but still the first method is used, with each uint16 being represented by a pair of RETLWs.

\$\endgroup\$
5
  • \$\begingroup\$ You might not notice it, but your question is essentially one long list of arguments against using PIC. The sensible approach would be to use a Cortex M and an array of uint16_t. \$\endgroup\$
    – Lundin
    Commented Feb 25, 2019 at 11:32
  • \$\begingroup\$ ... and then I'd need level shifters to interface to the 5V thing I'm connecting to :) \$\endgroup\$
    – anrieff
    Commented Feb 25, 2019 at 18:57
  • \$\begingroup\$ @anrieff It's pretty hard to find a Cortex M which isn't 3-5V tolerant. It's pretty much standard for MCUs since 10 years back. \$\endgroup\$
    – user206438
    Commented Feb 25, 2019 at 19:14
  • \$\begingroup\$ Kinetis E Cortex-M0+ is 5V tolerant. \$\endgroup\$
    – filo
    Commented Feb 28, 2019 at 7:24
  • \$\begingroup\$ @filo, didn't know such devices existed, these could be quite useful for a few projects. Many thanks! \$\endgroup\$
    – anrieff
    Commented Feb 28, 2019 at 10:25

3 Answers 3

3
\$\begingroup\$

You ask the question "How to define 14-bit lookup table for PIC/XC8 (if it's possible to do without assembly)?"

It is almost possible to do this in XC8 without assembly. Assembly must be used to define the table because XC8 does not have any C language extension to declare constant data in code space as packed 14-bit words.

To do this requires at least two source files.

C language file:

/*
 * File:     main.c
 * Author:   dan1138
 * Target:   PIC16F1829
 * Compiler: XC8 v2.00
 * IDE:      MPLABX v5.10
 *
 * Created on February 25, 2019, 12:24 AM
 */
#include <xc.h>
#include <stdint.h>
typedef unsigned char   uint7_t;

extern const uint7_t PWM_Table;
extern const uint7_t PWM_TableEnd;

volatile uint16_t Address_PWM_Table;

uint7_t ReadPWM_Table( void ) 
{
    uint8_t INTCON_save;
    uint7_t TableValue;

    INTCON_save = INTCON;
    EEADR = Address_PWM_Table >> 1;
    EECON1bits.CFGS = 0;
    EECON1bits.EEPGD = 1;
    INTCONbits.GIE = 0;
    EECON1bits.RD = 1;
    NOP();
    NOP();
    if (INTCON_save & 0x80) INTCONbits.GIE = 1;
    if (!(Address_PWM_Table & 1))
    {
        TableValue = EEDATL & 0x7F;
    }
    else 
    {
        TableValue = ( EEDATH << 1 );
        if (EEDATL & 0x80) TableValue |= 1;
    }
    Address_PWM_Table++;
    return TableValue;
}
/* 
 * Main application
 */
volatile uint7_t DataFromTable;
void main(void) 
{
    /* 
     * Main application loop
     */
    uint16_t TableElements;

    for(;;)
    {
        Address_PWM_Table = ((unsigned int)&PWM_Table)<<1;
        TableElements = (unsigned int)(&PWM_TableEnd-&PWM_Table)<<1;
        if (TableElements) do
        {
            DataFromTable = ReadPWM_Table();
        } while (TableElements--);
    }
}

Assembly language file:

; 
; File:     PWM_Table.as
; Author:   dan1138
; Target:   PIC16F1829
; Compiler: XC8 v2.00
; IDE:      MPLABX v5.10
; 
; Created on February 25, 2019, 12:24 AM
;   
; The DABS directive allows ASPIC code to "reserve" a range of code space
; and prevent the linker from locating anything there.
; 
; Using the psect "abs" option allows the ASPIC code to declare code and
; ignores the class bindings.
;
; Finally the "org" ASPIC directive can place the psect at where the DABS
; directive reserved the memory area.
;
; This example defines a PWM_Table of 13500 7-bit values.
; 
; Note: There is a drawback with this method. An absolute psect storage
; does not get counted as allocated space by the linker. This makes the
; storage usage map inaccurate.
; 
#include <xc.inc>

    global _PWM_Table,_PWM_TableEnd
;
; Pack two 7-bit values in one 14-bit instruction word
;
; Walk a one bit through all 7 bits of a table read.
;
    psect   PWM_Table,abs,class=PWMTAB,delta=2,noexec ; PIC10/12/16
    org       0x5a2
    DABS    0,0x5a2,6750  ; 0=code space, 0x5a2=start address, 6750=number of instruction words. These values must be numeric constants.
_PWM_Table: 
    rept (13500/12)
    dw 0x0101,0x0404,0x1010,0x1040,0x0410,0x0104
    endm
_PWM_TableEnd:
\$\endgroup\$
4
  • 1
    \$\begingroup\$ This looks about as good as it's going to get. +1. \$\endgroup\$ Commented Feb 25, 2019 at 11:07
  • \$\begingroup\$ Well this isn't really without assembly :) but I reckon that's possibly unavoidable. I implemented this yesterday and stumbled into other issues (e.g. you cannot have a psect with more than 2048 words like that, so you have to split it over several psects and several symbols; it's a pain, that's why I'm eagerly looking for a more "official" approach). \$\endgroup\$
    – anrieff
    Commented Feb 25, 2019 at 11:07
  • 1
    \$\begingroup\$ @anrieff I don't think it's going to get much better then this. The official approach would be to get a 18F which has a table read instruction for this. \$\endgroup\$ Commented Feb 25, 2019 at 11:19
  • \$\begingroup\$ Agreed. I guess nobody really envisioned a 16F to be used in this scenario :) \$\endgroup\$
    – anrieff
    Commented Feb 25, 2019 at 13:19
1
\$\begingroup\$

If the values in the table / array go from small to large (ie in order) then you can put them into excel, plot it and fit a trend line then get the equation. Then your code only needs to do the calculation rather than having a massive lookup table.

\$\endgroup\$
1
  • 1
    \$\begingroup\$ No, the data doesn't have a structure and can't be computed. Treat it as random. \$\endgroup\$
    – anrieff
    Commented Feb 24, 2019 at 21:47
0
\$\begingroup\$

So, with 101 possible values, you're bound to use at least 7 bits per data word.

You should feel absolutely free to pack eight data words into a 7 B struct; C does allow that:

struct eight_words 
{
  unsigned int word0 : 7;
  unsigned int word1 : 7;
  unsigned int word2 : 7;
  unsigned int word3 : 7;
  unsigned int word4 : 7;
  unsigned int word5 : 7;
  unsigned int word6 : 7;
  unsigned int word7 : 7;
} __attribute__ ((__packed__));
const struct eight_words table[13500/8] = { {.word0 = 0x00, .word1 = 0x23, …} , {.word0 = …}};

and the __packed__ attribute will make any decent compiler actually emit structures of size 7 Bytes.

Access then is super simple. You can either just use pointers to your eight_words struct:

// get the 1000. element
unsigned int index, offset, value;
index = 1000/8; // integer math!
offset = 1000 - index * 8;
switch(offset)
{
  case 0: value = table[index].word0; break;
  case 1: value = table[index].word1; break;
  …
  case 7: value = table[index].word7; break;
}

or you just do a bit of pointer arithmetic:

// get the 1000. element
uint64_t* array = (char*) table;
unsigned int index, offset, value;
index = 1000/8; // integer math!
offset = 1000 - index * 8;
value = (array[index*7] >> (offset*7)) & (0xF);

That should both amount to roughly the same assembly, a few bytes in machine code.

Note that PWM sequence does sound like you're trying to reproduce some band-limited signal. For such, good compressors exist. Especially since you're only planning to access the data sequentially, and since even of your 7 bits, 27 values don't occur and, probably, a few bit combos are way more likely than others:

Simply use some simple LZW or huffman (low dict size) compressor and decompressor. These are a couple hundred bytes in machine code.

\$\endgroup\$
9
  • \$\begingroup\$ I'm sorry, this is a good technique, but misses the core of the question completely. The packing style you suggest will again be translated in to a string of RETLWs, essentially using only 8 bits out of every 14-bit word. The array will again not fit in the progmem of the PIC16F1829. \$\endgroup\$
    – anrieff
    Commented Feb 24, 2019 at 19:59
  • \$\begingroup\$ No, it won't – RETLW is an instruction, and this is not code, but data.. What happens to constants is that the linker puts them in them some data section. \$\endgroup\$ Commented Feb 24, 2019 at 20:23
  • \$\begingroup\$ I mean the following: if you define a 200-entry lookup table of uint8_t, it will consume a bit more than 200 words of code space. This is because under the hood the compiler implements it with a string of RETLW instructions and a computed goto. So your code will consume more than 11813 words, larger than what the PIC in question has. \$\endgroup\$
    – anrieff
    Commented Feb 24, 2019 at 21:47
  • 1
    \$\begingroup\$ @marcus, dont be so quick to claim that, write the code and you will see that it ends up using retlw because that is the fastest instruction on the pic to do that... \$\endgroup\$
    – Cobusve
    Commented Feb 25, 2019 at 1:05
  • \$\begingroup\$ @Cobusve I must admit I would have just put the raw binary data in a file and used objcopy to make it linkable, or just straight away ld -r -b binary datafile.raw -o table.o and used that; I don't know the compiler Op uses well enough, but an -Os switch should exist? \$\endgroup\$ Commented Feb 25, 2019 at 6:45

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.