From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 76F22301CD4; Sun, 1 Jun 2025 23:36:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1748821011; cv=none; b=iUp813g6FEzNmvgscu0T+xlEZwFpDPgV/IAxMpXoyOK2DLZKcZOG98qvD/5ENRvacWCEiqZNUgnWAL4Ius7764k4EU/qT4Bl/VWlbqDB1cFhomVbGliPhOw3FZBEC1nIrppdOAeIPopCye5WX5OE419W/pjnJ8Ko2HXcDaoc7eU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1748821011; c=relaxed/simple; bh=EFzZa+Nch75GomygTs6THnTdW39Dbqe4N6p6BnzEhuA=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version; b=Qfn5xbTv0zILIgDDPfFTleg82ZdH8WnS5eVy7Ce+S+FZvFp+8piNcZ0kDKHISk6PLntc8PfL0grdTU1bRAEgF+rAqEZbUTPKOrXBTX0JMzpsKrvwCbrXB4oDFPrhou1IxVzUt5x+MOUUqqG7OMqVaYSSoUpG/+uLMacz7Vngc7Q= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=qPJ5Tfhc; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="qPJ5Tfhc" Received: by smtp.kernel.org (Postfix) with ESMTPSA id D3256C4CEE7; Sun, 1 Jun 2025 23:36:49 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1748821011; bh=EFzZa+Nch75GomygTs6THnTdW39Dbqe4N6p6BnzEhuA=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=qPJ5TfhczRONo95YX4N5e7mmH2Jm/u7v+3BgnWnvUMESbBzAlMQjuvuD+R9yU8s34 22qsE6RooPnJRwiIwB8pq2CO75ryC6twn4a5NoxXoHeGuGtHwsQnUqImr6YN9tCw0+ nR3c7+myJ1UiSosqZaEXaMWJlX4rnKmAB8iaWJYbmHauk+QFElfGY2Wq97/bz19cx5 qTHs1m5RujeFdfRWW14Kf7lUK5Jy2jXFcIrAnjIzDvpcXGwjGzkScmGocNQ7kmdv/1 bTyIBB+Q5O42OqaQTvXTSNqZJa6yqwwQwx+WkqhH3P4ZfTWyfA5FnHE498pJpu20PR yb23U/8f0z4Wg== From: Sasha Levin To: patches@lists.linux.dev, stable@vger.kernel.org Cc: Antonin Godard , Dmitry Baryshkov , Louis Chauvet , Sasha Levin , neil.armstrong@linaro.org, maarten.lankhorst@linux.intel.com, mripard@kernel.org, tzimmermann@suse.de, airlied@gmail.com, simona@ffwll.ch, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: [PATCH AUTOSEL 6.12 66/93] drm/panel: simple: Add POWERTIP PH128800T004-ZZA01 panel entry Date: Sun, 1 Jun 2025 19:33:33 -0400 Message-Id: <20250601233402.3512823-66-sashal@kernel.org> X-Mailer: git-send-email 2.39.5 In-Reply-To: <20250601233402.3512823-1-sashal@kernel.org> References: <20250601233402.3512823-1-sashal@kernel.org> Precedence: bulk X-Mailing-List: patches@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-stable: review X-Patchwork-Hint: Ignore X-stable-base: Linux 6.12.31 Content-Transfer-Encoding: 8bit From: Antonin Godard [ Upstream commit 6374a1005f20c1c2f7bbcc1bc735c2be4910a685 ] Add support for the POWERTIP PH128800T004-ZZA01 10.1" (1280x800) LCD-TFT panel. Its panel description is very much like the POWERTIP PH128800T006-ZHC01 configured below this one, only its timings are different. Signed-off-by: Antonin Godard Reviewed-by: Dmitry Baryshkov Link: https://patchwork.freedesktop.org/patch/msgid/20250311-add-powertip-ph128800t004-v1-2-7f95e6984cea@bootlin.com Signed-off-by: Louis Chauvet Signed-off-by: Sasha Levin --- NO This commit should NOT be backported to stable kernel trees based on the following analysis: **Commit Classification**: This is a new hardware support addition, not a bug fix. **Analysis of the commit**: 1. **Nature of Change**: The commit adds support for a brand new panel variant (POWERTIP PH128800T004-ZZA01) to the panel-simple driver. This is purely additive functionality for new hardware. 2. **Code Changes Review**: - Adds a new `powertip_ph128800t004_zza01_mode` structure with display timing parameters - Adds a new `powertip_ph128800t004_zza01` panel descriptor - Adds a new device tree compatible string entry in the `platform_of_match` table - The changes are isolated and only add new entries without modifying existing functionality 3. **Reference to Similar Commits**: All the provided historical examples of panel additions (Similar Commits #1, #2, #5) were marked as "Backport Status: NO", indicating that new panel support commits are typically not backported. 4. **Stable Tree Criteria Violation**: - This is new feature addition, not a bug fix - It doesn't address any existing user-affecting issues - It enables support for hardware that previously wasn't supported at all - No indication of fixing broken functionality 5. **Exception Analysis**: The only similar commit marked "YES" (Similar Commit #4) was specifically fixing missing display mode flags for an existing panel, which was a clear bug fix with a "Fixes:" tag. This current commit has no such characteristics. 6. **Risk Assessment**: While the change is low-risk and isolated, it doesn't meet the fundamental criteria for stable backporting as it's new hardware enablement rather than fixing existing functionality. The stable tree policy specifically excludes new hardware support unless it fixes existing broken functionality, which this commit does not do. drivers/gpu/drm/panel/panel-simple.c | 29 ++++++++++++++++++++++++++++ 1 file changed, 29 insertions(+) diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c index d041ff542a4ee..ed09ffc0e2b48 100644 --- a/drivers/gpu/drm/panel/panel-simple.c +++ b/drivers/gpu/drm/panel/panel-simple.c @@ -3685,6 +3685,32 @@ static const struct panel_desc pda_91_00156_a0 = { .bus_format = MEDIA_BUS_FMT_RGB888_1X24, }; +static const struct drm_display_mode powertip_ph128800t004_zza01_mode = { + .clock = 71150, + .hdisplay = 1280, + .hsync_start = 1280 + 48, + .hsync_end = 1280 + 48 + 32, + .htotal = 1280 + 48 + 32 + 80, + .vdisplay = 800, + .vsync_start = 800 + 9, + .vsync_end = 800 + 9 + 8, + .vtotal = 800 + 9 + 8 + 6, + .flags = DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC, +}; + +static const struct panel_desc powertip_ph128800t004_zza01 = { + .modes = &powertip_ph128800t004_zza01_mode, + .num_modes = 1, + .bpc = 8, + .size = { + .width = 216, + .height = 135, + }, + .bus_format = MEDIA_BUS_FMT_RGB888_1X7X4_SPWG, + .bus_flags = DRM_BUS_FLAG_DE_HIGH, + .connector_type = DRM_MODE_CONNECTOR_LVDS, +}; + static const struct drm_display_mode powertip_ph128800t006_zhc01_mode = { .clock = 66500, .hdisplay = 1280, @@ -4960,6 +4986,9 @@ static const struct of_device_id platform_of_match[] = { }, { .compatible = "pda,91-00156-a0", .data = &pda_91_00156_a0, + }, { + .compatible = "powertip,ph128800t004-zza01", + .data = &powertip_ph128800t004_zza01, }, { .compatible = "powertip,ph128800t006-zhc01", .data = &powertip_ph128800t006_zhc01, -- 2.39.5