On 2023-05-08 14:46:10, Jessica Zhang wrote: > > > On 5/7/2023 9:00 AM, Marijn Suijten wrote: > > On 2023-05-05 14:23:50, Jessica Zhang wrote: > >> Add DATA_COMPRESS feature flag to DPU INTF block. > >> > >> In DPU 7.x and later, DSC/DCE enablement registers have been moved from > >> PINGPONG to INTF. > >> > >> As core_rev (and related macros) was removed from the dpu_kms struct, the > >> most straightforward way to indicate the presence of this register would be > >> to have a feature flag. > > > > Irrelevant. Even though core_rev was still in mainline until recently, > > we always hardcoded the features in the catalog and only used core_rev > > to select a dpu_mdss_cfg catalog entry. There is no "if version >= X > > then enable feature Y" logic, this manually-enabled feature flag is the > > only, correct way to do it. > > Hi Marijn, > > Understood. Thanks if you can drop the paragraph. > FWIW, if we do find more register bit-level differences > between HW versions in the future, it might make more sense to keep the > HW catalog small and bring core_rev back, rather than keep adding these > kinds of small differences to caps. That is not up to me to decide, but I do agree that DPU is currently "one big mess" where lots of things are hardcoded in the catalog (which isn't a bad thing, these things won't change but it does make it harder on us than if we could dynamically state "every DPU between these two revisions"), and certain other things are/were read back from hardware registers. As well as the sub-block feature flags that pain us :) - Marijn > Thanks, > > Jessica Zhang <snip>
