Hi Fabrizio,
Thank you for the patch.
On Friday, 13 October 2017 18:22:19 EEST Fabrizio Castro wrote:
Add device tree bindings for r8a7743 and r8a7745 DUs. r8a7743 DU is similar to the one from r8a7791, r8a7745 DU is similar to the one from r8a7794.
Signed-off-by: Fabrizio Castro fabrizio.castro@bp.renesas.com Reviewed-by: Biju Das biju.das@bp.renesas.com
.../devicetree/bindings/display/renesas,du.txt | 30 ++++++++++--------- 1 file changed, 17 insertions(+), 13 deletions(-)
diff --git a/Documentation/devicetree/bindings/display/renesas,du.txt b/Documentation/devicetree/bindings/display/renesas,du.txt index 4bbd1e9..c520226 100644 --- a/Documentation/devicetree/bindings/display/renesas,du.txt +++ b/Documentation/devicetree/bindings/display/renesas,du.txt @@ -3,6 +3,8 @@ Required Properties:
- compatible: must be one of the following.
- "renesas,du-r8a7743" for R8A7743 (RZ/G1M) compatible DU
- "renesas,du-r8a7745" for R8A7745 (RZ/G1E) compatible DU
- "renesas,du-r8a7779" for R8A7779 (R-Car H1) compatible DU
- "renesas,du-r8a7790" for R8A7790 (R-Car H2) compatible DU
- "renesas,du-r8a7791" for R8A7791 (R-Car M2-W) compatible DU
@@ -27,10 +29,10 @@ Required Properties:
- clock-names: Name of the clocks. This property is model-dependent.
- R8A7779 uses a single functional clock. The clock doesn't need to be named.
- R8A779[0123456] use one functional clock per channel and one clock
per
LVDS encoder (if available). The functional clocks must be named"du.x"
with "x" being the channel numerical index. The LVDS clocks must benamed "lvds.x" with "x" being the LVDS encoder numerical index.
- R8A779[0123456] and R8A774[35] use one functional clock per channel
How about "All other DU instances use one functional clock..." ? I expect more entries to the added in the future, it would be nice not to have to modify this paragraph.
and
one clock per LVDS encoder (if available). The functional clocks mustbe
named "du.x" with "x" being the channel numerical index. The LVDSclocks
must be named "lvds.x" with "x" being the LVDS encoder numericalindex.
- In addition to the functional and encoder clocks, all DU versions also
support externally supplied pixel clocks. Those clocks are optional. When supplied they must be named "dclkin.x" with "x" being the input @@ -49,16 +51,18 @@ bindings specified in Documentation/devicetree/bindings/graph.txt. The following table lists for each supported model the port number corresponding to each DU output.
Port 0 Port1 Port2 Port3
Port0 Port1 Port2 Port3
-- - R8A7779 (H1) DPAD 0 DPAD 1 - -
- R8A7790 (H2) DPAD LVDS 0 LVDS 1 -
- R8A7791 (M2-W) DPAD LVDS 0 - -
- R8A7792 (V2H) DPAD 0 DPAD 1 - -
- R8A7793 (M2-N) DPAD LVDS 0 - -
- R8A7794 (E2) DPAD 0 DPAD 1 - -
- R8A7795 (H3) DPAD HDMI 0 HDMI 1 LVDS
- R8A7796 (M3-W) DPAD HDMI LVDS -
- R8A7743 (RZ/G1M) DPAD 0 LVDS 0 - -
When there's a single DPAD the table mentions "DPAD" instead of "DPAD 0". There's no specific reason for that other than a historical one, but to keep entries consistent I would either keep doing so, or rename DPAD to DPAD 0 in all existing entries. What would you prefer ?
If you're fine with these changes there's no need to submit a new version, I can fix when applying the patch to my tree.
- R8A7745 (RZ/G1E) DPAD 0 DPAD 1 - -
- R8A7779 (R-Car H1) DPAD 0 DPAD 1 - -
- R8A7790 (R-Car H2) DPAD LVDS 0 LVDS 1 -
- R8A7791 (R-Car M2-W) DPAD LVDS 0 - -
- R8A7792 (R-Car V2H) DPAD 0 DPAD 1 - -
- R8A7793 (R-Car M2-N) DPAD LVDS 0 - -
- R8A7794 (R-Car E2) DPAD 0 DPAD 1 - -
- R8A7795 (R-Car H3) DPAD HDMI 0 HDMI 1 LVDS
- R8A7796 (R-Car M3-W) DPAD HDMI LVDS -
Example: R8A7795 (R-Car H3) ES2.0 DU