[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