[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <pml7rfbkerzkx5df26x7kxn3tk2o7oqrkqnx26tzikeg53djn5@islb4hlm4ks7>
Date: Tue, 2 Sep 2025 19:29:48 +0200
From: Maxime Ripard <mripard@...nel.org>
To: Mike Looijmans <mike.looijmans@...ic.nl>
Cc: dri-devel@...ts.freedesktop.org,
Andrzej Hajda <andrzej.hajda@...el.com>, Conor Dooley <conor+dt@...nel.org>,
David Airlie <airlied@...il.com>, Jernej Skrabec <jernej.skrabec@...il.com>,
Jonas Karlman <jonas@...boo.se>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Laurent Pinchart <Laurent.pinchart@...asonboard.com>, Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Neil Armstrong <neil.armstrong@...aro.org>, Rob Herring <robh@...nel.org>, Robert Foss <rfoss@...nel.org>,
Simona Vetter <simona@...ll.ch>, Thomas Zimmermann <tzimmermann@...e.de>,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v4 1/2] dt-bindings: drm/bridge: ti-tmds181: Add TI
TMDS181 and SN65DP159 bindings
On Mon, Sep 01, 2025 at 04:29:01PM +0200, Mike Looijmans wrote:
> + ti,retimer-threshold-hz:
> + minimum: 25000000
> + maximum: 600000000
> + default: 200000000
> + description:
> + Cross-over point. Up until this pixel clock frequency
> + the chip remains in the low-power redriver mode. Above
> + the threshold the chip should operate in retimer mode.
Why should anyone want to tune this at the firmware level?
> + ti,dvi-mode:
> + type: boolean
> + description: Makes the DP159 chip operate in DVI mode.
Ditto. Both describe policy, not hardware.
> + ti,slew-rate:
> + $ref: /schemas/types.yaml#/definitions/uint32
> + minimum: 0
> + maximum: 3
> + default: 3
> + description: Set slew rate, 0 is slowest, 3 is fastest.
> +
> + ti,disable-equalizer:
> + type: boolean
> + description: Disable the equalizer (to save power).
Why shouldn't we disable all the time then? Again, it looks like a
policy, and not something that should be set in stone in the firmware.
> + ti,adaptive-equalizer:
> + type: boolean
> + description: Set the equalizer to adaptive mode.
Ditto.
Maxime
Download attachment "signature.asc" of type "application/pgp-signature" (274 bytes)
Powered by blists - more mailing lists