On Thu, Feb 9, 2023 at 11:38 PM Randy Dunlap <rdun...@infradead.org> 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