Em 04-11-2011 08:27, Hans Verkuil escreveu: > On Friday 04 November 2011 11:19:37 Mauro Carvalho Chehab wrote: >> Em 04-11-2011 08:13, Mauro Carvalho Chehab escreveu: >>> Em 04-11-2011 07:39, Hans Verkuil escreveu: >>>> Mauro, >>>> >>>> This fixes two compilation problems when using the media_build system. >>>> >>>> Both gspca and the solo driver have a header with the same name, and >>>> that clashes when using media_build. >>> >>> This the kind of patch that doesn't make much sense upstream. Granted, >>> the files weren't properly named, but there's not requirement upstream >>> that denies naming two different files with the same name. >>> >>> Btw, looking at both, it seems that they can be merged: both defines the >>> jpeg header. The basic difference is that, while gspca jpeg header can >>> have a size of either 556 or 589 bytes, the one at solo6x10 has 575 >>> bytes. >>> >>> IMHO, the proper fix is to make solo6x10 driver to use the gspca jpeg.h >>> header. Assuming that this driver would find his way out of staging, >>> then the jpeg.h file should also be moved to another place, like >>> include/linux/media, as I don't think that solo6x10 driver should be a >>> gspca sub-driver. >>> >>> Hans G, what do you think? >> >> The quantization tables are completely different on Solo driver. Merging >> them will probably be very messy. I withdraw the proposal of merging them. >> >> Your patch makes sense to me. > > OK. > > Perhaps the gspca jpeg.h header should be renamed to gspca-jpeg.h as well? I think so, as it currently doesn't implement a generic way of generating jpeg headers. > Since the solo driver is in staging I went for just renaming the header in > that driver rather than renaming a header in a non-staging driver. > >>>> And the solo driver uses an incorrect Makefile construct, which >>>> (somewhat mysteriously) skips the compilation of 90% of all media >>>> drivers. >>> >>> Hmm.. probably they're using "=" or ":=" instead of "+=". While this >>> works at leaf Makefiles, this breaks compilation when there's just one >>> Makefile, or when you add another thing to be compiled there. This is >>> something that requires a fix. > > That's indeed what I fixed in my second patch (:= -> +=). Ok. Regards, Mauro -- 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
