Re: [PATCH] drivers: media: platform: Makefile: Add build dependency for davinci/

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux