0

さまざまなファイルでこれらの警告が表示されます。私はそれを解決しようとしましたが、すべて（6つのシナリオすべて）に対して同じ警告が表示されます。

#define CMD_READ_ARRAY_FAST  0x0bU
uint64 page_size;
uint32 offset;
uint64 page_addr;
uint64 sector_size;
uint8_t cmd[5];
size_t len;
size_t actual;

1) page_addr = (uint64) (offset / page_size);

2)  cmd[0] = CMD_READ_ARRAY_FAST;
    cmd[1] = (uint8_t) (page_addr >> 8U);
    cmd[2] = (uint8_t) page_addr;
    cmd[3] = (uint8_t) (offset % page_size);
    cmd[4] = 0x00U;

3)page_addr = (uint64) (offset / page_size);
4)byte_addr = (uint64) (offset % page_size);
5)if ((offset % sector_size) || (len % sector_size)) 
6)cmd[1] = (uint8_t) ((offset / sector_size) + actual);

その理由を教えてください。

4

2 に答える 2

0

警告は、同じ式に異なる整数型を混在させ、暗黙のキャストに依存することに起因しています。MISRAであろうとなかろうと、暗黙のキャストは常に悪いものです。

次の修正により、すべての警告がクリアされることが期待されます。

1)

page_addr = (uint64_t)offset / page_size;

2) 3 列目以外はすべて問題ないようです。

cmd[3] = (uint8_t) ((uint64_t)offset % page_size);

3) 操作の前に uint32_t を uint64_t に再度キャストします。

4) 操作の前に uint32_t を uint64_t に再度キャストします。

5) ここにいくつかの MISRA 違反があります。長くて読めないものではなく、数行で書き直すのが最善です。例えば：

bool cond1 = ((uint64_t)offset % sector_size) != 0;
bool cond2 = ((uint64_t)len % sector_size) != 0;

if (cond1 || cond2)

6) これも、読みやすいままにするために、おそらく数行にする必要があります。

uint64_t tmp = (uint64_t)offset / sector_size;
cmd[1] = (uint8_t) (tmp + (uint64_t)actual);
于 2014-01-28T17:24:19.477 に答える
0

この方法で上記の問題を解決しました：

#define CMD_READ_ARRAY_FAST  0x0bU
uint64 page_size;
uint32 offset;
uint64 page_addr;
uint64 sector_size;
uint8_t cmd[5];
size_t len;
size_t actual;

1) page_addr = (uint64) (offset / page_size);
Ans: (uint64) ((uint64)offset / page_size);

2)  cmd[0] = CMD_READ_ARRAY_FAST;
    cmd[1] = (uint8_t) (page_addr >> 8U);
    cmd[2] = (uint8_t) page_addr;
    cmd[3] = (uint8_t) (offset % page_size);
    cmd[4] = 0x00U;

Ans:cmd[0] = CMD_READ_ARRAY_FAST;
    cmd[1] = (uint8_t) (page_addr >> 8U);
    cmd[2] = (uint8_t) page_addr;
    cmd[3] = (uint8_t) ((uint8_t)offset % page_size);
    cmd[4] = 0x00U;


3)page_addr = (uint64) (offset / page_size);
Ans: page_addr = (uint64) ((uint64)offset / page_size);

4)byte_addr = (uint64) (offset % page_size);
Ans: byte_addr = (uint64) ((uint64)offset % page_size);

5)if ((offset % sector_size) || (len % sector_size)) 
Ans: if (((uint64)offset % sector_size) || ((uint64)len % sector_size)) {

6)cmd[1] = (uint8_t) ((offset / sector_size) + actual);
Ans: cmd[1] = (uint8_t) (((uint8_t)offset / (uint8_t)sector_size) + (uint8_t)actual);
于 2014-01-24T09:29:47.637 に答える