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? 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 (:= -> +=). Hans -- 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