[PATCH v3 2/2] BeaglePlay: enable IT66121 HDMI bridge and video stack

[Moteen Shah]

Hey Shaurya,

On 21/04/26 18:51, Shaurya Rane wrote:
> Bring in DTS bootph-all tags for the DSS, I2C2, HDMI bridge and
> pinmux nodes, plus defconfig options for TIDSS, VIDEO_BRIDGE_ITE_IT66121,
> BMP and splash screen support.
>
> Signed-off-by: Shaurya Rane <ssrane_b23 at ee.vjti.ac.in>
> Reviewed-by: Simon Glass <sjg at chromium.org>
> ---
> Changes in v3:
>   - No functional changes; carry Reviewed-by tag from Simon Glass.
>
> Changes in v2:
>   - Split out of the original patch; add bootph-all DT fragments
>     alongside the defconfig options enabling TIDSS, the IT66121
>     bridge driver, and splash-screen/BMP support.
>
>   arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi | 42 ++++++++++++++++++++
>   configs/am62x_beagleplay_a53_defconfig       | 13 ++++++
>   2 files changed, 55 insertions(+)
>
> diff --git a/arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi b/arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi
> index 2a4f0e45365..b1ba62c20ad 100644
> --- a/arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi
> +++ b/arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi
> @@ -322,3 +322,45 @@
>   &cpsw_port2 {
>   	status = "disabled";
>   };
> +
> +/* ===== Display / HDMI support for U-Boot splash screen ===== */
> +
> +&main_pmx0 {
> +	bootph-all;


I don't think that you have to mention bootph-all in the parent node 
here as it is implied by the subnode(s) already below.

Refer [0] for more context.

> +
> +	dss0_pins_default {
> +		bootph-all;
> +	};
> +
> +	hdmi_gpio_pins_default {
> +		bootph-all;
> +	};
> +
> +	i2c2_1v8_pins_default {
> +		bootph-all;
> +	};
> +};
> +
> +&dss {
> +	bootph-all;
> +};
> +
> +&dss_ports {
> +	bootph-all;
> +
> +	port at 1 {
> +		bootph-all;
> +	};
> +};
> +
> +&main_i2c2 {
> +	bootph-all;
> +
> +	bridge-hdmi at 4c {
> +		bootph-all;
> +	};
> +};
> +
> +&main_gpio0 {
> +	bootph-all;
> +};
> diff --git a/configs/am62x_beagleplay_a53_defconfig b/configs/am62x_beagleplay_a53_defconfig
> index c9bc7cd8d39..bb4be38537c 100644
> --- a/configs/am62x_beagleplay_a53_defconfig
> +++ b/configs/am62x_beagleplay_a53_defconfig
> @@ -121,6 +121,19 @@ CONFIG_EXT4_WRITE=y
>   CONFIG_FS_FAT_MAX_CLUSTSIZE=16384
>   CONFIG_LZO=y
>   CONFIG_EFI_SET_TIME=y
> +CONFIG_DM_VIDEO=y
> +CONFIG_VIDEO=y
> +CONFIG_VIDEO_TIDSS=y
> +CONFIG_VIDEO_BRIDGE=y
> +CONFIG_VIDEO_BRIDGE_ITE_IT66121=y
> +CONFIG_BACKLIGHT_GPIO=y
> +CONFIG_SYS_WHITE_ON_BLACK=y
> +CONFIG_BMP=y
> +CONFIG_CMD_BMP=y
> +CONFIG_SPLASH_SCREEN=y
> +CONFIG_SPLASH_SCREEN_ALIGN=y
> +CONFIG_VIDEO_LOGO=y
> +CONFIG_I2C_EDID=y

Nitpick: This should come in alphabetical order, I think.

[0] https://lore.kernel.org/all/20250516114148.3862114-1-m-shah@ti.com/ 
for more context

Regards,
Moteen
>   
>   #include <configs/k3_efi_capsule.config>
>   #include <configs/am62x_a53_usbdfu.config>