Hello Martin and Miquel,

On 2019/4/12 6:00, Martin Blumenstingl wrote:
Use the recently introduced struct_size macro instead of open-coding
it's logic.
No functional changes.

Signed-off-by: Martin Blumenstingl <martin.blumensti...@googlemail.com>
---
  drivers/mtd/nand/raw/meson_nand.c | 3 +--
  1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/mtd/nand/raw/meson_nand.c 
b/drivers/mtd/nand/raw/meson_nand.c
index cb0b03e36a35..c1a6af57dab5 100644
--- a/drivers/mtd/nand/raw/meson_nand.c
+++ b/drivers/mtd/nand/raw/meson_nand.c
@@ -1242,8 +1242,7 @@ meson_nfc_nand_chip_init(struct device *dev,
                return -EINVAL;
        }
- meson_chip = devm_kzalloc(dev,
-                                 sizeof(*meson_chip) + (nsels * sizeof(u8)),
+       meson_chip = devm_kzalloc(dev, struct_size(meson_chip, sels, nsels),
                                  GFP_KERNEL);
Tested-by:Liang Yang <liang.y...@amlogic.com>
Acked-by: Liang Yang <liang.y...@amlogic.com>
        if (!meson_chip)
                return -ENOMEM;

Reply via email to