Received-SPF: pass (google.com: domain of linux-kernel+bounces-110170-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Message-ID: <95136c06-dd99-46c7-a672-200ffc0b188a@theobroma-systems.com>
Date: Thu, 21 Mar 2024 15:44:37 +0100
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 2/2] drm/panel: ltk050h3146w: drop duplicate commands from
 LTK050H3148W init
Content-Language: en-US
To: Heiko Stuebner <heiko@sntech.de>, neil.armstrong@linaro.org,
 quic_jesszhan@quicinc.com, sam@ravnborg.org
Cc: maarten.lankhorst@linux.intel.com, mripard@kernel.org,
 tzimmermann@suse.de, klaus.goger@theobroma-systems.com,
 dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org,
 Heiko Stuebner <heiko.stuebner@cherry.de>
References: <20240320131232.327196-1-heiko@sntech.de>
 <20240320131232.327196-2-heiko@sntech.de>
From: Quentin Schulz <quentin.schulz@theobroma-systems.com>
In-Reply-To: <20240320131232.327196-2-heiko@sntech.de>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?dFhJTDBXMm1vVHM1M3FsRmFUZ0IwR1ErZkE5UVJ0Nlg1UHRYdTM3eVgxcEpH?=
 =?utf-8?B?YllzU1RrTDJBbDVzQnc3cWw3SnVPZWFLZ1VhazR3aFJIN1YzdUJLWjZBcVJO?=
 =?utf-8?B?MzdweVhwdkxqK2FoNTFIRUJHcWIwMTEzZzR0UUh6TnYwUGQ3Y3p3bkZLM3dI?=
 =?utf-8?B?ckpCd1pvWnFMTndnVnIySmFuZnE3c283cDdia2ZIUVBHZzg4SjBDY0h6Y3hQ?=
 =?utf-8?B?aW1zSUhzUTVtRG1NWVgvSG5wS05rckVwREh6WlliWUVialJNcXltM01jNlU1?=
 =?utf-8?B?TVliOVdmeXhuVnNjMzBkOWE5MjhoK0U4MWdhSTJOWjFQZnRpeFdOODluVmI4?=
 =?utf-8?B?TmxDMGVrVEIwV2dwWUR1VGljcjhQS2lLamNWdG9lTmVndVlTQ1hUUFJZcTky?=
 =?utf-8?B?QVVBK0NnWDJUYm95elJCa2xrVFUzQk9xc04xQVM3MVBkOXZUTTJ5SWlzRm8r?=
 =?utf-8?B?cUlqWUJyRXEvY0xTYWV5OG1XeDhsSllOL0RaNHdIdVlpSTRnSm8reDlacXBH?=
 =?utf-8?B?VTFuNXdEUUs5VnhicEFsWVcxVFRRVkFJSzBJT010YkZWODFhcUVUSllUZ2ZL?=
 =?utf-8?B?cTdHVjFCREYwYzJqeW84UGRYWk1nUThFM3krZ2s3ZzVvU1hkcm9nVmNuN2Jx?=
 =?utf-8?B?SlljZzYyZkF6emQrcFdzR0FFVG1kaTBJOUxkUnk0TnIxQmpId0RGMTBSbVpI?=
 =?utf-8?B?dDVIbXBESm4vb21kT281Z0hMbnlodmJJQkdqWW5iSWVRazdPdFE5MmtZaG45?=
 =?utf-8?B?UXl3T3loYWpGdGdIMlNQclFzQ0JMaE9WYnBGWXM1UTFiQXRsUVJ3eXFaUVlW?=
 =?utf-8?B?MUtoaUZsL0E3ZU51ZDgzTUpuWFUxMkdNMm1kREJCQlg5RXV3Sytna1FUd0tp?=
 =?utf-8?B?ZE1kUVI1NnFaYk81UHhpNC8zclhaanFCR25BaXRXcFlpSGo0bDczRStFNkNM?=
 =?utf-8?B?aVBHcVRraEpRcEI3RFNXVnpKT21yQjF5Q1BXNm11WlhqK2NLNCswd0g5N1BY?=
 =?utf-8?B?bEVFZ0xySUFNaDZQMjNObi9OMkt2VXYwbVcxT0EveDdEMHptMSs1cFdYQmZi?=
 =?utf-8?B?T1M1a2VXeFNZVDdmQmFUeFFrbDhiWVNqMjMyYjFWOVNRZU81WEgwRlhCWWMx?=
 =?utf-8?B?cjZrMjVuM3FXbDhNbW5NMEdaSEdBSnM2VC9ZcVlwL3UzKzVNNlVPSGxiRU1z?=
 =?utf-8?B?K2k5eW5pcytlUEdhaFBHdThYOWo5QUVnQTFxZnlRUXBwdDNiSVU4M2FRbDVx?=
 =?utf-8?B?WTlpa251Q3lFMkYzdEl2K2tjZjlBMmI5NlM0UU9Md0NyWEhONWVqbCtaYkd3?=
 =?utf-8?B?MDFLRkd6MkJJUDVWMDB5d25DVTY5WnhSZzJuWjJBNUdpT0VDNFQ4NnVNOGJi?=
 =?utf-8?B?M2pPaUpFNGMrT3Nkd09aTmIzSW5uandadzJFQ3hyTUowVG9yeUR0Z0twOTZt?=
 =?utf-8?B?ekoyaHBzWUFwMkVQYmlEcld0NyszUFN0cDFIVEVlZWYwWlBOdlVsNklwdHNS?=
 =?utf-8?B?cTlMbGM2eUlmLzIxTWREaENUWkZlNjdMRTJnWm51N0ZPc3pTQnRoVldYcWpT?=
 =?utf-8?B?WmJEQkEwZWIwTkRPenI0OERoVitjZGpGVkhBOXMxd3d2OWF4L01JOERmSXNK?=
 =?utf-8?B?MHRENlJZcDR6a1hFREJNbnJ3ZWFJSllvczhNZm90ZWVYNXdqcGZ5NDBMTXI2?=
 =?utf-8?B?YzNCM2Fxb2JNaVh4Y01ma2tiM2dyTUw3djJFY3l4dGU3aE9LU2hWTUJlV3Zh?=
 =?utf-8?B?UlhGallXN0lOcVk1RzBnRjE3MGREYW5reUVHL0pxQ1ZsOVF2QjZJaiswek0x?=
 =?utf-8?B?L3VWQWU4TUF4UWRHUFpYMEVFajN0NVZXdnMxYnRORExsMUIzSnVDMWlQZTE3?=
 =?utf-8?B?cGVRaEk2WHlDNnpSTXlQQzRVa1NUamE2Q2luRmRCYW81UWgvVmRIODZ1cDlH?=
 =?utf-8?B?VWk5eEFXMkUzV3ovVm83bXVHZk00MkxiOG5zaW9LS2ViTlpZK01sbzJ6aGdp?=
 =?utf-8?B?QVJKUGI4Yk9NUFRUdEQzT1ZaWUtuUGR1bjN0N0dqU2RXeDF1ZG56K1llZXZw?=
 =?utf-8?B?Um03N29TTCtrdStTa2p5QW8yRXhpR2hhQ1EySktnYzBFKzREVGcwcS9mOXk0?=
 =?utf-8?B?aVhzZ0F1anNId2JzaUhualViNFcwM1JPVHM1Z05FN2ZsSXZ3UW9wRWNrRFhK?=
 =?utf-8?Q?vaG1soqs/v8gQXEp4f8eGXI=3D?=
