[PATCH 0/5] dtbs_check warning clean up against fsl,mxs-usbphy.yaml

[PATCH 0/5] dtbs_check warning clean up against fsl,mxs-usbphy.yaml

[Xu Yang]

Some changes to clean up dtbs_check warning against fsl,mxs-usbphy.yaml.

The first 4 patches are used to fix below warning, "nxp,sim" is only
needed by i.MX7ULP:
 - 'nxp,sim' is a required property

The last patch is used to fix below warning:
 - fsl,tx-d-cal: 5 is less than the minimum of 79

Xu Yang (5):
  dt-bindings: phy: mxs-usb-phy: add imx8qxp compatible
  arm64: dts: imx8dxl-ss-conn: change usbphy1 compatible
  arm64: dts: imx8qm: change usbphy1 compatible
  arm64: dts: imx8qxp: change usbphy1 compatible
  ARM: dts: imx6qdl: convert fsl,tx-d-cal to correct value

 Documentation/devicetree/bindings/phy/fsl,mxs-usbphy.yaml | 1 +
 arch/arm/boot/dts/nxp/imx/imx6qdl-var-som.dtsi            | 4 ++--
 arch/arm64/boot/dts/freescale/imx8dxl-ss-conn.dtsi        | 4 ++++
 arch/arm64/boot/dts/freescale/imx8qm-ss-conn.dtsi         | 4 ++++
 arch/arm64/boot/dts/freescale/imx8qxp-ss-conn.dtsi        | 4 ++++
 5 files changed, 15 insertions(+), 2 deletions(-)

-- 
2.34.1


-- 
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy

[PATCH 1/5] dt-bindings: phy: mxs-usb-phy: add imx8qxp compatible

[Xu Yang]

Add "fsl,imx8qxp-usbphy" compatible.

Signed-off-by: Xu Yang <xu.yang_2@nxp.com>
---
 Documentation/devicetree/bindings/phy/fsl,mxs-usbphy.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/phy/fsl,mxs-usbphy.yaml b/Documentation/devicetree/bindings/phy/fsl,mxs-usbphy.yaml
index ce665a2779b7..d01b7d187040 100644
--- a/Documentation/devicetree/bindings/phy/fsl,mxs-usbphy.yaml
+++ b/Documentation/devicetree/bindings/phy/fsl,mxs-usbphy.yaml
@@ -32,6 +32,7 @@ properties:
           - enum:
               - fsl,imx8dxl-usbphy
               - fsl,imx8qm-usbphy
+              - fsl,imx8qxp-usbphy
               - fsl,imx8ulp-usbphy
           - const: fsl,imx7ulp-usbphy
 
-- 
2.34.1


-- 
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy

Re: [PATCH 1/5] dt-bindings: phy: mxs-usb-phy: add imx8qxp compatible

[Conor Dooley]

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

On Thu, Sep 26, 2024 at 03:39:47PM +0800, Xu Yang wrote:
> Add "fsl,imx8qxp-usbphy" compatible.
> 
> Signed-off-by: Xu Yang <xu.yang_2@nxp.com>

Acked-by: Conor Dooley <conor.dooley@microchip.com>

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

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

-- 
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy

[PATCH 2/5] arm64: dts: imx8dxl-ss-conn: change usbphy1 compatible

[Xu Yang]

Make fsl,imx8dxl-usbphy compatible with fsl,imx7ulp-usbphy to fix below
warning:

/home/nxf75279/work/linux-next/arch/arm64/boot/dts/freescale/imx8dxl-evk.dtb: usbphy@5b100000: 'nxp,sim' is a required property
        from schema $id: http://devicetree.org/schemas/phy/fsl,mxs-usbphy.yaml#

Signed-off-by: Xu Yang <xu.yang_2@nxp.com>
---
 arch/arm64/boot/dts/freescale/imx8dxl-ss-conn.dtsi | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8dxl-ss-conn.dtsi b/arch/arm64/boot/dts/freescale/imx8dxl-ss-conn.dtsi
index 1e02b04494e9..9b114bed084b 100644
--- a/arch/arm64/boot/dts/freescale/imx8dxl-ss-conn.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8dxl-ss-conn.dtsi
@@ -138,6 +138,10 @@ &gpmi {
 	interrupts = <GIC_SPI 174 IRQ_TYPE_LEVEL_HIGH>;
 };
 
