[Intel-xe] [PATCH 1/3] drm/xe: Remove unused revid from firmware name

Lucas De Marchi lucas.demarchi at intel.com
Fri Mar 31 21:35:29 UTC 2023


On Mon, Mar 27, 2023 at 09:59:55AM -0700, Matt Roper wrote:
>On Thu, Mar 23, 2023 at 10:17:52PM -0700, Lucas De Marchi wrote:
>> The rev field is always 0 so it ends up never used. In i915 it was
>> introduced because of CML: up to rev 5 it reuses the guc and huc
>> firmware blobs from KBL. After that there is a specific firmware for
>> that platform.  This can be reintroduced later if ever needed.
>
>I doubt we'd ever need the revid again; more likely we'd want a way to
>select different firmwares for a given subplatform (which is something I
>think we need to add anyway for ADL-N).
>
>Reviewed-by: Matt Roper <matthew.d.roper at intel.com>

thanks, applied this first patch.

Lucas De Marchi


More information about the dri-devel mailing list