> -----Original Message----- > From: Dan Carpenter [mailto:error27@xxxxxxxxx] > Sent: Thursday, October 21, 2010 9:24 PM > To: Mauro Carvalho Chehab > Cc: Kyungmin Park; Sylwester Nawrocki; Marek Szyprowski; Pawel Osciak; > linux-media@xxxxxxxxxxxxxxx; kernel-janitors@xxxxxxxxxxxxxxx > Subject: [patch 2/3] V4L/DVB: s5p-fimc: make it compile > > The work_queue was partially removed in f93000ac11: "[media] s5p-fimc: > mem2mem driver refactoring and cleanup" but this bit was missed. Also > we need to include sched.h otherwise the compile fails with: > > drivers/media/video/s5p-fimc/fimc-core.c: > In function âfimc_capture_handlerâ: > drivers/media/video/s5p-fimc/fimc-core.c:286: > error: âTASK_NORMALâ undeclared (first use in this function) > > Signed-off-by: Dan Carpenter <error27@xxxxxxxxx> > --- > Compile tested only. :P > > diff --git a/drivers/media/video/s5p-fimc/fimc-core.h > b/drivers/media/video/s5p-fimc/fimc-core.h > index e3a7c6a..1c1437c 100644 > --- a/drivers/media/video/s5p-fimc/fimc-core.h > +++ b/drivers/media/video/s5p-fimc/fimc-core.h > @@ -14,6 +14,7 @@ > /*#define DEBUG*/ > > #include <linux/types.h> > +#include <linux/sched.h> > #include <media/videobuf-core.h> > #include <media/v4l2-device.h> > #include <media/v4l2-mem2mem.h> > diff --git a/drivers/media/video/s5p-fimc/fimc-core.c > b/drivers/media/video/s5p-fimc/fimc-core.c > index 8335045..cf9bc8e 100644 > --- a/drivers/media/video/s5p-fimc/fimc-core.c > +++ b/drivers/media/video/s5p-fimc/fimc-core.c > @@ -1593,12 +1593,6 @@ static int fimc_probe(struct platform_device > *pdev) > goto err_clk; > } > > - fimc->work_queue = create_workqueue(dev_name(&fimc->pdev->dev)); > - if (!fimc->work_queue) { > - ret = -ENOMEM; > - goto err_irq; > - } > - This code is properly removed in my original patch. But it has been added again during a merge conflict solving. Unfortunately I cannot identify the merge commit today in linux-next. As for sched.h, it needs a separate patch so I could handle it and add you as reported by it is OK. Regards, Sylwester > ret = fimc_register_m2m_device(fimc); > if (ret) > goto err_irq; -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
