tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git
staging-testing
head: e7b6875944e61bec586ede80b95727cd55381e1e
commit: e7b6875944e61bec586ede80b95727cd55381e1e [41/41] Staging: vt6655:
Replace camel case variable names.
If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <l...@intel.com>
cppcheck warnings: (new ones prefixed by >>)
>> drivers/staging/vt6655/baseband.c:1711:15: warning: The scope of the
>> variable 'preamble' can be reduced. [variableScope]
unsigned int preamble;
^
>> drivers/staging/vt6655/baseband.c:1912:16: warning: The scope of the
>> variable 'by_value' can be reduced. [variableScope]
>> unsigned char by_value;
^
drivers/staging/vt6655/baseband.c:1955:16: warning: The scope of the
variable 'by_value' can be reduced. [variableScope]
>> unsigned char by_value;
^
vim +/preamble +1711 drivers/staging/vt6655/baseband.c
1689
1690 /*--------------------- Export Variables --------------------------*/
1691 /*
1692 * Description: Calculate data frame transmitting time
1693 *
1694 * Parameters:
1695 * In:
1696 * by_preamble_type - Preamble Type
1697 * by_pkt_type - PK_TYPE_11A, PK_TYPE_11B, PK_TYPE_11GB,
PK_TYPE_11GA
1698 * cb_frame_length - Baseband Type
1699 * tx_rate - Tx Rate
1700 * Out:
1701 *
1702 * Return Value: FrameTime
1703 *
1704 */
1705 unsigned int bb_get_frame_time(unsigned char by_preamble_type,
1706 unsigned char by_pkt_type,
1707 unsigned int cb_frame_length,
1708 unsigned short tx_rate)
1709 {
1710 unsigned int frame_time;
> 1711 unsigned int preamble;
1712 unsigned int tmp;
1713 unsigned int rate_idx = (unsigned int)tx_rate;
1714 unsigned int rate = 0;
1715
1716 if (rate_idx > RATE_54M)
1717 return 0;
1718
1719 rate = (unsigned int)awcFrameTime[rate_idx];
1720
1721 if (rate_idx <= 3) { /* CCK mode */
1722 if (by_preamble_type == 1) /* Short */
1723 preamble = 96;
1724 else
1725 preamble = 192;
1726 frame_time = (cb_frame_length * 80) / rate; /* ????? */
1727 tmp = (frame_time * rate) / 80;
1728 if (cb_frame_length != tmp)
1729 frame_time++;
1730
1731 return preamble + frame_time;
1732 }
1733 frame_time = (cb_frame_length * 8 + 22) / rate; /* ???????? */
1734 tmp = ((frame_time * rate) - 22) / 8;
1735 if (cb_frame_length != tmp)
1736 frame_time++;
1737
1738 frame_time = frame_time * 4; /* ??????? */
1739 if (by_pkt_type != PK_TYPE_11A)
1740 frame_time += 6; /* ?????? */
1741
1742 return 20 + frame_time; /* ?????? */
1743 }
1744
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
