From: Niklas Cassel <cassel@kernel.org>
To: Manivannan Sadhasivam <manivannan.sadhasivam@oss.qualcomm.com>
Cc: Bjorn Helgaas <bhelgaas@google.com>,
Manivannan Sadhasivam <mani@kernel.org>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Bartosz Golaszewski <brgl@bgdev.pl>,
Damien Le Moal <dlemoal@kernel.org>,
Linus Walleij <linus.walleij@linaro.org>,
Bartosz Golaszewski <brgl@kernel.org>,
linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org,
devicetree@vger.kernel.org, linux-arm-msm@vger.kernel.org,
Stephan Gerhold <stephan.gerhold@linaro.org>,
Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>,
linux-pm@vger.kernel.org, linux-ide@vger.kernel.org,
Frank Li <Frank.Li@nxp.com>,
Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
Subject: Re: [PATCH v4 0/5] PCI: Add initial support for handling PCIe M.2 connectors in devicetree
Date: Tue, 30 Dec 2025 10:33:52 +0100 [thread overview]
Message-ID: <aVOcgDeOejO9m1zE@ryzen> (raw)
In-Reply-To: <20251228-pci-m2-v4-0-5684868b0d5f@oss.qualcomm.com>
Hello Mani,
On Sun, Dec 28, 2025 at 10:31:00PM +0530, Manivannan Sadhasivam wrote:
> The Mechanical Key M connector is used to connect SSDs to the host machine over
> PCIe/SATA interfaces. Due to the hardware constraints, this series only adds
> support for driving the PCIe interface of the connector in the kernel.
Since this series does not add any support for SATA, do we really want to
modify the SATA device tree binding?
I know that device tree describes the hardware, but if there is no software
that makes use of this, the SATA DT binding change feels a bit unnecessary.
Do we perhaps want to defer modifying the SATA DT binding change until the
corresponding change in software is added?
Kind regards,
Niklas
next prev parent reply other threads:[~2025-12-30 9:34 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-12-28 17:01 [PATCH v4 0/5] PCI: Add initial support for handling PCIe M.2 connectors in devicetree Manivannan Sadhasivam
2025-12-28 17:01 ` [PATCH v4 1/5] dt-bindings: ata: sata: Document the graph port Manivannan Sadhasivam
2025-12-28 17:01 ` [PATCH v4 2/5] dt-bindings: connector: Add PCIe M.2 Mechanical Key M connector Manivannan Sadhasivam
2025-12-28 17:01 ` [PATCH v4 3/5] PCI/pwrctrl: Add support for handling PCIe M.2 connectors Manivannan Sadhasivam
2025-12-28 17:01 ` [PATCH v4 4/5] PCI/pwrctrl: Create pwrctrl device if the graph port is found Manivannan Sadhasivam
2025-12-28 17:01 ` [PATCH v4 5/5] power: sequencing: Add the Power Sequencing driver for the PCIe M.2 connectors Manivannan Sadhasivam
2026-01-02 11:26 ` Bartosz Golaszewski
2026-01-07 9:38 ` Manivannan Sadhasivam
2026-01-07 9:51 ` Bartosz Golaszewski
2026-01-07 12:31 ` Manivannan Sadhasivam
2025-12-30 9:33 ` Niklas Cassel [this message]
2025-12-30 9:59 ` [PATCH v4 0/5] PCI: Add initial support for handling PCIe M.2 connectors in devicetree Manivannan Sadhasivam
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=aVOcgDeOejO9m1zE@ryzen \
--to=cassel@kernel.org \
--cc=Frank.Li@nxp.com \
--cc=bartosz.golaszewski@linaro.org \
--cc=bhelgaas@google.com \
--cc=brgl@bgdev.pl \
--cc=brgl@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dlemoal@kernel.org \
--cc=dmitry.baryshkov@oss.qualcomm.com \
--cc=krzk+dt@kernel.org \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=mani@kernel.org \
--cc=manivannan.sadhasivam@oss.qualcomm.com \
--cc=robh@kernel.org \
--cc=stephan.gerhold@linaro.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