[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for acpi/pwm/i915: Convert pwm-crc and i915 driver's PWM code to use the atomic PWM API

Patchwork patchwork at emeril.freedesktop.org
Tue Aug 25 10:25:44 UTC 2020


== Series Details ==

Series: acpi/pwm/i915: Convert pwm-crc and i915 driver's PWM code to use the atomic PWM API
URL   : https://patchwork.freedesktop.org/series/80976/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
ef85c003b00a ACPI / LPSS: Resume Cherry Trail PWM controller in no-irq phase
1d26b9fe569e ACPI / LPSS: Save Cherry Trail PWM ctx registers only once (at activation)
5dd4db0dc219 pwm: lpss: Fix off by one error in base_unit math in pwm_lpss_prepare()
a4e4727f40e6 pwm: lpss: Add range limit check for the base_unit register value
0d3912600257 pwm: lpss: Add pwm_lpss_prepare_enable() helper
-:45: CHECK:BOOL_COMPARISON: Using comparison to false is error prone
#45: FILE: drivers/pwm/pwm-lpss.c:137:
+	pwm_lpss_cond_enable(pwm, enable && lpwm->info->bypass == false);

-:50: CHECK:BOOL_COMPARISON: Using comparison to true is error prone
#50: FILE: drivers/pwm/pwm-lpss.c:142:
+	pwm_lpss_cond_enable(pwm, enable && lpwm->info->bypass == true);

total: 0 errors, 0 warnings, 2 checks, 67 lines checked
9fb3d675a6b6 pwm: lpss: Use pwm_lpss_apply() when restoring state on resume
2dda19e1952a pwm: crc: Fix period / duty_cycle times being off by a factor of 256
2061f911c93f pwm: crc: Fix off-by-one error in the clock-divider calculations
2e80db8eb3ff pwm: crc: Fix period changes not having any effect
04e01429ab69 pwm: crc: Enable/disable PWM output on enable/disable
f0758f16468b pwm: crc: Implement apply() method to support the new atomic PWM API
9a833ba135f1 pwm: crc: Implement get_state() method
ace83a7f4e48 drm/i915: panel: Add get_vbt_pwm_freq() helper
6c5200023ed2 drm/i915: panel: Honor the VBT PWM frequency for devs with an external PWM controller
5cba5633c584 drm/i915: panel: Honor the VBT PWM min setting for devs with an external PWM controller
871bbce513b9 drm/i915: panel: Use atomic PWM API for devs with an external PWM controller




More information about the Intel-gfx mailing list