Hi Andreas, On Fri, Aug 15, 2014 at 6:10 PM, Andreas Ruprecht <rupran@xxxxxxxxxxxx> wrote: > In the davinci/ subdirectory, all drivers but one depend on > CONFIG_ARCH_DAVINCI. The only exception, selected by CONFIG_VIDEO_DM6446_CCDC, > is also available on CONFIG_ARCH_OMAP3. > > Thus, it is not necessary to always descend into davinci/. It is sufficient to > do this only if CONFIG_ARCH_OMAP3 or CONFIG_ARCH_DAVINCI is selected. While the > latter is already present, this patch changes the dependency from obj-y to > obj-$(CONFIG_ARCH_OMAP3). > I have submitted a proper fix [1], so NACK. [1] https://patchwork.kernel.org/patch/4730111/ Regards, --Prabhakar Lad > Signed-off-by: Andreas Ruprecht <rupran@xxxxxxxxxxxx> > --- > drivers/media/platform/Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/platform/Makefile b/drivers/media/platform/Makefile > index e5269da..d32e79a 100644 > --- a/drivers/media/platform/Makefile > +++ b/drivers/media/platform/Makefile > @@ -47,7 +47,7 @@ obj-$(CONFIG_SOC_CAMERA) += soc_camera/ > > obj-$(CONFIG_VIDEO_RENESAS_VSP1) += vsp1/ > > -obj-y += davinci/ > +obj-$(CONFIG_ARCH_OMAP3) += davinci/ > > obj-$(CONFIG_ARCH_OMAP) += omap/ > > -- > 1.9.1 > > -- > 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 -- 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