X-OriginatorOrg: theobroma-systems.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 2495b3c0-c724-4627-492d-08dc49b570ad
X-MS-Exchange-CrossTenant-AuthSource: DU2PR04MB8536.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Mar 2024 14:44:40.9577
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 5e0e1b52-21b5-4e7b-83bb-514ec460677e
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: t3tWEaSOIA6y4oME+0peBNcuXtFzG2j4k9a1jYSSotRChcVpco7fFcUkt+sxilH5PoPsXzTzopEA3meUu+rxNJ9qSnfFm2wrKYByTIsoeS64rAML68MFJoUMFI8ndssx
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR04MB7831

Hi Heiko,

On 3/20/24 14:12, Heiko Stuebner wrote:
> From: Heiko Stuebner <heiko.stuebner@cherry.de>
> 
> The init sequence specifies the 0x11 and 0x29 dsi commands, which are
> the exit-sleep and display-on commands.
> 
> In the actual prepare step the driver already uses the appropriate
> function calls for those, so drop the duplicates.
> 
> Fixes: e5f9d543419c ("drm/panel: ltk050h3146w: add support for Leadtek LTK050H3148W-CTA6 variant")
> Signed-off-by: Heiko Stuebner <heiko.stuebner@cherry.de>

Reviewed-by: Quentin Schulz <quentin.schulz@theobroma-systems.com>

Thanks!
Quentin