* [PATCH v3] schemas: pci: Document PCIe T_POWER_ON
@ 2025-11-26 10:31 Krishna Chaitanya Chundru
2025-11-26 13:30 ` Manivannan Sadhasivam
2025-11-26 14:44 ` Bjorn Helgaas
0 siblings, 2 replies; 3+ messages in thread
From: Krishna Chaitanya Chundru @ 2025-11-26 10:31 UTC (permalink / raw)
To: andersson, robh, mani, krzk, helgaas
Cc: linux-arm-msm, devicetree, lpieralisi, kw, conor+dt, linux-pci,
linux-kernel, devicetree-spec, quic_vbadigan, krishna.chundru,
lukas
From PCIe r7, sec 5.5.4 & Table 5-11 in sec 5.5.5 T_POWER_ON is the
minimum amount of time(in us) that each component must wait in L1.2.Exit
after sampling CLKREQ# asserted before actively driving the interface to
ensure no device is ever actively driving into an unpowered component and
these values are based on the components and AC coupling capacitors used
in the connection linking the two components.
This property should be used to indicate the T_POWER_ON and drivers using
this property are responsible for parsing both the scale and the value of
T_POWER_ON to comply with the PCIe specification.
Signed-off-by: Krishna Chaitanya Chundru <krishna.chundru@oss.qualcomm.com>
---
Changes in v2:
- Move the property to pci-device.yaml so that it will be applicable to
endpoint devices also (Mani).
- Use latest spec (Lukas)
- Link to v2: https://lore.kernel.org/all/20251110112947.2071036-1-krishna.chundru@oss.qualcomm.com/
Changes in v1:
- Updated the commiit text (Mani).
- Link to v1: https://lore.kernel.org/all/20251110112550.2070659-1-krishna.chundru@oss.qualcomm.com/#t
dtschema/schemas/pci/pci-device.yaml | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/dtschema/schemas/pci/pci-device.yaml b/dtschema/schemas/pci/pci-device.yaml
index ca094a0..4baab71 100644
--- a/dtschema/schemas/pci/pci-device.yaml
+++ b/dtschema/schemas/pci/pci-device.yaml
@@ -63,6 +63,15 @@ properties:
description: GPIO controlled connection to WAKE# signal
maxItems: 1
+ t-power-on-us:
+ description:
+ The minimum amount of time that each component must wait in
+ L1.2.Exit after sampling CLKREQ# asserted before actively driving
+ the interface to ensure no device is ever actively driving into an
+ unpowered component. This value is based on the components and AC
+ coupling capacitors used in the connection linking the two
+ components(PCIe r7.0, sec 5.5.4).
+
required:
- reg
--
2.34.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH v3] schemas: pci: Document PCIe T_POWER_ON
2025-11-26 10:31 [PATCH v3] schemas: pci: Document PCIe T_POWER_ON Krishna Chaitanya Chundru
@ 2025-11-26 13:30 ` Manivannan Sadhasivam
2025-11-26 14:44 ` Bjorn Helgaas
1 sibling, 0 replies; 3+ messages in thread
From: Manivannan Sadhasivam @ 2025-11-26 13:30 UTC (permalink / raw)
To: Krishna Chaitanya Chundru
Cc: andersson, robh, krzk, helgaas, linux-arm-msm, devicetree,
lpieralisi, kw, conor+dt, linux-pci, linux-kernel,
devicetree-spec, quic_vbadigan, lukas
On Wed, Nov 26, 2025 at 04:01:12PM +0530, Krishna Chaitanya Chundru wrote:
> From PCIe r7, sec 5.5.4 & Table 5-11 in sec 5.5.5 T_POWER_ON is the
> minimum amount of time(in us) that each component must wait in L1.2.Exit
> after sampling CLKREQ# asserted before actively driving the interface to
> ensure no device is ever actively driving into an unpowered component and
> these values are based on the components and AC coupling capacitors used
> in the connection linking the two components.
>
> This property should be used to indicate the T_POWER_ON and drivers using
> this property are responsible for parsing both the scale and the value of
s/parsing/programming
> T_POWER_ON to comply with the PCIe specification.
>
> Signed-off-by: Krishna Chaitanya Chundru <krishna.chundru@oss.qualcomm.com>
Acked-by: Manivannan Sadhasivam <mani@kernel.org>
- Mani
> ---
> Changes in v2:
> - Move the property to pci-device.yaml so that it will be applicable to
> endpoint devices also (Mani).
> - Use latest spec (Lukas)
> - Link to v2: https://lore.kernel.org/all/20251110112947.2071036-1-krishna.chundru@oss.qualcomm.com/
> Changes in v1:
> - Updated the commiit text (Mani).
> - Link to v1: https://lore.kernel.org/all/20251110112550.2070659-1-krishna.chundru@oss.qualcomm.com/#t
>
> dtschema/schemas/pci/pci-device.yaml | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/dtschema/schemas/pci/pci-device.yaml b/dtschema/schemas/pci/pci-device.yaml
> index ca094a0..4baab71 100644
> --- a/dtschema/schemas/pci/pci-device.yaml
> +++ b/dtschema/schemas/pci/pci-device.yaml
> @@ -63,6 +63,15 @@ properties:
> description: GPIO controlled connection to WAKE# signal
> maxItems: 1
>
> + t-power-on-us:
> + description:
> + The minimum amount of time that each component must wait in
> + L1.2.Exit after sampling CLKREQ# asserted before actively driving
> + the interface to ensure no device is ever actively driving into an
> + unpowered component. This value is based on the components and AC
> + coupling capacitors used in the connection linking the two
> + components(PCIe r7.0, sec 5.5.4).
> +
> required:
> - reg
>
> --
> 2.34.1
>
--
மணிவண்ணன் சதாசிவம்
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v3] schemas: pci: Document PCIe T_POWER_ON
2025-11-26 10:31 [PATCH v3] schemas: pci: Document PCIe T_POWER_ON Krishna Chaitanya Chundru
2025-11-26 13:30 ` Manivannan Sadhasivam
@ 2025-11-26 14:44 ` Bjorn Helgaas
1 sibling, 0 replies; 3+ messages in thread
From: Bjorn Helgaas @ 2025-11-26 14:44 UTC (permalink / raw)
To: Krishna Chaitanya Chundru
Cc: andersson, robh, mani, krzk, linux-arm-msm, devicetree,
lpieralisi, kw, conor+dt, linux-pci, linux-kernel,
devicetree-spec, quic_vbadigan, lukas
On Wed, Nov 26, 2025 at 04:01:12PM +0530, Krishna Chaitanya Chundru wrote:
> From PCIe r7, sec 5.5.4 & Table 5-11 in sec 5.5.5 T_POWER_ON is the
> minimum amount of time(in us) that each component must wait in L1.2.Exit
> after sampling CLKREQ# asserted before actively driving the interface to
> ensure no device is ever actively driving into an unpowered component and
> these values are based on the components and AC coupling capacitors used
> in the connection linking the two components.
>
> This property should be used to indicate the T_POWER_ON and drivers using
> this property are responsible for parsing both the scale and the value of
> T_POWER_ON to comply with the PCIe specification.
Add space before open "(" above and in description below.
> Signed-off-by: Krishna Chaitanya Chundru <krishna.chundru@oss.qualcomm.com>
> ---
> Changes in v2:
> - Move the property to pci-device.yaml so that it will be applicable to
> endpoint devices also (Mani).
> - Use latest spec (Lukas)
> - Link to v2: https://lore.kernel.org/all/20251110112947.2071036-1-krishna.chundru@oss.qualcomm.com/
> Changes in v1:
> - Updated the commiit text (Mani).
> - Link to v1: https://lore.kernel.org/all/20251110112550.2070659-1-krishna.chundru@oss.qualcomm.com/#t
>
> dtschema/schemas/pci/pci-device.yaml | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/dtschema/schemas/pci/pci-device.yaml b/dtschema/schemas/pci/pci-device.yaml
> index ca094a0..4baab71 100644
> --- a/dtschema/schemas/pci/pci-device.yaml
> +++ b/dtschema/schemas/pci/pci-device.yaml
> @@ -63,6 +63,15 @@ properties:
> description: GPIO controlled connection to WAKE# signal
> maxItems: 1
>
> + t-power-on-us:
> + description:
> + The minimum amount of time that each component must wait in
> + L1.2.Exit after sampling CLKREQ# asserted before actively driving
> + the interface to ensure no device is ever actively driving into an
> + unpowered component. This value is based on the components and AC
> + coupling capacitors used in the connection linking the two
> + components(PCIe r7.0, sec 5.5.4).
> +
> required:
> - reg
>
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-11-26 14:44 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-11-26 10:31 [PATCH v3] schemas: pci: Document PCIe T_POWER_ON Krishna Chaitanya Chundru
2025-11-26 13:30 ` Manivannan Sadhasivam
2025-11-26 14:44 ` Bjorn Helgaas
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox