mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Frieder Hannenheim" <mail@fhannenheim.net>
To: "David Heidelberg" <david@ixit.cz>,
	"Frieder Hannenheim" <mail@fhannenheim.net>,
	"Frieder Hannenheim" <git@fhannenheim.net>,
	"Bjorn Andersson" <andersson@kernel.org>,
	"Konrad Dybcio" <konradybcio@kernel.org>,
	"Rob Herring" <robh@kernel.org>,
	"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
	"Conor Dooley" <conor+dt@kernel.org>
Cc: "Robert Oscilowski" <drgitx@gmail.com>,
	"Jean-François Bobier" <jf.bobier@gmail.com>,
	phone-devel@vger.kernel.org, linux-arm-msm@vger.kernel.org,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH RESEND v4 1/3] dt-bindings: arm: qcom: Document OnePlus 8T
Date: Mon, 09 Nov 2026 14:58:09 +0100	[thread overview]
Message-ID: <DMQQEMNZAC9R.2QNFULU5X2PAT@fhannenheim.net> (raw)
In-Reply-To: <67c44ce0-8dcf-48f1-abf3-f1dd81175a2f@ixit.cz>

On Sun Nov 8, 2026 at 10:03 PM CET, David Heidelberg wrote:
> On 08/11/2026 20:58, Frieder Hannenheim wrote:
>> On 08.11.26 21:52, David Heidelberg wrote:
>>> On 05/11/2026 20:20, Frieder Hannenheim wrote:
>>>> Add compatible for OnePlus 8T
>>>>
>>>> Signed-off-by: Frieder Hannenheim <git@fhannenheim.net>
>>>> ---
>>>>   Documentation/devicetree/bindings/arm/qcom.yaml | 1 +
>>>>   1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/ 
>>>> Documentation/devicetree/bindings/arm/qcom.yaml
>>>> index 50cc18a6ec5e..c21ef16e9ac8 100644
>>>> --- a/Documentation/devicetree/bindings/arm/qcom.yaml
>>>> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
>>>> @@ -1067,6 +1067,7 @@ properties:
>>>>         - items:
>>>>             - enum:
>>>> +              - oneplus,kebab
>>>
>>> How about oneplus,8t? We can use the non-so-much-confusing product name.
>>> Looking at existing device-trees it seems customary to use the code name 
>> of the product for phones. Examples include:
>> - xiaomi,lavender
>> - google,sargo
>> - oneplus,cheeseburger
>
> Yup, we had discussion with Krzysztof [1], that it could make more sense made 
> actual production name as a codename. My guess is Android codenames are used 
> before the product is released and the marketing name isn't known. Since we know 
> marketing names, which people use to identify the phones, we can use them to 
> make more clear which device is which.
>
> David
>
> [1] https://lore.kernel.org/all/0fd3e0a5-33f7-4bc6-9146-dcaf035c3ef5@kernel.org/

I think in the linked discussion it made sense to use the marketing name since
there where multiple devices with the same codename. This is not the case here
so I wouldn't stray from the precedent.

Frieder

  reply	other threads:[~2026-11-09 13:58 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-11-05 20:20 [PATCH RESEND v4 0/3] Add " Frieder Hannenheim
2026-11-05 20:20 ` [PATCH RESEND v4 1/3] dt-bindings: arm: qcom: Document " Frieder Hannenheim
2026-11-08 20:52   ` David Heidelberg
2026-11-08 20:58     ` Frieder Hannenheim
2026-11-08 21:03       ` David Heidelberg
2026-11-09 13:58         ` Frieder Hannenheim [this message]
2026-11-09 11:06       ` Konrad Dybcio
2026-11-05 20:20 ` [PATCH RESEND v4 2/3] arm64: dts: qcom: Add sm8250-oneplus-common.dtsi Frieder Hannenheim
2026-11-08 21:05   ` David Heidelberg
2026-11-09 10:36     ` Frieder Hannenheim
2026-11-05 20:20 ` [PATCH RESEND v4 3/3] arm64: dts: qcom: oneplus-kebab: new device Frieder Hannenheim
2026-11-09 11:08   ` Konrad Dybcio

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=DMQQEMNZAC9R.2QNFULU5X2PAT@fhannenheim.net \
    --to=mail@fhannenheim.net \
    --cc=andersson@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=david@ixit.cz \
    --cc=devicetree@vger.kernel.org \
    --cc=drgitx@gmail.com \
    --cc=git@fhannenheim.net \
    --cc=jf.bobier@gmail.com \
    --cc=konradybcio@kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=phone-devel@vger.kernel.org \
    --cc=robh@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®