On 12/15/20 9:07 AM, Patrick Talbert wrote:
On Tue, Dec 15, 2020 at 2:07 PM GitLab Bridge on behalf of
redhat-patchlab <cki-gitlab(a)redhat.com> wrote:
>
> From: Fedora Kernel Team <kernel-team(a)fedoraproject.org>
>
> Hi,
>
> As part of the ongoing rebase effort, the following configuration
> options need to be reviewed.
>
> As a reminder, the ARK configuration flow involves moving unreviewed
> configuration options from the pending directory to the ark directory.
> In the diff below, options are removed from the pending directory and
> added to the ark hierarchy. The final options that need to be ACKed
> are the files that are being added to the ark hierarchy.
>
> If the value for a file that is added should be changed, please reply
> with a better option.
>
> CONFIG_VIDEO_CCS:
David, Gopal,
Is this something either of you can comment on?
I would think defaulting to not set is the correct approach for now.
Acked-by: David Arcari <darcari(a)redhat.com>
Patrick
>
> This is a generic driver for MIPI CCS, SMIA++ and SMIA compliant
> camera sensors.
>
> Symbol: VIDEO_CCS [=n]
> Type : tristate
> Defined at drivers/media/i2c/ccs/Kconfig:2
> Prompt: MIPI CCS/SMIA++/SMIA sensor support
> Depends on: MEDIA_SUPPORT [=m] && I2C [=y] && VIDEO_V4L2 [=m]
&& HAVE_CLK [=y]
> Visible if: MEDIA_SUPPORT [=m] && I2C [=y] && VIDEO_V4L2 [=m]
&& HAVE_CLK [=y] && MEDIA_CAMERA_SUPPORT [=y]
> Location:
> -> Device Drivers
> -> Multimedia support (MEDIA_SUPPORT [=m])
> -> Media ancillary drivers
> -> Camera sensor devices
> Selects: MEDIA_CONTROLLER [=y] && VIDEO_V4L2_SUBDEV_API [=n] &&
VIDEO_CCS_PLL [=n] && V4L2_FWNODE [=n]
>
> ---
>
> CONFIG_VIDEO_OV02A10:
>
> This is a Video4Linux2 sensor driver for the OmniVision
> OV02A10 camera.
>
> To compile this driver as a module, choose M here: the
> module will be called ov02a10.
>
> Symbol: VIDEO_OV02A10 [=n]
> Type : tristate
> Defined at drivers/media/i2c/Kconfig:828
> Prompt: OmniVision OV02A10 sensor support
> Depends on: MEDIA_SUPPORT [=m] && VIDEO_V4L2 [=m] && I2C [=y]
> Visible if: MEDIA_SUPPORT [=m] && VIDEO_V4L2 [=m] && I2C [=y]
&& MEDIA_CAMERA_SUPPORT [=y]
> Location:
> -> Device Drivers
> -> Multimedia support (MEDIA_SUPPORT [=m])
> -> Media ancillary drivers
> -> Camera sensor devices
> Selects: MEDIA_CONTROLLER [=y] && VIDEO_V4L2_SUBDEV_API [=n] &&
V4L2_FWNODE [=n]
>
> ---
>
> CONFIG_VIDEO_OV9734:
>
> This is a Video4Linux2 sensor driver for the OmniVision
> OV9734 camera.
>
> To compile this driver as a module, choose M here: the
> module's name is ov9734.
>
> Symbol: VIDEO_OV9734 [=n]
> Type : tristate
> Defined at drivers/media/i2c/Kconfig:1067
> Prompt: OmniVision OV9734 sensor support
> Depends on: MEDIA_SUPPORT [=m] && VIDEO_V4L2 [=m] && I2C [=y]
&& (ACPI [=y] || COMPILE_TEST [=n])
> Visible if: MEDIA_SUPPORT [=m] && VIDEO_V4L2 [=m] && I2C [=y]
&& (ACPI [=y] || COMPILE_TEST [=n]) && MEDIA_CAMERA_SUPPORT [=y]
> Location:
> -> Device Drivers
> -> Multimedia support (MEDIA_SUPPORT [=m])
> -> Media ancillary drivers
> -> Camera sensor devices
> Selects: MEDIA_CONTROLLER [=y] && VIDEO_V4L2_SUBDEV_API [=n] &&
V4L2_FWNODE [=n]
>
> ---
>
> Signed-off-by: Fedora Kernel Team <kernel-team(a)fedoraproject.org>
> ---
> .../configs/common/generic/CONFIG_VIDEO_CCS | 1 +
> .../common/generic/CONFIG_VIDEO_OV02A10 | 1 +
> .../common/generic/CONFIG_VIDEO_OV9734 | 1 +
> .../pending-common/generic/CONFIG_VIDEO_CCS | 21 ----------------
> .../generic/CONFIG_VIDEO_OV02A10 | 24 -------------------
> .../generic/CONFIG_VIDEO_OV9734 | 24 -------------------
> 6 files changed, 3 insertions(+), 69 deletions(-)
> create mode 100644 redhat/configs/common/generic/CONFIG_VIDEO_CCS
> create mode 100644 redhat/configs/common/generic/CONFIG_VIDEO_OV02A10
> create mode 100644 redhat/configs/common/generic/CONFIG_VIDEO_OV9734
> delete mode 100644 redhat/configs/pending-common/generic/CONFIG_VIDEO_CCS
> delete mode 100644 redhat/configs/pending-common/generic/CONFIG_VIDEO_OV02A10
> delete mode 100644 redhat/configs/pending-common/generic/CONFIG_VIDEO_OV9734
>
> diff --git a/redhat/configs/common/generic/CONFIG_VIDEO_CCS
b/redhat/configs/common/generic/CONFIG_VIDEO_CCS
> new file mode 100644
> index 000000000000..0812c0b6bb5c
> --- /dev/null
> +++ b/redhat/configs/common/generic/CONFIG_VIDEO_CCS
> @@ -0,0 +1 @@
> +# CONFIG_VIDEO_CCS is not set
> diff --git a/redhat/configs/common/generic/CONFIG_VIDEO_OV02A10
b/redhat/configs/common/generic/CONFIG_VIDEO_OV02A10
> new file mode 100644
> index 000000000000..e7120589ee56
> --- /dev/null
> +++ b/redhat/configs/common/generic/CONFIG_VIDEO_OV02A10
> @@ -0,0 +1 @@
> +# CONFIG_VIDEO_OV02A10 is not set
> diff --git a/redhat/configs/common/generic/CONFIG_VIDEO_OV9734
b/redhat/configs/common/generic/CONFIG_VIDEO_OV9734
> new file mode 100644
> index 000000000000..53715cae2f85
> --- /dev/null
> +++ b/redhat/configs/common/generic/CONFIG_VIDEO_OV9734
> @@ -0,0 +1 @@
> +# CONFIG_VIDEO_OV9734 is not set
> diff --git a/redhat/configs/pending-common/generic/CONFIG_VIDEO_CCS
b/redhat/configs/pending-common/generic/CONFIG_VIDEO_CCS
> deleted file mode 100644
> index 57199497610e..000000000000
> --- a/redhat/configs/pending-common/generic/CONFIG_VIDEO_CCS
> +++ /dev/null
> @@ -1,21 +0,0 @@
> -# CONFIG_VIDEO_CCS:
> -#
> -# This is a generic driver for MIPI CCS, SMIA++ and SMIA compliant
> -# camera sensors.
> -#
> -# Symbol: VIDEO_CCS [=n]
> -# Type : tristate
> -# Defined at drivers/media/i2c/ccs/Kconfig:2
> -# Prompt: MIPI CCS/SMIA++/SMIA sensor support
> -# Depends on: MEDIA_SUPPORT [=m] && I2C [=y] && VIDEO_V4L2 [=m]
&& HAVE_CLK [=y]
> -# Visible if: MEDIA_SUPPORT [=m] && I2C [=y] && VIDEO_V4L2 [=m]
&& HAVE_CLK [=y] && MEDIA_CAMERA_SUPPORT [=y]
> -# Location:
> -# -> Device Drivers
> -# -> Multimedia support (MEDIA_SUPPORT [=m])
> -# -> Media ancillary drivers
> -# -> Camera sensor devices
> -# Selects: MEDIA_CONTROLLER [=y] && VIDEO_V4L2_SUBDEV_API [=n] &&
VIDEO_CCS_PLL [=n] && V4L2_FWNODE [=n]
> -#
> -#
> -#
> -# CONFIG_VIDEO_CCS is not set
> diff --git a/redhat/configs/pending-common/generic/CONFIG_VIDEO_OV02A10
b/redhat/configs/pending-common/generic/CONFIG_VIDEO_OV02A10
> deleted file mode 100644
> index 935c179ab020..000000000000
> --- a/redhat/configs/pending-common/generic/CONFIG_VIDEO_OV02A10
> +++ /dev/null
> @@ -1,24 +0,0 @@
> -# CONFIG_VIDEO_OV02A10:
> -#
> -# This is a Video4Linux2 sensor driver for the OmniVision
> -# OV02A10 camera.
> -#
> -# To compile this driver as a module, choose M here: the
> -# module will be called ov02a10.
> -#
> -# Symbol: VIDEO_OV02A10 [=n]
> -# Type : tristate
> -# Defined at drivers/media/i2c/Kconfig:828
> -# Prompt: OmniVision OV02A10 sensor support
> -# Depends on: MEDIA_SUPPORT [=m] && VIDEO_V4L2 [=m] && I2C [=y]
> -# Visible if: MEDIA_SUPPORT [=m] && VIDEO_V4L2 [=m] && I2C [=y]
&& MEDIA_CAMERA_SUPPORT [=y]
> -# Location:
> -# -> Device Drivers
> -# -> Multimedia support (MEDIA_SUPPORT [=m])
> -# -> Media ancillary drivers
> -# -> Camera sensor devices
> -# Selects: MEDIA_CONTROLLER [=y] && VIDEO_V4L2_SUBDEV_API [=n] &&
V4L2_FWNODE [=n]
> -#
> -#
> -#
> -# CONFIG_VIDEO_OV02A10 is not set
> diff --git a/redhat/configs/pending-common/generic/CONFIG_VIDEO_OV9734
b/redhat/configs/pending-common/generic/CONFIG_VIDEO_OV9734
> deleted file mode 100644
> index 576a5bfff73d..000000000000
> --- a/redhat/configs/pending-common/generic/CONFIG_VIDEO_OV9734
> +++ /dev/null
> @@ -1,24 +0,0 @@
> -# CONFIG_VIDEO_OV9734:
> -#
> -# This is a Video4Linux2 sensor driver for the OmniVision
> -# OV9734 camera.
> -#
> -# To compile this driver as a module, choose M here: the
> -# module's name is ov9734.
> -#
> -# Symbol: VIDEO_OV9734 [=n]
> -# Type : tristate
> -# Defined at drivers/media/i2c/Kconfig:1067
> -# Prompt: OmniVision OV9734 sensor support
> -# Depends on: MEDIA_SUPPORT [=m] && VIDEO_V4L2 [=m] && I2C [=y]
&& (ACPI [=y] || COMPILE_TEST [=n])
> -# Visible if: MEDIA_SUPPORT [=m] && VIDEO_V4L2 [=m] && I2C [=y]
&& (ACPI [=y] || COMPILE_TEST [=n]) && MEDIA_CAMERA_SUPPORT [=y]
> -# Location:
> -# -> Device Drivers
> -# -> Multimedia support (MEDIA_SUPPORT [=m])
> -# -> Media ancillary drivers
> -# -> Camera sensor devices
> -# Selects: MEDIA_CONTROLLER [=y] && VIDEO_V4L2_SUBDEV_API [=n] &&
V4L2_FWNODE [=n]
> -#
> -#
> -#
> -# CONFIG_VIDEO_OV9734 is not set
> --
> GitLab
> _______________________________________________
> kernel mailing list -- kernel(a)lists.fedoraproject.org
> To unsubscribe send an email to kernel-leave(a)lists.fedoraproject.org
> Fedora Code of Conduct:
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
> List Guidelines:
https://fedoraproject.org/wiki/Mailing_list_guidelines
> List Archives:
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org