diff options
| author | Colin Ian King <colin.king@canonical.com> | 2014-02-11 09:38:53 +0000 |
|---|---|---|
| committer | Lee Jones <lee.jones@linaro.org> | 2014-03-13 22:21:48 +0000 |
| commit | 39a85bcbfc54d602934e2657c146c299d71b27ba (patch) | |
| tree | ae7c658afbc7ec6a6f0a840c01d47c2ccc244df2 | |
| parent | 0414855fdc4a40da05221fc6062cccbc0c30f169 (diff) | |
mfd: omap-usb-tll: Fix cppcheck sizeof warning
Static analysis from cppcheck issued the following warning:
[drivers/mfd/omap-usb-tll.c:255]: (warning) Found calculation
inside sizeof().
The current size calculation is not obvious and is easy to
miscomprehend, so re-work the size of the allocation based
on the size of the struct pointer and quantity to allocate.
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
| -rw-r--r-- | drivers/mfd/omap-usb-tll.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mfd/omap-usb-tll.c b/drivers/mfd/omap-usb-tll.c index 5ee50f779ef..532eacab6b4 100644 --- a/drivers/mfd/omap-usb-tll.c +++ b/drivers/mfd/omap-usb-tll.c @@ -252,7 +252,7 @@ static int usbtll_omap_probe(struct platform_device *pdev) break; } - tll->ch_clk = devm_kzalloc(dev, sizeof(struct clk * [tll->nch]), + tll->ch_clk = devm_kzalloc(dev, sizeof(struct clk *) * tll->nch, GFP_KERNEL); if (!tll->ch_clk) { ret = -ENOMEM; |