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

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/

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/
>
>

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/

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
>