diff options
author | Fabio Estevam <fabio.estevam@freescale.com> | 2014-11-06 21:44:34 -0500 |
---|---|---|
committer | Wolfram Sang <wsa@the-dreams.de> | 2014-11-10 03:34:54 -0500 |
commit | d4ffeecb3bb73a7fe3aa58a013b1b036996a75db (patch) | |
tree | 5c71180e969c07d34df68cbcf73d196a68a13106 /drivers/i2c/busses/i2c-imx.c | |
parent | 206c5f60a3d902bc4b56dab2de3e88de5eb06108 (diff) |
i2c: imx+mxs: Use the preferred form for passing a size of a struct
According to Documentation/CodingStyle - Chapter 14:
"The preferred form for passing a size of a struct is the following:
p = kmalloc(sizeof(*p), ...);
The alternative form where struct name is spelled out hurts readability and
introduces an opportunity for a bug when the pointer variable type is changed
but the corresponding sizeof that is passed to a memory allocator is not."
So do it as recommeded.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
Diffstat (limited to 'drivers/i2c/busses/i2c-imx.c')
-rw-r--r-- | drivers/i2c/busses/i2c-imx.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/i2c/busses/i2c-imx.c b/drivers/i2c/busses/i2c-imx.c index e9fb7cf78612..3cbc09262326 100644 --- a/drivers/i2c/busses/i2c-imx.c +++ b/drivers/i2c/busses/i2c-imx.c | |||
@@ -668,8 +668,7 @@ static int i2c_imx_probe(struct platform_device *pdev) | |||
668 | if (IS_ERR(base)) | 668 | if (IS_ERR(base)) |
669 | return PTR_ERR(base); | 669 | return PTR_ERR(base); |
670 | 670 | ||
671 | i2c_imx = devm_kzalloc(&pdev->dev, sizeof(struct imx_i2c_struct), | 671 | i2c_imx = devm_kzalloc(&pdev->dev, sizeof(*i2c_imx), GFP_KERNEL); |
672 | GFP_KERNEL); | ||
673 | if (!i2c_imx) | 672 | if (!i2c_imx) |
674 | return -ENOMEM; | 673 | return -ENOMEM; |
675 | 674 | ||