Skip to contentSkip to conversation

staging: iio added missing "()" in kmalloc

PATCH In review anish 2 participants 2 messages Jun 6, 2011

Subsystems (3)
Staging IIO Staging - industrial io
lore
Share & tools
lore

anish

author
#0 lore raw

PATCH 2/2 staging: iio added missing "()" in kmalloc

From: anish kumar <[email protected]>

"()" was missing from kmalloc,so added it.

Signed-off-by anish kumar <[email protected]>
---
 drivers/staging/iio/industrialio-core.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
drivers/staging/iio/industrialio-core.c +1 -1
index 3a82414..5251332 100644
@@ -136,7 +136,7 @@ static irqreturn_t iio_interrupt_handler(int irq, void *_int_info)
 
 static struct iio_interrupt *iio_allocate_interrupt(void)
 {
-	struct iio_interrupt *i = kmalloc(sizeof *i, GFP_KERNEL);
+	struct iio_interrupt *i = kmalloc(sizeof(*i), GFP_KERNEL);
 	if (i) {
 		spin_lock_init(&i->ev_list_lock);
 		INIT_LIST_HEAD(&i->ev_list);
Signature · 1 line
1.7.0.4
#1 lore raw

Hi,

Quoted 4 lines · anish

From: anish kumar <[email protected]>

"()" was missing from kmalloc,so added it.

This is not required per-se, is it fixing a bug of some sort ?

- Arnaud

Quoted 31 lines
Signed-off-by anish kumar <[email protected]>

---
 drivers/staging/iio/industrialio-core.c |    2 +-

 1 files changed, 1 insertions(+), 1 deletions(-)
drivers/staging/iio/industrialio-core.c +1 -2
index 3a82414..5251332 100644
@@ -136,7 +136,7 @@ static irqreturn_t iio_interrupt_handler(int irq, void *_int_info)
 
  static struct iio_interrupt *iio_allocate_interrupt(void)
  {
-       struct iio_interrupt *i = kmalloc(sizeof *i, GFP_KERNEL);
+       struct iio_interrupt *i = kmalloc(sizeof(*i), GFP_KERNEL);
        if (i) {
                spin_lock_init(&i->ev_list_lock);
                INIT_LIST_HEAD(&i->ev_list);
--

1.7.0.4

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/