Skip to contentSkip to conversation

pxa-camera: build error 2.6.32-rc4

PATCH In review Jonathan Cameron 2 participants 4 messages Oct 12, 2009 Media (V4L/DVB)

lore
Share & tools
lore
#0 lore raw

Error was

drivers/media/video/pxa_camera.c: In function 'pxa_camera_wakeup':
drivers/media/video/pxa_camera.c:683: error: 'TASK_NORMAL' undeclared (first use in this function) drivers/media/video/pxa_camera.c:683: error: (Each undeclared identifier is reported only once drivers/media/video/pxa_camera.c:683: error: for each function it appears in.)
CC [M] drivers/pcmcia/soc_common.o

Line in question is

        wake_up(&vb->done);

in pxa_camera_wakeup.

Looks like issue is lack of inclusion of sched.h.

Right now I'm having trouble tracking down why this became and issue since 2.6.32-rc3.

Might be related to
Staging: comedi: Add include of <linux/sched.h> to fix build which is down to removal of sched.h from poll.h
commmit a99bbaf5ee6bad1aca0c88ea65ec6e5373e86184
Looks like media/v4l2-dev.h and others include poll.h so I'm guessing we were original getting it from there.

I'm happy to post a patch but was wondering if anyone else has seen this or has tracked down exactly what changed, not to mention if this is a more general problem? (or for that matter already fixed and I just missed it.)

Thanks

Jonathan

#1 lore raw
Quoted 24 lines · Jonathan Cameron

Error was

drivers/media/video/pxa_camera.c: In function 'pxa_camera_wakeup':
drivers/media/video/pxa_camera.c:683: error: 'TASK_NORMAL' undeclared (first use in this function) drivers/media/video/pxa_camera.c:683: error: (Each undeclared identifier is reported only once drivers/media/video/pxa_camera.c:683: error: for each function it appears in.)
CC [M] drivers/pcmcia/soc_common.o

Line in question is

        wake_up(&vb->done);

in pxa_camera_wakeup.

Looks like issue is lack of inclusion of sched.h.

Right now I'm having trouble tracking down why this became and issue since 2.6.32-rc3.

Might be related to
Staging: comedi: Add include of <linux/sched.h> to fix build which is down to removal of sched.h from poll.h
commmit a99bbaf5ee6bad1aca0c88ea65ec6e5373e86184
Looks like media/v4l2-dev.h and others include poll.h so I'm guessing we were original getting it from there.

Quoted 3 lines

I'm happy to post a patch but was wondering if anyone else has seen this or has tracked down exactly what changed, not to mention if this is a more general problem? (or for that matter already fixed and I just missed it.)

Just read lkml, Ingo Molnar has posted a whole set of patches (in reply to Linus' 2.6.32-rc4 thread) dealing with this issue in a number of other places. This one isn't there however.

Jonathan

#2 lore raw

PATCH 1/1 pxa-camera: Fix missing sched.h

linux/sched.h include was removed form linux/poll.h by commmit a99bbaf5ee6bad1aca0c88ea65ec6e5373e86184

Required for wakeup call.

---
 drivers/media/video/pxa_camera.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
drivers/media/video/pxa_camera.c +1 -0
index 6952e96..5d01dcf 100644
@@ -26,6 +26,7 @@
 #include <linux/device.h>
 #include <linux/platform_device.h>
 #include <linux/clk.h>
+#include <linux/sched.h>
 
 #include <media/v4l2-common.h>
 #include <media/v4l2-dev.h>
Signature · 1 line
1.6.3.3
Acked-by on v1 1/1
#3 lore raw

Re: [PATCH] pxa-camera: Fix missing sched.h

Quoted 6 lines · Jonathan Cameron

linux/sched.h include was removed form linux/poll.h by commmit a99bbaf5ee6bad1aca0c88ea65ec6e5373e86184

Required for wakeup call.

Mauro, can you take it from here with my ack for -rc5 or do I have to pull it through my tree?

Thanks
Guennadi

Quoted 19 lines
---
 drivers/media/video/pxa_camera.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
drivers/media/video/pxa_camera.c +1 -0
index 6952e96..5d01dcf 100644
@@ -26,6 +26,7 @@
 #include <linux/device.h>
 #include <linux/platform_device.h>
 #include <linux/clk.h>
+#include <linux/sched.h>
 
 #include <media/v4l2-common.h>
 #include <media/v4l2-dev.h>
Signature · 1 line
1.6.3.3

---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/