Re: [PATCH v2] arm64: defconfig: Add CDNS_DSI and CDNS_PHY config

[Judith Mendez <jm@ti.com>]

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@ti.com>

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

Reviewed-by: Judith Mendez <jm@ti.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