i2c: Improve size determinations
authorMarkus Elfring <elfring@users.sourceforge.net>
Fri, 2 Feb 2018 13:50:09 +0000 (14:50 +0100)
committerWolfram Sang <wsa@kernel.org>
Wed, 7 Jun 2023 12:12:11 +0000 (14:12 +0200)
Replace the specification of a data structure by a pointer dereference
as the parameter for the operator "sizeof" to make the corresponding
size determination a bit safer according to the Linux coding style
convention.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Wolfram Sang <wsa@kernel.org>
drivers/i2c/busses/i2c-nomadik.c
drivers/i2c/busses/i2c-sh7760.c

index 05eaae5..5004b9d 100644 (file)
@@ -970,7 +970,7 @@ static int nmk_i2c_probe(struct amba_device *adev, const struct amba_id *id)
        struct i2c_vendor_data *vendor = id->data;
        u32 max_fifo_threshold = (vendor->fifodepth / 2) - 1;
 
-       dev = devm_kzalloc(&adev->dev, sizeof(struct nmk_i2c_dev), GFP_KERNEL);
+       dev = devm_kzalloc(&adev->dev, sizeof(*dev), GFP_KERNEL);
        if (!dev) {
                ret = -ENOMEM;
                goto err_no_mem;
index 5a72fa7..1ad2a26 100644 (file)
@@ -443,7 +443,7 @@ static int sh7760_i2c_probe(struct platform_device *pdev)
                goto out0;
        }
 
-       id = kzalloc(sizeof(struct cami2c), GFP_KERNEL);
+       id = kzalloc(sizeof(*id), GFP_KERNEL);
        if (!id) {
                ret = -ENOMEM;
                goto out0;