| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-Id: <175011004935.2433563.8726528182523156685.robh@kernel.org> Date: Mon, 16 Jun 2025 16:46:51 -0500 From: "Rob Herring (Arm)" <robh@...nel.org> To: Pritam Manohar Sutar <pritam.sutar@...sung.com> Cc: rosa.pila@...sung.com, s.nawrocki@...sung.com, linux-samsung-soc@...r.kernel.org, conor+dt@...nel.org, linux-kernel@...r.kernel.org, andre.draszik@...aro.org, linux-phy@...ts.infradead.org, vkoul@...nel.org, krzk+dt@...nel.org, faraz.ata@...sung.com, selvarasu.g@...sung.com, kauschluss@...root.org, ivo.ivanov.ivanov1@...il.com, dev.tailor@...sung.com, devicetree@...r.kernel.org, kishon@...nel.org, peter.griffin@...aro.org, muhammed.ali@...sung.com, linux-arm-kernel@...ts.infradead.org, alim.akhtar@...sung.com, m.szyprowski@...sung.com Subject: Re: [PATCH v3 0/9] initial usbdrd phy support for Exynosautov920 soc On Fri, 13 Jun 2025 11:26:04 +0530, Pritam Manohar Sutar wrote: > This SoC has a single USB 3.1 DRD combo phy and three USB2.0 only > DRD phy controllers > > - Combo phy supports USB3.1 SSP+(10Gbps) protocol and is backwards > compatible to the USB3.0 SS(5Gbps). 'Add-on USB2.0' phy is added > to support USB2.0 HS(480Mbps), FS(12Mbps) and LS(1.5Mbps) data > rates. These two phys are combined to form a combo phy as mentioned > below. > > USB30DRD_0 port > > +-----------------------------------------------------+ > | | > | (combo) USB PHY controller | > | +-----------------------------------------+ | > | | USB HSPHY | | > | | (samsung,exynosautov920-usbdrd-hsphy) | | > | +-----------------------------------------+ | > | | > | +---------------------------------------------+ | > | | USB SSPHY | | > | | (samsung,exynosautov920-usb31drd-ssphy) | | > | +---------------------------------------------+ | > | | > +-----------------------------------------------------+ > | | > | USBDRD30 Link | > | Controller | > | | > +-----------------------------------------------------+ > > - USB2.0 phy supports only UTMI+ interface. USB2.0DRD phy > is very similar to the existing Exynos850 support in this driver. > > USB20DRD_0/1/2 ports > > > +---------------------------------------------------+ > | | > | USB PHY controller | > | +-----------------------------------------+ | > | | USB HSPHY | | > | | (samsung,exynosautov920-usbdrd-phy) | | > | +-----------------------------------------+ | > | | > +---------------------------------------------------+ > | | > | USBDRD20_* Link | > | Controller | > | | > +---------------------------------------------------+ > > This patchset only supports device mode and same is verified with > as NCM device with below configfs commands > > changelog > ---------- > Changes in v2: > - Used standard GENMASK() and FIELD_GET() to get the major version > from controller version register. > link for v1: https://lore.kernel.org/linux-phy/20250514134813.380807-1-pritam.sutar@samsung.com/ > > Changes in v3: > - Updated dt-bindings for USB2.0 only. > - Added dt-bindings for combo phy. > - Added implementation for combo phy (SS and HS phy). > - Added added DTS nodes for all the phys. > link for v2: https://lore.kernel.org/linux-phy/20250516102650.2144487-1-pritam.sutar@samsung.com/ > > Pritam Manohar Sutar (9): > dt-bindings: phy: samsung,usb3-drd-phy: add ExynosAutov920 HS phy > compatible > phy: exyons5-usbdrd: support HS phy for ExynosAutov920 > arm64: dts: exynos: ExynosAutov920: add USB and USB-phy nodes > dt-bindings: phy: samsung,usb3-drd-phy: add ExynosAutov920 combo HS > phy > phy: exyons5-usbdrd: support HS combo phy for ExynosAutov920 > arm64: dts: exynos: ExynosAutov920: add USB and USB HS combo phy nodes > dt-bindings: phy: samsung,usb3-drd-phy: add ExynosAutov920 combo SS > phy > phy: exyons5-usbdrd: support SS combo phy for ExynosAutov920 > arm64: dts: exynos: ExynosAutov920: add USB and USB SS combo phy nodes > > .../bindings/phy/samsung,usb3-drd-phy.yaml | 6 + > .../boot/dts/exynos/exynosautov920-sadk.dts | 53 ++ > .../arm64/boot/dts/exynos/exynosautov920.dtsi | 155 +++++ > drivers/phy/samsung/phy-exynos5-usbdrd.c | 529 ++++++++++++++++++ > 4 files changed, 743 insertions(+) > > -- > 2.34.1 > > > My bot found new DTB warnings on the .dts files added or changed in this series. Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings are fixed by another series. Ultimately, it is up to the platform maintainer whether these warnings are acceptable or not. No need to reply unless the platform maintainer has comments. If you already ran DT checks and didn't see these error(s), then make sure dt-schema is up to date: pip3 install dtschema --upgrade This patch series was applied (using b4) to base: Base: attempting to guess base-commit... Base: tags/v6.16-rc1-6-g8a22d9e79cf0 (exact match) If this is not the correct base, please add 'base-commit' tag (or use b4 which does this automatically) New warnings running 'make CHECK_DTBS=y for arch/arm64/boot/dts/exynos/' for 20250613055613.866909-1-pritam.sutar@...sung.com: arch/arm64/boot/dts/exynos/exynosautov920-sadk.dtb: usb@...00000 (samsung,exynosautov920-dwusb3): 'vdd33-supply' is a required property from schema $id: http://devicetree.org/schemas/usb/samsung,exynos-dwc3.yaml# arch/arm64/boot/dts/exynos/exynosautov920-sadk.dtb: usb@...00000 (samsung,exynosautov920-dwusb3): 'vdd33-supply' is a required property from schema $id: http://devicetree.org/schemas/usb/samsung,exynos-dwc3.yaml# arch/arm64/boot/dts/exynos/exynosautov920-sadk.dtb: usb@...00000 (samsung,exynosautov920-dwusb3): 'vdd33-supply' is a required property from schema $id: http://devicetree.org/schemas/usb/samsung,exynos-dwc3.yaml# arch/arm64/boot/dts/exynos/exynosautov920-sadk.dtb: usb@...00000 (samsung,exynosautov920-dwusb3): 'vdd33-supply' is a required property from schema $id: http://devicetree.org/schemas/usb/samsung,exynos-dwc3.yaml#
Powered by blists - more mailing lists