[PATCH v4 1/2] dt-bindings: display: bridge: tc358867: Document default DP preemphasis

Marek Vasut marex at denx.de
Tue Jul 16 18:29:46 UTC 2024


On 7/10/24 5:33 PM, Rob Herring (Arm) wrote:
> 
> On Mon, 08 Jul 2024 17:01:13 +0200, Marek Vasut wrote:
>> Document default DP port preemphasis configurable via new DT property
>> "toshiba,pre-emphasis". This is useful in case the DP link properties
>> are known and starting link training from preemphasis setting of 0 dB
>> is not useful. The preemphasis can be set separately for both DP lanes
>> in range 0=0dB, 1=3.5dB, 2=6dB .
>>
>> This is an endpoint property, not a port property, because the TC9595
>> datasheet does mention that the DP might operate in some sort of split
>> mode, where each DP lane is used to feed one display, so in that case
>> there might be two endpoints.
>>
>> Signed-off-by: Marek Vasut <marex at denx.de>
>> ---
>> Cc: Alexander Stein <alexander.stein at ew.tq-group.com>
>> Cc: Andrzej Hajda <andrzej.hajda at intel.com>
>> Cc: Conor Dooley <conor+dt at kernel.org>
>> Cc: Daniel Vetter <daniel at ffwll.ch>
>> Cc: David Airlie <airlied at gmail.com>
>> Cc: Jernej Skrabec <jernej.skrabec at gmail.com>
>> Cc: Jonas Karlman <jonas at kwiboo.se>
>> Cc: Krzysztof Kozlowski <krzk+dt at kernel.org>
>> Cc: Laurent Pinchart <Laurent.pinchart at ideasonboard.com>
>> Cc: Lucas Stach <l.stach at pengutronix.de>
>> Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
>> Cc: Maxime Ripard <mripard at kernel.org>
>> Cc: Neil Armstrong <neil.armstrong at linaro.org>
>> Cc: Rob Herring <robh at kernel.org>
>> Cc: Robert Foss <rfoss at kernel.org>
>> Cc: Thomas Zimmermann <tzimmermann at suse.de>
>> Cc: devicetree at vger.kernel.org
>> Cc: dri-devel at lists.freedesktop.org
>> Cc: kernel at dh-electronics.com
>> ---
>> V2: - Fix the type to u8 array
>>      - Fix the enum items to match what they represent
>> V3: - Update commit message, expand on this being an endpoint property
>> V4: - Fix ref: /schemas/graph.yaml#/$defs/port-base and add unevaluatedProperties
>> ---
>>   .../display/bridge/toshiba,tc358767.yaml      | 21 ++++++++++++++++++-
>>   1 file changed, 20 insertions(+), 1 deletion(-)
>>
> 
> Reviewed-by: Rob Herring (Arm) <robh at kernel.org>

If there are no objections, I will apply these two patches to drm-misc 
soon ?


More information about the dri-devel mailing list