ASoC: Intel: fix array_size.cocci warnings
authorkbuild test robot <fengguang.wu@intel.com>
Sun, 12 Apr 2015 18:16:21 +0000 (02:16 +0800)
committerMark Brown <broonie@kernel.org>
Sun, 12 Apr 2015 18:47:15 +0000 (19:47 +0100)
sound/soc/intel/haswell/sst-haswell-ipc.c:646:28-29: WARNING: Use ARRAY_SIZE

 Use ARRAY_SIZE instead of dividing sizeof array with sizeof an element

Semantic patch information:
 This makes an effort to find cases where ARRAY_SIZE can be used such as
 where there is a division of sizeof the array by the sizeof its first
 element or by any indexed element or the element type. It replaces the
 division of the two sizeofs by ARRAY_SIZE.

Generated by: scripts/coccinelle/misc/array_size.cocci

CC: Jie Yang <yang.jie@intel.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/intel/haswell/sst-haswell-ipc.c

index d75f09e..344a1e9 100644 (file)
@@ -466,7 +466,7 @@ static void hsw_fw_ready(struct sst_hsw *hsw, u32 header)
                /* log the FW version info got from the mailbox here. */
                memcpy(fw_info, fw_ready.fw_info, fw_ready.fw_info_size);
                pinfo = &fw_info[0];
-               for (i = 0; i < sizeof(tmp) / sizeof(char *); i++)
+               for (i = 0; i < ARRAY_SIZE(tmp); i++)
                        tmp[i] = strsep(&pinfo, " ");
                dev_info(hsw->dev, "FW loaded, mailbox readback FW info: type %s, - "
                        "version: %s.%s, build %s, source commit id: %s\n",