linux-kernel - Re: [PATCH v2] arm64: defconfig: Add CDNS_DSI and CDNS

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] [day] [month] [year] [list]

Message-ID: <ac10eeeb-ab9e-4eaf-bf7c-7810c7ba64a5@ti.com>
Date: Wed, 23 Apr 2025 10:23:04 -0500
From: Judith Mendez <jm@...com>
To: Jayesh Choudhary <j-choudhary@...com>, <catalin.marinas@....com>,
        <will@...nel.org>, <nm@...com>, <linux-kernel@...r.kernel.org>
CC: <bjorn.andersson@....qualcomm.com>, <krzysztof.kozlowski@...aro.org>,
        <geert+renesas@...der.be>, <lumag@...nel.org>,
        <nfraprado@...labora.com>, <quic_tdas@...cinc.com>,
        <elinor.montmasson@...oirfairelinux.com>, <ebiggers@...gle.com>,
        <ross.burton@....com>, <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH v2] arm64: defconfig: Add CDNS_DSI and CDNS_PHY config

Hi Jayesh,

On 4/16/25 1:29 PM, Jayesh Choudhary wrote:
> Enable configs for cdns-dsi and cdns-dphy drivers required for DSI
> display on the following TI platforms:
> J721E-EVM, J721E-BeagleBone-AI64, J721S2-EVM, AM68-SK, J722S-EVM,
> BeagleY-AI, AM62P5-SK, J784S4-EVM and J742S2-EVM
> 
> The dsi wrapper driver used by all the above mentioned platforms,
> cdns-dsi-j721e is enabled by default if cdns-dsi config is enabled.
> 
> Signed-off-by: Jayesh Choudhary <j-choudhary@...com>

LGTM, thanks for adding AM62P5-SK to patch description.

Reviewed-by: Judith Mendez <jm@...com>

> ---
> 
> Changelog v1->v2:
> - Change commit message to add the platforms that needs cadence dsi
>    and dphy driver enabled.
> 
> Link to v1 patch:
> <https://lore.kernel.org/linux-arm-kernel/20250411095043.272488-1-j-choudhary@ti.com/>
> 
>   arch/arm64/configs/defconfig | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index 5c85e5da152f..4c0eb2711bbe 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -938,6 +938,7 @@ CONFIG_DRM_TI_SN65DSI86=m
>   CONFIG_DRM_ANALOGIX_ANX7625=m
>   CONFIG_DRM_I2C_ADV7511=m
>   CONFIG_DRM_I2C_ADV7511_AUDIO=y
> +CONFIG_DRM_CDNS_DSI=m
>   CONFIG_DRM_CDNS_MHDP8546=m
>   CONFIG_DRM_IMX8MP_DW_HDMI_BRIDGE=m
>   CONFIG_DRM_DW_HDMI_AHB_AUDIO=m
> @@ -1579,6 +1580,7 @@ CONFIG_PHY_CAN_TRANSCEIVER=m
>   CONFIG_PHY_NXP_PTN3222=m
>   CONFIG_PHY_SUN4I_USB=y
>   CONFIG_PHY_CADENCE_TORRENT=m
> +CONFIG_PHY_CADENCE_DPHY=m
>   CONFIG_PHY_CADENCE_DPHY_RX=m
>   CONFIG_PHY_CADENCE_SIERRA=m
>   CONFIG_PHY_CADENCE_SALVO=m