[PATCH] riscv: config: Set MEDIA_SUPPORT_FILTER

Hal Feng hal.feng at starfivetech.com
Tue Apr 8 22:47:44 PDT 2025


> On 04.04.25 14:23, Joel Stanley wrote:
> 
> When Hal enabled MEIDA_SUPPORT in order to add the
> VIDEO_CADENCE_CSI2RX driver in commit d8a7d89abb09 ("riscv: defconfig:
> Enable StarFive JH7110 drivers"), it turned on a few hundred (254) other
> MEDIA drivers.
> 
> The subsystem has a option for this very case that keeps the other drivers
> disabled: MEDIA_SUPPORT_FILTER. Turn that on, and
> MEDIA_PLATFORM_SUPPORT, so the CSI2RX driver stays enabled.
> 
> Signed-off-by: Joel Stanley <joel at jms.id.au>
> ---
> The filter option only appears in a small handful of arm32 defconfigs (and I
> maintain a few of them). That's because MEDIA_SUPPORT_FILTER is normally
> enabled unless EXPERT is on.
> 
> The root of the issue is the riscv defconfig selects EXPERT. Most arch-wide
> defconfig don't have that on. Need to get to the bottom of that, and perhaps
> disable EXPERT?
> 
> Signed-off-by: Joel Stanley <joel at jms.id.au>
> ---
>  arch/riscv/configs/defconfig | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/arch/riscv/configs/defconfig b/arch/riscv/configs/defconfig index
> 0f7dcbe3c45b..c34fcf2c6a2f 100644
> --- a/arch/riscv/configs/defconfig
> +++ b/arch/riscv/configs/defconfig
> @@ -182,6 +182,8 @@ CONFIG_REGULATOR_FIXED_VOLTAGE=y
> CONFIG_REGULATOR_AXP20X=y  CONFIG_REGULATOR_GPIO=y
> CONFIG_MEDIA_SUPPORT=m
> +CONFIG_MEDIA_SUPPORT_FILTER=y
> +CONFIG_MEDIA_PLATFORM_SUPPORT=y
>  CONFIG_VIDEO_CADENCE_CSI2RX=m
>  CONFIG_DRM=m
>  CONFIG_DRM_RADEON=m

Reviewed-by: Hal Feng <hal.feng at starfivetech.com>

Best regards,
Hal



More information about the linux-riscv mailing list