linux-kernel - Re: [PATCH v2 21/38] media: platform: camss: move config to its own file

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAG3jFyt8H=X0DTeu9AHu5meOZj2eXAzX+i1dBvPbNZHmHqTTCw@mail.gmail.com>
Date:   Sat, 12 Mar 2022 11:07:39 +0100
From:   Robert Foss <robert.foss@...aro.org>
To:     Mauro Carvalho Chehab <mchehab@...nel.org>
Cc:     Niklas Söderlund 
        <niklas.soderlund+renesas@...natech.se>,
        Andy Gross <agross@...nel.org>,
        Bjorn Andersson <bjorn.andersson@...aro.org>,
        Dmitry Osipenko <digetx@...il.com>,
        Hans Verkuil <hverkuil-cisco@...all.nl>,
        Laurent Pinchart <laurent.pinchart@...asonboard.com>,
        Ming Qian <ming.qian@....com>,
        Todor Tomov <todor.too@...il.com>,
        linux-arm-msm@...r.kernel.org, linux-kernel@...r.kernel.org,
        linux-media@...r.kernel.org
Subject: Re: [PATCH v2 21/38] media: platform: camss: move config to its own file

On Fri, 11 Mar 2022 at 15:09, Mauro Carvalho Chehab <mchehab@...nel.org> wrote:
>
> In order to better organize the platform/Kconfig, place
> camss-specific config stuff on a separate Kconfig file.
>
> Signed-off-by: Mauro Carvalho Chehab <mchehab@...nel.org>
> Signed-off-by: Mauro Carvalho Chehab <mchehab@...nel.org>
> ---
>
> To avoid mailbombing on a large number of people, only mailing lists were C/C on the cover.
> See [PATCH v2 00/38] at: https://lore.kernel.org/all/cover.1647006877.git.mchehab@kernel.org/
>
>  drivers/media/platform/Kconfig            | 11 +----------
>  drivers/media/platform/qcom/camss/Kconfig |  9 +++++++++
>  2 files changed, 10 insertions(+), 10 deletions(-)
>  create mode 100644 drivers/media/platform/qcom/camss/Kconfig
>
> diff --git a/drivers/media/platform/Kconfig b/drivers/media/platform/Kconfig
> index f65eefa15dda..19ca01d2f841 100644
> --- a/drivers/media/platform/Kconfig
> +++ b/drivers/media/platform/Kconfig
> @@ -61,6 +61,7 @@ source "drivers/media/platform/mtk-vcodec/Kconfig"
>  source "drivers/media/platform/mtk-vpu/Kconfig"
>  source "drivers/media/platform/omap3isp/Kconfig"
>  source "drivers/media/platform/omap/Kconfig"
> +source "drivers/media/platform/qcom/camss/Kconfig"
>
>  source "drivers/media/platform/aspeed/Kconfig"
>
> @@ -78,16 +79,6 @@ config VIDEO_MUX
>
>  source "drivers/media/platform/intel/Kconfig"
>
> -config VIDEO_QCOM_CAMSS
> -       tristate "Qualcomm V4L2 Camera Subsystem driver"
> -       depends on V4L_PLATFORM_DRIVERS
> -       depends on VIDEO_V4L2
> -       depends on (ARCH_QCOM && IOMMU_DMA) || COMPILE_TEST
> -       select MEDIA_CONTROLLER
> -       select VIDEO_V4L2_SUBDEV_API
> -       select VIDEOBUF2_DMA_SG
> -       select V4L2_FWNODE
> -
>  config VIDEO_S3C_CAMIF
>         tristate "Samsung S3C24XX/S3C64XX SoC Camera Interface driver"
>         depends on V4L_PLATFORM_DRIVERS
> diff --git a/drivers/media/platform/qcom/camss/Kconfig b/drivers/media/platform/qcom/camss/Kconfig
> new file mode 100644
> index 000000000000..56be91578706
> --- /dev/null
> +++ b/drivers/media/platform/qcom/camss/Kconfig
> @@ -0,0 +1,9 @@
> +config VIDEO_QCOM_CAMSS
> +       tristate "Qualcomm V4L2 Camera Subsystem driver"
> +       depends on V4L_PLATFORM_DRIVERS
> +       depends on VIDEO_V4L2
> +       depends on (ARCH_QCOM && IOMMU_DMA) || COMPILE_TEST
> +       select MEDIA_CONTROLLER
> +       select VIDEO_V4L2_SUBDEV_API
> +       select VIDEOBUF2_DMA_SG
> +       select V4L2_FWNODE
> --
> 2.35.1
>

Reviewed-by: Robert Foss <robert.foss@...aro.org>