On Thu, Apr 20, 2023 at 01:05:27PM -0700, Umesh Nerlige Ramappa wrote: > On Thu, Mar 16, 2023 at 01:25:48PM -0700, Radhakrishna Sripada wrote: > > From: Vinay Belgaumkar <vinay.belgaumkar@xxxxxxxxx> > > > > If BIOS enables/disables C6, i915 should do the same. Also, retain > > this value across driver reloads. This is needed only for MTL as > > of now due to an existing bug in OA which needs C6 disabled for > > it to function. BIOS behavior is also different across platforms > > in terms of how C6 is enabled. > > > > v2: Review comments (Umesh) > > v3: Cache the C6 enable value for all MTL. The OA WA is needed only > > for A/B step, but we don't need to check for that here. > > v4: Rename to mtl_check_bios_c6_setup() > > > > Signed-off-by: Vinay Belgaumkar <vinay.belgaumkar@xxxxxxxxx> > > Signed-off-by: Radhakrishna Sripada <radhakrishna.sripada@xxxxxxxxx> > > This one's already pushed with the MTL OA series. Thank you for the update. Skipped this patch in the latest rev at [1]. [1] https://patchwork.freedesktop.org/series/115292/#rev3 - Radhakrishna Sripada > > Regards, > Umesh