在Arduino上实现CRC8以写入MLX90614

更新：我甚至无法复制图8和图9所示的SMBus PEC

所以我把arduino和Melexis温度传感器连接起来，除了CRC检查无法正常工作之外，一切正常

虽然我的软件忽略了数据包错误代码，但我已经成功地完成了读取操作，但是我尝试了很多CRC8的实现来检查PEC字节，但都没有用。我现在使用的代码块来自OneWire：

uint8_t OneWire::crc8(const uint8_t *addr, uint8_t len)
{
    uint8_t crc = 0;

    while (len--) {
        uint8_t inbyte = *addr++;
        for (uint8_t i = 8; i; i--) {
            uint8_t mix = (crc ^ inbyte) & 0x01;
            crc >>= 1;
            if (mix) crc ^= 0x8C;
            inbyte >>= 1;
        }
    }
    return crc;
}

我重写它只考虑了一个字节：

int smbCRC(int message) {

    uint8_t crc = 0;

  uint8_t inbyte = message & 0xFF;
  for (uint8_t i = 8; i; i--) {
    uint8_t mix = (crc ^ inbyte) & 0x01;
    crc >>= 1;
    if (mix) crc ^= 0x8C;
    inbyte >>= 1;
  }

    return crc;
}

但其CRC与例如图8中的MLX数据表的CRC不匹配。当我打印带有CRC8的int时，如下所示：

int message = 0x3aD2;
lcd.print(String(message,HEX) + " " + String(smbCRC(message),HEX));

---

我得到3ad2 eb，尽管数据表上说正确的PEC为0x30。我哪里做错了？似乎这可能是由于CRC的错误实现或我对CRC输入的错误假设造成的，我不确定从哪里开始故障排除。

请参阅以下链接的参考：

Arduino的功能代码：

byte c8( byte x ){
  for( byte i = 8; i--; ) {
    x = ( x << 1 ) ^ ( x & 128 ? 7 : 0 );
  }
  return x;
}

void setup() {
  Serial.begin( 9600 );
  int msg = 0x3AD2;
  Serial.print( '0x' );
  Serial.print( c8( msg ), HEX );
  // '0x30' is displayed on Serial Monitor
}
void loop() {}

---

由于参数x是强类型的字节uint8_t，因此在0x3AD2的情况下，诸如0x3AD2之类的字大小数据将被截断为字节大小0xD2。

我没有检查您的CRC实现，但MLX数据表中有错误，或者至少写得不好。您必须包括所有I2C帧的数据，以用于PEC的计算，而不仅仅是回复的数据。 对于读字命令，必须包括[SA_W，command，SA_R，LSB，MSB]，对于写字命令，必须包括[SA_W，command，LSB，MSB]

因此，对于他们的第一个示例，必须在[0xB4，0x07，0xB5，0xD2，0x3A]上进行计算，而不仅仅是在[0xD2，0x3A]上进行计算，这样可以得到预期的0x30

下面是一个简单的CRC C C实现，带有非Arduino的查找表，但它必须非常容易适应：

static const uint8_t crc_table[] = {
    0x00, 0x07, 0x0e, 0x09, 0x1c, 0x1b, 0x12, 0x15, 0x38, 0x3f, 0x36, 0x31,
    0x24, 0x23, 0x2a, 0x2d, 0x70, 0x77, 0x7e, 0x79, 0x6c, 0x6b, 0x62, 0x65,
    0x48, 0x4f, 0x46, 0x41, 0x54, 0x53, 0x5a, 0x5d, 0xe0, 0xe7, 0xee, 0xe9,
    0xfc, 0xfb, 0xf2, 0xf5, 0xd8, 0xdf, 0xd6, 0xd1, 0xc4, 0xc3, 0xca, 0xcd,
    0x90, 0x97, 0x9e, 0x99, 0x8c, 0x8b, 0x82, 0x85, 0xa8, 0xaf, 0xa6, 0xa1,
    0xb4, 0xb3, 0xba, 0xbd, 0xc7, 0xc0, 0xc9, 0xce, 0xdb, 0xdc, 0xd5, 0xd2,
    0xff, 0xf8, 0xf1, 0xf6, 0xe3, 0xe4, 0xed, 0xea, 0xb7, 0xb0, 0xb9, 0xbe,
    0xab, 0xac, 0xa5, 0xa2, 0x8f, 0x88, 0x81, 0x86, 0x93, 0x94, 0x9d, 0x9a,
    0x27, 0x20, 0x29, 0x2e, 0x3b, 0x3c, 0x35, 0x32, 0x1f, 0x18, 0x11, 0x16,
    0x03, 0x04, 0x0d, 0x0a, 0x57, 0x50, 0x59, 0x5e, 0x4b, 0x4c, 0x45, 0x42,
    0x6f, 0x68, 0x61, 0x66, 0x73, 0x74, 0x7d, 0x7a, 0x89, 0x8e, 0x87, 0x80,
    0x95, 0x92, 0x9b, 0x9c, 0xb1, 0xb6, 0xbf, 0xb8, 0xad, 0xaa, 0xa3, 0xa4,
    0xf9, 0xfe, 0xf7, 0xf0, 0xe5, 0xe2, 0xeb, 0xec, 0xc1, 0xc6, 0xcf, 0xc8,
    0xdd, 0xda, 0xd3, 0xd4, 0x69, 0x6e, 0x67, 0x60, 0x75, 0x72, 0x7b, 0x7c,
    0x51, 0x56, 0x5f, 0x58, 0x4d, 0x4a, 0x43, 0x44, 0x19, 0x1e, 0x17, 0x10,
    0x05, 0x02, 0x0b, 0x0c, 0x21, 0x26, 0x2f, 0x28, 0x3d, 0x3a, 0x33, 0x34,
    0x4e, 0x49, 0x40, 0x47, 0x52, 0x55, 0x5c, 0x5b, 0x76, 0x71, 0x78, 0x7f,
    0x6a, 0x6d, 0x64, 0x63, 0x3e, 0x39, 0x30, 0x37, 0x22, 0x25, 0x2c, 0x2b,
    0x06, 0x01, 0x08, 0x0f, 0x1a, 0x1d, 0x14, 0x13, 0xae, 0xa9, 0xa0, 0xa7,
    0xb2, 0xb5, 0xbc, 0xbb, 0x96, 0x91, 0x98, 0x9f, 0x8a, 0x8d, 0x84, 0x83,
    0xde, 0xd9, 0xd0, 0xd7, 0xc2, 0xc5, 0xcc, 0xcb, 0xe6, 0xe1, 0xe8, 0xef,
    0xfa, 0xfd, 0xf4, 0xf3
};

uint8_t
crc8(uint8_t *p, uint8_t len)
{
        uint16_t i;
        uint16_t crc = 0x0;

        while (len--) {
                i = (crc ^ *p++) & 0xFF;
                crc = (crc_table[i] ^ (crc << 8)) & 0xFF;
        }

        return crc & 0xFF;
}

---

可以让编译器为您找出查找表，如下所示：

#include <stdio.h>
#include <stdint.h>

/* * *
 * Just change this define to whatever polynomial is in use
 */
#define  CRC1B(b)       ( (uint8_t)((b)<<1) ^ ((b)&0x80? 0x07 : 0) ) // MS first

/* * *
 * 8+1 entry enum lookup table define
 */
#define  CRC(b)         CRC_##b          // or CRC8B(b)

enum {
    CRC(0x01) = CRC1B(0x80),
    CRC(0x02) = CRC1B(CRC(0x01)),
    CRC(0x04) = CRC1B(CRC(0x02)),
    CRC(0x08) = CRC1B(CRC(0x04)),
    CRC(0x10) = CRC1B(CRC(0x08)),
    CRC(0x20) = CRC1B(CRC(0x10)),
    CRC(0x40) = CRC1B(CRC(0x20)),
    CRC(0x80) = CRC1B(CRC(0x40)),
    // Add 0x03 to optimise in CRCTAB1
    CRC(0x03) = CRC(0x02)^CRC(0x01)
};

/* * *
 * Build a 256 byte CRC constant lookup table, built from from a reduced constant
 * lookup table, namely CRC of each bit, 0x00 to 0x80. These will be defined as
 * enumerations to take it easy on the compiler. This depends on the relation:
 *   CRC(a^b) = CRC(a)^CRC(b)
 * In other words, we can build up each byte CRC as the xor of the CRC of each bit.
 * So CRC(0x05) = CRC(0x04)^CRC(0x01). We include the CRC of 0x03 for a little more
 * optimisation, since CRCTAB1 can use it instead of CRC(0x01)^CRC(0x02), again a
 * little easier on the compiler.
 */

#define  CRCTAB1(ex)    CRC(0x01)ex, CRC(0x02)ex,  CRC(0x03)ex,
#define  CRCTAB2(ex)    CRCTAB1(ex)  CRC(0x04)ex,  CRCTAB1(^CRC(0x04)ex)
#define  CRCTAB3(ex)    CRCTAB2(ex)  CRC(0x08)ex,  CRCTAB2(^CRC(0x08)ex)
#define  CRCTAB4(ex)    CRCTAB3(ex)  CRC(0x10)ex,  CRCTAB3(^CRC(0x10)ex)
#define  CRCTAB5(ex)    CRCTAB4(ex)  CRC(0x20)ex,  CRCTAB4(^CRC(0x20)ex)
#define  CRCTAB6(ex)    CRCTAB5(ex)  CRC(0x40)ex,  CRCTAB5(^CRC(0x40)ex)

/* * *
 * This is the final lookup table. It is rough on the compiler, but generates the
 * required lookup table automagically at compile time.
 */
static const uint8_t crc_table[256] = { 0, CRCTAB6() CRC(0x80), CRCTAB6(^CRC(0x80)) };

uint8_t crc8(uint8_t *p, uint8_t len)
{
    uint8_t crc = 0x0;

    while (len--) {
        crc = crc_table[crc ^ *p++];
    }

    return crc;
}

void main( void )
{
    int i, j;

    printf("static const uint8_t crc_table[] = {");
    for (i = 0; i < 0x10; i++)
    {
        printf("\n   ");
        for (j = 0; j < 0x10; j++)
        {
            printf( " 0x%02x,", crc_table[i*0x10+j] );
        }
    }
    printf("\n};\n\n");
}

---

这适用于我问题中的数据，但数据表中的下一个示例是0xC807->0x48，而此函数返回0x15。您完全正确！我使用计算器精确地计算出数据表中数字中的CRC示例所考虑的内容，并发现命令、地址和数据都是计算的一部分。所以主要的问题是我没有为算法提供足够的数据。你用什么多项式表示来创建这个查找表？