All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Troy Mitchell" <troy.mitchell@linux.spacemit.com>
To: "Sarah Emery" <sarah.emery@canonical.com>
Cc: "Rob Herring" <robh@kernel.org>,
	"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
	"Conor Dooley" <conor+dt@kernel.org>,
	"Paul Walmsley" <pjw@kernel.org>,
	"Palmer Dabbelt" <palmer@dabbelt.com>,
	"Albert Ou" <aou@eecs.berkeley.edu>,
	"Alexandre Ghiti" <alex@ghiti.fr>, "Yixun Lan" <dlan@kernel.org>,
	"Guodong Xu" <docular.xu@gmail.com>,
	"Yangyu Chen" <cyy@cyyself.name>,
	"Troy Mitchell" <troy.mitchell@linux.spacemit.com>,
	<devicetree@vger.kernel.org>, <linux-riscv@lists.infradead.org>,
	<spacemit@lists.linux.dev>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 2/2] riscv: dts: spacemit: Add K3 CoM260 IFX TQ support
Date: Fri, 13 Nov 2026 08:39:58 +0000	[thread overview]
Message-ID: <DMTY56PU7CP2.JGIZ9XDXII9E@linux.spacemit.com> (raw)
In-Reply-To: <fcd2fc41-5af8-4256-8296-a5435fdf41db@canonical.com>


[-- Attachment #1.1: Type: text/plain, Size: 1513 bytes --]

On Tue, Nov 10, 2026 at 01:46:28PM +0100, Sarah Emery wrote:
> My understanding is that the nodes from CSB1-N10SPK3, the "K3 CoM260 TQ"
> and the "K3 CoM260 IFX TQ" are all the same: a "K3 CoM260" module on a
> "TQ" or "IFX TQ" carrier board.
>
> Is there two different carrier boards name "TQ" and "IFX TQ"? Or is it
> just the same thing, called differently in this patch and in the EEPROM?

They are the same carrier board. The only difference is the power scheme
of the CPU clusters. The bootloader programs those regulators at boot,
and upstream Linux currently keeps the voltages it leaves. Support for
the regulators is not upstream yet, so the two boards cannot be told
apart in the device tree for now.

> I'm not sure how devices out there are referencing themselves. But if it
> is in fact one device and the EEPROM calls it "k3-com260-tq", this patch
> will not work for device-tree matching.
>
> What do you think? Can you clarify the situation?

You are right, thanks for pointing this out. I only had an IFX TQ board
when I sent the series, so I did not notice that the TQ nodes identify
themselves as k3_com260_tq in the EEPROM.

I will restructure the series as k3-com260-tq.dtsi holding the shared
carrier description, with k3-com260-tq.dts and k3-com260-ifx-tq.dts on
top of it that only set model and compatible. I will add
k3-com260-tq-power.dtsi and k3-com260-ifx-tq-power.dtsi for the two
power schemes later.

Yixun, what do you think?

-- 
Troy Mitchell

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 273 bytes --]

[-- Attachment #2: Type: text/plain, Size: 161 bytes --]

_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

WARNING: multiple messages have this Message-ID (diff)
From: "Troy Mitchell" <troy.mitchell@linux.spacemit.com>
To: "Sarah Emery" <sarah.emery@canonical.com>
Cc: "Rob Herring" <robh@kernel.org>,
	"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
	"Conor Dooley" <conor+dt@kernel.org>,
	"Paul Walmsley" <pjw@kernel.org>,
	"Palmer Dabbelt" <palmer@dabbelt.com>,
	"Albert Ou" <aou@eecs.berkeley.edu>,
	"Alexandre Ghiti" <alex@ghiti.fr>, "Yixun Lan" <dlan@kernel.org>,
	"Guodong Xu" <docular.xu@gmail.com>,
	"Yangyu Chen" <cyy@cyyself.name>,
	"Troy Mitchell" <troy.mitchell@linux.spacemit.com>,
	<devicetree@vger.kernel.org>, <linux-riscv@lists.infradead.org>,
	<spacemit@lists.linux.dev>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 2/2] riscv: dts: spacemit: Add K3 CoM260 IFX TQ support
