* [PATCH] [media] platform: Order the Makefile alphabetically
@ 2017-03-07 13:39 Maxime Ripard
2017-03-20 10:22 ` Maxime Ripard
2017-03-27 8:52 ` Laurent Pinchart
0 siblings, 2 replies; 4+ messages in thread
From: Maxime Ripard @ 2017-03-07 13:39 UTC (permalink / raw)
To: Mauro Carvalho Chehab
Cc: linux-media, linux-kernel, Laurent Pinchart, Maxime Ripard
The Makefile was a free for all without a clear order defined. Sort all the
options based on the Kconfig symbol.
Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
---
drivers/media/platform/Makefile | 89 ++++++++++++++---------------------------
1 file changed, 31 insertions(+), 58 deletions(-)
diff --git a/drivers/media/platform/Makefile b/drivers/media/platform/Makefile
index 349ddf6a69da..bd5dc3068cbc 100644
--- a/drivers/media/platform/Makefile
+++ b/drivers/media/platform/Makefile
@@ -1,73 +1,46 @@
#
# Makefile for the video capture/playback device drivers.
#
+ccflags-y += -I$(srctree)/drivers/media/i2c
-obj-$(CONFIG_VIDEO_M32R_AR_M64278) += arv.o
-
-obj-$(CONFIG_VIDEO_VIA_CAMERA) += via-camera.o
-obj-$(CONFIG_VIDEO_CAFE_CCIC) += marvell-ccic/
-obj-$(CONFIG_VIDEO_MMP_CAMERA) += marvell-ccic/
-
-obj-$(CONFIG_VIDEO_OMAP3) += omap3isp/
-obj-$(CONFIG_VIDEO_PXA27x) += pxa_camera.o
-
-obj-$(CONFIG_VIDEO_VIU) += fsl-viu.o
-
-obj-$(CONFIG_VIDEO_VIVID) += vivid/
-obj-$(CONFIG_VIDEO_VIM2M) += vim2m.o
-
-obj-$(CONFIG_VIDEO_TI_VPE) += ti-vpe/
-
-obj-$(CONFIG_VIDEO_TI_CAL) += ti-vpe/
-
-obj-$(CONFIG_VIDEO_MX2_EMMAPRP) += mx2_emmaprp.o
+obj-y += omap/
+obj-$(CONFIG_ARCH_DAVINCI) += davinci/
+obj-$(CONFIG_BLACKFIN) += blackfin/
+obj-$(CONFIG_DVB_C8SECTPFE) += sti/c8sectpfe/
+obj-$(CONFIG_SOC_CAMERA) += soc_camera/
+obj-$(CONFIG_VIDEO_AM437X_VPFE) += am437x/
+obj-$(CONFIG_VIDEO_ATMEL_ISC) += atmel/
+obj-$(CONFIG_VIDEO_CAFE_CCIC) += marvell-ccic/
obj-$(CONFIG_VIDEO_CODA) += coda/
-
-obj-$(CONFIG_VIDEO_SH_VEU) += sh_veu.o
-
+obj-$(CONFIG_VIDEO_M32R_AR_M64278) += arv.o
+obj-$(CONFIG_VIDEO_MEDIATEK_MDP) += mtk-mdp/
+obj-$(CONFIG_VIDEO_MEDIATEK_VCODEC) += mtk-vcodec/
+obj-$(CONFIG_VIDEO_MEDIATEK_VPU) += mtk-vpu/
obj-$(CONFIG_VIDEO_MEM2MEM_DEINTERLACE) += m2m-deinterlace.o
-
+obj-$(CONFIG_VIDEO_MMP_CAMERA) += marvell-ccic/
+obj-$(CONFIG_VIDEO_MX2_EMMAPRP) += mx2_emmaprp.o
+obj-$(CONFIG_VIDEO_OMAP3) += omap3isp/
+obj-$(CONFIG_VIDEO_PXA27x) += pxa_camera.o
+obj-$(CONFIG_VIDEO_RCAR_VIN) += rcar-vin/
+obj-$(CONFIG_VIDEO_RENESAS_FCP) += rcar-fcp.o
+obj-$(CONFIG_VIDEO_RENESAS_FDP1) += rcar_fdp1.o
+obj-$(CONFIG_VIDEO_RENESAS_JPU) += rcar_jpu.o
+obj-$(CONFIG_VIDEO_RENESAS_VSP1) += vsp1/
obj-$(CONFIG_VIDEO_S3C_CAMIF) += s3c-camif/
obj-$(CONFIG_VIDEO_SAMSUNG_EXYNOS4_IS) += exynos4-is/
obj-$(CONFIG_VIDEO_SAMSUNG_S5P_JPEG) += s5p-jpeg/
obj-$(CONFIG_VIDEO_SAMSUNG_S5P_MFC) += s5p-mfc/
-
obj-$(CONFIG_VIDEO_SAMSUNG_S5P_G2D) += s5p-g2d/
obj-$(CONFIG_VIDEO_SAMSUNG_EXYNOS_GSC) += exynos-gsc/
-
+obj-$(CONFIG_VIDEO_SH_VEU) += sh_veu.o
+obj-$(CONFIG_VIDEO_SH_VOU) += sh_vou.o
obj-$(CONFIG_VIDEO_STI_BDISP) += sti/bdisp/
-obj-$(CONFIG_VIDEO_STI_HVA) += sti/hva/
-obj-$(CONFIG_DVB_C8SECTPFE) += sti/c8sectpfe/
-
obj-$(CONFIG_VIDEO_STI_DELTA) += sti/delta/
-
-obj-$(CONFIG_BLACKFIN) += blackfin/
-
-obj-$(CONFIG_ARCH_DAVINCI) += davinci/
-
-obj-$(CONFIG_VIDEO_SH_VOU) += sh_vou.o
-
-obj-$(CONFIG_SOC_CAMERA) += soc_camera/
-
-obj-$(CONFIG_VIDEO_RENESAS_FCP) += rcar-fcp.o
-obj-$(CONFIG_VIDEO_RENESAS_FDP1) += rcar_fdp1.o
-obj-$(CONFIG_VIDEO_RENESAS_JPU) += rcar_jpu.o
-obj-$(CONFIG_VIDEO_RENESAS_VSP1) += vsp1/
-
-obj-y += omap/
-
-obj-$(CONFIG_VIDEO_AM437X_VPFE) += am437x/
-
+obj-$(CONFIG_VIDEO_STI_HVA) += sti/hva/
+obj-$(CONFIG_VIDEO_TI_CAL) += ti-vpe/
+obj-$(CONFIG_VIDEO_TI_VPE) += ti-vpe/
+obj-$(CONFIG_VIDEO_VIA_CAMERA) += via-camera.o
+obj-$(CONFIG_VIDEO_VIM2M) += vim2m.o
+obj-$(CONFIG_VIDEO_VIU) += fsl-viu.o
+obj-$(CONFIG_VIDEO_VIVID) += vivid/
obj-$(CONFIG_VIDEO_XILINX) += xilinx/
-
-obj-$(CONFIG_VIDEO_RCAR_VIN) += rcar-vin/
-
-obj-$(CONFIG_VIDEO_ATMEL_ISC) += atmel/
-
-ccflags-y += -I$(srctree)/drivers/media/i2c
-
-obj-$(CONFIG_VIDEO_MEDIATEK_VPU) += mtk-vpu/
-
-obj-$(CONFIG_VIDEO_MEDIATEK_VCODEC) += mtk-vcodec/
-
-obj-$(CONFIG_VIDEO_MEDIATEK_MDP) += mtk-mdp/
--
2.11.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] [media] platform: Order the Makefile alphabetically
2017-03-07 13:39 [PATCH] [media] platform: Order the Makefile alphabetically Maxime Ripard
@ 2017-03-20 10:22 ` Maxime Ripard
2017-03-27 8:27 ` Maxime Ripard
2017-03-27 8:52 ` Laurent Pinchart
1 sibling, 1 reply; 4+ messages in thread
From: Maxime Ripard @ 2017-03-20 10:22 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: linux-media, linux-kernel, Laurent Pinchart
[-- Attachment #1: Type: text/plain, Size: 404 bytes --]
Hi Mauro,
On Tue, Mar 07, 2017 at 02:39:28PM +0100, Maxime Ripard wrote:
> The Makefile was a free for all without a clear order defined. Sort all the
> options based on the Kconfig symbol.
>
> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
Any news on this one?
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] [media] platform: Order the Makefile alphabetically
2017-03-20 10:22 ` Maxime Ripard
@ 2017-03-27 8:27 ` Maxime Ripard
0 siblings, 0 replies; 4+ messages in thread
From: Maxime Ripard @ 2017-03-27 8:27 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: linux-media, linux-kernel, Laurent Pinchart
[-- Attachment #1: Type: text/plain, Size: 497 bytes --]
On Mon, Mar 20, 2017 at 11:22:02AM +0100, Maxime Ripard wrote:
> Hi Mauro,
>
> On Tue, Mar 07, 2017 at 02:39:28PM +0100, Maxime Ripard wrote:
> > The Makefile was a free for all without a clear order defined. Sort all the
> > options based on the Kconfig symbol.
> >
> > Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
>
> Any news on this one?
Ping?
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] [media] platform: Order the Makefile alphabetically
2017-03-07 13:39 [PATCH] [media] platform: Order the Makefile alphabetically Maxime Ripard
2017-03-20 10:22 ` Maxime Ripard
@ 2017-03-27 8:52 ` Laurent Pinchart
1 sibling, 0 replies; 4+ messages in thread
From: Laurent Pinchart @ 2017-03-27 8:52 UTC (permalink / raw)
To: Maxime Ripard; +Cc: Mauro Carvalho Chehab, linux-media, linux-kernel
Hi Maxime,
Thank you for the patch.
On Tuesday 07 Mar 2017 14:39:28 Maxime Ripard wrote:
> The Makefile was a free for all without a clear order defined. Sort all the
> options based on the Kconfig symbol.
I've been annoyed in the past by not knowing where to put entries for new
drivers. I think this is a good improvement.
> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
> drivers/media/platform/Makefile | 89 ++++++++++++-------------------------
> 1 file changed, 31 insertions(+), 58 deletions(-)
>
> diff --git a/drivers/media/platform/Makefile
> b/drivers/media/platform/Makefile index 349ddf6a69da..bd5dc3068cbc 100644
> --- a/drivers/media/platform/Makefile
> +++ b/drivers/media/platform/Makefile
> @@ -1,73 +1,46 @@
> #
> # Makefile for the video capture/playback device drivers.
> #
> +ccflags-y += -I$(srctree)/drivers/media/i2c
>
> -obj-$(CONFIG_VIDEO_M32R_AR_M64278) += arv.o
> -
> -obj-$(CONFIG_VIDEO_VIA_CAMERA) += via-camera.o
> -obj-$(CONFIG_VIDEO_CAFE_CCIC) += marvell-ccic/
> -obj-$(CONFIG_VIDEO_MMP_CAMERA) += marvell-ccic/
> -
> -obj-$(CONFIG_VIDEO_OMAP3) += omap3isp/
> -obj-$(CONFIG_VIDEO_PXA27x) += pxa_camera.o
> -
> -obj-$(CONFIG_VIDEO_VIU) += fsl-viu.o
> -
> -obj-$(CONFIG_VIDEO_VIVID) += vivid/
> -obj-$(CONFIG_VIDEO_VIM2M) += vim2m.o
> -
> -obj-$(CONFIG_VIDEO_TI_VPE) += ti-vpe/
> -
> -obj-$(CONFIG_VIDEO_TI_CAL) += ti-vpe/
> -
> -obj-$(CONFIG_VIDEO_MX2_EMMAPRP) += mx2_emmaprp.o
> +obj-y += omap/
> +obj-$(CONFIG_ARCH_DAVINCI) += davinci/
> +obj-$(CONFIG_BLACKFIN) += blackfin/
> +obj-$(CONFIG_DVB_C8SECTPFE) += sti/c8sectpfe/
> +obj-$(CONFIG_SOC_CAMERA) += soc_camera/
> +obj-$(CONFIG_VIDEO_AM437X_VPFE) += am437x/
> +obj-$(CONFIG_VIDEO_ATMEL_ISC) += atmel/
> +obj-$(CONFIG_VIDEO_CAFE_CCIC) += marvell-ccic/
> obj-$(CONFIG_VIDEO_CODA) += coda/
> -
> -obj-$(CONFIG_VIDEO_SH_VEU) += sh_veu.o
> -
> +obj-$(CONFIG_VIDEO_M32R_AR_M64278) += arv.o
> +obj-$(CONFIG_VIDEO_MEDIATEK_MDP) += mtk-mdp/
> +obj-$(CONFIG_VIDEO_MEDIATEK_VCODEC) += mtk-vcodec/
> +obj-$(CONFIG_VIDEO_MEDIATEK_VPU) += mtk-vpu/
> obj-$(CONFIG_VIDEO_MEM2MEM_DEINTERLACE) += m2m-deinterlace.o
> -
> +obj-$(CONFIG_VIDEO_MMP_CAMERA) += marvell-ccic/
> +obj-$(CONFIG_VIDEO_MX2_EMMAPRP) += mx2_emmaprp.o
> +obj-$(CONFIG_VIDEO_OMAP3) += omap3isp/
> +obj-$(CONFIG_VIDEO_PXA27x) += pxa_camera.o
> +obj-$(CONFIG_VIDEO_RCAR_VIN) += rcar-vin/
> +obj-$(CONFIG_VIDEO_RENESAS_FCP) += rcar-fcp.o
> +obj-$(CONFIG_VIDEO_RENESAS_FDP1) += rcar_fdp1.o
> +obj-$(CONFIG_VIDEO_RENESAS_JPU) += rcar_jpu.o
> +obj-$(CONFIG_VIDEO_RENESAS_VSP1) += vsp1/
> obj-$(CONFIG_VIDEO_S3C_CAMIF) += s3c-camif/
> obj-$(CONFIG_VIDEO_SAMSUNG_EXYNOS4_IS) += exynos4-is/
> obj-$(CONFIG_VIDEO_SAMSUNG_S5P_JPEG) += s5p-jpeg/
> obj-$(CONFIG_VIDEO_SAMSUNG_S5P_MFC) += s5p-mfc/
> -
> obj-$(CONFIG_VIDEO_SAMSUNG_S5P_G2D) += s5p-g2d/
> obj-$(CONFIG_VIDEO_SAMSUNG_EXYNOS_GSC) += exynos-gsc/
> -
> +obj-$(CONFIG_VIDEO_SH_VEU) += sh_veu.o
> +obj-$(CONFIG_VIDEO_SH_VOU) += sh_vou.o
> obj-$(CONFIG_VIDEO_STI_BDISP) += sti/bdisp/
> -obj-$(CONFIG_VIDEO_STI_HVA) += sti/hva/
> -obj-$(CONFIG_DVB_C8SECTPFE) += sti/c8sectpfe/
> -
> obj-$(CONFIG_VIDEO_STI_DELTA) += sti/delta/
> -
> -obj-$(CONFIG_BLACKFIN) += blackfin/
> -
> -obj-$(CONFIG_ARCH_DAVINCI) += davinci/
> -
> -obj-$(CONFIG_VIDEO_SH_VOU) += sh_vou.o
> -
> -obj-$(CONFIG_SOC_CAMERA) += soc_camera/
> -
> -obj-$(CONFIG_VIDEO_RENESAS_FCP) += rcar-fcp.o
> -obj-$(CONFIG_VIDEO_RENESAS_FDP1) += rcar_fdp1.o
> -obj-$(CONFIG_VIDEO_RENESAS_JPU) += rcar_jpu.o
> -obj-$(CONFIG_VIDEO_RENESAS_VSP1) += vsp1/
> -
> -obj-y += omap/
> -
> -obj-$(CONFIG_VIDEO_AM437X_VPFE) += am437x/
> -
> +obj-$(CONFIG_VIDEO_STI_HVA) += sti/hva/
> +obj-$(CONFIG_VIDEO_TI_CAL) += ti-vpe/
> +obj-$(CONFIG_VIDEO_TI_VPE) += ti-vpe/
> +obj-$(CONFIG_VIDEO_VIA_CAMERA) += via-camera.o
> +obj-$(CONFIG_VIDEO_VIM2M) += vim2m.o
> +obj-$(CONFIG_VIDEO_VIU) += fsl-viu.o
> +obj-$(CONFIG_VIDEO_VIVID) += vivid/
> obj-$(CONFIG_VIDEO_XILINX) += xilinx/
> -
> -obj-$(CONFIG_VIDEO_RCAR_VIN) += rcar-vin/
> -
> -obj-$(CONFIG_VIDEO_ATMEL_ISC) += atmel/
> -
> -ccflags-y += -I$(srctree)/drivers/media/i2c
> -
> -obj-$(CONFIG_VIDEO_MEDIATEK_VPU) += mtk-vpu/
> -
> -obj-$(CONFIG_VIDEO_MEDIATEK_VCODEC) += mtk-vcodec/
> -
> -obj-$(CONFIG_VIDEO_MEDIATEK_MDP) += mtk-mdp/
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-03-27 8:51 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-03-07 13:39 [PATCH] [media] platform: Order the Makefile alphabetically Maxime Ripard
2017-03-20 10:22 ` Maxime Ripard
2017-03-27 8:27 ` Maxime Ripard
2017-03-27 8:52 ` Laurent Pinchart
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).