[PATCH 09/12] drm/i915: Use REG_BIT() & co. for BDW+ EU/slice fuse bits
Jani Nikula
jani.nikula at linux.intel.com
Tue Feb 25 07:52:41 UTC 2025
On Mon, 24 Feb 2025, Andi Shyti <andi.shyti at linux.intel.com> wrote:
> Hi Ville,
>
> On Wed, Feb 12, 2025 at 01:19:37AM +0200, Ville Syrjala wrote:
>> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
>>
>> Convert the BDW+ EU/slice fuse bits to the modern REG_BIT()/etc.
>> style.
>
> using REG_BIT() and co. doesn't alway make it more readable. In
> some of the cases below I would have preferred not to use it.
Interesting. I read through the patch and I thought all of it was
good. Care to elaborate?
BR,
Jani.
>
> But on the other hand we keep consistency and bit operations are
> not supposed to be easy-readable, anyway.
>
> Reviewed-by: Andi Shyti <andi.shyti at linux.intel.com>
>
> Thanks,
> Andi
--
Jani Nikula, Intel
More information about the Intel-gfx
mailing list