Re: [PATCH v2 2/2] phy: qualcomm: phy-qcom-eusb2-repeater: rework reg override handler

From: Dmitry Baryshkov
Date: Fri Apr 25 2025 - 15:42:03 EST


On Wed, Apr 16, 2025 at 08:02:01PM +0800, Pengyu Luo wrote:
> In downstream tree, many registers need to be overridden, it varies
> from devices and platforms, with these registers getting more, adding
> a handler for this is helpful.

It should be noted that previously all values were applied during _init
phase, before checking the status etc. Now the overrides are programmed
from the set_mode. Should you still program sane defaults at the init
stage too?

BTW, is there a real need to override those for the platform you are
working on? Could you please provide some details, maybe in the cover
letter.

> Signed-off-by: Pengyu Luo <mitltlatltl@xxxxxxxxx>
> ---
> .../phy/qualcomm/phy-qcom-eusb2-repeater.c | 105 +++++++++++++++---
> 1 file changed, 92 insertions(+), 13 deletions(-)

--
With best wishes
Dmitry