From: "Yann Sionneau" <yann.sionneau@vates.tech>
To: uli@fpond.eu, cip-dev@lists.cip-project.org
Cc: m.szyprowski@samsung.com
Subject: Re: [cip-dev] 4.19-y-cip missing backport?
Date: Sun, 28 Dec 2025 21:23:15 +0000 [thread overview]
Message-ID: <0d89d301-20dd-42e6-a588-c7d2c616df2c@vates.tech> (raw)
In-Reply-To: <1534957620.311452.1766834410980@webmail.strato.de>
On 12/27/25 12:20, Ulrich Hecht via lists.cip-project.org wrote:
>
>> On 12/23/2025 3:19 PM CET Yann Sionneau via lists.cip-project.org <yann.sionneau=vates.tech@lists.cip-project.org> wrote:
>> I noticed that 4.19.325 (as well as the latest -cip126 release) has this
>> backport:
>>
>> commit 2b8f2afa311c722a90f00fb2960e6deb4f5100a5
>> Author: Marek Szyprowski <m.szyprowski@samsung.com>
>> Date: Thu Aug 29 07:30:28 2019 +0200
>>
>> usb: dwc3: remove generic PHY calibrate() calls
>>
>> [ Upstream commit a0a465569b45e3690be155c96fb54603d6904f41 ]
>>
>>
>> But it does not have this:
>>
>> commit 34c7ed72f4f04e73901ca6ec2e1db6c040495935
>> Author: Marek Szyprowski <m.szyprowski@samsung.com>
>> Date: Thu Aug 29 07:30:27 2019 +0200
>>
>> usb: core: phy: add support for PHY calibration
>>
>>
>> So it seems nobody calls phy_calibrate() anymore.
>> I must add to this report that I didn't notice any crash or debug anything.
>> I was just skimming through the 4.19.323 ChangeLog when I noticed this.
>>
>> So maybe someone needs to confirm that this is a real issue, but it's
>> suspicious at least.
>
> Thank you for reporting this. I'm not quite sure what to do about it, though. The options are:
>
> 1. Backport "usb: core: phy: add support for PHY calibration". I like this one the least because it potentially affects all drivers, none of which have any issues ATM.
> 2. Revert "usb: dwc3: remove generic PHY calibrate() calls". Better in terms of regression risk, but still may have unwanted side effects for dwc3.
> 3. Do nothing. Least likely to introduce regressions but leaves the code in a known-bad state, although one that may not cause any real-life issue.
>
> Any opinions, anyone?
>
> CU
> Uli
>
Hello Uli,
Thanks for your answer, I think you're right about the 3 options.
Let's pull Marek Szyprowski into this so that he can give us his opinion
since he's the Author of those patches.
I'm putting him in CC.
Regards,
--
--
Yann Sionneau | Vates XCP-ng Developer
XCP-ng & Xen Orchestra - Vates solutions
web: https://vates.tech
next prev parent reply other threads:[~2025-12-28 21:23 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-12-23 14:19 4.19-y-cip missing backport? Yann Sionneau
2025-12-27 11:20 ` [cip-dev] " Ulrich Hecht
2025-12-28 21:23 ` Yann Sionneau [this message]
2025-12-31 7:23 ` Marek Szyprowski
2026-01-02 2:09 ` Ulrich Hecht
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=0d89d301-20dd-42e6-a588-c7d2c616df2c@vates.tech \
--to=yann.sionneau@vates.tech \
--cc=cip-dev@lists.cip-project.org \
--cc=m.szyprowski@samsung.com \
--cc=uli@fpond.eu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox