Re: [GIT PATCHES] V4L/DVB fixes for 2.6.26

On Wed, May 14, 2008 at 11:49:10AM -0300, Mauro Carvalho Chehab wrote:
...
PS.: There are yet a number of other Kconfig potential breakages at V4L/DVB. I'm
currently working on fixing those issues. Basically, what users do is to select
I2C, DVB and V4L as module. This works fine, but more complex scenarios where
you mix 'M' and 'Y' inside the subsystem generally cause compilation breakage.
Those scenarios are more theorical, since there's not much practical sense on
having a DVB driver foo as module, and V4L driver bar as in-kernel. However,
the better is to not allow compilation of the scenarios that don't work.

The main trouble at drivers/media Kbuild is that several rules there assumed that
"select" would check the "depends on" dependencies of the selected drivers.
However, this feature doesn't exist at the current Kbuild implementation. Even
if implemented, I suspect that this will generate circular dependency errors on
some cases.
...

The basic problem is that drivers/media/ does the most fancy kconfig
stuff in the kernel since it tries to both have very fine grained
dependencies and offer a usable kconfig UI to the user, which results
in very complicated dependencies.

We are not getting this solved by any changes in the kconfig
implementation.

Thinking about reasonable ways to reduce the problem space:

Where could we reduce the complexity without big disadvantages?

Could we e.g. let VIDEO_DEV select I2C which would remove all the
fiddling with I2C dependencies (which is a bigger part of recent
problems)?

I can make a patch for it after this pull went into Linus' tree if it is
considered an acceptable option.

Cheers,
Mauro.
...

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Relevant Pages

  • Re: [patch] MTD: fix DOC2000/2001/2001PLUS build error
    ... DiskOnChip driver. ... yeah, i mis-analyzed the point of breakage - and my Kconfig hack simply ... I come across this frequently -- and I just look at the Kconfig file ... to see the dependencies of the option I want to enable. ...
    (Linux-Kernel)
  • Re: kms in defconfig
    ... Linus Torvalds wrote: ... i could imagine a kconfig knob to specify an optional ... suplies and maintains his script ... the dependencies before to enable/disable drivers), ...
    (Linux-Kernel)
  • Re: Ingo, no more kconfig patches
    ... and the .config but not quick kconfig patches that are often wrong ... || driver's dependencies with the current tools in a way that doesnt ... and that's exactly what happens with Roman's patch: ... select's) is worked around in the driver space: ...
    (Linux-Kernel)
  • Re: [GIT PATCHES] V4L/DVB fixes for 2.6.26
    ... There are yet a number of other Kconfig potential breakages at V4L/DVB. ... having a DVB driver foo as module, and V4L driver bar as in-kernel. ... dependencies and offer a usable kconfig UI to the user, ... It would be nice if you could help on fixing those issues. ...
    (Linux-Kernel)
  • Re: build #337 failed for 2.6.24-rc1-gb1d08ac In function `usbnet_set_settings:
    ... Notice how only the ASIX kconfig depended on NET_ETHERNET... ... reverse dependencies didn't capture the complete dependency ... I'm all for cleanups that make things consistent. ... There had been need of rain for many days. ...
    (Linux-Kernel)