Date: Fri, 13 Nov 2026 08:39:58 +0000	[thread overview]
Message-ID: <DMTY56PU7CP2.JGIZ9XDXII9E@linux.spacemit.com> (raw)
In-Reply-To: <fcd2fc41-5af8-4256-8296-a5435fdf41db@canonical.com>

[-- Attachment #1: Type: text/plain, Size: 1513 bytes --]

On Tue, Nov 10, 2026 at 01:46:28PM +0100, Sarah Emery wrote:
> My understanding is that the nodes from CSB1-N10SPK3, the "K3 CoM260 TQ"
> and the "K3 CoM260 IFX TQ" are all the same: a "K3 CoM260" module on a
> "TQ" or "IFX TQ" carrier board.
>
> Is there two different carrier boards name "TQ" and "IFX TQ"? Or is it
> just the same thing, called differently in this patch and in the EEPROM?

They are the same carrier board. The only difference is the power scheme
of the CPU clusters. The bootloader programs those regulators at boot,
and upstream Linux currently keeps the voltages it leaves. Support for
the regulators is not upstream yet, so the two boards cannot be told
apart in the device tree for now.

> I'm not sure how devices out there are referencing themselves. But if it
> is in fact one device and the EEPROM calls it "k3-com260-tq", this patch
> will not work for device-tree matching.
>
> What do you think? Can you clarify the situation?

You are right, thanks for pointing this out. I only had an IFX TQ board
when I sent the series, so I did not notice that the TQ nodes identify
themselves as k3_com260_tq in the EEPROM.

I will restructure the series as k3-com260-tq.dtsi holding the shared
carrier description, with k3-com260-tq.dts and k3-com260-ifx-tq.dts on
top of it that only set model and compatible. I will add
k3-com260-tq-power.dtsi and k3-com260-ifx-tq-power.dtsi for the two
power schemes later.

Yixun, what do you think?

-- 
Troy Mitchell

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 273 bytes --]

  reply	other threads:[~2026-11-13  8:40 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-11-05  8:29 [PATCH 0/2] riscv: dts: spacemit: Add K3 CoM260 IFX TQ support Troy Mitchell
2026-11-05  8:29 ` Troy Mitchell
2026-11-05  8:29 ` [PATCH 1/2] dt-bindings: riscv: spacemit: Add K3 CoM260 IFX TQ Troy Mitchell
2026-11-05  8:29   ` Troy Mitchell
2026-11-05  8:29 ` [PATCH 2/2] riscv: dts: spacemit: Add K3 CoM260 IFX TQ support Troy Mitchell
2026-11-05  8:29   ` Troy Mitchell
2026-11-10 12:46   ` Sarah Emery
2026-11-10 12:46     ` Sarah Emery
2026-11-13  8:39     ` Troy Mitchell [this message]
2026-11-13  8:39       ` Troy Mitchell

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=DMTY56PU7CP2.JGIZ9XDXII9E@linux.spacemit.com \
    --to=troy.mitchell@linux.spacemit.com \
    --cc=alex@ghiti.fr \
    --cc=aou@eecs.berkeley.edu \
    --cc=conor+dt@kernel.org \
    --cc=cyy@cyyself.name \
    --cc=devicetree@vger.kernel.org \
    --cc=dlan@kernel.org \
    --cc=docular.xu@gmail.com \
    --cc=krzk+dt@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-riscv@lists.infradead.org \
    --cc=palmer@dabbelt.com \
    --cc=pjw@kernel.org \
    --cc=robh@kernel.org \
    --cc=sarah.emery@canonical.com \
    --cc=spacemit@lists.linux.dev \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.