[PATCH 3/8] video: tegra: Rework some of the driver dependencies

[Svyatoslav Ryhel]

ср, 6 серп. 2025 р. о 17:56 Tom Rini <trini at konsulko.com> пише:
>
> Looking these drivers over, all of them cannot build without access to
> some platform specific header files. Express those requirements in
> Kconfig as well. Furthermore, update the logic a bit more to reflect
> which parts are optional when other drivers are enabled and which parts
> cannot be enabled (meaningfully) by themselves.
>
> Signed-off-by: Tom Rini <trini at konsulko.com>
> ---
> Cc: Svyatoslav Ryhel <clamor95 at gmail.com>
> ---
>  configs/star_defconfig      |  1 +
>  drivers/video/tegra/Kconfig | 19 ++++++++++---------
>  2 files changed, 11 insertions(+), 9 deletions(-)
>

Thank you for cc-ing me in. Maybe it would be better/cleaner to gate
entire /tegra dir with ARCH_TEGRA dependency, after all, drivers in
there are all Tegra specific. With said or as is LGTM.

Reviewed-by: Svyatoslav Ryhel <clamor95 at gmail.com>

> diff --git a/configs/star_defconfig b/configs/star_defconfig
> index 38cbdec1c3a0..1547f724f062 100644
> --- a/configs/star_defconfig
> +++ b/configs/star_defconfig
> @@ -89,4 +89,5 @@ CONFIG_VIDEO_LCD_LG_LH400WV3=y
>  CONFIG_VIDEO_LCD_HITACHI_TX10D07VM0BAA=y
>  CONFIG_BACKLIGHT_AAT2870=y
>  CONFIG_VIDEO_BRIDGE=y
> +CONFIG_VIDEO_TEGRA=y
>  CONFIG_TEGRA_8BIT_CPU_BRIDGE=y
> diff --git a/drivers/video/tegra/Kconfig b/drivers/video/tegra/Kconfig
> index f32972d937e5..8bc29f2838bd 100644
> --- a/drivers/video/tegra/Kconfig
> +++ b/drivers/video/tegra/Kconfig
> @@ -1,10 +1,12 @@
>  config HOST1X_TEGRA
> -       bool "NVIDIA Tegra host1x BUS support"
> -       depends on SIMPLE_BUS
> +       bool
> +       select SIMPLE_BUS
> +       help
> +         NVIDIA Tegra host1x BUS support
>
>  config VIDEO_TEGRA
>         bool "Enable Display Controller support on Tegra devices"
> -       depends on OF_CONTROL
> +       depends on OF_CONTROL && ARCH_TEGRA
>         select HOST1X_TEGRA
>         help
>            Enable support for Display Controller found in Tegra SoC. The
> @@ -17,7 +19,7 @@ config VIDEO_TEGRA
>
>  config VIDEO_DSI_TEGRA
>         bool "Enable DSI controller support on Tegra devices"
> -       depends on VIDEO_BRIDGE && PANEL && DM_GPIO
> +       depends on VIDEO_BRIDGE && PANEL && DM_GPIO && ARCH_TEGRA
>         select VIDEO_TEGRA
>         select VIDEO_MIPI_DSI
>         help
> @@ -27,7 +29,7 @@ config VIDEO_DSI_TEGRA
>
>  config VIDEO_HDMI_TEGRA
>         bool "Enable HDMI support on Tegra devices"
> -       depends on VIDEO_BRIDGE && DM_I2C
> +       depends on VIDEO_BRIDGE && DM_I2C && ARCH_TEGRA
>         select I2C_EDID
>         select VIDEO_TEGRA
>         help
> @@ -36,16 +38,14 @@ config VIDEO_HDMI_TEGRA
>
>  config TEGRA_BACKLIGHT_PWM
>         bool "Enable Tegra DC PWM backlight support"
> -       depends on BACKLIGHT
> -       select VIDEO_TEGRA
> +       depends on BACKLIGHT && VIDEO_TEGRA
>         help
>            Enable support for the Display Controller dependent PWM backlight
>            found in the Tegra SoC and usually used with DSI panels.
>
>  config TEGRA_8BIT_CPU_BRIDGE
>         bool "Enable 8 bit panel communication protocol for Tegra 20/30"
> -       depends on VIDEO_BRIDGE && DM_GPIO
> -       select VIDEO_TEGRA
> +       depends on VIDEO_BRIDGE && DM_GPIO && VIDEO_TEGRA
>         select VIDEO_MIPI_DSI
>         help
>            Tegra 20 and Tegra 30 feature 8 bit CPU driver panel control
> @@ -54,6 +54,7 @@ config TEGRA_8BIT_CPU_BRIDGE
>
>  config VIDEO_TEGRA124
>         bool "Enable video support on Tegra124"
> +       depends on ARCH_TEGRA
>         imply VIDEO_DAMAGE
>         help
>            Tegra124 supports many video output options including eDP and
> --
> 2.43.0
>