RE: [PATCH v3] drm/i915/display: use x100 version for full version and release

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 30 Oct 2024, "Kandpal, Suraj" <suraj.kandpal@xxxxxxxxx> wrote:
>> -----Original Message-----
>> From: Intel-gfx <intel-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> On Behalf Of Jani
>> Nikula
>> Sent: Tuesday, October 29, 2024 9:26 PM
>> To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx; intel-xe@xxxxxxxxxxxxxxxxxxxxx
>> Cc: Nikula, Jani <jani.nikula@xxxxxxxxx>; ville.syrjala@xxxxxxxxxxxxxxx; Roper,
>> Matthew D <matthew.d.roper@xxxxxxxxx>; De Marchi, Lucas
>> <lucas.demarchi@xxxxxxxxx>
>> Subject: [PATCH v3] drm/i915/display: use x100 version for full version and
>> release
>>
>> Use x100, or ver * 100 + rel, versions for full IP version checks, similar to what
>> xe driver does:
>>
>> - Replace IP_VER(14, 1) inline with 1401, etc.
>>
>> - Convert DISPLAY_VER_FULL() to DISPLAY_VERx100()
>>
>> - Convert IS_DISPLAY_VER_FULL() to IS_DISPLAY_VERx100()
>>
>> - Convert IS_DISPLAY_VER_STEP() to IS_DISPLAY_VERx100_STEP()
>>
>> This makes ver.rel versions easier to use, follows the xe driver pattern, and
>> drops the dependency on the IP_VER() macro.
>>
>> v2: Rebase, drop IP_VER() from xe compat headers
>>
>> v3: Rebase
>>
>> Cc: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
>> Acked-by: Matt Roper <matthew.d.roper@xxxxxxxxx>
>> Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx>
>
> LGTM,
> Reviewed-by: Suraj Kandpal <suraj.kandpal@xxxxxxxxx>

Thanks for the review, pushed to din.

BR,
Jani.


-- 
Jani Nikula, Intel




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux