Messages in this thread | | | Date | Mon, 21 Feb 2011 09:36:40 +0200 | From | Felipe Balbi <> | Subject | Re: [PATCH resend] video: omap24xxcam: Fix compilation |
| |
Hi,
On Sat, Feb 19, 2011 at 06:04:58PM +0200, David Cohen wrote: > > I have to disagree. The fundamental problem is the circular dependency > > between those two files: > > > > sched.h uses wait_queue_head_t defined in wait.h > > wait.h uses TASK_* defined in sched.h > > > > So, IMO the real fix would be clear out the circular dependency. Maybe > > introducing <linux/task.h> to define those TASK_* symbols and include > > that on sched.h and wait.h > > > > Just dig a quick and dirty to try it out and works like a charm > > We have 2 problems: > - omap24xxcam compilation broken > - circular dependency between sched.h and wait.h > > To fix the broken compilation we can do what the rest of the kernel is > doing, which is to include sched.h. > Then, the circular dependency is fixed by some different approach > which would probably change *all* current usage of TASK_*.
considering that 1 is caused by 2 I would fix 2.
> IMO, there's no need to create a dependency between those issues.
There's no dependency between them, it's just that the root cause for this problem is a circular dependency between wait.h and sched.h
-- balbi
| |