Re: [RFC PATCH 7/7] media: ipu7: add Makefile and Kconfig for IPU7

[Bingbu Cao <bingbu.cao@linux.intel.com>]

All,

On 2/21/25 3:52 PM, bingbu.cao@intel.com wrote:
> From: Bingbu Cao <bingbu.cao@intel.com>
> 
> Add Kconfig and Makefile for IPU7 driver and also update
> the Makefile to build the IPU7 driver.
> 
> Signed-off-by: Bingbu Cao <bingbu.cao@intel.com>
> ---
>  drivers/media/pci/intel/Kconfig       |  1 +
>  drivers/media/pci/intel/Makefile      |  1 +
>  drivers/media/pci/intel/ipu7/Kconfig  | 18 ++++++++++++++++++
>  drivers/media/pci/intel/ipu7/Makefile | 23 +++++++++++++++++++++++
>  4 files changed, 43 insertions(+)
>  create mode 100644 drivers/media/pci/intel/ipu7/Kconfig
>  create mode 100644 drivers/media/pci/intel/ipu7/Makefile
> 
> diff --git a/drivers/media/pci/intel/Kconfig b/drivers/media/pci/intel/Kconfig
> index d9fcddce028b..948cda08fff5 100644
> --- a/drivers/media/pci/intel/Kconfig
> +++ b/drivers/media/pci/intel/Kconfig
> @@ -2,6 +2,7 @@
>  
>  source "drivers/media/pci/intel/ipu3/Kconfig"
>  source "drivers/media/pci/intel/ipu6/Kconfig"
> +source "drivers/media/pci/intel/ipu7/Kconfig"
>  source "drivers/media/pci/intel/ivsc/Kconfig"
>  
>  config IPU_BRIDGE
> diff --git a/drivers/media/pci/intel/Makefile b/drivers/media/pci/intel/Makefile
> index 3a2cc6567159..ff0fea13422d 100644
> --- a/drivers/media/pci/intel/Makefile
> +++ b/drivers/media/pci/intel/Makefile
> @@ -6,3 +6,4 @@ obj-$(CONFIG_IPU_BRIDGE) += ipu-bridge.o
>  obj-y	+= ipu3/
>  obj-y	+= ivsc/
>  obj-$(CONFIG_VIDEO_INTEL_IPU6)	+= ipu6/
> +obj-$(CONFIG_VIDEO_INTEL_IPU7)	+= ipu7/
> diff --git a/drivers/media/pci/intel/ipu7/Kconfig b/drivers/media/pci/intel/ipu7/Kconfig
> new file mode 100644
> index 000000000000..b759d5e6c3a9
> --- /dev/null
> +++ b/drivers/media/pci/intel/ipu7/Kconfig
> @@ -0,0 +1,18 @@
> +config VIDEO_INTEL_IPU7
> +	tristate "Intel IPU7 driver"
> +	depends on ACPI || COMPILE_TEST
> +	depends on VIDEO_DEV
> +	depends on X86 && X86_64 && HAS_DMA

depends on X86 && HAS_DMA

> +	depends on IPU_BRIDGE || !IPU_BRIDGE
> +	select AUXILIARY_BUS
> +	select IOMMU_IOVA
> +	select VIDEO_V4L2_SUBDEV_API
> +	select MEDIA_CONTROLLER
> +	select VIDEOBUF2_DMA_CONTIG

This should be VIDEOBUF2_DMA_SG, I forgot to amend it.

> +	select V4L2_FWNODE
> +	help
> +	  This is the 7th Gen Intel Image Processing Unit, found in Intel SoCs
> +	  and used for capturing images and video from camera sensors.
> +
> +	  To compile this driver, say Y here! It contains 2 modules -
> +	  intel_ipu7 and intel_ipu7_isys.
> diff --git a/drivers/media/pci/intel/ipu7/Makefile b/drivers/media/pci/intel/ipu7/Makefile
> new file mode 100644
> index 000000000000..9a15d119026a
> --- /dev/null
> +++ b/drivers/media/pci/intel/ipu7/Makefile
> @@ -0,0 +1,23 @@
> +# SPDX-License-Identifier: GPL-2.0
> +# Copyright (c) 2017 - 2024 Intel Corporation.
> +
> +intel-ipu7-objs				+= ipu7.o \
> +					   ipu7-bus.o \
> +					   ipu7-dma.o \
> +					   ipu7-mmu.o \
> +					   ipu7-buttress.o \
> +					   ipu7-cpd.o \
> +					   ipu7-syscom.o \
> +					   ipu7-boot.o
> +
> +obj-$(CONFIG_VIDEO_INTEL_IPU7)		+= intel-ipu7.o
> +
> +intel-ipu7-isys-objs			+= ipu7-isys.o \
> +					   ipu7-isys-csi2.o \
> +					   ipu7-isys-csi-phy.o \
> +					   ipu7-fw-isys.o \
> +					   ipu7-isys-video.o \
> +					   ipu7-isys-queue.o \
> +					   ipu7-isys-subdev.o
> +
> +obj-$(CONFIG_VIDEO_INTEL_IPU7)		+= intel-ipu7-isys.o
> 

-- 
Best regards,
Bingbu Cao