On Fri, 7 Jun 2024 18:25:37 +0300 Jani Nikula <[email protected]> wrote:
> Range is a bit odd name for what really is stride. Rename. Switch to > u32 while at it. > > Cc: Zhenyu Wang <[email protected]> > Cc: Zhi Wang <[email protected]> > Cc: [email protected] > Signed-off-by: Jani Nikula <[email protected]> > --- > drivers/gpu/drm/i915/gvt/handlers.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/gvt/handlers.c > b/drivers/gpu/drm/i915/gvt/handlers.c index > b005ab0104ee..f79dd6cfc75b 100644 --- > a/drivers/gpu/drm/i915/gvt/handlers.c +++ > b/drivers/gpu/drm/i915/gvt/handlers.c @@ -887,12 +887,12 @@ static > unsigned int calc_index(unsigned int offset, i915_reg_t _start, u32 > start = i915_mmio_reg_offset(_start); u32 next = > i915_mmio_reg_offset(_next); u32 end = i915_mmio_reg_offset(_end); > - unsigned int range = next - start; > + u32 stride = next - start; > > if (offset < start || offset > end) > return INVALID_INDEX; > offset -= start; > - return offset / range; > + return offset / stride; > } > > #define FDI_RX_CTL_TO_PIPE(offset) \ Reviewed-by: Zhi Wang <[email protected]>
