public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
* [PATCH 1/2] dt-bindings: connector: add pd-disable dependency
@ 2026-03-12  7:16 Xu Yang
  2026-03-12  7:16 ` [PATCH 2/2] arm64: dts: freescale: imx8mp-moduline-display-106: add typec-power-opmode property Xu Yang
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Xu Yang @ 2026-03-12  7:16 UTC (permalink / raw)
  To: robh, krzk+dt, conor+dt, Frank.Li, s.hauer, kernel, festevam,
	amitsd, gregkh, kyletso
  Cc: devicetree, linux-kernel, imx, linux-arm-kernel

When Power Delivery is not supported, the source is unable to obtain the
current capability from the Source PDO. As a result, typec-power-opmode
needs to be added to advertise such capability.

Fixes: 7a4440bc0d86 ("dt-bindings: connector: Add pd-disable property")
Signed-off-by: Xu Yang <xu.yang_2@nxp.com>
---
 Documentation/devicetree/bindings/connector/usb-connector.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/connector/usb-connector.yaml b/Documentation/devicetree/bindings/connector/usb-connector.yaml
index 901986de3e2b..671d7775b635 100644
--- a/Documentation/devicetree/bindings/connector/usb-connector.yaml
+++ b/Documentation/devicetree/bindings/connector/usb-connector.yaml
@@ -335,6 +335,7 @@ properties:
     maximum: 100000
 
 dependencies:
+  pd-disable: [typec-power-opmode]
   sink-vdos-v1: [ sink-vdos ]
   sink-vdos: [ sink-vdos-v1 ]
 
-- 
2.34.1



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 2/2] arm64: dts: freescale: imx8mp-moduline-display-106: add typec-power-opmode property
  2026-03-12  7:16 [PATCH 1/2] dt-bindings: connector: add pd-disable dependency Xu Yang
@ 2026-03-12  7:16 ` Xu Yang
  2026-03-12 17:47 ` [PATCH 1/2] dt-bindings: connector: add pd-disable dependency Conor Dooley
  2026-03-26 13:36 ` (subset) " Frank Li
  2 siblings, 0 replies; 6+ messages in thread
From: Xu Yang @ 2026-03-12  7:16 UTC (permalink / raw)
  To: robh, krzk+dt, conor+dt, Frank.Li, s.hauer, kernel, festevam,
	amitsd, gregkh, kyletso
  Cc: devicetree, linux-kernel, imx, linux-arm-kernel

To avoid dtbs_check warning, add typec-power-opmode property.

Signed-off-by: Xu Yang <xu.yang_2@nxp.com>
---
 .../imx8mp-tx8p-ml81-moduline-display-106-av101hdt-a10.dtso      | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8mp-tx8p-ml81-moduline-display-106-av101hdt-a10.dtso b/arch/arm64/boot/dts/freescale/imx8mp-tx8p-ml81-moduline-display-106-av101hdt-a10.dtso
index e3965caca6be..c6fc5d5b1e5f 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-tx8p-ml81-moduline-display-106-av101hdt-a10.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mp-tx8p-ml81-moduline-display-106-av101hdt-a10.dtso
@@ -77,6 +77,7 @@ connector {
 		compatible = "usb-c-connector";
 		data-role = "host";
 		pd-disable;
+		typec-power-opmode = "default";
 		vbus-supply = <&reg_vbus>;
 
 		port {
-- 
2.34.1



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/2] dt-bindings: connector: add pd-disable dependency
  2026-03-12  7:16 [PATCH 1/2] dt-bindings: connector: add pd-disable dependency Xu Yang
  2026-03-12  7:16 ` [PATCH 2/2] arm64: dts: freescale: imx8mp-moduline-display-106: add typec-power-opmode property Xu Yang
@ 2026-03-12 17:47 ` Conor Dooley
  2026-03-13  1:58   ` Xu Yang
  2026-03-26 13:36 ` (subset) " Frank Li
  2 siblings, 1 reply; 6+ messages in thread
From: Conor Dooley @ 2026-03-12 17:47 UTC (permalink / raw)
  To: Xu Yang
  Cc: robh, krzk+dt, conor+dt, Frank.Li, s.hauer, kernel, festevam,
	amitsd, gregkh, kyletso, devicetree, linux-kernel, imx,
	linux-arm-kernel

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

On Thu, Mar 12, 2026 at 03:16:08PM +0800, Xu Yang wrote:
> When Power Delivery is not supported, the source is unable to obtain the
> current capability from the Source PDO. As a result, typec-power-opmode
> needs to be added to advertise such capability.

What's the impact on any users of making this a dependency?
I assume nothing negative, since this will only impact systems that
don't work properly at present?

> 
> Fixes: 7a4440bc0d86 ("dt-bindings: connector: Add pd-disable property")
> Signed-off-by: Xu Yang <xu.yang_2@nxp.com>
> ---
>  Documentation/devicetree/bindings/connector/usb-connector.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/connector/usb-connector.yaml b/Documentation/devicetree/bindings/connector/usb-connector.yaml
> index 901986de3e2b..671d7775b635 100644
> --- a/Documentation/devicetree/bindings/connector/usb-connector.yaml
> +++ b/Documentation/devicetree/bindings/connector/usb-connector.yaml
> @@ -335,6 +335,7 @@ properties:
>      maximum: 100000
>  
>  dependencies:
> +  pd-disable: [typec-power-opmode]
>    sink-vdos-v1: [ sink-vdos ]
>    sink-vdos: [ sink-vdos-v1 ]
>  
> -- 
> 2.34.1
> 

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/2] dt-bindings: connector: add pd-disable dependency
  2026-03-12 17:47 ` [PATCH 1/2] dt-bindings: connector: add pd-disable dependency Conor Dooley
@ 2026-03-13  1:58   ` Xu Yang
  2026-03-13 17:25     ` Conor Dooley
  0 siblings, 1 reply; 6+ messages in thread
From: Xu Yang @ 2026-03-13  1:58 UTC (permalink / raw)
  To: Conor Dooley
  Cc: robh, krzk+dt, conor+dt, Frank.Li, s.hauer, kernel, festevam,
	amitsd, gregkh, kyletso, devicetree, linux-kernel, imx,
	linux-arm-kernel

On Thu, Mar 12, 2026 at 05:47:32PM +0000, Conor Dooley wrote:
> On Thu, Mar 12, 2026 at 03:16:08PM +0800, Xu Yang wrote:
> > When Power Delivery is not supported, the source is unable to obtain the
> > current capability from the Source PDO. As a result, typec-power-opmode
> > needs to be added to advertise such capability.
> 
> What's the impact on any users of making this a dependency?
> I assume nothing negative, since this will only impact systems that
> don't work properly at present?

Yes, right. Most of the user satisfy this requirement,
expect imx8mp-tx8p-ml81-moduline-display-106-av101hdt-a10.dtso.
I suppose this user can't work well, so patch#2 is added.

Thanks,
Xu Yang


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/2] dt-bindings: connector: add pd-disable dependency
  2026-03-13  1:58   ` Xu Yang
