[PATCH] drm/amd/display: fix DCC settings for DCN3

Marek Olšák maraeo at gmail.com
Thu Sep 30 17:33:36 UTC 2021


The name is kind of correct. It means "64B with no 128B cache line
straddling", which really means just 64B independent blocks with a small
modification to support DCC image stores.  They are not true 128B
independent blocks.

Marek

On Thu, Sep 30, 2021 at 12:35 PM Joshua Ashton <joshua at froggi.es> wrote:

> Can we please add documentation for this enum?
>
> This was not necessarily a typo, but me misunderstanding and stuff it
> working in my testing.
>
> I guess I don't understand why hubp_ind_block_64b_no_128bcl is for 64b
> && 128b when it specifically says "no_128" in the name.
>
> Is there something about it I am missing or is it just misleading naming?
>
> - Joshie 🐸✨
>
> On 9/30/21 17:14, Marek Olšák wrote:
> > I've also amended the version bump that I forgot to do:
> >
> > -#define KMS_DRIVER_MINOR       43
> > +#define KMS_DRIVER_MINOR       44
> >
> > Marek
> >
> > On Thu, Sep 30, 2021 at 12:06 PM Alex Deucher <alexdeucher at gmail.com
> > <mailto:alexdeucher at gmail.com>> wrote:
> >
> >     Acked-by: Alex Deucher <alexander.deucher at amd.com
> >     <mailto:alexander.deucher at amd.com>>
> >
> >     On Thu, Sep 30, 2021 at 11:50 AM Marek Olšák <maraeo at gmail.com
> >     <mailto:maraeo at gmail.com>> wrote:
> >      >
> >      > Hi,
> >      >
> >      > Just discovered this typo. Please review.
> >      >
> >      > Thanks,
> >      > Marek
> >
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20210930/18e87365/attachment.htm>


More information about the amd-gfx mailing list