From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-oi2-f5.google.com (mail-oi2-f5.google.com [74.125.231.197]) (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 90A4C3BED30 for ; Mon, 7 Sep 2026 23:27:27 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.231.197 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788823648; cv=none; b=bv53WICvXtZwy7zVw/8MfsI2QY92Po6uHuMCx2ODx/O3bP5KUQ8HGxh26zPM6bxy5wWm7oNBv8x+BAIIW9zwK2hx1DD3LdAvJbXc5eshrobOMR0MY5bOZrUdUsVodJG5arcQ1en8TbWix4rBT+4roH9GXa31uiVbEB2hSjrwDUo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788823648; c=relaxed/simple; bh=/zq62zwl2SYqbX3nWntxxVcw7qJL9ZY4f2IiFrSdXUo=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=TrGVZWNQzGsTOx/U2afjWT30qQbGxTGTo9VYfcq/UmWe+ZF/83NX8Rs/zeAsDcY8Xn3khk9PLwDdVWcFcx8tCEHaduVvLgv5IEWGSuDaJ2qx44WXwxp9isIziqMMWgnD+Yf2IWuvF6w6vDjx+lXMfQpBzo50PoNoAU+g5PJczjw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=IYFn/DU7; arc=none smtp.client-ip=74.125.231.197 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="IYFn/DU7" Received: by mail-oi2-f5.google.com with SMTP id 5614622812f47-4b3759b7ff0so1488761b6e.1 for ; Mon, 07 Sep 2026 16:27:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1788823646; x=1789428446; darn=vger.kernel.org; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:from:to:cc:subject:date:message-id:reply-to :content-type; bh=S2mXpnCZ25GxwnmGohlXqYGxtFFG4TCx9pnjWCRTcOc=; b=IYFn/DU7zmkUxhRouGb8RcpsQGZynVRMaqeV+ux64GLJpSCQtvSaVIKqdcb/07ZYH0 lKnu9ikN4grTsKzIU/E+HoNtYHYR3Z8eIkBSlrB02rmE7ekmgT3kr9UHiXfE+LiVfrIi heGTNsRK3z+T7cDzBUEDZf2lMwB1Srhp+4YSd4odyzNKtT8/6qJ2fjs6IYjc9AHXS8fQ UhEvGjZWMThVbAoSmVdaE7zMzkk28TyPFYYiobkQxxamWc5JcJ9ycirucD7E7PUxWPhU QHxglUqGawnJk+Sk0TG9kcE16NnaPZJThakVhnwZ+6sj70QI6hBP5GhQ8gA0MHFhZ8b5 LmFQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1788823646; x=1789428446; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=S2mXpnCZ25GxwnmGohlXqYGxtFFG4TCx9pnjWCRTcOc=; b=siBzt0vV4MAmjHcS597OlYkuajUL4lV7tyeoXN8tuUfsHGTmdB3+5u2WAvlMpqyUYv cGO0qlCYC1SrfmHXLz6v7Y81N1fsx/uUVw7CLilI8a/ypgy/nDLCXxDVcKJWtfusEK4G bHqIPL1b8xexHPV6/SF0BBJud77Hfj6ANHzPkqmpCA2XE9taId9EANagjNpWgwrBeR7a GYu+kLk0CXBd5hzc8WDU4bvDB9DSlCefCFX3tPJuE9hiZ5pY1YhfDCx+y4B1ecPQe4m9 ijzLiqY4H9yxeoQkYHagWYcc9zMlXn0Sfl9X7xLhvsmuYc2iqNkHwf0HPsAaCzzrphOa 3AWg== X-Forwarded-Encrypted: i=1; AKwUvBwPuXUYEXVWlhSbhgjJF+psl+x8j2Md/a8yHFwt07HNjpXwd0SYBPiAtBHYGhEqszOIzE7n2ZgYD5A26qs=@vger.kernel.org X-Gm-Message-State: AFuF++mKQQReOycMbcPqbQ+mMFQwUz9ylEPEvCYVxP8NxU59ChRvQb3q fQMFsA3de2lASOBfDRkSuGG2zeOT4Fdg3Kt/zYbRlNZFFzIzqwq+G5eE X-Gm-Gg: AYBFou3D9Lsi0iNddrLEMYvIWWP3jptRmwJrPRT7NzzalT3TdHf9ciVB1D7qySAOq+y xDs9/z2hps67z/+RVq29VtI7lz5xpbPF7GQUClnrK/9noErQtE4fTpOO6rNeVj+aFmEhbENzXkE 8VlLMDxOG8LhduvxqZ+KqEPK6F5q0Rf47QIDnSR0f6DVyWrZ9l6g+JjvXTKyqWHcyZxLKFiOdiY it9ezcqq138k688sUYdhgdbvr/la6Zb97/0QCdFKuoypNwYVsPmEfPeZITKk/gSBwTk+oCIMwYL XFiQAZyHeiDZKjxs2g1u5zTRVpWalst859nKHpK8hu0YYmCP7FN+rx0g7UVeRIVig+IZs8+UAjX tfk33K2jj+sUVcbr7NdSvAUkrUuRdXuFHMvGfmBkRWrHx1xnX7Vak9UQ0C0wCgd+/cTKmWcLF0M SQMg3gNBV/KLrdDEg4aXqDNu3DwOhMuKUr3i7y4/L3EOYoFJOJzIxjCiHBHEF4Gi7gPXkag2BRV NBMLKE= X-Received: by 2002:a05:6808:e8b:b0:4b9:a829:f00a with SMTP id 5614622812f47-4b9a829f91dmr13256254b6e.22.1788823646304; Mon, 07 Sep 2026 16:27:26 -0700 (PDT) Received: from ?IPV6:2600:8804:5716:d800::b712? ([2600:8804:5716:d800::b712]) by smtp.gmail.com with ESMTPSA id 5614622812f47-4b96babc2c3sm13310503b6e.0.2026.09.07.16.27.23 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 07 Sep 2026 16:27:24 -0700 (PDT) Message-ID: <668b6dce-32f1-478b-b78b-06f8b5d7e5e8@gmail.com> Date: Mon, 7 Sep 2026 18:27:22 -0500 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH RFC 0/2] Novatek NT51021 DSI panel IC driver To: Nickolay Goppen , Neil Armstrong , Jessica Zhang , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Alexey Minnekhanov References: <20260829-qcom-novatek-nt51021-panels-v1-0-cc1ca61bd261@mainlining.org> <20260907060129.120497-1-ryanbrue.dev@gmail.com> <54719b3e-d882-4b0c-b477-1e12dcf65133@mainlining.org> Content-Language: en-US From: Ryan Brue In-Reply-To: <54719b3e-d882-4b0c-b477-1e12dcf65133@mainlining.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit On 9/7/26 1:42 AM, Nickolay Goppen wrote: > > 07.09.2026 09:01, Ryan Brue wrote: >> One question on the binding, which is as much about my patch as yours. >> Where does "boe,tv101wum-nx0" come from? Xiaomi's panel dtsi >> (dsi-panel-nt51021-boe-10wu-video.dtsi) gives only >> >>      qcom,mdss-dsi-panel-name = "NT51021_BOE_BOE10"; >> >> with no part number, and the register sequence in your driver matches >> that file byte for byte. TV101WUM-NX0 is a real BOE part, and its >> datasheet active area of 216.576 x 135.36 mm agrees nicely with the >> 135 x 216 in that dtsi, so the attribution looks reasonable. But if it >> is inferred rather than read off the panel or a BOM, it may be worth >> saying so in the commit message, or naming the compatible after the >> board the way panel-novatek-nt36523.c does for >> xiaomi,elish-boe-nt36523. >> >> I ask because my module has no identity I can establish at all. It >> answers no DSI read, DCS or generic, before or after sleep-out; the >> vendor kernel and bootloader name it only "BOE", chosen from two board >> ID straps; and its vendor-declared physical height is arithmetically >> impossible, since 1200/136 and 1920/221 disagree by 1.5%. So I am >> naming mine after the board, and would rather not accidentally imply it >> is or is not your part. > > I've found these part numbers on the photos of a replacement panels > from the AliExpress. > > Also, for TV101WUM-NX0, I have a confirmation from Alexey Min, who > disassembled > Xiaomi Mi Pad 4 Plus and took a photo of the panel's part number. > I searched around on AliExpress for my panel, and could not find anything about the BOE part number for mine. Oh well, since there's some precedent for naming unknown BOE glass after the device that has it, I am hoping that will be OK for my case. >> If it is useful, I have the Fire HD 10 panel ready as a variant on top >> of your series: two patches, binding and driver, adding >> amazon,suez-boe-nt51021. The only change it needs in your code is >> making mode_flags per variant. My vendor asks for sync-event video mode >> with a continuously running HS clock rather than burst, and your two >> panels keep exactly the flags they have now. Burst worked on my panel >> too, so this is about matching what the module is characterised for, >> not a fix. > I will try to do that. Can I add you to the maintainers of this panel > driver? Yes, you may. Once you get the v2 out, I'll get a patch ready on top of that for the Fire Tablet's BOE panel! >> I will hold that until your v2, since Neil's renaming will touch every >> identifier it adds. Happy to test v2 on this hardware when you post it. >> >> Thanks for doing this first. >> >> Ryan >