Using arithmetic operators with booleans is confusing. Switch to logical operators. Cc: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx> Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx> --- drivers/gpu/drm/i915/intel_dp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c index 4e7b8d..ef4992f 100644 --- a/drivers/gpu/drm/i915/intel_dp.c +++ b/drivers/gpu/drm/i915/intel_dp.c @@ -5094,7 +5094,7 @@ static void icl_update_tc_port_type(struct drm_i915_private *dev_priv, enum port port = intel_dig_port->base.port; enum tc_port_type old_type = intel_dig_port->tc_type; - WARN_ON(is_legacy + is_typec + is_tbt != 1); + WARN_ON(is_legacy || is_typec || !is_tbt); if (is_legacy) intel_dig_port->tc_type = TC_PORT_LEGACY; -- 2.20.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