+&usbphy1 {
+	compatible = "fsl,imx8dxl-usbphy", "fsl,imx7ulp-usbphy";
+};
+
 &usdhc1 {
 	compatible = "fsl,imx8dxl-usdhc", "fsl,imx8qxp-usdhc";
 	interrupts = <GIC_SPI 138 IRQ_TYPE_LEVEL_HIGH>;
-- 
2.34.1


-- 
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy

Re: [PATCH 2/5] arm64: dts: imx8dxl-ss-conn: change usbphy1 compatible

[Shawn Guo]

On Thu, Sep 26, 2024 at 03:39:48PM +0800, Xu Yang wrote:
> Make fsl,imx8dxl-usbphy compatible with fsl,imx7ulp-usbphy to fix below
> warning:
> 
> /home/nxf75279/work/linux-next/arch/arm64/boot/dts/freescale/imx8dxl-evk.dtb: usbphy@5b100000: 'nxp,sim' is a required property

A relative path might be a better fit:

arch/arm64/boot/dts/freescale/imx8dxl-evk.dtb: usbphy@5b100000: 'nxp,sim' is a required property

I applied all 4 DTS patches with this fix-up.

Shawn

>         from schema $id: http://devicetree.org/schemas/phy/fsl,mxs-usbphy.yaml#
> 
> Signed-off-by: Xu Yang <xu.yang_2@nxp.com>
> ---
>  arch/arm64/boot/dts/freescale/imx8dxl-ss-conn.dtsi | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/freescale/imx8dxl-ss-conn.dtsi b/arch/arm64/boot/dts/freescale/imx8dxl-ss-conn.dtsi
> index 1e02b04494e9..9b114bed084b 100644
> --- a/arch/arm64/boot/dts/freescale/imx8dxl-ss-conn.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8dxl-ss-conn.dtsi
> @@ -138,6 +138,10 @@ &gpmi {
>  	interrupts = <GIC_SPI 174 IRQ_TYPE_LEVEL_HIGH>;
>  };
>  
> +&usbphy1 {
> +	compatible = "fsl,imx8dxl-usbphy", "fsl,imx7ulp-usbphy";
> +};
> +
>  &usdhc1 {
>  	compatible = "fsl,imx8dxl-usdhc", "fsl,imx8qxp-usdhc";
>  	interrupts = <GIC_SPI 138 IRQ_TYPE_LEVEL_HIGH>;
> -- 
> 2.34.1
> 


-- 
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy

[PATCH 3/5] arm64: dts: imx8qm: change usbphy1 compatible

[Xu Yang]

Make fsl,imx8qm-usbphy compatible with fsl,imx7ulp-usbphy to fix below
warning:

/home/nxf75279/work/linux-next/arch/arm64/boot/dts/freescale/imx8qm-apalis-eval.dtb: usbphy@5b100000: 'nxp,sim' is a required property
        from schema $id: http://devicetree.org/schemas/phy/fsl,mxs-usbphy.yaml#

Signed-off-by: Xu Yang <xu.yang_2@nxp.com>
---
 arch/arm64/boot/dts/freescale/imx8qm-ss-conn.dtsi | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8qm-ss-conn.dtsi b/arch/arm64/boot/dts/freescale/imx8qm-ss-conn.dtsi
index 545e175c88b3..ccf9f510e0f8 100644
--- a/arch/arm64/boot/dts/freescale/imx8qm-ss-conn.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8qm-ss-conn.dtsi
@@ -4,6 +4,10 @@
  *	Dong Aisheng <aisheng.dong@nxp.com>
  */
 
+&usbphy1 {
+	compatible = "fsl,imx8qm-usbphy", "fsl,imx7ulp-usbphy";
+};
+
 &fec1 {
 	compatible = "fsl,imx8qm-fec", "fsl,imx6sx-fec";
 	iommus = <&smmu 0x12 0x7f80>;
-- 
2.34.1


-- 
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy

[PATCH 4/5] arm64: dts: imx8qxp: change usbphy1 compatible

[Xu Yang]

Make fsl,imx8qxp-usbphy compatible with fsl,imx7ulp-usbphy to fix below
warning:

/home/nxf75279/work/linux-next/arch/arm64/boot/dts/freescale/imx8qxp-colibri-aster.dtb: usbphy@5b100000: 'nxp,sim' is a required property
        from schema $id: http://devicetree.org/schemas/phy/fsl,mxs-usbphy.yaml#

Signed-off-by: Xu Yang <xu.yang_2@nxp.com>
---
 arch/arm64/boot/dts/freescale/imx8qxp-ss-conn.dtsi | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8qxp-ss-conn.dtsi b/arch/arm64/boot/dts/freescale/imx8qxp-ss-conn.dtsi
index 46da21af3702..4eb48ad48745 100644
--- a/arch/arm64/boot/dts/freescale/imx8qxp-ss-conn.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8qxp-ss-conn.dtsi
@@ -4,6 +4,10 @@
  *	Dong Aisheng <aisheng.dong@nxp.com>
  */
 
+&usbphy1 {
+	compatible = "fsl,imx8qxp-usbphy", "fsl,imx7ulp-usbphy";
+};
+
 &usdhc1 {
 	compatible = "fsl,imx8qxp-usdhc", "fsl,imx7d-usdhc";
 };
-- 
2.34.1


-- 
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy

[PATCH 5/5] ARM: dts: imx6qdl: convert fsl,tx-d-cal to correct value

[Xu Yang]

make dtbs_check has below warning:

/home/nxf75279/work/usb/arch/arm/boot/dts/nxp/imx/imx6q-var-mx6customboard.dtb: usbphy@20c9000: fsl,tx-d-cal: 5 is less than the minimum of 79
        from schema $id: http://devicetree.org/schemas/phy/fsl,mxs-usbphy.yaml#

According to schema, valid value of fsl,tx-d-cal is from 79 to 119. This
will convert register raw value <0x5> to corresponding <106>.

Signed-off-by: Xu Yang <xu.yang_2@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx6qdl-var-som.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-var-som.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-var-som.dtsi
index a1ea33c4eeb7..d3396cb81912 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-var-som.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-var-som.dtsi
@@ -529,11 +529,11 @@ &usbotg {
 };
 
 &usbphy1 {
-	fsl,tx-d-cal = <0x5>;
+	fsl,tx-d-cal = <106>;
 };
 
 &usbphy2 {
-	fsl,tx-d-cal = <0x5>;
+	fsl,tx-d-cal = <106>;
 };
 
 &usdhc1 {
-- 
2.34.1


-- 
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy

Re: [PATCH 0/5] dtbs_check warning clean up against fsl,mxs-usbphy.yaml

[Vinod Koul]

On Thu, 26 Sep 2024 15:39:46 +0800, Xu Yang wrote:
> Some changes to clean up dtbs_check warning against fsl,mxs-usbphy.yaml.
> 
> The first 4 patches are used to fix below warning, "nxp,sim" is only
> needed by i.MX7ULP:
>  - 'nxp,sim' is a required property
> 
> The last patch is used to fix below warning:
>  - fsl,tx-d-cal: 5 is less than the minimum of 79
> 
> [...]

Applied, thanks!

[1/5] dt-bindings: phy: mxs-usb-phy: add imx8qxp compatible
      commit: 76b4f2a5cec59e842de4b5989eb990a2ee8cedf3
[2/5] arm64: dts: imx8dxl-ss-conn: change usbphy1 compatible
      (no commit info)
[3/5] arm64: dts: imx8qm: change usbphy1 compatible
      (no commit info)
[4/5] arm64: dts: imx8qxp: change usbphy1 compatible
      (no commit info)
[5/5] ARM: dts: imx6qdl: convert fsl,tx-d-cal to correct value
      (no commit info)

Best regards,
-- 
~Vinod



-- 
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy

Re: [PATCH 0/5] dtbs_check warning clean up against fsl,mxs-usbphy.yaml

[Alexander Stein]

Hi Vinod,

Am Donnerstag, 17. Oktober 2024, 17:36:12 CET schrieb Vinod Koul:
> 
> On Thu, 26 Sep 2024 15:39:46 +0800, Xu Yang wrote:
> > Some changes to clean up dtbs_check warning against fsl,mxs-usbphy.yaml.
> > 
> > The first 4 patches are used to fix below warning, "nxp,sim" is only
> > needed by i.MX7ULP:
> >  - 'nxp,sim' is a required property
> > 
> > The last patch is used to fix below warning:
> >  - fsl,tx-d-cal: 5 is less than the minimum of 79
> > 
> > [...]
> 
> Applied, thanks!
> 
> [1/5] dt-bindings: phy: mxs-usb-phy: add imx8qxp compatible
>       commit: 76b4f2a5cec59e842de4b5989eb990a2ee8cedf3
> [2/5] arm64: dts: imx8dxl-ss-conn: change usbphy1 compatible
>       (no commit info)
> [3/5] arm64: dts: imx8qm: change usbphy1 compatible
>       (no commit info)
> [4/5] arm64: dts: imx8qxp: change usbphy1 compatible
>       (no commit info)
> [5/5] ARM: dts: imx6qdl: convert fsl,tx-d-cal to correct value
>       (no commit info)

Did maybe something go wrong while merging? Only patch 1 has a commit SHA1
in this mail.
Also only patch 1 is available in linux-next

Thanks and best regards,
Alexander
-- 
TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
Amtsgericht München, HRB 105018
Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
http://www.tq-group.com/



-- 
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy

Re: [PATCH 0/5] dtbs_check warning clean up against fsl,mxs-usbphy.yaml

[Frank Li]

On Tue, Oct 29, 2024 at 04:15:54PM +0100, Alexander Stein wrote:
> Hi Vinod,
>
> Am Donnerstag, 17. Oktober 2024, 17:36:12 CET schrieb Vinod Koul:
> >
> > On Thu, 26 Sep 2024 15:39:46 +0800, Xu Yang wrote:
> > > Some changes to clean up dtbs_check warning against fsl,mxs-usbphy.yaml.
> > >
> > > The first 4 patches are used to fix below warning, "nxp,sim" is only
> > > needed by i.MX7ULP:
> > >  - 'nxp,sim' is a required property
> > >
> > > The last patch is used to fix below warning:
> > >  - fsl,tx-d-cal: 5 is less than the minimum of 79
> > >
> > > [...]
> >
> > Applied, thanks!
> >
> > [1/5] dt-bindings: phy: mxs-usb-phy: add imx8qxp compatible
> >       commit: 76b4f2a5cec59e842de4b5989eb990a2ee8cedf3
> > [2/5] arm64: dts: imx8dxl-ss-conn: change usbphy1 compatible
> >       (no commit info)
> > [3/5] arm64: dts: imx8qm: change usbphy1 compatible
> >       (no commit info)
> > [4/5] arm64: dts: imx8qxp: change usbphy1 compatible
> >       (no commit info)
> > [5/5] ARM: dts: imx6qdl: convert fsl,tx-d-cal to correct value
> >       (no commit info)
>
> Did maybe something go wrong while merging? Only patch 1 has a commit SHA1
> in this mail.
> Also only patch 1 is available in linux-next

Supposed script report issue. Only dt-bindings go to through DMA tree.
Other dts should go through shawn's tree.

Still wait for shawn pick dts patches.

Frank

>
> Thanks and best regards,
> Alexander
> --
> TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
> Amtsgericht München, HRB 105018
> Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
> http://www.tq-group.com/
>
>

-- 
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy

Re: [PATCH 0/5] dtbs_check warning clean up against fsl, mxs-usbphy.yaml

[Alexander Stein]

Hi Frank,

Am Dienstag, 29. Oktober 2024, 16:29:07 CET schrieb Frank Li:
> On Tue, Oct 29, 2024 at 04:15:54PM +0100, Alexander Stein wrote:
> > Hi Vinod,
> >
> > Am Donnerstag, 17. Oktober 2024, 17:36:12 CET schrieb Vinod Koul:
> > >
> > > On Thu, 26 Sep 2024 15:39:46 +0800, Xu Yang wrote:
> > > > Some changes to clean up dtbs_check warning against fsl,mxs-usbphy.yaml.
> > > >
> > > > The first 4 patches are used to fix below warning, "nxp,sim" is only
> > > > needed by i.MX7ULP:
> > > >  - 'nxp,sim' is a required property
> > > >
> > > > The last patch is used to fix below warning:
> > > >  - fsl,tx-d-cal: 5 is less than the minimum of 79
> > > >
> > > > [...]
> > >
> > > Applied, thanks!
> > >
> > > [1/5] dt-bindings: phy: mxs-usb-phy: add imx8qxp compatible
> > >       commit: 76b4f2a5cec59e842de4b5989eb990a2ee8cedf3
> > > [2/5] arm64: dts: imx8dxl-ss-conn: change usbphy1 compatible
> > >       (no commit info)
> > > [3/5] arm64: dts: imx8qm: change usbphy1 compatible
> > >       (no commit info)
> > > [4/5] arm64: dts: imx8qxp: change usbphy1 compatible
> > >       (no commit info)
> > > [5/5] ARM: dts: imx6qdl: convert fsl,tx-d-cal to correct value
> > >       (no commit info)
> >
> > Did maybe something go wrong while merging? Only patch 1 has a commit SHA1
> > in this mail.
> > Also only patch 1 is available in linux-next
> 
> Supposed script report issue. Only dt-bindings go to through DMA tree.
> Other dts should go through shawn's tree.
> 
> Still wait for shawn pick dts patches.

Aah, thanks for clearing up.

Best regards,
Alexander
-- 
TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
Amtsgericht München, HRB 105018
Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
http://www.tq-group.com/



-- 
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy