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 <[email protected]>
Reviewed-by: Simon Glass <[email protected]>
---
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@1 {
+ bootph-all;
+ };
+};
+
+&main_i2c2 {
+ bootph-all;
+
+ bridge-hdmi@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/[email protected]/
for more context
Regards,
Moteen
#include <configs/k3_efi_capsule.config>
#include <configs/am62x_a53_usbdfu.config>