U-boot bug - missing #ifdef guard in apalis-imx6.c

Technical Support
, ,
2

Hi, I’m the Toradex AI Assistant. The Toradex Support team will follow up with you shortly!

Did you like my answer?

:+1: :-1:

Hello,

Thank you for bringing this issue to our attention. You’ve correctly identified a potential problem with the U-Boot configuration for the Apalis iMX6 module, where the absence of an #ifdef guard for CONFIG_IMX_HDMI in the displays[] array can lead to compile errors if CONFIG_VIDEO is enabled without CONFIG_IMX_HDMI being defined.

To address this, you can modify the U-Boot source code by adding an #ifdef guard around the HDMI display entry. This ensures that HDMI-related functions, such as detect_hdmi, are only included when CONFIG_IMX_HDMI is defined. Here’s how you can implement this fix:

  1. Locate the displays[] array in the apalis_imx6.c file within the U-Boot source directory.
  2. Surround the HDMI display entry with #ifdef CONFIG_IMX_HDMI and #endif preprocessor directives.

For example:

#ifdef CONFIG_IMX_HDMI
    // HDMI display entry
    {
        .detect = detect_hdmi,
        // other HDMI-specific configurations
    },
#endif

This change will prevent the compilation of HDMI-related code unless CONFIG_IMX_HDMI is explicitly defined, thus avoiding the compile errors you encountered.

If you are maintaining a custom U-Boot build, please ensure to test this modification thoroughly to confirm that it resolves the issue without introducing new problems.

Best regards,
Toradex AI Assistant

Please, also check if these links can help you solve your problem:

Compiling U-Boot for Verdin AM62.