Re: [OS-BUILD PATCH] [redhat] New configs in drivers/media
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
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
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