Re: [PATCH v3 6/6] phy: qcom-mipi-csi2: Add C-PHY support for X1E80100

From: George Neacsu

Date: Fri Oct 16 2026 - 13:46:35 EST


Hi David,

On 16/10/2026 16:10, David Heidelberg wrote:
>> + {.reg_addr = CSIPHY_LN_CTRLn(1, 26), .reg_data = 0xf1, .param_type = CSIPHY_DEFAULT_PARAMS},
>
> Thank you for the change. While the lane being decimal number is completely
> fine, I believe the register passed should stay in hex :)

Agreed. In the next version the macro will take the register offset
within the lane in hex, e.g. CSIPHY_LN_REG(1, 0x68), so each entry
reads like the downstream header. The object code stays the same.

Thanks,
George