[PATCH 1/1] PM: runtime: change stub pm_runtime_get_if_active() return 1 when !CONFIG_PM
Dan Carpenter
error27 at gmail.com
Thu Oct 29 11:59:04 PDT 2026
On Thu, Oct 29, 2026 at 12:30:37PM -0400, Frank.Li at oss.nxp.com wrote:
> Pattern C: ret > 0 / ret <= 0 / ret != 1 checks -- FIXED
> drivers/usb/core/offload.c: checks (!= 1), old -EINVAL caused
> -EBUSY even with !CONFIG_PM. Fixed: now returns 1, proceeds.
> drivers/accel/ivpu/ivpu_job.c: drm_WARN_ON(<= 0), old -EINVAL
> spuriously triggered WARN. Fixed: 1 > 0, no warn.
> drivers/gpu/drm/i915/intel_runtime_pm.c: checks (<= 0), old
> -EINVAL triggered error path. Fixed: 1 > 0, correct path.
> drivers/accel/ivpu/ivpu_sysfs.c: (> 0), old skipped freq read.
> Fixed: proceeds correctly without CONFIG_PM.
> drivers/gpu/drm/xe/xe_pm.c: wraps (> 0), all xe callers
> (xe_bo.c, xe_shrinker.c, xe_vsec.c, xe_ggtt.c, xe_guc_ct.c,
> xe_pxp.c) now correctly treat !CONFIG_PM as active.
> drivers/net/ipa/ipa_smp2p.c: power_on = (> 0), now correctly true.
> drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c: (> 0), now correctly
> executes stall register update.
> drivers/staging/media/ipu7/ipu7-buttress.c: active (<= 0) early
> return now skipped, correctly proceeds.
Pattern C is mostly bugs so this patch should have a Fixes tag since
it fixes bugs.
regards,
dan carpenter
More information about the linux-arm-kernel
mailing list