From nobody Fri Dec 19 09:44:14 2025 Received: from mail.tuxedocomputers.com (mail.tuxedocomputers.com [157.90.84.7]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 01FF9226D1B for ; Fri, 11 Apr 2025 17:18:11 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=157.90.84.7 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744391895; cv=none; b=uok7JrwsfdB5eWVYD/cFvA0nNnS4FMLWJmzdkklzGOWi9CH4GzY9Fyl+vFyKeSVDy9X4lS7k+eJuWoeGfgbwVt0aHHK9hW/u37XhEXWF74PCE39kn29pI96M0ryxMp86Ta3bZ13C58jSxSgGqbqXhw0JEvaWSdHn4tdc2EcXlcw= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744391895; c=relaxed/simple; bh=kSg1yxImtDViKukVSV71HKAEdDTA2YQxsnjjTalcRbo=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=KqfWa38R0B0Agk0FIkQRbEW768ZGo0/Xw0Otv9A2uSWeNp2JMzqYwGDERnalsA2mP6Nd2IB2pB+r4W/siPMDFziVAwt/WaNdZWAY5s3dkAEffy2Kl0jbsVr3QNmmLN/C46YuxncOO+N08h4jhyXz1TGZIG0LPxCqjgVUk/DYcSc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=tuxedocomputers.com; spf=pass smtp.mailfrom=tuxedocomputers.com; dkim=pass (1024-bit key) header.d=tuxedocomputers.com header.i=@tuxedocomputers.com header.b=VbY9FhuJ; arc=none smtp.client-ip=157.90.84.7 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=tuxedocomputers.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=tuxedocomputers.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=tuxedocomputers.com header.i=@tuxedocomputers.com header.b="VbY9FhuJ" Received: from wse-pc.fritz.box (host-88-217-226-44.customer.m-online.net [88.217.226.44]) (Authenticated sender: wse@tuxedocomputers.com) by mail.tuxedocomputers.com (Postfix) with ESMTPA id 0240C2FC0057; Fri, 11 Apr 2025 19:18:02 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tuxedocomputers.com; s=default; t=1744391883; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=jvjWVlP9H3/pq4vkbNFgzNEUHg3ze8OlwhtXkulXxCM=; b=VbY9FhuJP+ruznSEX8AwohySD0gqx5yOFm4q9ywfaBh0/HmMB5ucm2ThEXDy3Alyl+/1AP IyOU4CzeZUB9XvduXd/CDFbFtIN8Upwk9Ss30gWL0Z3wl9jYQC4o1k1ULqGtT7xnAeWIuC 4aTkavyZbScNk4DYTJa8WjCk8Mb0j9w= Authentication-Results: mail.tuxedocomputers.com; auth=pass smtp.auth=wse@tuxedocomputers.com smtp.mailfrom=wse@tuxedocomputers.com From: Werner Sembach To: jani.nikula@linux.intel.com, rodrigo.vivi@intel.com, joonas.lahtinen@linux.intel.com, tursulin@ursulin.net, airlied@gmail.com, simona@ffwll.ch Cc: intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Werner Sembach Subject: [PATCH 1/1] drm/i915/display: Add quirk to force backlight type on some TUXEDO devices Date: Fri, 11 Apr 2025 17:55:08 +0200 Message-ID: <20250411171756.142777-2-wse@tuxedocomputers.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20250411171756.142777-1-wse@tuxedocomputers.com> References: <20250411171756.142777-1-wse@tuxedocomputers.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" The display backlight on TUXEDO DX1708 and InsanityBook 15 v1 with panels AUO 12701 and AUO 12701 must be forced to INTEL_DP_AUX_BACKLIGHT_ON to be able to control the brightness. This could already be archived via a module parameter, but this patch adds a quirk to apply this by default on the mentioned device + panel combinations. Signed-off-by: Werner Sembach Cc: stable@vger.kernel.org --- .../drm/i915/display/intel_dp_aux_backlight.c | 14 ++++++- drivers/gpu/drm/i915/display/intel_quirks.c | 42 +++++++++++++++++++ drivers/gpu/drm/i915/display/intel_quirks.h | 1 + 3 files changed, 56 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c b/driver= s/gpu/drm/i915/display/intel_dp_aux_backlight.c index c846ef4acf5b2..0cac04c98a3e3 100644 --- a/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c +++ b/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c @@ -34,12 +34,14 @@ * for some reason. */ =20 +#include #include "i915_utils.h" #include "intel_backlight.h" #include "intel_display_core.h" #include "intel_display_types.h" #include "intel_dp.h" #include "intel_dp_aux_backlight.h" +#include "intel_quirks.h" =20 /* * DP AUX registers for Intel's proprietary HDR backlight interface. We de= fine @@ -607,11 +609,21 @@ int intel_dp_aux_init_backlight_funcs(struct intel_co= nnector *connector) struct drm_device *dev =3D connector->base.dev; struct intel_panel *panel =3D &connector->panel; bool try_intel_interface =3D false, try_vesa_interface =3D false; + int enable_dpcd_backlight; + u32 pnl_id; =20 /* Check the VBT and user's module parameters to figure out which * interfaces to probe */ - switch (display->params.enable_dpcd_backlight) { + enable_dpcd_backlight =3D display->params.enable_dpcd_backlight; + if (enable_dpcd_backlight =3D=3D INTEL_DP_AUX_BACKLIGHT_AUTO && + intel_has_quirk(display, QUIRK_AUO_12701_21229_ENABLE_DPCD_BACKLIGHT)= ) { + pnl_id =3D drm_edid_get_panel_id(panel->fixed_edid); + if (pnl_id =3D=3D drm_edid_encode_panel_id('A', 'U', 'O', 0x319d) || + pnl_id =3D=3D drm_edid_encode_panel_id('A', 'U', 'O', 0x52ed)) + enable_dpcd_backlight =3D INTEL_DP_AUX_BACKLIGHT_ON; + } + switch (enable_dpcd_backlight) { case INTEL_DP_AUX_BACKLIGHT_OFF: return -ENODEV; case INTEL_DP_AUX_BACKLIGHT_AUTO: diff --git a/drivers/gpu/drm/i915/display/intel_quirks.c b/drivers/gpu/drm/= i915/display/intel_quirks.c index 8b30e9fd936e7..6f724e5712664 100644 --- a/drivers/gpu/drm/i915/display/intel_quirks.c +++ b/drivers/gpu/drm/i915/display/intel_quirks.c @@ -78,6 +78,12 @@ static void quirk_fw_sync_len(struct intel_dp *intel_dp) drm_info(display->drm, "Applying Fast Wake sync pulse count quirk\n"); } =20 +static void quirk_auo_12701_21229_enable_dpcd_backlight(struct intel_displ= ay *display) +{ + intel_set_quirk(display, QUIRK_AUO_12701_21229_ENABLE_DPCD_BACKLIGHT); + drm_info(display->drm, "Applying Display AUO Model 12701 and 21229 Enable= DPCD Backlight quirk\n"); +} + struct intel_quirk { int device; int subsystem_vendor; @@ -118,6 +124,12 @@ static int intel_dmi_no_pps_backlight(const struct dmi= _system_id *id) return 1; } =20 +static int intel_dmi_auo_12701_21229_enable_dpcd_backlight(const struct dm= i_system_id *id) +{ + DRM_INFO("Display AUO model 12701 and 21229 DPCD backlight control on %s\= n", id->ident); + return 1; +} + static const struct intel_dmi_quirk intel_dmi_quirks[] =3D { { .dmi_id_list =3D &(const struct dmi_system_id[]) { @@ -164,6 +176,36 @@ static const struct intel_dmi_quirk intel_dmi_quirks[]= =3D { }, .hook =3D quirk_no_pps_backlight_power_hook, }, + { + .dmi_id_list =3D &(const struct dmi_system_id[]) { + { + .callback =3D intel_dmi_auo_12701_21229_enable_dpcd_backlight, + .ident =3D "TUXEDO DX1708", + .matches =3D {DMI_MATCH(DMI_BOARD_NAME, "N8xEJEK"), + }, + }, + { + .callback =3D intel_dmi_auo_12701_21229_enable_dpcd_backlight, + .ident =3D "TUXEDO InsanityBook 15 v1", + .matches =3D {DMI_MATCH(DMI_BOARD_NAME, "P95_HP"), + }, + }, + { + .callback =3D intel_dmi_auo_12701_21229_enable_dpcd_backlight, + .ident =3D "TUXEDO InsanityBook 15 v1", + .matches =3D {DMI_MATCH(DMI_BOARD_NAME, "P95_HR"), + }, + }, + { + .callback =3D intel_dmi_auo_12701_21229_enable_dpcd_backlight, + .ident =3D "TUXEDO InsanityBook 15 v1", + .matches =3D {DMI_MATCH(DMI_BOARD_NAME, "P95_HP,HR,HQ"), + }, + }, + { } + }, + .hook =3D quirk_auo_12701_21229_enable_dpcd_backlight, + }, }; =20 static struct intel_quirk intel_quirks[] =3D { diff --git a/drivers/gpu/drm/i915/display/intel_quirks.h b/drivers/gpu/drm/= i915/display/intel_quirks.h index cafdebda75354..38bdbd65d7efb 100644 --- a/drivers/gpu/drm/i915/display/intel_quirks.h +++ b/drivers/gpu/drm/i915/display/intel_quirks.h @@ -20,6 +20,7 @@ enum intel_quirk_id { QUIRK_LVDS_SSC_DISABLE, QUIRK_NO_PPS_BACKLIGHT_POWER_HOOK, QUIRK_FW_SYNC_LEN, + QUIRK_AUO_12701_21229_ENABLE_DPCD_BACKLIGHT, }; =20 void intel_init_quirks(struct intel_display *display); --=20 2.43.0