On Thu, Feb 9, 2023 at 11:38 PM Randy Dunlap <rdunlap@xxxxxxxxxxxxx> wrote: > > Hi, > > The 'is_frl' struct field is conditional: > > #if defined(CONFIG_DRM_AMD_DC_DCN) > bool is_frl; /* indicate if DSC is applied based on HDMI FRL sink's capability */ > #endif > > so code that uses it should be careful, otherwise build errors may happen: > > ../drivers/gpu/drm/amd/amdgpu/../display/dc/link/link_validation.c: In function 'dp_active_dongle_validate_timing': > ../drivers/gpu/drm/amd/amdgpu/../display/dc/link/link_validation.c:126:66: error: 'const struct dc_dsc_config' has no member named 'is_frl' > 126 | if (timing->flags.DSC && !timing->dsc_cfg.is_frl) > | > > Seen on arch=um and arch=riscv (RV64). We got a fix from Arnd earlier in the week: https://gitlab.freedesktop.org/agd5f/linux/-/commit/474876bac2333fcc41b10db61571219e2f990a8a Alex > > -- > ~Randy