@ 2026-03-13 17:25     ` Conor Dooley
  0 siblings, 0 replies; 6+ messages in thread
From: Conor Dooley @ 2026-03-13 17:25 UTC (permalink / raw)
  To: Xu Yang
  Cc: robh, krzk+dt, conor+dt, Frank.Li, s.hauer, kernel, festevam,
	amitsd, gregkh, kyletso, devicetree, linux-kernel, imx,
	linux-arm-kernel

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

On Fri, Mar 13, 2026 at 09:58:18AM +0800, Xu Yang wrote:
> On Thu, Mar 12, 2026 at 05:47:32PM +0000, Conor Dooley wrote:
> > On Thu, Mar 12, 2026 at 03:16:08PM +0800, Xu Yang wrote:
> > > When Power Delivery is not supported, the source is unable to obtain the
> > > current capability from the Source PDO. As a result, typec-power-opmode
> > > needs to be added to advertise such capability.
> > 
> > What's the impact on any users of making this a dependency?
> > I assume nothing negative, since this will only impact systems that
> > don't work properly at present?
> 
> Yes, right. Most of the user satisfy this requirement,
> expect imx8mp-tx8p-ml81-moduline-display-106-av101hdt-a10.dtso.
> I suppose this user can't work well, so patch#2 is added.

Okay.
Acked-by: Conor Dooley <conor.dooley@microchip.com>
pw-bot: not-applicable

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: (subset) [PATCH 1/2] dt-bindings: connector: add pd-disable dependency
  2026-03-12  7:16 [PATCH 1/2] dt-bindings: connector: add pd-disable dependency Xu Yang
  2026-03-12  7:16 ` [PATCH 2/2] arm64: dts: freescale: imx8mp-moduline-display-106: add typec-power-opmode property Xu Yang
  2026-03-12 17:47 ` [PATCH 1/2] dt-bindings: connector: add pd-disable dependency Conor Dooley
@ 2026-03-26 13:36 ` Frank Li
  2 siblings, 0 replies; 6+ messages in thread
From: Frank Li @ 2026-03-26 13:36 UTC (permalink / raw)
  To: robh, krzk+dt, conor+dt, s.hauer, kernel, festevam, amitsd,
	gregkh, kyletso, Xu Yang
  Cc: devicetree, linux-kernel, imx, linux-arm-kernel


On Thu, 12 Mar 2026 15:16:08 +0800, Xu Yang wrote:
> When Power Delivery is not supported, the source is unable to obtain the
> current capability from the Source PDO. As a result, typec-power-opmode
> needs to be added to advertise such capability.
> 
> 

Applied, thanks!

[2/2] arm64: dts: freescale: imx8mp-moduline-display-106: add typec-power-opmode property
      commit: 057756df6770c033eb8ebbc9b0066f6602e99b08

Best regards,
-- 
Frank Li <Frank.Li@nxp.com>



^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2026-03-26 13:36 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-03-12  7:16 [PATCH 1/2] dt-bindings: connector: add pd-disable dependency Xu Yang
2026-03-12  7:16 ` [PATCH 2/2] arm64: dts: freescale: imx8mp-moduline-display-106: add typec-power-opmode property Xu Yang
2026-03-12 17:47 ` [PATCH 1/2] dt-bindings: connector: add pd-disable dependency Conor Dooley
2026-03-13  1:58   ` Xu Yang
2026-03-13 17:25     ` Conor Dooley
2026-03-26 13:36 ` (subset) " Frank Li

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox