* [PATCH v2 1/3] dt-bindings: phy: qcom,snps-eusb2-repeater: Add squelch param update
2025-12-04 4:46 [PATCH v2 0/3] Add eusb2 repeater squelch detect parameter override Krishna Kurapati
@ 2025-12-04 4:46 ` Krishna Kurapati
2025-12-05 20:55 ` Dmitry Baryshkov
2025-12-04 4:46 ` [PATCH v2 2/3] phy: qualcomm: phy-qcom-eusb2-repeater: Add squelch detect " Krishna Kurapati
2025-12-04 4:46 ` [PATCH v2 3/3] arm64: dts: qcom: sm8750-mtp: Add eusb2 repeater tuning parameters Krishna Kurapati
2 siblings, 1 reply; 15+ messages in thread
From: Krishna Kurapati @ 2025-12-04 4:46 UTC (permalink / raw)
To: Vinod Koul, Kishon Vijay Abraham I, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson, Konrad Dybcio,
Luca Weiss, Neil Armstrong, Johan Hovold, Pengyu Luo
Cc: linux-arm-msm, linux-phy, devicetree, linux-kernel,
Krishna Kurapati
Add squelch detect parameter update for synopsys eusb2 repeater.
Signed-off-by: Krishna Kurapati <krishna.kurapati@oss.qualcomm.com>
---
.../devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml b/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml
index 5bf0d6c9c025..f2afcf0e986a 100644
--- a/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml
+++ b/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml
@@ -59,6 +59,14 @@ properties:
minimum: 0
maximum: 7
+ qcom,squelch-detector-bp:
+ description:
+ This adjusts the voltage level for the threshold used to detect valid
+ high-speed data.
+ minimum: -6000
+ maximum: 1000
+ multipleOf: 1000
+
required:
- compatible
- reg
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread* Re: [PATCH v2 1/3] dt-bindings: phy: qcom,snps-eusb2-repeater: Add squelch param update
2025-12-04 4:46 ` [PATCH v2 1/3] dt-bindings: phy: qcom,snps-eusb2-repeater: Add squelch param update Krishna Kurapati
@ 2025-12-05 20:55 ` Dmitry Baryshkov
2025-12-06 11:02 ` Krzysztof Kozlowski
0 siblings, 1 reply; 15+ messages in thread
From: Dmitry Baryshkov @ 2025-12-05 20:55 UTC (permalink / raw)
To: Krishna Kurapati
Cc: Vinod Koul, Kishon Vijay Abraham I, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson, Konrad Dybcio,
Luca Weiss, Neil Armstrong, Johan Hovold, Pengyu Luo,
linux-arm-msm, linux-phy, devicetree, linux-kernel
On Thu, Dec 04, 2025 at 10:16:42AM +0530, Krishna Kurapati wrote:
> Add squelch detect parameter update for synopsys eusb2 repeater.
>
> Signed-off-by: Krishna Kurapati <krishna.kurapati@oss.qualcomm.com>
> ---
> .../devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml b/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml
> index 5bf0d6c9c025..f2afcf0e986a 100644
> --- a/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml
> +++ b/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml
> @@ -59,6 +59,14 @@ properties:
> minimum: 0
> maximum: 7
>
> + qcom,squelch-detector-bp:
-uV? -mV?
> + description:
> + This adjusts the voltage level for the threshold used to detect valid
> + high-speed data.
> + minimum: -6000
> + maximum: 1000
> + multipleOf: 1000
> +
> required:
> - compatible
> - reg
> --
> 2.34.1
>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v2 1/3] dt-bindings: phy: qcom,snps-eusb2-repeater: Add squelch param update
2025-12-05 20:55 ` Dmitry Baryshkov
@ 2025-12-06 11:02 ` Krzysztof Kozlowski
2025-12-06 16:10 ` Krishna Kurapati
2025-12-09 4:36 ` Krishna Kurapati
0 siblings, 2 replies; 15+ messages in thread
From: Krzysztof Kozlowski @ 2025-12-06 11:02 UTC (permalink / raw)
To: Dmitry Baryshkov
Cc: Krishna Kurapati, Vinod Koul, Kishon Vijay Abraham I, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson, Konrad Dybcio,
Luca Weiss, Neil Armstrong, Johan Hovold, Pengyu Luo,
linux-arm-msm, linux-phy, devicetree, linux-kernel
On Fri, Dec 05, 2025 at 10:55:36PM +0200, Dmitry Baryshkov wrote:
> On Thu, Dec 04, 2025 at 10:16:42AM +0530, Krishna Kurapati wrote:
> > Add squelch detect parameter update for synopsys eusb2 repeater.
> >
> > Signed-off-by: Krishna Kurapati <krishna.kurapati@oss.qualcomm.com>
> > ---
> > .../devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml | 8 ++++++++
> > 1 file changed, 8 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml b/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml
> > index 5bf0d6c9c025..f2afcf0e986a 100644
> > --- a/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml
> > +++ b/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml
> > @@ -59,6 +59,14 @@ properties:
> > minimum: 0
> > maximum: 7
> >
> > + qcom,squelch-detector-bp:
>
> -uV? -mV?
If thi sis adjustment in ratio, then bp is correct. Some sort of
explanation in commit msg would be nice, but I don't want to stall the
patchset for that.
Assuming this is indeed ratio without actual SI units:
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v2 1/3] dt-bindings: phy: qcom,snps-eusb2-repeater: Add squelch param update
2025-12-06 11:02 ` Krzysztof Kozlowski
@ 2025-12-06 16:10 ` Krishna Kurapati
2025-12-09 4:36 ` Krishna Kurapati
1 sibling, 0 replies; 15+ messages in thread
From: Krishna Kurapati @ 2025-12-06 16:10 UTC (permalink / raw)
To: Krzysztof Kozlowski, Dmitry Baryshkov
Cc: Vinod Koul, Kishon Vijay Abraham I, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson, Konrad Dybcio,
Luca Weiss, Neil Armstrong, Johan Hovold, Pengyu Luo,
linux-arm-msm, linux-phy, devicetree, linux-kernel
On 12/6/2025 4:32 PM, Krzysztof Kozlowski wrote:
> On Fri, Dec 05, 2025 at 10:55:36PM +0200, Dmitry Baryshkov wrote:
>> On Thu, Dec 04, 2025 at 10:16:42AM +0530, Krishna Kurapati wrote:
>>> Add squelch detect parameter update for synopsys eusb2 repeater.
>>>
>>> Signed-off-by: Krishna Kurapati <krishna.kurapati@oss.qualcomm.com>
>>> ---
>>> .../devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml | 8 ++++++++
>>> 1 file changed, 8 insertions(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml b/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml
>>> index 5bf0d6c9c025..f2afcf0e986a 100644
>>> --- a/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml
>>> +++ b/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml
>>> @@ -59,6 +59,14 @@ properties:
>>> minimum: 0
>>> maximum: 7
>>>
>>> + qcom,squelch-detector-bp:
>>
>> -uV? -mV?
>
> If thi sis adjustment in ratio, then bp is correct. Some sort of
> explanation in commit msg would be nice, but I don't want to stall the
> patchset for that.
>
> Assuming this is indeed ratio without actual SI units:
>
Yes, this register values indicates % change in squelch limit. Hence I
used basis points instead of uV or mV. Will update commit text accordingly.
Regards,
Krishna,
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
>
> Best regards,
> Krzysztof
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v2 1/3] dt-bindings: phy: qcom,snps-eusb2-repeater: Add squelch param update
2025-12-06 11:02 ` Krzysztof Kozlowski
2025-12-06 16:10 ` Krishna Kurapati
@ 2025-12-09 4:36 ` Krishna Kurapati
2025-12-16 8:18 ` Krishna Kurapati
1 sibling, 1 reply; 15+ messages in thread
From: Krishna Kurapati @ 2025-12-09 4:36 UTC (permalink / raw)
To: Krzysztof Kozlowski, Dmitry Baryshkov
Cc: Vinod Koul, Kishon Vijay Abraham I, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson, Konrad Dybcio,
Luca Weiss, Neil Armstrong, Johan Hovold, Pengyu Luo,
linux-arm-msm, linux-phy, devicetree, linux-kernel
On 12/6/2025 4:32 PM, Krzysztof Kozlowski wrote:
> On Fri, Dec 05, 2025 at 10:55:36PM +0200, Dmitry Baryshkov wrote:
>> On Thu, Dec 04, 2025 at 10:16:42AM +0530, Krishna Kurapati wrote:
>>> Add squelch detect parameter update for synopsys eusb2 repeater.
>>>
>>> Signed-off-by: Krishna Kurapati <krishna.kurapati@oss.qualcomm.com>
>>> ---
>>> .../devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml | 8 ++++++++
>>> 1 file changed, 8 insertions(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml b/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml
>>> index 5bf0d6c9c025..f2afcf0e986a 100644
>>> --- a/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml
>>> +++ b/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml
>>> @@ -59,6 +59,14 @@ properties:
>>> minimum: 0
>>> maximum: 7
>>>
>>> + qcom,squelch-detector-bp:
>>
>> -uV? -mV?
>
> If thi sis adjustment in ratio, then bp is correct. Some sort of
> explanation in commit msg would be nice, but I don't want to stall the
> patchset for that.
>
> Assuming this is indeed ratio without actual SI units:
>
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
>
Thanks for the RB Krzysztof,
Can I update the following as commit text and retain your RB when
sending v3:
"Add squelch detect parameter update for synopsys eusb2 repeater. The
values (indicated in basis-points) depict a percentage change with
respect to the nominal value."
Regards,
Krishna,
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v2 1/3] dt-bindings: phy: qcom,snps-eusb2-repeater: Add squelch param update
2025-12-09 4:36 ` Krishna Kurapati
@ 2025-12-16 8:18 ` Krishna Kurapati
0 siblings, 0 replies; 15+ messages in thread
From: Krishna Kurapati @ 2025-12-16 8:18 UTC (permalink / raw)
To: Krzysztof Kozlowski, Dmitry Baryshkov, Krzysztof Kozlowski
Cc: Vinod Koul, Kishon Vijay Abraham I, Rob Herring, Conor Dooley,
Bjorn Andersson, Konrad Dybcio, Luca Weiss, Neil Armstrong,
Johan Hovold, Pengyu Luo, linux-arm-msm, linux-phy, devicetree,
linux-kernel
On 12/9/2025 10:06 AM, Krishna Kurapati wrote:
>
>
> On 12/6/2025 4:32 PM, Krzysztof Kozlowski wrote:
>> On Fri, Dec 05, 2025 at 10:55:36PM +0200, Dmitry Baryshkov wrote:
>>> On Thu, Dec 04, 2025 at 10:16:42AM +0530, Krishna Kurapati wrote:
>>>> Add squelch detect parameter update for synopsys eusb2 repeater.
>>>>
>>>> Signed-off-by: Krishna Kurapati <krishna.kurapati@oss.qualcomm.com>
>>>> ---
>>>> .../devicetree/bindings/phy/qcom,snps-eusb2-repeater.yaml | 8 ++++
>>>> ++++
>>>> 1 file changed, 8 insertions(+)
>>>>
>>>> diff --git a/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-
>>>> repeater.yaml b/Documentation/devicetree/bindings/phy/qcom,snps-
>>>> eusb2-repeater.yaml
>>>> index 5bf0d6c9c025..f2afcf0e986a 100644
>>>> --- a/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-
>>>> repeater.yaml
>>>> +++ b/Documentation/devicetree/bindings/phy/qcom,snps-eusb2-
>>>> repeater.yaml
>>>> @@ -59,6 +59,14 @@ properties:
>>>> minimum: 0
>>>> maximum: 7
>>>> + qcom,squelch-detector-bp:
>>>
>>> -uV? -mV?
>>
>> If thi sis adjustment in ratio, then bp is correct. Some sort of
>> explanation in commit msg would be nice, but I don't want to stall the
>> patchset for that.
>>
>> Assuming this is indeed ratio without actual SI units:
>>
>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
>>
>
> Thanks for the RB Krzysztof,
>
> Can I update the following as commit text and retain your RB when
> sending v3:
>
> "Add squelch detect parameter update for synopsys eusb2 repeater. The
> values (indicated in basis-points) depict a percentage change with
> respect to the nominal value."
>
Hi Krzysztof,
Can you help let me know if the above commit message can be added with
retaining your RB tag.
Regards,
Krishna,
^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH v2 2/3] phy: qualcomm: phy-qcom-eusb2-repeater: Add squelch detect param update
2025-12-04 4:46 [PATCH v2 0/3] Add eusb2 repeater squelch detect parameter override Krishna Kurapati
2025-12-04 4:46 ` [PATCH v2 1/3] dt-bindings: phy: qcom,snps-eusb2-repeater: Add squelch param update Krishna Kurapati
@ 2025-12-04 4:46 ` Krishna Kurapati
2025-12-05 13:31 ` Konrad Dybcio
2025-12-04 4:46 ` [PATCH v2 3/3] arm64: dts: qcom: sm8750-mtp: Add eusb2 repeater tuning parameters Krishna Kurapati
2 siblings, 1 reply; 15+ messages in thread
From: Krishna Kurapati @ 2025-12-04 4:46 UTC (permalink / raw)
To: Vinod Koul, Kishon Vijay Abraham I, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson, Konrad Dybcio,
Luca Weiss, Neil Armstrong, Johan Hovold, Pengyu Luo
Cc: linux-arm-msm, linux-phy, devicetree, linux-kernel,
Krishna Kurapati
Add support for overriding Squelch Detect parameter.
Signed-off-by: Krishna Kurapati <krishna.kurapati@oss.qualcomm.com>
---
.../phy/qualcomm/phy-qcom-eusb2-repeater.c | 22 +++++++++++++++++++
1 file changed, 22 insertions(+)
diff --git a/drivers/phy/qualcomm/phy-qcom-eusb2-repeater.c b/drivers/phy/qualcomm/phy-qcom-eusb2-repeater.c
index 651a12b59bc8..a75b37d4e16d 100644
--- a/drivers/phy/qualcomm/phy-qcom-eusb2-repeater.c
+++ b/drivers/phy/qualcomm/phy-qcom-eusb2-repeater.c
@@ -37,6 +37,17 @@
#define EUSB2_TUNE_EUSB_EQU 0x5A
#define EUSB2_TUNE_EUSB_HS_COMP_CUR 0x5B
+static const int squelch_detector[] = {
+ [0] = -6000,
+ [1] = -5000,
+ [2] = -4000,
+ [3] = -3000,
+ [4] = -2000,
+ [5] = -1000,
+ [6] = 0,
+ [7] = 1000,
+};
+
struct eusb2_repeater_init_tbl_reg {
unsigned int reg;
unsigned int value;
@@ -120,7 +131,9 @@ static int eusb2_repeater_init(struct phy *phy)
struct regmap *regmap = rptr->regmap;
u32 base = rptr->base;
u32 poll_val;
+ s32 dt_val;
int ret;
+ int i;
u8 val;
ret = regulator_bulk_enable(rptr->cfg->num_vregs, rptr->vregs);
@@ -147,6 +160,15 @@ static int eusb2_repeater_init(struct phy *phy)
if (!of_property_read_u8(np, "qcom,tune-res-fsdif", &val))
regmap_write(regmap, base + EUSB2_TUNE_RES_FSDIF, val);
+ if (!of_property_read_s32(np, "qcom,squelch-detector-bp", &dt_val)) {
+ for (i = 0; i < ARRAY_SIZE(squelch_detector); i++) {
+ if (squelch_detector[i] == dt_val) {
+ regmap_write(regmap, base + EUSB2_TUNE_SQUELCH_U, i);
+ break;
+ }
+ }
+ }
+
/* Wait for status OK */
ret = regmap_read_poll_timeout(regmap, base + EUSB2_RPTR_STATUS, poll_val,
poll_val & RPTR_OK, 10, 5);
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread* Re: [PATCH v2 2/3] phy: qualcomm: phy-qcom-eusb2-repeater: Add squelch detect param update
2025-12-04 4:46 ` [PATCH v2 2/3] phy: qualcomm: phy-qcom-eusb2-repeater: Add squelch detect " Krishna Kurapati
@ 2025-12-05 13:31 ` Konrad Dybcio
2025-12-06 16:12 ` Krishna Kurapati
0 siblings, 1 reply; 15+ messages in thread
From: Konrad Dybcio @ 2025-12-05 13:31 UTC (permalink / raw)
To: Krishna Kurapati, Vinod Koul, Kishon Vijay Abraham I, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson, Konrad Dybcio,
Luca Weiss, Neil Armstrong, Johan Hovold, Pengyu Luo
Cc: linux-arm-msm, linux-phy, devicetree, linux-kernel
On 12/4/25 5:46 AM, Krishna Kurapati wrote:
> Add support for overriding Squelch Detect parameter.
>
> Signed-off-by: Krishna Kurapati <krishna.kurapati@oss.qualcomm.com>
> ---
> .../phy/qualcomm/phy-qcom-eusb2-repeater.c | 22 +++++++++++++++++++
> 1 file changed, 22 insertions(+)
>
> diff --git a/drivers/phy/qualcomm/phy-qcom-eusb2-repeater.c b/drivers/phy/qualcomm/phy-qcom-eusb2-repeater.c
> index 651a12b59bc8..a75b37d4e16d 100644
> --- a/drivers/phy/qualcomm/phy-qcom-eusb2-repeater.c
> +++ b/drivers/phy/qualcomm/phy-qcom-eusb2-repeater.c
> @@ -37,6 +37,17 @@
> #define EUSB2_TUNE_EUSB_EQU 0x5A
> #define EUSB2_TUNE_EUSB_HS_COMP_CUR 0x5B
>
> +static const int squelch_detector[] = {
> + [0] = -6000,
> + [1] = -5000,
> + [2] = -4000,
> + [3] = -3000,
> + [4] = -2000,
> + [5] = -1000,
> + [6] = 0,
> + [7] = 1000,
> +};
I forgot to ask, do these values apply to all of the supported repeaters?
i.e.
PM8550B
PMIV0104
SMB2360
Konrad
^ permalink raw reply [flat|nested] 15+ messages in thread* Re: [PATCH v2 2/3] phy: qualcomm: phy-qcom-eusb2-repeater: Add squelch detect param update
2025-12-05 13:31 ` Konrad Dybcio
@ 2025-12-06 16:12 ` Krishna Kurapati
2025-12-08 12:54 ` Konrad Dybcio
0 siblings, 1 reply; 15+ messages in thread
From: Krishna Kurapati @ 2025-12-06 16:12 UTC (permalink / raw)
To: Konrad Dybcio
Cc: linux-arm-msm, linux-phy, Vinod Koul, Kishon Vijay Abraham I,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson,
Konrad Dybcio, Luca Weiss, Neil Armstrong, Johan Hovold,
Pengyu Luo, devicetree, linux-kernel
On 12/5/2025 7:01 PM, Konrad Dybcio wrote:
> On 12/4/25 5:46 AM, Krishna Kurapati wrote:
>> Add support for overriding Squelch Detect parameter.
>>
>> Signed-off-by: Krishna Kurapati <krishna.kurapati@oss.qualcomm.com>
>> ---
>> .../phy/qualcomm/phy-qcom-eusb2-repeater.c | 22 +++++++++++++++++++
>> 1 file changed, 22 insertions(+)
>>
>> diff --git a/drivers/phy/qualcomm/phy-qcom-eusb2-repeater.c b/drivers/phy/qualcomm/phy-qcom-eusb2-repeater.c
>> index 651a12b59bc8..a75b37d4e16d 100644
>> --- a/drivers/phy/qualcomm/phy-qcom-eusb2-repeater.c
>> +++ b/drivers/phy/qualcomm/phy-qcom-eusb2-repeater.c
>> @@ -37,6 +37,17 @@
>> #define EUSB2_TUNE_EUSB_EQU 0x5A
>> #define EUSB2_TUNE_EUSB_HS_COMP_CUR 0x5B
>>
>> +static const int squelch_detector[] = {
>> + [0] = -6000,
>> + [1] = -5000,
>> + [2] = -4000,
>> + [3] = -3000,
>> + [4] = -2000,
>> + [5] = -1000,
>> + [6] = 0,
>> + [7] = 1000,
>> +};
>
> I forgot to ask, do these values apply to all of the supported repeaters?
>
I checked smb2360/pmih010x_eusb2_repeater and pm8550b repeaters
(SM8{5/6/7}50/ Kaanapali/ Hamoa). For all of them, the above table is same.
Regards,
Krishna,
> i.e.
>
> PM8550B
> PMIV0104
> SMB2360
>
> Konrad
^ permalink raw reply [flat|nested] 15+ messages in thread* Re: [PATCH v2 2/3] phy: qualcomm: phy-qcom-eusb2-repeater: Add squelch detect param update
2025-12-06 16:12 ` Krishna Kurapati
@ 2025-12-08 12:54 ` Konrad Dybcio
0 siblings, 0 replies; 15+ messages in thread
From: Konrad Dybcio @ 2025-12-08 12:54 UTC (permalink / raw)
To: Krishna Kurapati
Cc: linux-arm-msm, linux-phy, Vinod Koul, Kishon Vijay Abraham I,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson,
Konrad Dybcio, Luca Weiss, Neil Armstrong, Johan Hovold,
Pengyu Luo, devicetree, linux-kernel
On 12/6/25 5:12 PM, Krishna Kurapati wrote:
>
>
> On 12/5/2025 7:01 PM, Konrad Dybcio wrote:
>> On 12/4/25 5:46 AM, Krishna Kurapati wrote:
>>> Add support for overriding Squelch Detect parameter.
>>>
>>> Signed-off-by: Krishna Kurapati <krishna.kurapati@oss.qualcomm.com>
>>> ---
>>> .../phy/qualcomm/phy-qcom-eusb2-repeater.c | 22 +++++++++++++++++++
>>> 1 file changed, 22 insertions(+)
>>>
>>> diff --git a/drivers/phy/qualcomm/phy-qcom-eusb2-repeater.c b/drivers/phy/qualcomm/phy-qcom-eusb2-repeater.c
>>> index 651a12b59bc8..a75b37d4e16d 100644
>>> --- a/drivers/phy/qualcomm/phy-qcom-eusb2-repeater.c
>>> +++ b/drivers/phy/qualcomm/phy-qcom-eusb2-repeater.c
>>> @@ -37,6 +37,17 @@
>>> #define EUSB2_TUNE_EUSB_EQU 0x5A
>>> #define EUSB2_TUNE_EUSB_HS_COMP_CUR 0x5B
>>> +static const int squelch_detector[] = {
>>> + [0] = -6000,
>>> + [1] = -5000,
>>> + [2] = -4000,
>>> + [3] = -3000,
>>> + [4] = -2000,
>>> + [5] = -1000,
>>> + [6] = 0,
>>> + [7] = 1000,
>>> +};
>>
>> I forgot to ask, do these values apply to all of the supported repeaters?
>>
>
> I checked smb2360/pmih010x_eusb2_repeater and pm8550b repeaters (SM8{5/6/7}50/ Kaanapali/ Hamoa). For all of them, the above table is same.
Thanks
Konrad
^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH v2 3/3] arm64: dts: qcom: sm8750-mtp: Add eusb2 repeater tuning parameters
2025-12-04 4:46 [PATCH v2 0/3] Add eusb2 repeater squelch detect parameter override Krishna Kurapati
2025-12-04 4:46 ` [PATCH v2 1/3] dt-bindings: phy: qcom,snps-eusb2-repeater: Add squelch param update Krishna Kurapati
2025-12-04 4:46 ` [PATCH v2 2/3] phy: qualcomm: phy-qcom-eusb2-repeater: Add squelch detect " Krishna Kurapati
@ 2025-12-04 4:46 ` Krishna Kurapati
2025-12-04 8:55 ` Abel Vesa
2025-12-05 13:33 ` Konrad Dybcio
2 siblings, 2 replies; 15+ messages in thread
From: Krishna Kurapati @ 2025-12-04 4:46 UTC (permalink / raw)
To: Vinod Koul, Kishon Vijay Abraham I, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson, Konrad Dybcio,
Luca Weiss, Neil Armstrong, Johan Hovold, Pengyu Luo
Cc: linux-arm-msm, linux-phy, devicetree, linux-kernel,
Krishna Kurapati
Add eusb2 repeater tuning parameters for MTP platform.
Signed-off-by: Krishna Kurapati <krishna.kurapati@oss.qualcomm.com>
---
arch/arm64/boot/dts/qcom/sm8750-mtp.dts | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/sm8750-mtp.dts b/arch/arm64/boot/dts/qcom/sm8750-mtp.dts
index c8cb521b4c26..b9a4d1c99816 100644
--- a/arch/arm64/boot/dts/qcom/sm8750-mtp.dts
+++ b/arch/arm64/boot/dts/qcom/sm8750-mtp.dts
@@ -1041,6 +1041,10 @@ wifi@0 {
&pmih0108_eusb2_repeater {
status = "okay";
+ qcom,tune-usb2-preem = /bits/ 8 <0x3>;
+ qcom,tune-usb2-amplitude = /bits/ 8 <0xa>;
+ qcom,squelch-detector-bp = <(-2000)>;
+
vdd18-supply = <&vreg_l15b_1p8>;
vdd3-supply = <&vreg_l5b_3p1>;
};
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread* Re: [PATCH v2 3/3] arm64: dts: qcom: sm8750-mtp: Add eusb2 repeater tuning parameters
2025-12-04 4:46 ` [PATCH v2 3/3] arm64: dts: qcom: sm8750-mtp: Add eusb2 repeater tuning parameters Krishna Kurapati
@ 2025-12-04 8:55 ` Abel Vesa
2025-12-05 13:33 ` Konrad Dybcio
1 sibling, 0 replies; 15+ messages in thread
From: Abel Vesa @ 2025-12-04 8:55 UTC (permalink / raw)
To: Krishna Kurapati
Cc: Vinod Koul, Kishon Vijay Abraham I, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson, Konrad Dybcio,
Luca Weiss, Neil Armstrong, Johan Hovold, Pengyu Luo,
linux-arm-msm, linux-phy, devicetree, linux-kernel
On 25-12-04 10:16:44, Krishna Kurapati wrote:
> Add eusb2 repeater tuning parameters for MTP platform.
>
> Signed-off-by: Krishna Kurapati <krishna.kurapati@oss.qualcomm.com>
LGTM.
Reviewed-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v2 3/3] arm64: dts: qcom: sm8750-mtp: Add eusb2 repeater tuning parameters
2025-12-04 4:46 ` [PATCH v2 3/3] arm64: dts: qcom: sm8750-mtp: Add eusb2 repeater tuning parameters Krishna Kurapati
2025-12-04 8:55 ` Abel Vesa
@ 2025-12-05 13:33 ` Konrad Dybcio
2025-12-06 16:09 ` Krishna Kurapati
1 sibling, 1 reply; 15+ messages in thread
From: Konrad Dybcio @ 2025-12-05 13:33 UTC (permalink / raw)
To: Krishna Kurapati, Vinod Koul, Kishon Vijay Abraham I, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson, Konrad Dybcio,
Luca Weiss, Neil Armstrong, Johan Hovold, Pengyu Luo
Cc: linux-arm-msm, linux-phy, devicetree, linux-kernel
On 12/4/25 5:46 AM, Krishna Kurapati wrote:
> Add eusb2 repeater tuning parameters for MTP platform.
>
> Signed-off-by: Krishna Kurapati <krishna.kurapati@oss.qualcomm.com>
> ---
> arch/arm64/boot/dts/qcom/sm8750-mtp.dts | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/sm8750-mtp.dts b/arch/arm64/boot/dts/qcom/sm8750-mtp.dts
> index c8cb521b4c26..b9a4d1c99816 100644
> --- a/arch/arm64/boot/dts/qcom/sm8750-mtp.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8750-mtp.dts
> @@ -1041,6 +1041,10 @@ wifi@0 {
> &pmih0108_eusb2_repeater {
> status = "okay";
>
> + qcom,tune-usb2-preem = /bits/ 8 <0x3>;
> + qcom,tune-usb2-amplitude = /bits/ 8 <0xa>;
> + qcom,squelch-detector-bp = <(-2000)>;
This happens to be the only(?) node in this file that has status first
instead of last. May I ask you to fix that up, since it's even visible
in the diff?
Also, does the QRD also need a similar update?
Konrad
> +
> vdd18-supply = <&vreg_l15b_1p8>;
> vdd3-supply = <&vreg_l5b_3p1>;
> };
^ permalink raw reply [flat|nested] 15+ messages in thread* Re: [PATCH v2 3/3] arm64: dts: qcom: sm8750-mtp: Add eusb2 repeater tuning parameters
2025-12-05 13:33 ` Konrad Dybcio
@ 2025-12-06 16:09 ` Krishna Kurapati
0 siblings, 0 replies; 15+ messages in thread
From: Krishna Kurapati @ 2025-12-06 16:09 UTC (permalink / raw)
To: Konrad Dybcio
Cc: linux-arm-msm, linux-phy, Vinod Koul, Kishon Vijay Abraham I,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson,
Konrad Dybcio, Luca Weiss, Neil Armstrong, Johan Hovold,
Pengyu Luo, devicetree, linux-kernel
On 12/5/2025 7:03 PM, Konrad Dybcio wrote:
> On 12/4/25 5:46 AM, Krishna Kurapati wrote:
>> Add eusb2 repeater tuning parameters for MTP platform.
>>
>> Signed-off-by: Krishna Kurapati <krishna.kurapati@oss.qualcomm.com>
>> ---
>> arch/arm64/boot/dts/qcom/sm8750-mtp.dts | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/sm8750-mtp.dts b/arch/arm64/boot/dts/qcom/sm8750-mtp.dts
>> index c8cb521b4c26..b9a4d1c99816 100644
>> --- a/arch/arm64/boot/dts/qcom/sm8750-mtp.dts
>> +++ b/arch/arm64/boot/dts/qcom/sm8750-mtp.dts
>> @@ -1041,6 +1041,10 @@ wifi@0 {
>> &pmih0108_eusb2_repeater {
>> status = "okay";
>>
>> + qcom,tune-usb2-preem = /bits/ 8 <0x3>;
>> + qcom,tune-usb2-amplitude = /bits/ 8 <0xa>;
>> + qcom,squelch-detector-bp = <(-2000)>;
>
> This happens to be the only(?) node in this file that has status first
> instead of last. May I ask you to fix that up, since it's even visible
> in the diff?
>
> Also, does the QRD also need a similar update?
>
Will fix it up in next revision.
And for QRD, I don't see any updated override params in qrd downstream.
Regards,
Krishna,
> Konrad
>
>> +
>> vdd18-supply = <&vreg_l15b_1p8>;
>> vdd3-supply = <&vreg_l5b_3p1>;
>> };
^ permalink raw reply [flat|nested] 15+ messages in thread