Hans Verkuil <hverk...@xs4all.nl> writes: > Linking soc_mediabus into this driver causes multiple definition linker > warnings > if soc_camera is also enabled: > > > drivers/media/platform/soc_camera/built-in.o:(___ksymtab+soc_mbus_image_size+0x0): > multiple definition of `__ksymtab_soc_mbus_image_size' > > drivers/media/platform/soc_camera/soc_mediabus.o:(___ksymtab+soc_mbus_image_size+0x0): > first defined here >>> drivers/media/platform/soc_camera/built-in.o:(___ksymtab+soc_mbus_samples_per_pixel+0x0): >>> multiple definition of `__ksymtab_soc_mbus_samples_per_pixel' > > drivers/media/platform/soc_camera/soc_mediabus.o:(___ksymtab+soc_mbus_samples_per_pixel+0x0): > first defined here > drivers/media/platform/soc_camera/built-in.o: In function > `soc_mbus_config_compatible': > (.text+0x3840): multiple definition of `soc_mbus_config_compatible' > drivers/media/platform/soc_camera/soc_mediabus.o:(.text+0x134): first > defined here > > Since we really don't want to have to use any of the soc-camera code this > patch > copies the relevant code and data structures from soc_mediabus and renames it > to pxa_mbus_*. > > The large table of formats has been culled a bit, removing formats that are > not supported > by this driver. > > Signed-off-by: Hans Verkuil <hans.verk...@cisco.com> > Cc: Robert Jarzmik <robert.jarz...@free.fr>
Hi Hans, I wonder why you chose to copy-paste this code instead of adding in the Kconfig a "depends on !SOC_CAMERA". Any specific reason ? As this will have to be dealt with later anyway as you pointed out earlier, this format translation I mean, I was wondering if this was the best approach. Cheers. -- Robert -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
