Re: [OS-BUILD PATCH] [redhat] New configs in drivers/media

2021-04-14 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/906#note_551891693

@jmflinuxtx - this changes common, do you want to ack this for Fedora?
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@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
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCH] [redhat] New configs in drivers/media

2020-12-15 Thread David Arcari

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  wrote:


From: Fedora Kernel Team 

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 



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 
---
  .../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 ..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 ..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 ..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 

Re: [OS-BUILD PATCH] [redhat] New configs in drivers/media

2020-12-15 Thread Patrick Talbert
On Tue, Dec 15, 2020 at 2:07 PM GitLab Bridge on behalf of
redhat-patchlab  wrote:
>
> From: Fedora Kernel Team 
>
> 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?

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 
> ---
>  .../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 ..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 ..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 ..53715cae2f85
> --- /dev/null
> +++ b/redhat/configs/common/generic/CONFIG_VIDEO_OV9734
> @@ -0,0 +1 @@
> +# CONFIG_VIDEO_OV9734 is not set
> diff --git