| Message ID | 1652723410-1630-1-git-send-email-quic_kriskura@quicinc.com |
|---|---|
| Headers | show |
| Series | Add QCOM SNPS PHY overriding params support | expand |
On 16/05/2022 19:50, Krishna Kurapati wrote: > Added support for overriding tuning parameters in QCOM SNPS PHY > from device tree. This parameter tuning is required to tune the > hs signal on dp/dm lines for electrical compliance to be successful. I commented at your v4 - please use proper PATCH subject. It is added automatically (with version) for example by git format-patch. Your mails escape regular filters - any reason why doing like that? More difficult for reviewers? Best regards, Krzysztof
Added support for overriding tuning parameters in QCOM SNPS PHY from device tree. This parameter tuning is required to tune the hs signal on dp/dm lines for electrical compliance to be successful. changes in v5: Fixed nitpicks in code. Added minimum and maximum for each parameter added in dt-bindings. Added proper suffixes to each parameter as per dtschema. changes in v4: Fixed nitpicks in code. Initial compliance test results showed overshoot in the middle of eye diagram. The current dt values were put in place to correct it and fix overshoot issue. changes in v3: Added support for phy tuning parameters to be represented in bps and corresponding register values to be written are obtained by traversing through data map declared in the driver. changes in v2: Reading the individual fields in each overriding register from device tree. Krishna Kurapati (2): phy: qcom-snps: Add support for overriding phy tuning parameters arm64: dts: qcom: sc7280: Update SNPS Phy params for SC7280 IDP device Sandeep Maheswaram (1): dt-bindings: phy: qcom,usb-snps-femto-v2: Add phy override params bindings .../bindings/phy/qcom,usb-snps-femto-v2.yaml | 105 ++++++++ arch/arm64/boot/dts/qcom/sc7280-idp.dtsi | 6 + drivers/phy/qualcomm/phy-qcom-snps-femto-v2.c | 268 ++++++++++++++++++++- 3 files changed, 377 insertions(+), 2 deletions(-)