Re: [PATCH v6 1/2] dt-bindings: ufs: Document static TX Equalization settings properties

From: Krzysztof Kozlowski

Date: Tue Jun 09 2026 - 03:13:38 EST


On 31/05/2026 06:48, Can Guo wrote:
>>>> is the minimal encoding that covers both.
>>> Again, why do you need to encode '0'?
>> The tuple is still needed because Precoding is configured per
>> transmitter-receiver pair,
>> so each lane has two independent states:
>> - Host_TX -> Device_RX
>> - Device_TX -> Host_RX
>> A lane-only enabled list cannot represent directional combinations
>> like lane0 =
>> (on, off) vs (off, on).
> How about we split into two properties, something like below?
> tx-precode-enable-g6-host-lanes = <0 1>
> tx-precode-enable-g6-device-lanes = <1>
>
> Only listed lanes are enabled; unlisted lanes are disabled by default.
>
> Are you OK with this approach?

Yes, I do prefer this, because we don't have empty entries (<0, 0>, <0,
1> ....).

Best regards,
Krzysztof