[PATCH] drm/amd/display: fix DCC settings for DCN3
Joshua Ashton
joshua at froggi.es
Thu Sep 30 17:37:09 UTC 2021
Thanks for the info!
- Joshie 🐸✨
On 9/30/21 18:33, Marek Olšák wrote:
> 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
> <mailto: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>
> > <mailto: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>
> > <mailto: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>
> > <mailto:maraeo at gmail.com <mailto:maraeo at gmail.com>>> wrote:
> > >
> > > Hi,
> > >
> > > Just discovered this typo. Please review.
> > >
> > > Thanks,
> > > Marek
> >
>
More information about the amd-gfx
mailing list