From mboxrd@z Thu Jan  1 00:00:00 1970
From: Claude Code Review Bot <claude-review@example.com>
To: dri-devel-reviews@example.com
Subject: Claude review: dt-bindings: rockchip: analogix-dp: Add data-lanes example
Date: Mon, 25 May 2026 20:15:52 +1000
Message-ID: <review-patch2-20260521114459.1394264-3-damon.ding@rock-chips.com>
In-Reply-To: <20260521114459.1394264-3-damon.ding@rock-chips.com>
References: <20260521114459.1394264-1-damon.ding@rock-chips.com> <20260521114459.1394264-3-damon.ding@rock-chips.com>
X-Mailer: Claude Code Patch Reviewer
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0

Patch Review

**Status: Fine**

Simple addition of `data-lanes =3D <0 1>;` to the example endpoint:
```
edp_out_panel: endpoint {
+   data-lanes =3D <0 1>;
    remote-endpoint =3D <&panel_in_edp>;
};
```

This is an identity mapping with only 2 lanes, so it doesn't demonstrate la=
ne reordering. Consider using a non-trivial example like `data-lanes =3D <1=
 0>;` to show actual lane swapping =E2=80=94 that would better illustrate t=
he feature's purpose.

---

---
Generated by Claude Code Patch Reviewer