Re: [CBUS PATCH 1/1] CBUS: Fix broken compilation

[Carlos Chinea <carlos.chinea@nokia.com>]

Ok, sorry for the noise. I missed Alexander Shishkin patch which took
care of this already.

On Wed, 2010-05-05 at 21:00 +0200, Balbi Felipe (Nokia-D/Helsinki)
wrote:
> hi again,
> 
> 
> On Wed, May 05, 2010 at 05:05:29PM +0200, Chinea Carlos (Nokia-D/Helsinki) wrote:
> >Include explicitly slab.h in:
> >	drivers/cbus/cbus.c
> >	drivers/cbus/retu-headset.c
> >	drivers/cbus/retu-user.c
> >	drivers/cbus/retu-wdt.c
> >	drivers/cbus/tahvo-user.c
> >
> >Compile tested with n8x0_defconfig
> >
> >Signed-off-by: Carlos Chinea <carlos.chinea@nokia.com>
> 
> I see now the problem.
> 
> >---
> > drivers/cbus/cbus.c         |    1 +
> > drivers/cbus/retu-headset.c |    1 +
> > drivers/cbus/retu-user.c    |    1 +
> > drivers/cbus/retu-wdt.c     |    1 +
> > drivers/cbus/tahvo-user.c   |    1 +
> > 5 files changed, 5 insertions(+), 0 deletions(-)
> >
> >diff --git a/drivers/cbus/cbus.c b/drivers/cbus/cbus.c
> >index 00c3c32..2c74d6e 100644
> >--- a/drivers/cbus/cbus.c
> >+++ b/drivers/cbus/cbus.c
> >@@ -30,6 +30,7 @@
> > #include <linux/spinlock.h>
> > #include <linux/gpio.h>
> > #include <linux/platform_device.h>
> >+#include <linux/slab.h>
> 
> this is already included in line 32 of that file.
> 
> >diff --git a/drivers/cbus/retu-headset.c b/drivers/cbus/retu-headset.c
> >index e798bc2..645679c 100644
> >--- a/drivers/cbus/retu-headset.c
> >+++ b/drivers/cbus/retu-headset.c
> >@@ -25,6 +25,7 @@
> > #include <linux/delay.h>
> > #include <linux/input.h>
> > #include <linux/platform_device.h>
> >+#include <linux/slab.h>
> 
> already on line 25, right above #include <linux/delay.h>
> 
> >diff --git a/drivers/cbus/retu-user.c b/drivers/cbus/retu-user.c
> >index 0f35dc5..7bc97d4 100644
> >--- a/drivers/cbus/retu-user.c
> >+++ b/drivers/cbus/retu-user.c
> >@@ -33,6 +33,7 @@
> > #include <linux/spinlock.h>
> > #include <linux/sched.h>
> > #include <linux/mutex.h>
> >+#include <linux/slab.h>
> 
> already on line 26
> 
> >diff --git a/drivers/cbus/retu-wdt.c b/drivers/cbus/retu-wdt.c
> >index 35932dd..fda8b35 100644
> >--- a/drivers/cbus/retu-wdt.c
> >+++ b/drivers/cbus/retu-wdt.c
> >@@ -35,6 +35,7 @@
> > #include <linux/platform_device.h>
> > #include <linux/miscdevice.h>
> > #include <linux/watchdog.h>
> >+#include <linux/slab.h>
> 
> already on line 25
> 
> >diff --git a/drivers/cbus/tahvo-user.c b/drivers/cbus/tahvo-user.c
> >index c0e8daf..b28b015 100644
> >--- a/drivers/cbus/tahvo-user.c
> >+++ b/drivers/cbus/tahvo-user.c
> >@@ -33,6 +33,7 @@
> > #include <linux/spinlock.h>
> > #include <linux/sched.h>
> > #include <linux/mutex.h>
> >+#include <linux/slab.h>
> 
> already on line 26.
> 
> NAK to this patch.
> 
-- 
Carlos Chinea <carlos.chinea@nokia.com>