login
Header Space

 
 

Re: Linux 2.6.26-rc1

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Robin Holt <holt@...>
Cc: Linus Torvalds <torvalds@...>, Linux Kernel Mailing List <linux-kernel@...>, Mauro Carvalho Chehab <mchehab@...>
Date: Sunday, May 4, 2008 - 12:33 pm

On Sunday 04 May 2008 17:09:04 Robin Holt wrote:

a) He's already on CC
b) No config I have posted was invalid
c) There are two (possibly three) entirely separate issues

You've got the wrong end of the stick here. I'm not saying V4L isn't required 
for MEDIA_TUNER, but MEDIA_TUNER should not be a dependency of V4L and so 
should NOT be unconditionally enabled.

The first issue is obviously the build error, which can be resolved by adding 
a dep on I2C to MEDIA_TUNER, but given the recent churn there I'll leave that 
to Mauro.

The second issue is the fact that the MEDIA_TUNER option is transparent and 
not user selectable, building code that is probably not required in some 
kernel configurations (tuner code depends on V4L, but surely not the other 
way around.. MEDIA_TUNER should be able to be n when VIDEO_DEV is y).

The final issue is more of a matter of personal taste, which refers to the 
MEDIA_TUNER_CUSTOMIZE option, something that I think should be inverted and 
turned into a menu like most other device driver sections. So rather than 
MEDIA_TUNER_CUSTOMIZE, why not just have MEDIA_TUNER be visible to the user, 
and if Y, let them select tuners, and if N, no tuners are built.

-- 
Cheers,
Alistair.

137/1 Warrender Park Road, Edinburgh, UK.
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Linux 2.6.26-rc1, Linus Torvalds, (Sat May 3, 3:38 pm)
Re: Scheduling problems ? [ Was: Linux 2.6.26-rc1 ], Ingo Molnar, (Tue May 6, 9:11 am)
Re: Linux 2.6.26-rc1, Andreas Mohr, (Mon May 5, 1:39 am)
Re: Linux 2.6.26-rc1, Ingo Molnar, (Mon May 5, 3:30 am)
Re: Linux 2.6.26-rc1, Stephen Rothwell, (Mon May 5, 7:51 pm)
Re: Linux 2.6.26-rc1, Ingo Molnar, (Mon May 5, 8:17 pm)
Re: Linux 2.6.26-rc1, Stephen Rothwell, (Tue May 6, 1:48 am)
Re: Linux 2.6.26-rc1, David Miller, (Mon May 5, 7:55 pm)
Re: Linux 2.6.26-rc1, Stephen Rothwell, (Tue May 6, 1:35 am)
Re: Linux 2.6.26-rc1, David Miller, (Tue May 6, 2:56 am)
Re: Linux 2.6.26-rc1, Stephen Rothwell, (Tue May 6, 3:26 am)
Re: Linux 2.6.26-rc1, Ingo Molnar, (Mon May 5, 8:21 pm)
Re: Linux 2.6.26-rc1, Andrew Morton, (Mon May 5, 8:16 pm)
Re: Linux 2.6.26-rc1, Stephen Rothwell, (Tue May 6, 1:57 am)
Re: Linux 2.6.26-rc1, Mauro Carvalho Chehab, (Mon May 5, 4:14 pm)
Re: Linux 2.6.26-rc1, Ingo Molnar, (Tue May 6, 9:07 am)
Re: Linux 2.6.26-rc1, Ingo Molnar, (Wed May 7, 4:35 am)
Re: Linux 2.6.26-rc1, Jan Engelhardt, (Sun May 4, 2:40 pm)
[patch] sysfs: build fix, Ingo Molnar, (Sun May 4, 3:29 am)
Re: Linux 2.6.26-rc1, Alistair John Strachan, (Sat May 3, 5:47 pm)
Re: Linux 2.6.26-rc1, Alistair John Strachan, (Sat May 3, 5:53 pm)
Re: Linux 2.6.26-rc1, Robin Holt, (Sun May 4, 8:30 am)
Re: Linux 2.6.26-rc1, Alistair John Strachan, (Sun May 4, 9:56 am)
Re: Linux 2.6.26-rc1, Robin Holt, (Sun May 4, 12:09 pm)
Re: Linux 2.6.26-rc1, Mauro Carvalho Chehab, (Mon May 5, 5:04 pm)
Re: Linux 2.6.26-rc1, Alistair John Strachan, (Tue May 6, 4:02 am)
Re: Linux 2.6.26-rc1, Mauro Carvalho Chehab, (Tue May 6, 6:31 am)
Re: Linux 2.6.26-rc1, Stefan Richter, (Tue May 6, 10:34 am)
Re: Linux 2.6.26-rc1, Mauro Carvalho Chehab, (Tue May 6, 3:13 pm)
Re: Linux 2.6.26-rc1, Stefan Richter, (Tue May 6, 3:44 pm)
Re: Linux 2.6.26-rc1, Mauro Carvalho Chehab, (Tue May 6, 4:06 pm)
Re: Linux 2.6.26-rc1, Stefan Richter, (Tue May 6, 4:47 pm)
Re: Linux 2.6.26-rc1, Stefan Richter, (Tue May 6, 7:48 pm)
Re: Linux 2.6.26-rc1, Mauro Carvalho Chehab, (Tue May 6, 6:39 pm)
Re: Linux 2.6.26-rc1, Stefan Richter, (Tue May 6, 7:36 pm)
Re: Linux 2.6.26-rc1, Mauro Carvalho Chehab, (Thu May 8, 8:48 am)
Re: Linux 2.6.26-rc1, Stefan Richter, (Thu May 8, 10:11 am)
Re: Linux 2.6.26-rc1, Stefan Richter, (Tue May 6, 10:54 am)
Re: Linux 2.6.26-rc1, Alistair John Strachan, (Sun May 4, 12:33 pm)
speck-geostationary