> -----Original Message----- > From: Tolakanahalli Pradeep, Madhumitha > <madhumitha.tolakanahalli.pradeep@xxxxxxxxx> > Sent: Wednesday, December 8, 2021 8:25 AM > To: Srivatsa, Anusha <anusha.srivatsa@xxxxxxxxx>; De Marchi, Lucas > <lucas.demarchi@xxxxxxxxx> > Cc: intel-gfx@xxxxxxxxxxxxxxxxxxxxx > Subject: Re: [PATCH] drm/i915/dmc: Change DMC FW size on ADL- > P > > @Lucas Thanks for the R-b. > > The BSpec is currently missing size related info, we're working on getting it > added. > > Strange indeed that v2.12 was above 24kB. > > @Anusha, do you recall any size related issues for v2.12? No. I don’t recall any size related issue for any dmc version till 2.14 Anusha > - Madhumitha > > On Mon, 2021-12-06 at 22:44 -0800, Lucas De Marchi wrote: > > On Mon, Dec 06, 2021 at 06:37:18PM -0800, Madhumitha Tolakanahalli > > Pradeep wrote: > > > Increase the size of DMC on ADL-P to account for support of new > > > features in the current/upcoming DMC versions. > > > > I was trying to find anything related on Bspec 49193 and 49194, but > > didn't find anything related to size. > > > > However I see adl-p 2.12 firmware is already above the previous 24kB. > > How did we ever loaded DMC? Yes, this is not the file size, but rather > > the payload size, but AFAICS the rest should account for less than 1k, > > so it doesn't make a real difference. > > > > For this specific change: > > > > > > Reviewed-by: Lucas De Marchi <lucas.demarchi@xxxxxxxxx> > > > > thanks > > Lucas De Marchi >