Hey Xin,
This series no longer applies to drm-misc/drm-misc-next, please rebase it.
On Wed, 24 Mar 2021 at 08:52, Xin Ji xji@analogixsemi.com wrote:
On Sun, Mar 21, 2021 at 02:00:38PM +0200, Laurent Pinchart wrote:
Hi Xin,
Thank you for the patch.
On Fri, Mar 19, 2021 at 02:32:39PM +0800, Xin Ji wrote:
Add 'bus-type' and 'data-lanes' define for port0. Define DP tx lane0, lane1 swing register array define, and audio enable flag.
Signed-off-by: Xin Ji xji@analogixsemi.com
.../display/bridge/analogix,anx7625.yaml | 58 ++++++++++++++++++- 1 file changed, 57 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml index c789784efe30..3f54d5876982 100644 --- a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml +++ b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml @@ -34,6 +34,26 @@ properties: description: used for reset chip control, RESET_N pin B7. maxItems: 1
- analogix,lane0-swing:
- $ref: /schemas/types.yaml#/definitions/uint32-array
- minItems: 1
- maxItems: 20
- description:
an array of swing register setting for DP tx lane0 PHY, please don'tadd this property, or contact vendor.DT properties need to be documented. Contacting the vendor doesn't count as documentation I'm afraid.
Hi Laurent Pinchart, thanks for your comment. For the DP phy swing setting, it is hard to describe in here, needs to refer the anx7625 datasheet and programming guide. Basically, no need to change the DP phy swing setting.
Laurent is right. But if the value practically is a constant, you can move the swing register into the driver. It should still be documented as well as possible, but we can be a little bit more flexible.
@@ -73,6 +123,10 @@ examples: enable-gpios = <&pio 45 GPIO_ACTIVE_HIGH>; reset-gpios = <&pio 73 GPIO_ACTIVE_HIGH>;
analogix,audio-enable;analogix,lane0-swing = <0x14 0x54 0x64 0x74 0x29 0x7b 0x77 0x5b>;analogix,lane1-swing = <0x14 0x54 0x64 0x74 0x29 0x7b 0x77 0x5b>;ports { #address-cells = <1>; #size-cells = <0>;