On Thu, Jun 25, 2026 at 01:44:42PM +0200, Andi Shyti wrote: > intel_has_gpu_reset() returns logically correct values by > returning a function pointer when GPU reset is supported and > NULL otherwise. However, as a boolean helper, it is more > appropriate to return explicit true or false values. > > Signed-off-by: Andi Shyti <[email protected]> > --- > drivers/gpu/drm/i915/gt/intel_reset.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/gt/intel_reset.c > b/drivers/gpu/drm/i915/gt/intel_reset.c > index b2cf672564dd..a3f198646e1d 100644 > --- a/drivers/gpu/drm/i915/gt/intel_reset.c > +++ b/drivers/gpu/drm/i915/gt/intel_reset.c > @@ -797,9 +797,9 @@ static int __intel_gt_reset(struct intel_gt *gt, > intel_engine_mask_t engine_mask > bool intel_has_gpu_reset(const struct intel_gt *gt) > { > if (!gt->i915->params.reset) > - return NULL; > + return false;
I know the logic remains the same but isn't this Fixes material? Raag > - return intel_get_gpu_reset(gt); > + return !!intel_get_gpu_reset(gt); > } > > bool intel_has_reset_engine(const struct intel_gt *gt) > -- > 2.53.0 >
