[PATCH v2 2/2] phy: cadence: cdns-dphy: Update calibration wait time for startup state machine
Devarsh Thakkar
devarsht at ti.com
Wed Apr 30 00:36:49 PDT 2025
Hi Tomi,
On 02/04/25 17:12, Tomi Valkeinen wrote:
> Hi,
>
Thanks for the review.
> On 26/03/2025 17:23, Devarsh Thakkar wrote:
>> @@ -405,6 +406,8 @@ static int cdns_dphy_configure(struct phy *phy,
>> union phy_configure_opts *opts)
>> reg = FIELD_PREP(DPHY_BAND_CFG_LEFT_BAND, band_ctrl) |
>> FIELD_PREP(DPHY_BAND_CFG_RIGHT_BAND, band_ctrl);
>> writel(reg, dphy->regs + DPHY_BAND_CFG);
>> + writel(FIELD_PREP(DPHY_CMN_SSM_CAL_WAIT_TIME, 0x14) |
>> DPHY_CMN_SSM_EN | DPHY_CMN_TX_MODE_EN,
>> + dphy->regs + DPHY_CMN_SSM);
>
> This sounds like a TI specific characterized value, but the function
> here is a generic one. Also, is the value same for all TI SoCs? Or is it
> per-soc?
>
I see same characterized value for calibration wait time for each TI SoC
but as discussed offline, I will use read-modify-write to preserve the
reset value which was characterized for the corresponding platform so
that if any other platform is using a different characterized reset
value for calibration wait time, it would handle that case as well.
Regards
Devarsh
More information about the linux-phy
mailing list