[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <65c923cda564f308eae5f24191afc1c15c500ea9.camel@ndufresne.ca>
Date: Fri, 07 Nov 2025 15:30:15 -0500
From: Nicolas Dufresne <nicolas@...fresne.ca>
To: yassine.ouaissa@...egrodvt.com, Mauro Carvalho Chehab
<mchehab@...nel.org>, Michael Tretter <m.tretter@...gutronix.de>,
Pengutronix Kernel Team <kernel@...gutronix.de>, Michal Simek
<michal.simek@....com>, Rob Herring <robh@...nel.org>, Krzysztof Kozlowski
<krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>
Cc: linux-kernel@...r.kernel.org, linux-media@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, devicetree@...r.kernel.org
Subject: Re: [PATCH v4 1/4] media: allegro-dvt: Move the current driver to a
subdirectory
Hi,
Le mercredi 16 juillet 2025 à 14:55 +0000, Yassine Ouaissa via B4 Relay a écrit :
> From: Yassine Ouaissa <yassine.ouaissa@...egrodvt.com>
>
> In preparation for the upcoming driver update, we need to relocate the
> current driver.
> This will help ensure a clean transition and avoid any potential
> conflicts with the new driver.
> This patch is crucial for keeping our directory organized and
> facilitating a smooth integration of the new driver.
>
> Signed-off-by: Yassine Ouaissa <yassine.ouaissa@...egrodvt.com>
> ---
> MAINTAINERS | 2 +-
> drivers/media/platform/allegro-dvt/Kconfig | 16 +---------------
> drivers/media/platform/allegro-dvt/Makefile | 5 +----
> drivers/media/platform/allegro-dvt/zynqmp/Kconfig | 17 +++++++++++++++++
> drivers/media/platform/allegro-dvt/zynqmp/Makefile | 6 ++++++
> .../platform/allegro-dvt/{ => zynqmp}/allegro-core.c | 0
> .../platform/allegro-dvt/{ => zynqmp}/allegro-mail.c | 0
> .../platform/allegro-dvt/{ => zynqmp}/allegro-mail.h | 0
> .../media/platform/allegro-dvt/{ => zynqmp}/nal-h264.c | 0
> .../media/platform/allegro-dvt/{ => zynqmp}/nal-h264.h | 0
> .../media/platform/allegro-dvt/{ => zynqmp}/nal-hevc.c | 0
> .../media/platform/allegro-dvt/{ => zynqmp}/nal-hevc.h | 0
> .../media/platform/allegro-dvt/{ => zynqmp}/nal-rbsp.c | 0
> .../media/platform/allegro-dvt/{ => zynqmp}/nal-rbsp.h | 0
> 14 files changed, 26 insertions(+), 20 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 60bba48f5479a025f9da3eaf9dbacb67a194df07..2cea337bd426f203a8dd8f5f2689f8091137175e 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -817,7 +817,7 @@ R: Pengutronix Kernel Team <kernel@...gutronix.de>
> L: linux-media@...r.kernel.org
> S: Maintained
> F: Documentation/devicetree/bindings/media/allegro,al5e.yaml
> -F: drivers/media/platform/allegro-dvt/
> +F: drivers/media/platform/allegro-dvt/zynqmp
If you can get Micheal approval, you can then carry over this for the followup versions:
Reviewed-by: Nicolas Dufresne <nicolas.dufresne@...labora.com>
cheers,
Nicolas
>
> ALLIED VISION ALVIUM CAMERA DRIVER
> M: Tommaso Merciai <tomm.merciai@...il.com>
> diff --git a/drivers/media/platform/allegro-dvt/Kconfig b/drivers/media/platform/allegro-dvt/Kconfig
> index 2182e1277568a407f51a23ea437811c50b1183c8..e9008614c27b9490d1cd29fab887977a1918ede4 100644
> --- a/drivers/media/platform/allegro-dvt/Kconfig
> +++ b/drivers/media/platform/allegro-dvt/Kconfig
> @@ -2,18 +2,4 @@
>
> comment "Allegro DVT media platform drivers"
>
> -config VIDEO_ALLEGRO_DVT
> - tristate "Allegro DVT Video IP Core"
> - depends on V4L_MEM2MEM_DRIVERS
> - depends on VIDEO_DEV
> - depends on ARCH_ZYNQMP || COMPILE_TEST
> - select V4L2_MEM2MEM_DEV
> - select VIDEOBUF2_DMA_CONTIG
> - select REGMAP_MMIO
> - help
> - Support for the encoder video IP core by Allegro DVT. This core is
> - found for example on the Xilinx ZynqMP SoC in the EV family and is
> - called VCU in the reference manual.
> -
> - To compile this driver as a module, choose M here: the module
> - will be called allegro.
> +source "drivers/media/platform/allegro-dvt/zynqmp/Kconfig"
> diff --git a/drivers/media/platform/allegro-dvt/Makefile b/drivers/media/platform/allegro-dvt/Makefile
> index 66108a3037747020d549bc0a427881e0667a3f0a..d2aa6875edcf7760901996aac4d5ac98282cce20 100644
> --- a/drivers/media/platform/allegro-dvt/Makefile
> +++ b/drivers/media/platform/allegro-dvt/Makefile
> @@ -1,6 +1,3 @@
> # SPDX-License-Identifier: GPL-2.0
>
> -allegro-objs := allegro-core.o allegro-mail.o
> -allegro-objs += nal-rbsp.o nal-h264.o nal-hevc.o
> -
> -obj-$(CONFIG_VIDEO_ALLEGRO_DVT) += allegro.o
> +obj-y += zynqmp/
> diff --git a/drivers/media/platform/allegro-dvt/zynqmp/Kconfig b/drivers/media/platform/allegro-dvt/zynqmp/Kconfig
> new file mode 100644
> index 0000000000000000000000000000000000000000..0a0a697c420da47f87f05153a2dbfbe5d3ccf988
> --- /dev/null
> +++ b/drivers/media/platform/allegro-dvt/zynqmp/Kconfig
> @@ -0,0 +1,17 @@
> +# SPDX-License-Identifier: GPL-2.0-only
> +
> +config VIDEO_ALLEGRO_DVT
> + tristate "Allegro DVT Video IP Core for ZynqMP"
> + depends on V4L_MEM2MEM_DRIVERS
> + depends on VIDEO_DEV
> + depends on ARCH_ZYNQMP || COMPILE_TEST
> + select V4L2_MEM2MEM_DEV
> + select VIDEOBUF2_DMA_CONTIG
> + select REGMAP_MMIO
> + help
> + Support for the encoder video IP core by Allegro DVT. This core is
> + found for example on the Xilinx ZynqMP SoC in the EV family and is
> + called VCU in the reference manual.
> +
> + To compile this driver as a module, choose M here: the module
> + will be called allegro.
> diff --git a/drivers/media/platform/allegro-dvt/zynqmp/Makefile b/drivers/media/platform/allegro-dvt/zynqmp/Makefile
> new file mode 100644
> index 0000000000000000000000000000000000000000..66108a3037747020d549bc0a427881e0667a3f0a
> --- /dev/null
> +++ b/drivers/media/platform/allegro-dvt/zynqmp/Makefile
> @@ -0,0 +1,6 @@
> +# SPDX-License-Identifier: GPL-2.0
> +
> +allegro-objs := allegro-core.o allegro-mail.o
> +allegro-objs += nal-rbsp.o nal-h264.o nal-hevc.o
> +
> +obj-$(CONFIG_VIDEO_ALLEGRO_DVT) += allegro.o
> diff --git a/drivers/media/platform/allegro-dvt/allegro-core.c b/drivers/media/platform/allegro-dvt/zynqmp/allegro-core.c
> similarity index 100%
> rename from drivers/media/platform/allegro-dvt/allegro-core.c
> rename to drivers/media/platform/allegro-dvt/zynqmp/allegro-core.c
> diff --git a/drivers/media/platform/allegro-dvt/allegro-mail.c b/drivers/media/platform/allegro-dvt/zynqmp/allegro-mail.c
> similarity index 100%
> rename from drivers/media/platform/allegro-dvt/allegro-mail.c
> rename to drivers/media/platform/allegro-dvt/zynqmp/allegro-mail.c
> diff --git a/drivers/media/platform/allegro-dvt/allegro-mail.h b/drivers/media/platform/allegro-dvt/zynqmp/allegro-mail.h
> similarity index 100%
> rename from drivers/media/platform/allegro-dvt/allegro-mail.h
> rename to drivers/media/platform/allegro-dvt/zynqmp/allegro-mail.h
> diff --git a/drivers/media/platform/allegro-dvt/nal-h264.c b/drivers/media/platform/allegro-dvt/zynqmp/nal-h264.c
> similarity index 100%
> rename from drivers/media/platform/allegro-dvt/nal-h264.c
> rename to drivers/media/platform/allegro-dvt/zynqmp/nal-h264.c
> diff --git a/drivers/media/platform/allegro-dvt/nal-h264.h b/drivers/media/platform/allegro-dvt/zynqmp/nal-h264.h
> similarity index 100%
> rename from drivers/media/platform/allegro-dvt/nal-h264.h
> rename to drivers/media/platform/allegro-dvt/zynqmp/nal-h264.h
> diff --git a/drivers/media/platform/allegro-dvt/nal-hevc.c b/drivers/media/platform/allegro-dvt/zynqmp/nal-hevc.c
> similarity index 100%
> rename from drivers/media/platform/allegro-dvt/nal-hevc.c
> rename to drivers/media/platform/allegro-dvt/zynqmp/nal-hevc.c
> diff --git a/drivers/media/platform/allegro-dvt/nal-hevc.h b/drivers/media/platform/allegro-dvt/zynqmp/nal-hevc.h
> similarity index 100%
> rename from drivers/media/platform/allegro-dvt/nal-hevc.h
> rename to drivers/media/platform/allegro-dvt/zynqmp/nal-hevc.h
> diff --git a/drivers/media/platform/allegro-dvt/nal-rbsp.c b/drivers/media/platform/allegro-dvt/zynqmp/nal-rbsp.c
> similarity index 100%
> rename from drivers/media/platform/allegro-dvt/nal-rbsp.c
> rename to drivers/media/platform/allegro-dvt/zynqmp/nal-rbsp.c
> diff --git a/drivers/media/platform/allegro-dvt/nal-rbsp.h b/drivers/media/platform/allegro-dvt/zynqmp/nal-rbsp.h
> similarity index 100%
> rename from drivers/media/platform/allegro-dvt/nal-rbsp.h
> rename to drivers/media/platform/allegro-dvt/zynqmp/nal-rbsp.h
Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)
Powered by blists - more mailing lists