On Tue, Feb 08, 2022 at 02:45:11AM -0800, Lucas De Marchi wrote: > Add helpers on top of iosys_map_read_field() / > iosys_map_write_field() functions so they always use the right > arguments and make code easier to read. > > Cc: Matt Roper <matthew.d.roper@xxxxxxxxx> > Cc: Thomas Hellström <thomas.hellstrom@xxxxxxxxxxxxxxx> > Cc: Daniel Vetter <daniel@xxxxxxxx> > Cc: John Harrison <John.C.Harrison@xxxxxxxxx> > Cc: Matthew Brost <matthew.brost@xxxxxxxxx> > Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@xxxxxxxxx> Reviewed-by: Matt Atwood <matthew.s.atwood@xxxxxxxxx> > Signed-off-by: Lucas De Marchi <lucas.demarchi@xxxxxxxxx> > --- > drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c > index 13671b186908..9bf9096b8337 100644 > --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c > +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c > @@ -60,6 +60,13 @@ struct __guc_ads_blob { > struct guc_mmio_reg regset[0]; > } __packed; > > +#define ads_blob_read(guc_, field_) \ > + iosys_map_rd_field(&(guc_)->ads_map, 0, struct __guc_ads_blob, field_) > + > +#define ads_blob_write(guc_, field_, val_) \ > + iosys_map_wr_field(&(guc_)->ads_map, 0, struct __guc_ads_blob, \ > + field_, val_) > + > static u32 guc_ads_regset_size(struct intel_guc *guc) > { > GEM_BUG_ON(!guc->ads_regset_size); > -- > 2.35.1 >