Re: [PATCH 4/5] drm/xe/xe_ras: Add structures and commands for get counter

[Raag Jadav <raag.jadav@intel.com>]

On Mon, Apr 06, 2026 at 10:53:13AM +0530, Tauro, Riana wrote:
> On 4/1/2026 1:28 PM, Raag Jadav wrote:
> > On Fri, Mar 20, 2026 at 03:55:59PM +0530, Riana Tauro wrote:

...

> > > +	/** @flags: Info queue control flags */
> > > +	u32 flags:8;
> > Why not u8?
> 
> flags, compression type and reserved are part of u32.
> 
> It's in-line with the firmware definition.
> 
> Also it is not recommended to mix the types if the bit fields belong to the
> same type.

By who? I thought it was just matter of readability since we already have
__packed in place? Anyway, I'll leave it upto you.

Raag

> > > +	/** @compression_type: Compression type used for this chunk */
> > > +	u32 compression_type:4;
> > Ditto.
> > 
> > > +	/** @reserved: Reserved for future use */
> > > +	u32 reserved:20;
> > > +	/** @checksum: Checksum of the chunk data */
> > > +	u32 checksum;
> > > +} __packed;