Re: [v4l-dvb-maintainer] -mm: cx88-blackbird.c: unused code re-added

From: Michael Krufky
Date: Fri Nov 17 2006 - 08:54:17 EST


Adrian Bunk wrote:

On Tue, Nov 14, 2006 at 01:41:25AM -0800, Andrew Morton wrote:


...
Changes since 2.6.19-rc5-mm2:
...
git-dvb.patch
...
git trees
...



Why does this patch re-add the still unused cx88_ioctl_hook and cx88_ioctl_translator in drivers/media/video/cx88/cx88-blackbird.c I removed a few months ago?


Adrian,

I have added those hooks back into the driver, because they are needed for the cx88-ivtv ioctl emulation layer. However, I have instructed Mauro not to merge that patch upstream -- it is meant to stay at our mercurial repository only, since the vanilla kernel does not need to use the cx88-ivtv ioctl emulation layer.

It looks like somehow these got into -mm ... I think that is because Andrew's scripts pull from Mauro's devel branch in his git tree.

The problem commit is: de513acdbafc48c52017255fee02af727dcfcc32

It appears that Mauro has folded my patch into Steve's patch, which he should not have done. :-( What a mess!

I will send a new patch to Mauro later on today to be applied to his git devel branch, which will remove those hooks again.

As soon as ivtv gets merged into our tree, that's when we will be able to remove cx88-ivtv alltogether, as there will no longer be any need to support the ivtv API.

regards,

Mike Krufky

-
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/