From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id EFDAFC54E41 for ; Wed, 6 Mar 2024 09:23:48 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 08F6E10E868; Wed, 6 Mar 2024 09:23:48 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.b="Zhv+RRBJ"; dkim-atps=neutral Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.14]) by gabe.freedesktop.org (Postfix) with ESMTPS id E2CEC10E868 for ; Wed, 6 Mar 2024 09:23:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1709717028; x=1741253028; h=from:to:cc:subject:in-reply-to:references:date: message-id:mime-version; bh=8azOrkKOqGN52FwT3QPVf8+BwT58YaCcbX8PIF4gYek=; b=Zhv+RRBJ++jFZCskWTbxZGrjbVFzJg05q/jXT0l0ZdFMkMO3U+h3P2XX 1YIGL2HL+pJ6U2RlqhtQlrAiZ8fDEl4kQGSz4MpACDiQX97sw3mrvFssD NGAVUkL0tetdVkX/SKm/Er+juVq0eV/ps34ko3wxsYaoemV+n/V0zgkx7 azBcAaeBJTtY5fhvjfEXHtZbwpqaE9QNTn1gjFpM9rCwAM+2UxMKMHRJu sc4QokcQ0Ng02Fc/62FUgXMovMzc3wuV1dkxmNFVTuWUdL+FaGyASMy80 oPkqOCQkabBqVYUnfvyuCb20le1a34aKuTb/L4tZc9ZN1RtcMwZWaGSuY g==; X-IronPort-AV: E=McAfee;i="6600,9927,11004"; a="8135414" X-IronPort-AV: E=Sophos;i="6.06,207,1705392000"; d="scan'208";a="8135414" Received: from fmviesa010.fm.intel.com ([10.60.135.150]) by orvoesa106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 Mar 2024 01:23:46 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.06,207,1705392000"; d="scan'208";a="9635982" Received: from rjongalo-mobl2.ger.corp.intel.com (HELO localhost) ([10.252.33.211]) by fmviesa010-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 Mar 2024 01:23:41 -0800 From: Jani Nikula To: Hsin-Yi Wang , Dmitry Baryshkov , Douglas Anderson Cc: Neil Armstrong , Jessica Zhang , Sam Ravnborg , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Daniel Vetter , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v4 4/5] drm/panel-edp: Match edp_panels with panel identity In-Reply-To: <20240306004347.974304-5-hsinyi@chromium.org> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <20240306004347.974304-1-hsinyi@chromium.org> <20240306004347.974304-5-hsinyi@chromium.org> Date: Wed, 06 Mar 2024 11:23:37 +0200 Message-ID: <87msrbzpba.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" On Tue, 05 Mar 2024, Hsin-Yi Wang wrote: > It's found that some panels have variants that they share the same panel id > although their EDID and names are different. When matching generic edp > panels, we should first match with both panel identity, which contains both > panel id and panel name. If not found, match with panel id only. Do you want to start matching also with name, for all panels? That's totally up to you, but that's the big functional change here. BR, Jani. > > Signed-off-by: Hsin-Yi Wang > --- > v3->v4: combine name and id to identity. > --- > drivers/gpu/drm/panel/panel-edp.c | 45 ++++++++++++++++--------------- > 1 file changed, 24 insertions(+), 21 deletions(-) > > diff --git a/drivers/gpu/drm/panel/panel-edp.c b/drivers/gpu/drm/panel/panel-edp.c > index d094cfc43da8..fb70e97a2e71 100644 > --- a/drivers/gpu/drm/panel/panel-edp.c > +++ b/drivers/gpu/drm/panel/panel-edp.c > @@ -210,15 +210,12 @@ struct panel_desc { > * struct edp_panel_entry - Maps panel ID to delay / panel name. > */ > struct edp_panel_entry { > - /** @panel_id: 32-bit ID for panel, encoded with drm_edid_encode_panel_id(). */ > - u32 panel_id; > + /** @ident: edid identity used for panel matching. */ > + const struct drm_edid_ident ident; > > /** @delay: The power sequencing delays needed for this panel. */ > const struct panel_delay *delay; > > - /** @name: Name of this panel (for printing to logs). */ > - const char *name; > - > /** @override_edid_mode: Override the mode obtained by edid. */ > const struct drm_display_mode *override_edid_mode; > }; > @@ -691,7 +688,7 @@ static int detected_panel_show(struct seq_file *s, void *data) > else if (!p->detected_panel) > seq_puts(s, "HARDCODED\n"); > else > - seq_printf(s, "%s\n", p->detected_panel->name); > + seq_printf(s, "%s\n", p->detected_panel->ident.name); > > return 0; > } > @@ -761,7 +758,7 @@ static void panel_edp_parse_panel_timing_node(struct device *dev, > dev_err(dev, "Reject override mode: No display_timing found\n"); > } > > -static const struct edp_panel_entry *find_edp_panel(u32 panel_id); > +static const struct edp_panel_entry *find_edp_panel(u32 panel_id, const struct drm_edid *edid); > > static int generic_edp_panel_probe(struct device *dev, struct panel_edp *panel) > { > @@ -799,7 +796,6 @@ static int generic_edp_panel_probe(struct device *dev, struct panel_edp *panel) > base_block = drm_edid_read_base_block(panel->ddc); > if (base_block) { > panel_id = drm_edid_get_panel_id(base_block); > - drm_edid_free(base_block); > } else { > dev_err(dev, "Couldn't identify panel via EDID\n"); > ret = -EIO; > @@ -807,7 +803,9 @@ static int generic_edp_panel_probe(struct device *dev, struct panel_edp *panel) > } > drm_edid_decode_panel_id(panel_id, vend, &product_id); > > - panel->detected_panel = find_edp_panel(panel_id); > + panel->detected_panel = find_edp_panel(panel_id, base_block); > + > + drm_edid_free(base_block); > > /* > * We're using non-optimized timings and want it really obvious that > @@ -840,7 +838,7 @@ static int generic_edp_panel_probe(struct device *dev, struct panel_edp *panel) > panel->detected_panel = ERR_PTR(-EINVAL); > } else { > dev_info(dev, "Detected %s %s (%#06x)\n", > - vend, panel->detected_panel->name, product_id); > + vend, panel->detected_panel->ident.name, product_id); > > /* Update the delay; everything else comes from EDID */ > desc->delay = *panel->detected_panel->delay; > @@ -1930,17 +1928,21 @@ static const struct panel_delay delay_200_500_e50_po2e200 = { > > #define EDP_PANEL_ENTRY(vend_chr_0, vend_chr_1, vend_chr_2, product_id, _delay, _name) \ > { \ > - .name = _name, \ > - .panel_id = drm_edid_encode_panel_id(vend_chr_0, vend_chr_1, vend_chr_2, \ > - product_id), \ > + .ident = { \ > + .name = _name, \ > + .panel_id = drm_edid_encode_panel_id(vend_chr_0, vend_chr_1, vend_chr_2, \ > + product_id), \ > + }, \ > .delay = _delay \ > } > > #define EDP_PANEL_ENTRY2(vend_chr_0, vend_chr_1, vend_chr_2, product_id, _delay, _name, _mode) \ > { \ > - .name = _name, \ > - .panel_id = drm_edid_encode_panel_id(vend_chr_0, vend_chr_1, vend_chr_2, \ > - product_id), \ > + .ident = { \ > + .name = _name, \ > + .panel_id = drm_edid_encode_panel_id(vend_chr_0, vend_chr_1, vend_chr_2, \ > + product_id), \ > + }, \ > .delay = _delay, \ > .override_edid_mode = _mode \ > } > @@ -2087,15 +2089,16 @@ static const struct edp_panel_entry edp_panels[] = { > { /* sentinal */ } > }; > > -static const struct edp_panel_entry *find_edp_panel(u32 panel_id) > +static const struct edp_panel_entry *find_edp_panel(u32 panel_id, const struct drm_edid *edid) > { > const struct edp_panel_entry *panel; > > - if (!panel_id) > - return NULL; > + for (panel = edp_panels; panel->ident.panel_id; panel++) > + if (drm_edid_match_identity(edid, &panel->ident)) > + return panel; > > - for (panel = edp_panels; panel->panel_id; panel++) > - if (panel->panel_id == panel_id) > + for (panel = edp_panels; panel->ident.panel_id; panel++) > + if (panel->ident.panel_id == panel_id) > return panel; > > return NULL; -- Jani Nikula, Intel