vanvught
commented
4 years ago The gcc compiler does generate the correct output. This is mostly a cleanup action. It would make the code easier to maintain not having unneeded error's generated by cppcheck
pi@nuc-i5:~/workspace$ find . -name *.h | xargs grep '<< 31'
./lib-h3/src/h3_spi_internal.h:#define GC_SRST (1 << 31) ///< Soft Reset
./lib-h3/src/h3_spi_internal.h:#define TC_XCH (1 << 31) ///< Initiate transfer
./lib-h3/src/h3_spi_internal.h:#define FC_TX_RST (1 << 31) ///< TX FIFO Reset
./lib-h3/src/h3_spi_internal.h:#define FS_TB_WR (1 << 31) ///< TX FIFO Write Buffer Write Enable
./lib-h3/include/h3_ccu.h:#define CCU_THS_CLK_SCLK_GATING (1 << 31)pi@nuc-i5:/development/workspace$ find . -name *.c | xargs grep '<< 31)'
./lib-spiflash/src/h3/spi_flash.c: H3_CCU->SPI0_CLK = (1 << 31) | (0x01 << 24); // Clock is ON, P0
./lib-h3/device/emac/emac.c:#define TX_CTL0_TX_EN (1 << 31)
./lib-h3/device/emac/emac.c:#define RX_CTL0_RX_EN (1 << 31)
./lib-h3/device/emac/emac.c: desc_p->status = (1 << 31);
./lib-h3/device/emac/emac.c: desc_p->status = (1 << 31);
./lib-h3/device/emac/emac.c: if (!(status & (1 << 31))) {
./lib-h3/device/emac/emac.c: desc_p->st |= (1 << 31);
./lib-h3/device/emac/emac.c: desc_p->status = (1 << 31);
./lib-h3/device/emac/emac.c: value |= (1 << 31);/* mandatory */
./lib-h3/device/emac/emac.c: desc_p->status |= (1 << 31);
./lib-h3/device/mmc/mmc.c: cmd.cmdarg = (mode << 31) | 0xffffff;
./lib-h3/device/mmc/mmc_bsp.c:#define SDMMC_CLK_SCLK_GATING (1 << 31) ///< 1 = Clock is ON. SCLK = Clock source/Divider N/ Divider M
./lib-h3/device/mmc/mmc_bsp.c:#define GCTL_FIFO_AC_MODE_AHB (1 << 31) ///< FIFO Access Mode AHB bus
./lib-h3/device/mmc/mmc_bsp.c:#define CKC_CCLK_MASK_DATA0 (1 << 31) ///< Mask Data0 when update clock
./lib-h3/device/mmc/mmc_bsp.c:#define CMD_CMD_LOAD (1 << 31) ///< Start Command
./lib-h3/src/h3_spi.c: H3_CCU->SPI0_CLK = (1 << 31) | (0x01 << 24); // Clock is ON, P0
./lib-h3/src/h3_spi.c: H3_CCU->SPI1_CLK = (1 << 31) | (0x01 << 24); // Clock is ON, P0
./lib-h3/src/h3_spi.c: H3_CCU->SPI0_CLK &= ~(1 << 31); // Clock is OFF
./lib-h3/src/h3_spi.c: H3_CCU->SPI1_CLK &= ~(1 << 31); // Clock is OFF
./lib-h3/src/h3_cpu.c:#define PLL_ENABLE (1 << 31)
./lib-h3/src/h3_codec.c:#define SCLK_1X_GATING (1 << 31)
./lib-h3/src/h3_codec.c:#define PLL_ENABLE (1 << 31)
./lib-h3/src/h3_timer.c:#define SCLK_GATING (1 << 31)
./lib-h3/src/h3_ccu.c:#define PLL_ENABLE (1 << 31) ///< The PLL output is (24MHz*N*K)/M
cppcheck --includes-file=cppcheck.includes-file --file-list=cppcheck.file-list -DDEBUG=1 -DH3=1 -DORANGE_PI=1 2> ./cpp_check.txt