Message ID | 1414162887-14279-3-git-send-email-festevam@gmail.com (mailing list archive) |
---|---|
State | Accepted |
Commit | cea82d8af3986508a05949cfbb6ad8e99ffc15eb |
Headers | show |
On Fri, Oct 24, 2014 at 01:01:27PM -0200, Fabio Estevam wrote: > From: Fabio Estevam <fabio.estevam@freescale.com> > > 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. > > Cc: Charles Keepax <ckeepax@opensource.wolfsonmicro.com> > Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com> > --- Acked-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com> Thanks, Charles
diff --git a/sound/soc/codecs/wm8731.c b/sound/soc/codecs/wm8731.c index eebb328..2c9f2a7 100644 --- a/sound/soc/codecs/wm8731.c +++ b/sound/soc/codecs/wm8731.c @@ -680,8 +680,7 @@ static int wm8731_spi_probe(struct spi_device *spi) struct wm8731_priv *wm8731; int ret; - wm8731 = devm_kzalloc(&spi->dev, sizeof(struct wm8731_priv), - GFP_KERNEL); + wm8731 = devm_kzalloc(&spi->dev, sizeof(*wm8731), GFP_KERNEL); if (wm8731 == NULL) return -ENOMEM;