* [PATCH v4 00/11] drm/bridge: tc358767: Add DSI-to-DPI mode support
@ 2022-03-28 21:43 Marek Vasut
2022-03-28 21:43 ` Marek Vasut
` (11 more replies)
0 siblings, 12 replies; 19+ messages in thread
From: Marek Vasut @ 2022-03-28 21:43 UTC (permalink / raw)
To: dri-devel
Cc: Marek Vasut, Neil Armstrong, Jonas Karlman, Laurent Pinchart,
Sam Ravnborg, Maxime Ripard
The TC358767/TC358867/TC9595 are all capable of operating in multiple
modes, DPI-to-(e)DP, DSI-to-(e)DP, DSI-to-DPI. Clean up the driver,
switch to atomic ops, and add support for the DSI-to-DPI mode in
addition to already supported DPI-to-(e)DP mode.
Cc: Jonas Karlman <jonas@kwiboo.se>
Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Cc: Maxime Ripard <maxime@cerno.tech>
Cc: Neil Armstrong <narmstrong@baylibre.com>
Cc: Sam Ravnborg <sam@ravnborg.org>
To: dri-devel@lists.freedesktop.org
Marek Vasut (11):
dt-bindings: display: bridge: tc358867: Document DPI output support
dt-bindings: display: bridge: tc358867: Document DSI data-lanes
property
drm/bridge: tc358767: Change tc_ prefix to tc_edp_ for (e)DP specific
functions
drm/bridge: tc358767: Convert to atomic ops
drm/bridge: tc358767: Implement atomic_check callback
drm/bridge: tc358767: Move (e)DP bridge endpoint parsing into
dedicated function
drm/bridge: tc358767: Wrap (e)DP aux I2C registration into
tc_aux_link_setup()
drm/bridge: tc358767: Move bridge ops setup into
tc_probe_edp_bridge_endpoint()
drm/bridge: tc358767: Detect bridge mode from connected endpoints in
DT
drm/bridge: tc358767: Split tc_set_video_mode() into common and (e)DP
part
drm/bridge: tc358767: Add DSI-to-DPI mode support
.../display/bridge/toshiba,tc358767.yaml | 22 +-
drivers/gpu/drm/bridge/Kconfig | 1 +
drivers/gpu/drm/bridge/tc358767.c | 566 ++++++++++++++++--
3 files changed, 529 insertions(+), 60 deletions(-)
--
2.35.1
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH v4 01/11] dt-bindings: display: bridge: tc358867: Document DPI output support
2022-03-28 21:43 [PATCH v4 00/11] drm/bridge: tc358767: Add DSI-to-DPI mode support Marek Vasut
@ 2022-03-28 21:43 ` Marek Vasut
2022-03-28 21:43 ` Marek Vasut
` (10 subsequent siblings)
11 siblings, 0 replies; 19+ messages in thread
From: Marek Vasut @ 2022-03-28 21:43 UTC (permalink / raw)
To: dri-devel
Cc: Marek Vasut, Jonas Karlman, Neil Armstrong, devicetree,
Rob Herring, Laurent Pinchart, Sam Ravnborg, Maxime Ripard
The TC358767/TC358867/TC9595 are all capable of operating in multiple
modes, DPI-to-(e)DP, DSI-to-(e)DP, DSI-to-DPI. Document support for the
DPI output port, which can now be connected both as input and output.
Acked-by: Rob Herring <robh@kernel.org>
Reviewed-by: Lucas Stach <l.stach@pengutronix.de>
Tested-by: Lucas Stach <l.stach@pengutronix.de> # In both DPI to eDP and DSI to DPI mode.
Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Jonas Karlman <jonas@kwiboo.se>
Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Cc: Maxime Ripard <maxime@cerno.tech>
Cc: Neil Armstrong <narmstrong@baylibre.com>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Sam Ravnborg <sam@ravnborg.org>
Cc: devicetree@vger.kernel.org
To: dri-devel@lists.freedesktop.org
---
V2: - Rebase on next-20220217
V3: - No change
V4: - Add AB by Rob, RB/TB by Lucas
---
.../devicetree/bindings/display/bridge/toshiba,tc358767.yaml | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml b/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
index f1541cc052977..5cfda6f2ba69c 100644
--- a/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
@@ -61,8 +61,8 @@ properties:
port@1:
$ref: /schemas/graph.yaml#/properties/port
description: |
- DPI input port. The remote endpoint phandle should be a
- reference to a valid DPI output endpoint node
+ DPI input/output port. The remote endpoint phandle should be a
+ reference to a valid DPI output or input endpoint node.
port@2:
$ref: /schemas/graph.yaml#/properties/port
--
2.35.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v4 01/11] dt-bindings: display: bridge: tc358867: Document DPI output support
@ 2022-03-28 21:43 ` Marek Vasut
0 siblings, 0 replies; 19+ messages in thread
From: Marek Vasut @ 2022-03-28 21:43 UTC (permalink / raw)
To: dri-devel
Cc: l.stach, Marek Vasut, Rob Herring, Jonas Karlman,
Laurent Pinchart, Maxime Ripard, Neil Armstrong, Rob Herring,
Sam Ravnborg, devicetree
The TC358767/TC358867/TC9595 are all capable of operating in multiple
modes, DPI-to-(e)DP, DSI-to-(e)DP, DSI-to-DPI. Document support for the
DPI output port, which can now be connected both as input and output.
Acked-by: Rob Herring <robh@kernel.org>
Reviewed-by: Lucas Stach <l.stach@pengutronix.de>
Tested-by: Lucas Stach <l.stach@pengutronix.de> # In both DPI to eDP and DSI to DPI mode.
Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Jonas Karlman <jonas@kwiboo.se>
Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Cc: Maxime Ripard <maxime@cerno.tech>
Cc: Neil Armstrong <narmstrong@baylibre.com>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Sam Ravnborg <sam@ravnborg.org>
Cc: devicetree@vger.kernel.org
To: dri-devel@lists.freedesktop.org
---
V2: - Rebase on next-20220217
V3: - No change
V4: - Add AB by Rob, RB/TB by Lucas
---
.../devicetree/bindings/display/bridge/toshiba,tc358767.yaml | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml b/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
index f1541cc052977..5cfda6f2ba69c 100644
--- a/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
@@ -61,8 +61,8 @@ properties:
port@1:
$ref: /schemas/graph.yaml#/properties/port
description: |
- DPI input port. The remote endpoint phandle should be a
- reference to a valid DPI output endpoint node
+ DPI input/output port. The remote endpoint phandle should be a
+ reference to a valid DPI output or input endpoint node.
port@2:
$ref: /schemas/graph.yaml#/properties/port
--
2.35.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v4 02/11] dt-bindings: display: bridge: tc358867: Document DSI data-lanes property
2022-03-28 21:43 [PATCH v4 00/11] drm/bridge: tc358767: Add DSI-to-DPI mode support Marek Vasut
@ 2022-03-28 21:43 ` Marek Vasut
2022-03-28 21:43 ` Marek Vasut
` (10 subsequent siblings)
11 siblings, 0 replies; 19+ messages in thread
From: Marek Vasut @ 2022-03-28 21:43 UTC (permalink / raw)
To: dri-devel
Cc: Marek Vasut, Jonas Karlman, Neil Armstrong, devicetree,
Rob Herring, Laurent Pinchart, Sam Ravnborg, Maxime Ripard
It is necessary to specify the number of connected/used DSI data lanes when
using the DSI input port of this bridge. Document the 'data-lanes' property
of the DSI input port.
Reviewed-by: Rob Herring <robh@kernel.org>
Acked-by: Lucas Stach <l.stach@pengutronix.de>
Tested-by: Lucas Stach <l.stach@pengutronix.de> # In both DPI to eDP and DSI to DPI mode.
Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Jonas Karlman <jonas@kwiboo.se>
Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Cc: Maxime Ripard <maxime@cerno.tech>
Cc: Neil Armstrong <narmstrong@baylibre.com>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Sam Ravnborg <sam@ravnborg.org>
Cc: devicetree@vger.kernel.org
To: dri-devel@lists.freedesktop.org
---
V3: - New patch
V4: - Add RB by Rob, AB/TB by Lucas
---
.../display/bridge/toshiba,tc358767.yaml | 18 +++++++++++++++++-
1 file changed, 17 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml b/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
index 5cfda6f2ba69c..ed280053ec62b 100644
--- a/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
@@ -53,11 +53,27 @@ properties:
properties:
port@0:
- $ref: /schemas/graph.yaml#/properties/port
+ $ref: /schemas/graph.yaml#/$defs/port-base
+ unevaluatedProperties: false
description: |
DSI input port. The remote endpoint phandle should be a
reference to a valid DSI output endpoint node
+ properties:
+ endpoint:
+ $ref: /schemas/media/video-interfaces.yaml#
+ unevaluatedProperties: false
+
+ properties:
+ data-lanes:
+ description: array of physical DSI data lane indexes.
+ minItems: 1
+ items:
+ - const: 1
+ - const: 2
+ - const: 3
+ - const: 4
+
port@1:
$ref: /schemas/graph.yaml#/properties/port
description: |
--
2.35.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v4 02/11] dt-bindings: display: bridge: tc358867: Document DSI data-lanes property
@ 2022-03-28 21:43 ` Marek Vasut
0 siblings, 0 replies; 19+ messages in thread
From: Marek Vasut @ 2022-03-28 21:43 UTC (permalink / raw)
To: dri-devel
Cc: l.stach, Marek Vasut, Rob Herring, Jonas Karlman,
Laurent Pinchart, Maxime Ripard, Neil Armstrong, Rob Herring,
Sam Ravnborg, devicetree
It is necessary to specify the number of connected/used DSI data lanes when
using the DSI input port of this bridge. Document the 'data-lanes' property
of the DSI input port.
Reviewed-by: Rob Herring <robh@kernel.org>
Acked-by: Lucas Stach <l.stach@pengutronix.de>
Tested-by: Lucas Stach <l.stach@pengutronix.de> # In both DPI to eDP and DSI to DPI mode.
Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Jonas Karlman <jonas@kwiboo.se>
Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Cc: Maxime Ripard <maxime@cerno.tech>
Cc: Neil Armstrong <narmstrong@baylibre.com>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Sam Ravnborg <sam@ravnborg.org>
Cc: devicetree@vger.kernel.org
To: dri-devel@lists.freedesktop.org
---
V3: - New patch
V4: - Add RB by Rob, AB/TB by Lucas
---
.../display/bridge/toshiba,tc358767.yaml | 18 +++++++++++++++++-
1 file changed, 17 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml b/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
index 5cfda6f2ba69c..ed280053ec62b 100644
--- a/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/toshiba,tc358767.yaml
@@ -53,11 +53,27 @@ properties:
properties:
port@0:
- $ref: /schemas/graph.yaml#/properties/port
+ $ref: /schemas/graph.yaml#/$defs/port-base
+ unevaluatedProperties: false
description: |
DSI input port. The remote endpoint phandle should be a
reference to a valid DSI output endpoint node
+ properties:
+ endpoint:
+ $ref: /schemas/media/video-interfaces.yaml#
+ unevaluatedProperties: false
+
+ properties:
+ data-lanes:
+ description: array of physical DSI data lane indexes.
+ minItems: 1
+ items:
+ - const: 1
+ - const: 2
+ - const: 3
+ - const: 4
+
port@1:
$ref: /schemas/graph.yaml#/properties/port
description: |
--
2.35.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v4 03/11] drm/bridge: tc358767: Change tc_ prefix to tc_edp_ for (e)DP specific functions
2022-03-28 21:43 [PATCH v4 00/11] drm/bridge: tc358767: Add DSI-to-DPI mode support Marek Vasut
2022-03-28 21:43 ` Marek Vasut
2022-03-28 21:43 ` Marek Vasut
@ 2022-03-28 21:43 ` Marek Vasut
2022-03-28 21:43 ` [PATCH v4 04/11] drm/bridge: tc358767: Convert to atomic ops Marek Vasut
` (8 subsequent siblings)
11 siblings, 0 replies; 19+ messages in thread
From: Marek Vasut @ 2022-03-28 21:43 UTC (permalink / raw)
To: dri-devel
Cc: Marek Vasut, Neil Armstrong, Jonas Karlman, Laurent Pinchart,
Sam Ravnborg, Maxime Ripard
These functions are specific to (e)DP output initialization and
operation, add specific tc_edp_ prefix to those functions to
discern them from DPI output functions that will be added later
in this series. No functional change.
Reviewed-by: Lucas Stach <l.stach@pengutronix.de>
Tested-by: Lucas Stach <l.stach@pengutronix.de> # In both DPI to eDP and DSI to DPI mode.
Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Jonas Karlman <jonas@kwiboo.se>
Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Cc: Maxime Ripard <maxime@cerno.tech>
Cc: Neil Armstrong <narmstrong@baylibre.com>
Cc: Sam Ravnborg <sam@ravnborg.org>
---
V2: - New patch
V3: - Add RB from Lucas
- Rename tc_bridge_funcs to tc_edp_bridge_funcs
V4: - Add TB from Lucas
---
drivers/gpu/drm/bridge/tc358767.c | 39 ++++++++++++++++---------------
1 file changed, 20 insertions(+), 19 deletions(-)
diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c
index c23e0abc65e8f..23e0280f484de 100644
--- a/drivers/gpu/drm/bridge/tc358767.c
+++ b/drivers/gpu/drm/bridge/tc358767.c
@@ -1164,7 +1164,7 @@ static int tc_main_link_disable(struct tc_data *tc)
return regmap_write(tc->regmap, DP0CTL, 0);
}
-static int tc_stream_enable(struct tc_data *tc)
+static int tc_edp_stream_enable(struct tc_data *tc)
{
int ret;
u32 value;
@@ -1219,7 +1219,7 @@ static int tc_stream_enable(struct tc_data *tc)
return 0;
}
-static int tc_stream_disable(struct tc_data *tc)
+static int tc_edp_stream_disable(struct tc_data *tc)
{
int ret;
@@ -1234,7 +1234,7 @@ static int tc_stream_disable(struct tc_data *tc)
return 0;
}
-static void tc_bridge_enable(struct drm_bridge *bridge)
+static void tc_edp_bridge_enable(struct drm_bridge *bridge)
{
struct tc_data *tc = bridge_to_tc(bridge);
int ret;
@@ -1251,7 +1251,7 @@ static void tc_bridge_enable(struct drm_bridge *bridge)
return;
}
- ret = tc_stream_enable(tc);
+ ret = tc_edp_stream_enable(tc);
if (ret < 0) {
dev_err(tc->dev, "main link stream start error: %d\n", ret);
tc_main_link_disable(tc);
@@ -1259,12 +1259,12 @@ static void tc_bridge_enable(struct drm_bridge *bridge)
}
}
-static void tc_bridge_disable(struct drm_bridge *bridge)
+static void tc_edp_bridge_disable(struct drm_bridge *bridge)
{
struct tc_data *tc = bridge_to_tc(bridge);
int ret;
- ret = tc_stream_disable(tc);
+ ret = tc_edp_stream_disable(tc);
if (ret < 0)
dev_err(tc->dev, "main link stream stop error: %d\n", ret);
@@ -1285,9 +1285,10 @@ static bool tc_bridge_mode_fixup(struct drm_bridge *bridge,
return true;
}
-static enum drm_mode_status tc_mode_valid(struct drm_bridge *bridge,
- const struct drm_display_info *info,
- const struct drm_display_mode *mode)
+static enum drm_mode_status
+tc_edp_mode_valid(struct drm_bridge *bridge,
+ const struct drm_display_info *info,
+ const struct drm_display_mode *mode)
{
struct tc_data *tc = bridge_to_tc(bridge);
u32 req, avail;
@@ -1395,8 +1396,8 @@ static const struct drm_connector_funcs tc_connector_funcs = {
.atomic_destroy_state = drm_atomic_helper_connector_destroy_state,
};
-static int tc_bridge_attach(struct drm_bridge *bridge,
- enum drm_bridge_attach_flags flags)
+static int tc_edp_bridge_attach(struct drm_bridge *bridge,
+ enum drm_bridge_attach_flags flags)
{
u32 bus_format = MEDIA_BUS_FMT_RGB888_1X24;
struct tc_data *tc = bridge_to_tc(bridge);
@@ -1448,18 +1449,18 @@ static int tc_bridge_attach(struct drm_bridge *bridge,
return ret;
}
-static void tc_bridge_detach(struct drm_bridge *bridge)
+static void tc_edp_bridge_detach(struct drm_bridge *bridge)
{
drm_dp_aux_unregister(&bridge_to_tc(bridge)->aux);
}
-static const struct drm_bridge_funcs tc_bridge_funcs = {
- .attach = tc_bridge_attach,
- .detach = tc_bridge_detach,
- .mode_valid = tc_mode_valid,
+static const struct drm_bridge_funcs tc_edp_bridge_funcs = {
+ .attach = tc_edp_bridge_attach,
+ .detach = tc_edp_bridge_detach,
+ .mode_valid = tc_edp_mode_valid,
.mode_set = tc_bridge_mode_set,
- .enable = tc_bridge_enable,
- .disable = tc_bridge_disable,
+ .enable = tc_edp_bridge_enable,
+ .disable = tc_edp_bridge_disable,
.mode_fixup = tc_bridge_mode_fixup,
.detect = tc_bridge_detect,
.get_edid = tc_get_edid,
@@ -1696,7 +1697,7 @@ static int tc_probe(struct i2c_client *client, const struct i2c_device_id *id)
tc->aux.transfer = tc_aux_transfer;
drm_dp_aux_init(&tc->aux);
- tc->bridge.funcs = &tc_bridge_funcs;
+ tc->bridge.funcs = &tc_edp_bridge_funcs;
if (tc->hpd_pin >= 0)
tc->bridge.ops |= DRM_BRIDGE_OP_DETECT;
tc->bridge.ops |= DRM_BRIDGE_OP_EDID;
--
2.35.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v4 04/11] drm/bridge: tc358767: Convert to atomic ops
2022-03-28 21:43 [PATCH v4 00/11] drm/bridge: tc358767: Add DSI-to-DPI mode support Marek Vasut
` (2 preceding siblings ...)
2022-03-28 21:43 ` [PATCH v4 03/11] drm/bridge: tc358767: Change tc_ prefix to tc_edp_ for (e)DP specific functions Marek Vasut
@ 2022-03-28 21:43 ` Marek Vasut
2022-03-28 21:43 ` [PATCH v4 05/11] drm/bridge: tc358767: Implement atomic_check callback Marek Vasut
` (7 subsequent siblings)
11 siblings, 0 replies; 19+ messages in thread
From: Marek Vasut @ 2022-03-28 21:43 UTC (permalink / raw)
To: dri-devel
Cc: Marek Vasut, Neil Armstrong, Jonas Karlman, Laurent Pinchart,
Sam Ravnborg, Maxime Ripard
Use the atomic version of the enable/disable operations to continue the
transition to the atomic API. This will be needed to access the mode
from the atomic state.
Reviewed-by: Lucas Stach <l.stach@pengutronix.de>
Tested-by: Lucas Stach <l.stach@pengutronix.de> # In both DPI to eDP and DSI to DPI mode.
Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Jonas Karlman <jonas@kwiboo.se>
Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Cc: Maxime Ripard <maxime@cerno.tech>
Cc: Neil Armstrong <narmstrong@baylibre.com>
Cc: Sam Ravnborg <sam@ravnborg.org>
---
V2: - New patch
V3: - Add RB from Lucas
V4: - Add TB from Lucas
---
drivers/gpu/drm/bridge/tc358767.c | 15 +++++++++++----
1 file changed, 11 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c
index 23e0280f484de..f88d8e616f7f8 100644
--- a/drivers/gpu/drm/bridge/tc358767.c
+++ b/drivers/gpu/drm/bridge/tc358767.c
@@ -1234,7 +1234,9 @@ static int tc_edp_stream_disable(struct tc_data *tc)
return 0;
}
-static void tc_edp_bridge_enable(struct drm_bridge *bridge)
+static void
+tc_edp_bridge_atomic_enable(struct drm_bridge *bridge,
+ struct drm_bridge_state *old_bridge_state)
{
struct tc_data *tc = bridge_to_tc(bridge);
int ret;
@@ -1259,7 +1261,9 @@ static void tc_edp_bridge_enable(struct drm_bridge *bridge)
}
}
-static void tc_edp_bridge_disable(struct drm_bridge *bridge)
+static void
+tc_edp_bridge_atomic_disable(struct drm_bridge *bridge,
+ struct drm_bridge_state *old_bridge_state)
{
struct tc_data *tc = bridge_to_tc(bridge);
int ret;
@@ -1459,11 +1463,14 @@ static const struct drm_bridge_funcs tc_edp_bridge_funcs = {
.detach = tc_edp_bridge_detach,
.mode_valid = tc_edp_mode_valid,
.mode_set = tc_bridge_mode_set,
- .enable = tc_edp_bridge_enable,
- .disable = tc_edp_bridge_disable,
+ .atomic_enable = tc_edp_bridge_atomic_enable,
+ .atomic_disable = tc_edp_bridge_atomic_disable,
.mode_fixup = tc_bridge_mode_fixup,
.detect = tc_bridge_detect,
.get_edid = tc_get_edid,
+ .atomic_duplicate_state = drm_atomic_helper_bridge_duplicate_state,
+ .atomic_destroy_state = drm_atomic_helper_bridge_destroy_state,
+ .atomic_reset = drm_atomic_helper_bridge_reset,
};
static bool tc_readable_reg(struct device *dev, unsigned int reg)
--
2.35.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v4 05/11] drm/bridge: tc358767: Implement atomic_check callback
2022-03-28 21:43 [PATCH v4 00/11] drm/bridge: tc358767: Add DSI-to-DPI mode support Marek Vasut
` (3 preceding siblings ...)
2022-03-28 21:43 ` [PATCH v4 04/11] drm/bridge: tc358767: Convert to atomic ops Marek Vasut
@ 2022-03-28 21:43 ` Marek Vasut
2022-03-28 21:43 ` [PATCH v4 06/11] drm/bridge: tc358767: Move (e)DP bridge endpoint parsing into dedicated function Marek Vasut
` (6 subsequent siblings)
11 siblings, 0 replies; 19+ messages in thread
From: Marek Vasut @ 2022-03-28 21:43 UTC (permalink / raw)
To: dri-devel
Cc: Marek Vasut, Neil Armstrong, Jonas Karlman, Laurent Pinchart,
Sam Ravnborg, Maxime Ripard
Implement .atomic_check callback which prevents user space from setting
unsupported mode. The tc_edp_common_atomic_check() variant is already
prepared for DSI-to-DPI mode addition, which has different frequency
limits.
Reviewed-by: Lucas Stach <l.stach@pengutronix.de>
Tested-by: Lucas Stach <l.stach@pengutronix.de> # In both DPI to eDP and DSI to DPI mode.
Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Jonas Karlman <jonas@kwiboo.se>
Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Cc: Maxime Ripard <maxime@cerno.tech>
Cc: Neil Armstrong <narmstrong@baylibre.com>
Cc: Sam Ravnborg <sam@ravnborg.org>
---
V2: - New patch
V3: - Drop edp from tc_edp_common_atomic_check,
s@\<tc_edp_common_atomic_check\>@tc_common_atomic_check@g
- Return -EINVAL in case clock frequency is too high
V4: - Add RB/TB by Lucas
---
drivers/gpu/drm/bridge/tc358767.c | 26 ++++++++++++++++++++++++++
1 file changed, 26 insertions(+)
diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c
index f88d8e616f7f8..e95153d9c1499 100644
--- a/drivers/gpu/drm/bridge/tc358767.c
+++ b/drivers/gpu/drm/bridge/tc358767.c
@@ -1289,6 +1289,31 @@ static bool tc_bridge_mode_fixup(struct drm_bridge *bridge,
return true;
}
+static int tc_common_atomic_check(struct drm_bridge *bridge,
+ struct drm_bridge_state *bridge_state,
+ struct drm_crtc_state *crtc_state,
+ struct drm_connector_state *conn_state,
+ const unsigned int max_khz)
+{
+ tc_bridge_mode_fixup(bridge, &crtc_state->mode,
+ &crtc_state->adjusted_mode);
+
+ if (crtc_state->adjusted_mode.clock > max_khz)
+ return -EINVAL;
+
+ return 0;
+}
+
+static int tc_edp_atomic_check(struct drm_bridge *bridge,
+ struct drm_bridge_state *bridge_state,
+ struct drm_crtc_state *crtc_state,
+ struct drm_connector_state *conn_state)
+{
+ /* DPI->(e)DP interface clock limitation: upto 154 MHz */
+ return tc_common_atomic_check(bridge, bridge_state, crtc_state,
+ conn_state, 154000);
+}
+
static enum drm_mode_status
tc_edp_mode_valid(struct drm_bridge *bridge,
const struct drm_display_info *info,
@@ -1463,6 +1488,7 @@ static const struct drm_bridge_funcs tc_edp_bridge_funcs = {
.detach = tc_edp_bridge_detach,
.mode_valid = tc_edp_mode_valid,
.mode_set = tc_bridge_mode_set,
+ .atomic_check = tc_edp_atomic_check,
.atomic_enable = tc_edp_bridge_atomic_enable,
.atomic_disable = tc_edp_bridge_atomic_disable,
.mode_fixup = tc_bridge_mode_fixup,
--
2.35.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v4 06/11] drm/bridge: tc358767: Move (e)DP bridge endpoint parsing into dedicated function
2022-03-28 21:43 [PATCH v4 00/11] drm/bridge: tc358767: Add DSI-to-DPI mode support Marek Vasut
` (4 preceding siblings ...)
2022-03-28 21:43 ` [PATCH v4 05/11] drm/bridge: tc358767: Implement atomic_check callback Marek Vasut
@ 2022-03-28 21:43 ` Marek Vasut
2022-03-28 21:43 ` [PATCH v4 07/11] drm/bridge: tc358767: Wrap (e)DP aux I2C registration into tc_aux_link_setup() Marek Vasut
` (5 subsequent siblings)
11 siblings, 0 replies; 19+ messages in thread
From: Marek Vasut @ 2022-03-28 21:43 UTC (permalink / raw)
To: dri-devel
Cc: Marek Vasut, Neil Armstrong, Jonas Karlman, Laurent Pinchart,
Sam Ravnborg, Maxime Ripard
The TC358767/TC358867/TC9595 are all capable of operating in multiple
modes, DPI-to-(e)DP, DSI-to-(e)DP, DSI-to-DPI. Only the first mode is
currently supported. In order to support the rest of the modes without
making the tc_probe() overly long, split the bridge endpoint parsing
into dedicated function, where the necessary logic to detect the bridge
mode based on which endpoints are connected, can be implemented.
Reviewed-by: Lucas Stach <l.stach@pengutronix.de>
Tested-by: Lucas Stach <l.stach@pengutronix.de> # In both DPI to eDP and DSI to DPI mode.
Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Jonas Karlman <jonas@kwiboo.se>
Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Cc: Maxime Ripard <maxime@cerno.tech>
Cc: Neil Armstrong <narmstrong@baylibre.com>
Cc: Sam Ravnborg <sam@ravnborg.org>
---
V2: - Rename tc_probe_bridge_mode() to tc_probe_edp_bridge_endpoint()
to better reflect that it parses the (e)DP output endpoint
V3: - Add RB from Lucas
V4: - Add TB from Lucas
---
drivers/gpu/drm/bridge/tc358767.c | 30 +++++++++++++++++++++---------
1 file changed, 21 insertions(+), 9 deletions(-)
diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c
index e95153d9c1499..0f24156543bae 100644
--- a/drivers/gpu/drm/bridge/tc358767.c
+++ b/drivers/gpu/drm/bridge/tc358767.c
@@ -1583,19 +1583,12 @@ static irqreturn_t tc_irq_handler(int irq, void *arg)
return IRQ_HANDLED;
}
-static int tc_probe(struct i2c_client *client, const struct i2c_device_id *id)
+static int tc_probe_edp_bridge_endpoint(struct tc_data *tc)
{
- struct device *dev = &client->dev;
+ struct device *dev = tc->dev;
struct drm_panel *panel;
- struct tc_data *tc;
int ret;
- tc = devm_kzalloc(dev, sizeof(*tc), GFP_KERNEL);
- if (!tc)
- return -ENOMEM;
-
- tc->dev = dev;
-
/* port@2 is the output port */
ret = drm_of_find_panel_or_bridge(dev->of_node, 2, 0, &panel, NULL);
if (ret && ret != -ENODEV)
@@ -1614,6 +1607,25 @@ static int tc_probe(struct i2c_client *client, const struct i2c_device_id *id)
tc->bridge.type = DRM_MODE_CONNECTOR_DisplayPort;
}
+ return ret;
+}
+
+static int tc_probe(struct i2c_client *client, const struct i2c_device_id *id)
+{
+ struct device *dev = &client->dev;
+ struct tc_data *tc;
+ int ret;
+
+ tc = devm_kzalloc(dev, sizeof(*tc), GFP_KERNEL);
+ if (!tc)
+ return -ENOMEM;
+
+ tc->dev = dev;
+
+ ret = tc_probe_edp_bridge_endpoint(tc);
+ if (ret)
+ return ret;
+
/* Shut down GPIO is optional */
tc->sd_gpio = devm_gpiod_get_optional(dev, "shutdown", GPIOD_OUT_HIGH);
if (IS_ERR(tc->sd_gpio))
--
2.35.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v4 07/11] drm/bridge: tc358767: Wrap (e)DP aux I2C registration into tc_aux_link_setup()
2022-03-28 21:43 [PATCH v4 00/11] drm/bridge: tc358767: Add DSI-to-DPI mode support Marek Vasut
` (5 preceding siblings ...)
2022-03-28 21:43 ` [PATCH v4 06/11] drm/bridge: tc358767: Move (e)DP bridge endpoint parsing into dedicated function Marek Vasut
@ 2022-03-28 21:43 ` Marek Vasut
2022-03-28 21:43 ` [PATCH v4 08/11] drm/bridge: tc358767: Move bridge ops setup into tc_probe_edp_bridge_endpoint() Marek Vasut
` (4 subsequent siblings)
11 siblings, 0 replies; 19+ messages in thread
From: Marek Vasut @ 2022-03-28 21:43 UTC (permalink / raw)
To: dri-devel
Cc: Marek Vasut, Neil Armstrong, Jonas Karlman, Laurent Pinchart,
Sam Ravnborg, Maxime Ripard
This bit of code is (e)DP and aux I2C link specific, move it into
tc_aux_link_setup() to permit cleaner addition of DSI-to-DPI mode.
No functional change.
Reviewed-by: Lucas Stach <l.stach@pengutronix.de>
Tested-by: Lucas Stach <l.stach@pengutronix.de> # In both DPI to eDP and DSI to DPI mode.
Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Jonas Karlman <jonas@kwiboo.se>
Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Cc: Maxime Ripard <maxime@cerno.tech>
Cc: Neil Armstrong <narmstrong@baylibre.com>
Cc: Sam Ravnborg <sam@ravnborg.org>
---
V2: - New patch
V3: - Add RB from Lucas
V4: - Add TB from Lucas
---
drivers/gpu/drm/bridge/tc358767.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c
index 0f24156543bae..8a8b495aa0ad1 100644
--- a/drivers/gpu/drm/bridge/tc358767.c
+++ b/drivers/gpu/drm/bridge/tc358767.c
@@ -656,6 +656,12 @@ static int tc_aux_link_setup(struct tc_data *tc)
if (ret)
goto err;
+ /* Register DP AUX channel */
+ tc->aux.name = "TC358767 AUX i2c adapter";
+ tc->aux.dev = tc->dev;
+ tc->aux.transfer = tc_aux_transfer;
+ drm_dp_aux_init(&tc->aux);
+
return 0;
err:
dev_err(tc->dev, "tc_aux_link_setup failed: %d\n", ret);
@@ -1736,12 +1742,6 @@ static int tc_probe(struct i2c_client *client, const struct i2c_device_id *id)
if (ret)
return ret;
- /* Register DP AUX channel */
- tc->aux.name = "TC358767 AUX i2c adapter";
- tc->aux.dev = tc->dev;
- tc->aux.transfer = tc_aux_transfer;
- drm_dp_aux_init(&tc->aux);
-
tc->bridge.funcs = &tc_edp_bridge_funcs;
if (tc->hpd_pin >= 0)
tc->bridge.ops |= DRM_BRIDGE_OP_DETECT;
--
2.35.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v4 08/11] drm/bridge: tc358767: Move bridge ops setup into tc_probe_edp_bridge_endpoint()
2022-03-28 21:43 [PATCH v4 00/11] drm/bridge: tc358767: Add DSI-to-DPI mode support Marek Vasut
` (6 preceding siblings ...)
2022-03-28 21:43 ` [PATCH v4 07/11] drm/bridge: tc358767: Wrap (e)DP aux I2C registration into tc_aux_link_setup() Marek Vasut
@ 2022-03-28 21:43 ` Marek Vasut
2022-03-28 21:43 ` [PATCH v4 09/11] drm/bridge: tc358767: Detect bridge mode from connected endpoints in DT Marek Vasut
` (3 subsequent siblings)
11 siblings, 0 replies; 19+ messages in thread
From: Marek Vasut @ 2022-03-28 21:43 UTC (permalink / raw)
To: dri-devel
Cc: Marek Vasut, Neil Armstrong, Jonas Karlman, Laurent Pinchart,
Sam Ravnborg, Maxime Ripard
The bridge ops are specific to the bridge configuration, move them
into tc_probe_edp_bridge_endpoint() to permit cleaner addition of
DSI-to-DPI mode. No functional change.
Reviewed-by: Lucas Stach <l.stach@pengutronix.de>
Tested-by: Lucas Stach <l.stach@pengutronix.de> # In both DPI to eDP and DSI to DPI mode.
Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Jonas Karlman <jonas@kwiboo.se>
Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Cc: Maxime Ripard <maxime@cerno.tech>
Cc: Neil Armstrong <narmstrong@baylibre.com>
Cc: Sam Ravnborg <sam@ravnborg.org>
---
V2: - New patch
V3: - No change
V4: - Add RB/TB from Lucas
---
drivers/gpu/drm/bridge/tc358767.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c
index 8a8b495aa0ad1..8c468f77618d2 100644
--- a/drivers/gpu/drm/bridge/tc358767.c
+++ b/drivers/gpu/drm/bridge/tc358767.c
@@ -1613,6 +1613,11 @@ static int tc_probe_edp_bridge_endpoint(struct tc_data *tc)
tc->bridge.type = DRM_MODE_CONNECTOR_DisplayPort;
}
+ tc->bridge.funcs = &tc_edp_bridge_funcs;
+ if (tc->hpd_pin >= 0)
+ tc->bridge.ops |= DRM_BRIDGE_OP_DETECT;
+ tc->bridge.ops |= DRM_BRIDGE_OP_EDID;
+
return ret;
}
@@ -1742,11 +1747,6 @@ static int tc_probe(struct i2c_client *client, const struct i2c_device_id *id)
if (ret)
return ret;
- tc->bridge.funcs = &tc_edp_bridge_funcs;
- if (tc->hpd_pin >= 0)
- tc->bridge.ops |= DRM_BRIDGE_OP_DETECT;
- tc->bridge.ops |= DRM_BRIDGE_OP_EDID;
-
tc->bridge.of_node = dev->of_node;
drm_bridge_add(&tc->bridge);
--
2.35.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v4 09/11] drm/bridge: tc358767: Detect bridge mode from connected endpoints in DT
2022-03-28 21:43 [PATCH v4 00/11] drm/bridge: tc358767: Add DSI-to-DPI mode support Marek Vasut
` (7 preceding siblings ...)
2022-03-28 21:43 ` [PATCH v4 08/11] drm/bridge: tc358767: Move bridge ops setup into tc_probe_edp_bridge_endpoint() Marek Vasut
@ 2022-03-28 21:43 ` Marek Vasut
2022-03-28 21:43 ` [PATCH v4 10/11] drm/bridge: tc358767: Split tc_set_video_mode() into common and (e)DP part Marek Vasut
` (2 subsequent siblings)
11 siblings, 0 replies; 19+ messages in thread
From: Marek Vasut @ 2022-03-28 21:43 UTC (permalink / raw)
To: dri-devel
Cc: Marek Vasut, Neil Armstrong, Jonas Karlman, Laurent Pinchart,
Sam Ravnborg, Maxime Ripard
The TC358767/TC358867/TC9595 are all capable of operating in multiple
modes, DPI-to-(e)DP, DSI-to-(e)DP, DSI-to-DPI. Only the first mode is
currently supported. It is possible to find out the mode in which the
bridge should be operated by testing connected endpoints in DT.
Port allocation:
port@0 - DSI input
port@1 - DPI input/output
port@2 - eDP output
Possible connections:
DPI -> port@1 -> port@2 -> eDP :: [port@0 is not connected]
DSI -> port@0 -> port@2 -> eDP :: [port@1 is not connected]
DSI -> port@0 -> port@1 -> DPI :: [port@2 is not connected]
Add function to determine the bridge mode based on connected endpoints.
Reviewed-by: Lucas Stach <l.stach@pengutronix.de>
Tested-by: Lucas Stach <l.stach@pengutronix.de> # In both DPI to eDP and DSI to DPI mode.
Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Jonas Karlman <jonas@kwiboo.se>
Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Cc: Maxime Ripard <maxime@cerno.tech>
Cc: Neil Armstrong <narmstrong@baylibre.com>
Cc: Sam Ravnborg <sam@ravnborg.org>
---
V2: - New patch
V3: - Add RB from Lucas
V4: - Add TB from Lucas
---
drivers/gpu/drm/bridge/tc358767.c | 46 ++++++++++++++++++++++++++++++-
1 file changed, 45 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c
index 8c468f77618d2..dd324cf16e4d0 100644
--- a/drivers/gpu/drm/bridge/tc358767.c
+++ b/drivers/gpu/drm/bridge/tc358767.c
@@ -1621,6 +1621,50 @@ static int tc_probe_edp_bridge_endpoint(struct tc_data *tc)
return ret;
}
+static int tc_probe_bridge_endpoint(struct tc_data *tc)
+{
+ struct device *dev = tc->dev;
+ struct of_endpoint endpoint;
+ struct device_node *node = NULL;
+ const u8 mode_dpi_to_edp = BIT(1) | BIT(2);
+ const u8 mode_dsi_to_edp = BIT(0) | BIT(2);
+ const u8 mode_dsi_to_dpi = BIT(0) | BIT(1);
+ u8 mode = 0;
+
+ /*
+ * Determine bridge configuration.
+ *
+ * Port allocation:
+ * port@0 - DSI input
+ * port@1 - DPI input/output
+ * port@2 - eDP output
+ *
+ * Possible connections:
+ * DPI -> port@1 -> port@2 -> eDP :: [port@0 is not connected]
+ * DSI -> port@0 -> port@2 -> eDP :: [port@1 is not connected]
+ * DSI -> port@0 -> port@1 -> DPI :: [port@2 is not connected]
+ */
+
+ for_each_endpoint_of_node(dev->of_node, node) {
+ of_graph_parse_endpoint(node, &endpoint);
+ if (endpoint.port > 2)
+ return -EINVAL;
+
+ mode |= BIT(endpoint.port);
+ }
+
+ if (mode == mode_dpi_to_edp)
+ return tc_probe_edp_bridge_endpoint(tc);
+ else if (mode == mode_dsi_to_dpi)
+ dev_warn(dev, "The mode DSI-to-DPI is not supported!\n");
+ else if (mode == mode_dsi_to_edp)
+ dev_warn(dev, "The mode DSI-to-(e)DP is not supported!\n");
+ else
+ dev_warn(dev, "Invalid mode (0x%x) is not supported!\n", mode);
+
+ return -EINVAL;
+}
+
static int tc_probe(struct i2c_client *client, const struct i2c_device_id *id)
{
struct device *dev = &client->dev;
@@ -1633,7 +1677,7 @@ static int tc_probe(struct i2c_client *client, const struct i2c_device_id *id)
tc->dev = dev;
- ret = tc_probe_edp_bridge_endpoint(tc);
+ ret = tc_probe_bridge_endpoint(tc);
if (ret)
return ret;
--
2.35.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v4 10/11] drm/bridge: tc358767: Split tc_set_video_mode() into common and (e)DP part
2022-03-28 21:43 [PATCH v4 00/11] drm/bridge: tc358767: Add DSI-to-DPI mode support Marek Vasut
` (8 preceding siblings ...)
2022-03-28 21:43 ` [PATCH v4 09/11] drm/bridge: tc358767: Detect bridge mode from connected endpoints in DT Marek Vasut
@ 2022-03-28 21:43 ` Marek Vasut
2022-03-29 1:54 ` kernel test robot
2022-03-29 2:05 ` kernel test robot
2022-03-28 21:43 ` [PATCH v4 11/11] drm/bridge: tc358767: Add DSI-to-DPI mode support Marek Vasut
2022-03-29 7:47 ` [PATCH v4 00/11] " Lucas Stach
11 siblings, 2 replies; 19+ messages in thread
From: Marek Vasut @ 2022-03-28 21:43 UTC (permalink / raw)
To: dri-devel
Cc: Marek Vasut, Neil Armstrong, Jonas Karlman, Laurent Pinchart,
Sam Ravnborg, Maxime Ripard
The tc_set_video_mode() sets up both common and (e)DP video mode settings of
the bridge chip. Split the function into tc_set_common_video_mode() to set
the common settings and tc_set_edp_video_mode() to set the (e)DP specific
settings. No functional change.
Reviewed-by: Lucas Stach <l.stach@pengutronix.de>
Tested-by: Lucas Stach <l.stach@pengutronix.de> # In both DPI to eDP and DSI to DPI mode.
Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Jonas Karlman <jonas@kwiboo.se>
Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Cc: Maxime Ripard <maxime@cerno.tech>
Cc: Neil Armstrong <narmstrong@baylibre.com>
Cc: Sam Ravnborg <sam@ravnborg.org>
---
V2: - New patch
V3: - Add RB from Lucas
V4: - Add TB from Lucas
---
drivers/gpu/drm/bridge/tc358767.c | 48 ++++++++++++++++++++++++-------
1 file changed, 37 insertions(+), 11 deletions(-)
diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c
index dd324cf16e4d0..5520c26d14b4c 100644
--- a/drivers/gpu/drm/bridge/tc358767.c
+++ b/drivers/gpu/drm/bridge/tc358767.c
@@ -734,11 +734,10 @@ static int tc_get_display_props(struct tc_data *tc)
return ret;
}
-static int tc_set_video_mode(struct tc_data *tc,
- const struct drm_display_mode *mode)
+static int tc_set_common_video_mode(struct tc_data *tc,
+ const struct drm_display_mode *mode)
{
int ret;
- int vid_sync_dly;
int max_tu_symbol;
int left_margin = mode->htotal - mode->hsync_end;
@@ -747,7 +746,6 @@ static int tc_set_video_mode(struct tc_data *tc,
int upper_margin = mode->vtotal - mode->vsync_end;
int lower_margin = mode->vsync_start - mode->vdisplay;
int vsync_len = mode->vsync_end - mode->vsync_start;
- u32 dp0_syncval;
u32 bits_per_pixel = 24;
u32 in_bw, out_bw;
@@ -818,8 +816,35 @@ static int tc_set_video_mode(struct tc_data *tc,
FIELD_PREP(COLOR_B, 99) |
ENI2CFILTER |
FIELD_PREP(COLOR_BAR_MODE, COLOR_BAR_MODE_BARS));
- if (ret)
- return ret;
+
+ return ret;
+}
+
+static int tc_set_edp_video_mode(struct tc_data *tc,
+ const struct drm_display_mode *mode)
+{
+ int ret;
+ int vid_sync_dly;
+ int max_tu_symbol;
+
+ int left_margin = mode->htotal - mode->hsync_end;
+ int hsync_len = mode->hsync_end - mode->hsync_start;
+ int upper_margin = mode->vtotal - mode->vsync_end;
+ int vsync_len = mode->vsync_end - mode->vsync_start;
+ u32 dp0_syncval;
+ u32 bits_per_pixel = 24;
+ u32 in_bw, out_bw;
+
+ /*
+ * Recommended maximum number of symbols transferred in a transfer unit:
+ * DIV_ROUND_UP((input active video bandwidth in bytes) * tu_size,
+ * (output active video bandwidth in bytes))
+ * Must be less than tu_size.
+ */
+
+ in_bw = mode->clock * bits_per_pixel / 8;
+ out_bw = tc->link.num_lanes * tc->link.rate;
+ max_tu_symbol = DIV_ROUND_UP(in_bw * TU_SIZE_RECOMMENDED, out_bw);
/* DP Main Stream Attributes */
vid_sync_dly = hsync_len + left_margin + mode->hdisplay;
@@ -869,10 +894,7 @@ static int tc_set_video_mode(struct tc_data *tc,
FIELD_PREP(MAX_TU_SYMBOL, max_tu_symbol) |
FIELD_PREP(TU_SIZE, TU_SIZE_RECOMMENDED) |
BPC_8);
- if (ret)
- return ret;
-
- return 0;
+ return ret;
}
static int tc_wait_link_training(struct tc_data *tc)
@@ -1185,7 +1207,11 @@ static int tc_edp_stream_enable(struct tc_data *tc)
return ret;
}
- ret = tc_set_video_mode(tc, &tc->mode);
+ ret = tc_set_common_video_mode(tc, &tc->mode);
+ if (ret)
+ return ret;
+
+ ret = tc_set_edp_video_mode(tc, &tc->mode);
if (ret)
return ret;
--
2.35.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v4 11/11] drm/bridge: tc358767: Add DSI-to-DPI mode support
2022-03-28 21:43 [PATCH v4 00/11] drm/bridge: tc358767: Add DSI-to-DPI mode support Marek Vasut
` (9 preceding siblings ...)
2022-03-28 21:43 ` [PATCH v4 10/11] drm/bridge: tc358767: Split tc_set_video_mode() into common and (e)DP part Marek Vasut
@ 2022-03-28 21:43 ` Marek Vasut
2022-03-29 7:47 ` [PATCH v4 00/11] " Lucas Stach
11 siblings, 0 replies; 19+ messages in thread
From: Marek Vasut @ 2022-03-28 21:43 UTC (permalink / raw)
To: dri-devel
Cc: Marek Vasut, Neil Armstrong, Jonas Karlman, Laurent Pinchart,
Sam Ravnborg, Maxime Ripard
The TC358767/TC358867/TC9595 are all capable of operating in multiple
modes, DPI-to-(e)DP, DSI-to-(e)DP, DSI-to-DPI. Add support for the
DSI-to-DPI mode.
This requires skipping most of the (e)DP initialization code, which is
currently a large part of this driver, hence it is better to have far
simpler separate tc_dpi_bridge_funcs and their implementation.
The configuration of DPI output is also much simpler. The configuration
of the DSI input is rather similar to the other TC bridge chips.
The Pixel PLL in DPI output mode does not have the 65..150 MHz limitation
imposed on the (e)DP output mode, so this limitation is skipped to permit
operating panels with far slower pixel clock, even below 9 MHz. This mode
of operation of the PLL is valid and tested.
The detection of bridge mode is now added into tc_probe_bridge_mode(),
where in case a DPI panel is found on port@1 endpoint@1, the mode is
assumed to be DSI-to-DPI. If (e)DP is detected on port@2, the mode is
assumed to be DPI-to-(e)DP.
The DSI-to-(e)DP mode is not supported due to lack of proper hardware,
but this would be some sort of mix between the two aforementioned modes.
Reviewed-by: Lucas Stach <l.stach@pengutronix.de>
Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Jonas Karlman <jonas@kwiboo.se>
Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Cc: Maxime Ripard <maxime@cerno.tech>
Cc: Neil Armstrong <narmstrong@baylibre.com>
Cc: Sam Ravnborg <sam@ravnborg.org>
---
V2: - Rebase on next-20220217 and new patches in this series
V3: - Drop edp from tc_edp_common_atomic_check,
s@\<tc_edp_common_atomic_check\>@tc_common_atomic_check@g
- Limit Pixel PLL output to 0-100 MHz for DPI and 150-650 MHz for eDP
- Drop VID_EN write from tc_dpi_stream_disable()
- Reduce PLL stabilization delay to 120..150us in tc_dpi_stream_enable()
- Call drm_bridge_remove() in case tc_mipi_dsi_host_attach() fails
- Check of_property_count_u32_elems() return code as int instead of u8
- Enable DP0/DP1 PLL for DSI-to-DPI mode too, they clock the internal
framebuffer and it is too slow if those PLLs are in bypass
V4: - Rebase on top of dropped drm/bridge: tc358767: Move hardware init to enable callback
- Use -1 in of_graph_get_endpoint_by_regs(.., .., -1) third param
- Select DRM_MIPI_DSI
- Add RB from Lucas
---
drivers/gpu/drm/bridge/Kconfig | 1 +
drivers/gpu/drm/bridge/tc358767.c | 358 +++++++++++++++++++++++++++++-
2 files changed, 348 insertions(+), 11 deletions(-)
diff --git a/drivers/gpu/drm/bridge/Kconfig b/drivers/gpu/drm/bridge/Kconfig
index 561e23727f6f5..536b3c3169918 100644
--- a/drivers/gpu/drm/bridge/Kconfig
+++ b/drivers/gpu/drm/bridge/Kconfig
@@ -280,6 +280,7 @@ config DRM_TOSHIBA_TC358767
select DRM_DP_HELPER
select DRM_KMS_HELPER
select REGMAP_I2C
+ select DRM_MIPI_DSI
select DRM_PANEL
help
Toshiba TC358767 eDP bridge chip driver.
diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c
index 5520c26d14b4c..13c56f23439aa 100644
--- a/drivers/gpu/drm/bridge/tc358767.c
+++ b/drivers/gpu/drm/bridge/tc358767.c
@@ -1,6 +1,12 @@
// SPDX-License-Identifier: GPL-2.0-or-later
/*
- * tc358767 eDP bridge driver
+ * TC358767/TC358867/TC9595 DSI/DPI-to-DPI/(e)DP bridge driver
+ *
+ * The TC358767/TC358867/TC9595 can operate in multiple modes.
+ * The following modes are supported:
+ * DPI->(e)DP -- supported
+ * DSI->DPI .... supported
+ * DSI->(e)DP .. NOT supported
*
* Copyright (C) 2016 CogentEmbedded Inc
* Author: Andrey Gusakov <andrey.gusakov@cogentembedded.com>
@@ -29,6 +35,7 @@
#include <drm/drm_bridge.h>
#include <drm/dp/drm_dp_helper.h>
#include <drm/drm_edid.h>
+#include <drm/drm_mipi_dsi.h>
#include <drm/drm_of.h>
#include <drm/drm_panel.h>
#include <drm/drm_print.h>
@@ -36,7 +43,35 @@
/* Registers */
-/* Display Parallel Interface */
+/* PPI layer registers */
+#define PPI_STARTPPI 0x0104 /* START control bit */
+#define PPI_LPTXTIMECNT 0x0114 /* LPTX timing signal */
+#define LPX_PERIOD 3
+#define PPI_LANEENABLE 0x0134
+#define PPI_TX_RX_TA 0x013c
+#define TTA_GET 0x40000
+#define TTA_SURE 6
+#define PPI_D0S_ATMR 0x0144
+#define PPI_D1S_ATMR 0x0148
+#define PPI_D0S_CLRSIPOCOUNT 0x0164 /* Assertion timer for Lane 0 */
+#define PPI_D1S_CLRSIPOCOUNT 0x0168 /* Assertion timer for Lane 1 */
+#define PPI_D2S_CLRSIPOCOUNT 0x016c /* Assertion timer for Lane 2 */
+#define PPI_D3S_CLRSIPOCOUNT 0x0170 /* Assertion timer for Lane 3 */
+#define PPI_START_FUNCTION BIT(0)
+
+/* DSI layer registers */
+#define DSI_STARTDSI 0x0204 /* START control bit of DSI-TX */
+#define DSI_LANEENABLE 0x0210 /* Enables each lane */
+#define DSI_RX_START BIT(0)
+
+/* Lane enable PPI and DSI register bits */
+#define LANEENABLE_CLEN BIT(0)
+#define LANEENABLE_L0EN BIT(1)
+#define LANEENABLE_L1EN BIT(2)
+#define LANEENABLE_L2EN BIT(1)
+#define LANEENABLE_L3EN BIT(2)
+
+/* Display Parallel Input Interface */
#define DPIPXLFMT 0x0440
#define VS_POL_ACTIVE_LOW (1 << 10)
#define HS_POL_ACTIVE_LOW (1 << 9)
@@ -48,6 +83,14 @@
#define DPI_BPP_RGB666 (1 << 0)
#define DPI_BPP_RGB565 (2 << 0)
+/* Display Parallel Output Interface */
+#define POCTRL 0x0448
+#define POCTRL_S2P BIT(7)
+#define POCTRL_PCLK_POL BIT(3)
+#define POCTRL_VS_POL BIT(2)
+#define POCTRL_HS_POL BIT(1)
+#define POCTRL_DE_POL BIT(0)
+
/* Video Path */
#define VPCTRL0 0x0450
#define VSDELAY GENMASK(31, 20)
@@ -247,6 +290,9 @@ struct tc_data {
struct drm_bridge *panel_bridge;
struct drm_connector connector;
+ struct mipi_dsi_device *dsi;
+ u8 dsi_lanes;
+
/* link settings */
struct tc_edp_link link;
@@ -469,10 +515,24 @@ static int tc_pxl_pll_en(struct tc_data *tc, u32 refclk, u32 pixelclock)
int mul, best_mul = 1;
int delta, best_delta;
int ext_div[] = {1, 2, 3, 5, 7};
+ int clk_min, clk_max;
int best_pixelclock = 0;
int vco_hi = 0;
u32 pxl_pllparam;
+ /*
+ * refclk * mul / (ext_pre_div * pre_div) should be in range:
+ * - DPI ..... 0 to 100 MHz
+ * - (e)DP ... 150 to 650 MHz
+ */
+ if (tc->bridge.type == DRM_MODE_CONNECTOR_DPI) {
+ clk_min = 0;
+ clk_max = 100000000;
+ } else {
+ clk_min = 150000000;
+ clk_max = 650000000;
+ }
+
dev_dbg(tc->dev, "PLL: requested %d pixelclock, ref %d\n", pixelclock,
refclk);
best_delta = pixelclock;
@@ -499,11 +559,7 @@ static int tc_pxl_pll_en(struct tc_data *tc, u32 refclk, u32 pixelclock)
continue;
clk = (refclk / ext_div[i_pre] / div) * mul;
- /*
- * refclk * mul / (ext_pre_div * pre_div)
- * should be in the 150 to 650 MHz range
- */
- if ((clk > 650000000) || (clk < 150000000))
+ if ((clk > clk_max) || (clk < clk_min))
continue;
clk = clk / ext_div[i_post];
@@ -820,6 +876,20 @@ static int tc_set_common_video_mode(struct tc_data *tc,
return ret;
}
+static int tc_set_dpi_video_mode(struct tc_data *tc,
+ const struct drm_display_mode *mode)
+{
+ u32 value = POCTRL_S2P;
+
+ if (tc->mode.flags & DRM_MODE_FLAG_NHSYNC)
+ value |= POCTRL_HS_POL;
+
+ if (tc->mode.flags & DRM_MODE_FLAG_NVSYNC)
+ value |= POCTRL_VS_POL;
+
+ return regmap_write(tc->regmap, POCTRL, value);
+}
+
static int tc_set_edp_video_mode(struct tc_data *tc,
const struct drm_display_mode *mode)
{
@@ -1192,6 +1262,85 @@ static int tc_main_link_disable(struct tc_data *tc)
return regmap_write(tc->regmap, DP0CTL, 0);
}
+static int tc_dpi_stream_enable(struct tc_data *tc)
+{
+ int ret;
+ u32 value;
+
+ dev_dbg(tc->dev, "enable video stream\n");
+
+ /* Setup PLL */
+ ret = tc_set_syspllparam(tc);
+ if (ret)
+ return ret;
+
+ /*
+ * Initially PLLs are in bypass. Force PLL parameter update,
+ * disable PLL bypass, enable PLL
+ */
+ ret = tc_pllupdate(tc, DP0_PLLCTRL);
+ if (ret)
+ return ret;
+
+ ret = tc_pllupdate(tc, DP1_PLLCTRL);
+ if (ret)
+ return ret;
+
+ /* Pixel PLL must always be enabled for DPI mode */
+ ret = tc_pxl_pll_en(tc, clk_get_rate(tc->refclk),
+ 1000 * tc->mode.clock);
+ if (ret)
+ return ret;
+
+ regmap_write(tc->regmap, PPI_D0S_CLRSIPOCOUNT, 3);
+ regmap_write(tc->regmap, PPI_D1S_CLRSIPOCOUNT, 3);
+ regmap_write(tc->regmap, PPI_D2S_CLRSIPOCOUNT, 3);
+ regmap_write(tc->regmap, PPI_D3S_CLRSIPOCOUNT, 3);
+ regmap_write(tc->regmap, PPI_D0S_ATMR, 0);
+ regmap_write(tc->regmap, PPI_D1S_ATMR, 0);
+ regmap_write(tc->regmap, PPI_TX_RX_TA, TTA_GET | TTA_SURE);
+ regmap_write(tc->regmap, PPI_LPTXTIMECNT, LPX_PERIOD);
+
+ value = ((LANEENABLE_L0EN << tc->dsi_lanes) - LANEENABLE_L0EN) |
+ LANEENABLE_CLEN;
+ regmap_write(tc->regmap, PPI_LANEENABLE, value);
+ regmap_write(tc->regmap, DSI_LANEENABLE, value);
+
+ ret = tc_set_common_video_mode(tc, &tc->mode);
+ if (ret)
+ return ret;
+
+ ret = tc_set_dpi_video_mode(tc, &tc->mode);
+ if (ret)
+ return ret;
+
+ /* Set input interface */
+ value = DP0_AUDSRC_NO_INPUT;
+ if (tc_test_pattern)
+ value |= DP0_VIDSRC_COLOR_BAR;
+ else
+ value |= DP0_VIDSRC_DSI_RX;
+ ret = regmap_write(tc->regmap, SYSCTRL, value);
+ if (ret)
+ return ret;
+
+ usleep_range(120, 150);
+
+ regmap_write(tc->regmap, PPI_STARTPPI, PPI_START_FUNCTION);
+ regmap_write(tc->regmap, DSI_STARTDSI, DSI_RX_START);
+
+ return 0;
+}
+
+static int tc_dpi_stream_disable(struct tc_data *tc)
+{
+ dev_dbg(tc->dev, "disable video stream\n");
+
+ tc_pxl_pll_dis(tc);
+
+ return 0;
+}
+
static int tc_edp_stream_enable(struct tc_data *tc)
{
int ret;
@@ -1266,6 +1415,34 @@ static int tc_edp_stream_disable(struct tc_data *tc)
return 0;
}
+static void
+tc_dpi_bridge_atomic_enable(struct drm_bridge *bridge,
+ struct drm_bridge_state *old_bridge_state)
+
+{
+ struct tc_data *tc = bridge_to_tc(bridge);
+ int ret;
+
+ ret = tc_dpi_stream_enable(tc);
+ if (ret < 0) {
+ dev_err(tc->dev, "main link stream start error: %d\n", ret);
+ tc_main_link_disable(tc);
+ return;
+ }
+}
+
+static void
+tc_dpi_bridge_atomic_disable(struct drm_bridge *bridge,
+ struct drm_bridge_state *old_bridge_state)
+{
+ struct tc_data *tc = bridge_to_tc(bridge);
+ int ret;
+
+ ret = tc_dpi_stream_disable(tc);
+ if (ret < 0)
+ dev_err(tc->dev, "main link stream stop error: %d\n", ret);
+}
+
static void
tc_edp_bridge_atomic_enable(struct drm_bridge *bridge,
struct drm_bridge_state *old_bridge_state)
@@ -1336,6 +1513,16 @@ static int tc_common_atomic_check(struct drm_bridge *bridge,
return 0;
}
+static int tc_dpi_atomic_check(struct drm_bridge *bridge,
+ struct drm_bridge_state *bridge_state,
+ struct drm_crtc_state *crtc_state,
+ struct drm_connector_state *conn_state)
+{
+ /* DSI->DPI interface clock limitation: upto 100 MHz */
+ return tc_common_atomic_check(bridge, bridge_state, crtc_state,
+ conn_state, 100000);
+}
+
static int tc_edp_atomic_check(struct drm_bridge *bridge,
struct drm_bridge_state *bridge_state,
struct drm_crtc_state *crtc_state,
@@ -1346,6 +1533,18 @@ static int tc_edp_atomic_check(struct drm_bridge *bridge,
conn_state, 154000);
}
+static enum drm_mode_status
+tc_dpi_mode_valid(struct drm_bridge *bridge,
+ const struct drm_display_info *info,
+ const struct drm_display_mode *mode)
+{
+ /* DPI interface clock limitation: upto 100 MHz */
+ if (mode->clock > 100000)
+ return MODE_CLOCK_HIGH;
+
+ return MODE_OK;
+}
+
static enum drm_mode_status
tc_edp_mode_valid(struct drm_bridge *bridge,
const struct drm_display_info *info,
@@ -1457,6 +1656,18 @@ static const struct drm_connector_funcs tc_connector_funcs = {
.atomic_destroy_state = drm_atomic_helper_connector_destroy_state,
};
+static int tc_dpi_bridge_attach(struct drm_bridge *bridge,
+ enum drm_bridge_attach_flags flags)
+{
+ struct tc_data *tc = bridge_to_tc(bridge);
+
+ if (!tc->panel_bridge)
+ return 0;
+
+ return drm_bridge_attach(tc->bridge.encoder, tc->panel_bridge,
+ &tc->bridge, flags);
+}
+
static int tc_edp_bridge_attach(struct drm_bridge *bridge,
enum drm_bridge_attach_flags flags)
{
@@ -1515,6 +1726,45 @@ static void tc_edp_bridge_detach(struct drm_bridge *bridge)
drm_dp_aux_unregister(&bridge_to_tc(bridge)->aux);
}
+#define MAX_INPUT_SEL_FORMATS 1
+
+static u32 *
+tc_dpi_atomic_get_input_bus_fmts(struct drm_bridge *bridge,
+ struct drm_bridge_state *bridge_state,
+ struct drm_crtc_state *crtc_state,
+ struct drm_connector_state *conn_state,
+ u32 output_fmt,
+ unsigned int *num_input_fmts)
+{
+ u32 *input_fmts;
+
+ *num_input_fmts = 0;
+
+ input_fmts = kcalloc(MAX_INPUT_SEL_FORMATS, sizeof(*input_fmts),
+ GFP_KERNEL);
+ if (!input_fmts)
+ return NULL;
+
+ /* This is the DSI-end bus format */
+ input_fmts[0] = MEDIA_BUS_FMT_RGB888_1X24;
+ *num_input_fmts = 1;
+
+ return input_fmts;
+}
+
+static const struct drm_bridge_funcs tc_dpi_bridge_funcs = {
+ .attach = tc_dpi_bridge_attach,
+ .mode_valid = tc_dpi_mode_valid,
+ .mode_set = tc_bridge_mode_set,
+ .atomic_check = tc_dpi_atomic_check,
+ .atomic_enable = tc_dpi_bridge_atomic_enable,
+ .atomic_disable = tc_dpi_bridge_atomic_disable,
+ .atomic_duplicate_state = drm_atomic_helper_bridge_duplicate_state,
+ .atomic_destroy_state = drm_atomic_helper_bridge_destroy_state,
+ .atomic_reset = drm_atomic_helper_bridge_reset,
+ .atomic_get_input_bus_fmts = tc_dpi_atomic_get_input_bus_fmts,
+};
+
static const struct drm_bridge_funcs tc_edp_bridge_funcs = {
.attach = tc_edp_bridge_attach,
.detach = tc_edp_bridge_detach,
@@ -1615,6 +1865,82 @@ static irqreturn_t tc_irq_handler(int irq, void *arg)
return IRQ_HANDLED;
}
+static int tc_mipi_dsi_host_attach(struct tc_data *tc)
+{
+ struct device *dev = tc->dev;
+ struct device_node *host_node;
+ struct device_node *endpoint;
+ struct mipi_dsi_device *dsi;
+ struct mipi_dsi_host *host;
+ const struct mipi_dsi_device_info info = {
+ .type = "tc358767",
+ .channel = 0,
+ .node = NULL,
+ };
+ int dsi_lanes, ret;
+
+ endpoint = of_graph_get_endpoint_by_regs(dev->of_node, 0, -1);
+ dsi_lanes = of_property_count_u32_elems(endpoint, "data-lanes");
+ host_node = of_graph_get_remote_port_parent(endpoint);
+ host = of_find_mipi_dsi_host_by_node(host_node);
+ of_node_put(host_node);
+ of_node_put(endpoint);
+
+ if (dsi_lanes < 0 || dsi_lanes > 4)
+ return -EINVAL;
+
+ if (!host)
+ return -EPROBE_DEFER;
+
+ dsi = mipi_dsi_device_register_full(host, &info);
+ if (IS_ERR(dsi))
+ return dev_err_probe(dev, PTR_ERR(dsi),
+ "failed to create dsi device\n");
+
+ tc->dsi = dsi;
+
+ tc->dsi_lanes = dsi_lanes;
+ dsi->lanes = tc->dsi_lanes;
+ dsi->format = MIPI_DSI_FMT_RGB888;
+ dsi->mode_flags = MIPI_DSI_MODE_VIDEO | MIPI_DSI_MODE_VIDEO_SYNC_PULSE;
+
+ ret = mipi_dsi_attach(dsi);
+ if (ret < 0) {
+ dev_err(dev, "failed to attach dsi to host: %d\n", ret);
+ return ret;
+ }
+
+ return 0;
+}
+
+static int tc_probe_dpi_bridge_endpoint(struct tc_data *tc)
+{
+ struct device *dev = tc->dev;
+ struct drm_panel *panel;
+ int ret;
+
+ /* port@1 is the DPI input/output port */
+ ret = drm_of_find_panel_or_bridge(dev->of_node, 1, 0, &panel, NULL);
+ if (ret && ret != -ENODEV)
+ return ret;
+
+ if (panel) {
+ struct drm_bridge *panel_bridge;
+
+ panel_bridge = devm_drm_panel_bridge_add(dev, panel);
+ if (IS_ERR(panel_bridge))
+ return PTR_ERR(panel_bridge);
+
+ tc->panel_bridge = panel_bridge;
+ tc->bridge.type = DRM_MODE_CONNECTOR_DPI;
+ tc->bridge.funcs = &tc_dpi_bridge_funcs;
+
+ return 0;
+ }
+
+ return ret;
+}
+
static int tc_probe_edp_bridge_endpoint(struct tc_data *tc)
{
struct device *dev = tc->dev;
@@ -1682,7 +2008,7 @@ static int tc_probe_bridge_endpoint(struct tc_data *tc)
if (mode == mode_dpi_to_edp)
return tc_probe_edp_bridge_endpoint(tc);
else if (mode == mode_dsi_to_dpi)
- dev_warn(dev, "The mode DSI-to-DPI is not supported!\n");
+ return tc_probe_dpi_bridge_endpoint(tc);
else if (mode == mode_dsi_to_edp)
dev_warn(dev, "The mode DSI-to-(e)DP is not supported!\n");
else
@@ -1813,15 +2139,25 @@ static int tc_probe(struct i2c_client *client, const struct i2c_device_id *id)
}
}
- ret = tc_aux_link_setup(tc);
- if (ret)
- return ret;
+ if (tc->bridge.type != DRM_MODE_CONNECTOR_DPI) { /* (e)DP output */
+ ret = tc_aux_link_setup(tc);
+ if (ret)
+ return ret;
+ }
tc->bridge.of_node = dev->of_node;
drm_bridge_add(&tc->bridge);
i2c_set_clientdata(client, tc);
+ if (tc->bridge.type == DRM_MODE_CONNECTOR_DPI) { /* DPI output */
+ ret = tc_mipi_dsi_host_attach(tc);
+ if (ret) {
+ drm_bridge_remove(&tc->bridge);
+ return ret;
+ }
+ }
+
return 0;
}
--
2.35.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH v4 10/11] drm/bridge: tc358767: Split tc_set_video_mode() into common and (e)DP part
2022-03-28 21:43 ` [PATCH v4 10/11] drm/bridge: tc358767: Split tc_set_video_mode() into common and (e)DP part Marek Vasut
@ 2022-03-29 1:54 ` kernel test robot
2022-03-29 2:05 ` kernel test robot
1 sibling, 0 replies; 19+ messages in thread
From: kernel test robot @ 2022-03-29 1:54 UTC (permalink / raw)
To: Marek Vasut, dri-devel
Cc: Marek Vasut, kbuild-all, Jonas Karlman, Neil Armstrong,
Laurent Pinchart, Sam Ravnborg, Maxime Ripard
Hi Marek,
I love your patch! Perhaps something to improve:
[auto build test WARNING on drm/drm-next]
[also build test WARNING on drm-intel/for-linux-next drm-tip/drm-tip drm-exynos/exynos-drm-next tegra-drm/drm/tegra/for-next v5.17 next-20220328]
[cannot apply to airlied/drm-next]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/intel-lab-lkp/linux/commits/Marek-Vasut/drm-bridge-tc358767-Add-DSI-to-DPI-mode-support/20220329-054627
base: git://anongit.freedesktop.org/drm/drm drm-next
config: xtensa-allyesconfig (https://download.01.org/0day-ci/archive/20220329/202203290927.vwyJHoKX-lkp@intel.com/config)
compiler: xtensa-linux-gcc (GCC) 11.2.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/intel-lab-lkp/linux/commit/294c331d772539772ae87613ae1990860bbbc6ff
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Marek-Vasut/drm-bridge-tc358767-Add-DSI-to-DPI-mode-support/20220329-054627
git checkout 294c331d772539772ae87613ae1990860bbbc6ff
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=xtensa SHELL=/bin/bash drivers/gpu/drm/bridge/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/gpu/drm/bridge/tc358767.c: In function 'tc_set_common_video_mode':
>> drivers/gpu/drm/bridge/tc358767.c:741:13: warning: variable 'max_tu_symbol' set but not used [-Wunused-but-set-variable]
741 | int max_tu_symbol;
| ^~~~~~~~~~~~~
vim +/max_tu_symbol +741 drivers/gpu/drm/bridge/tc358767.c
7caff0fc4296eba Andrey Gusakov 2016-07-13 736
294c331d7725397 Marek Vasut 2022-03-28 737 static int tc_set_common_video_mode(struct tc_data *tc,
63f8f3badf799c8 Laurent Pinchart 2018-04-06 738 const struct drm_display_mode *mode)
7caff0fc4296eba Andrey Gusakov 2016-07-13 739 {
7caff0fc4296eba Andrey Gusakov 2016-07-13 740 int ret;
7caff0fc4296eba Andrey Gusakov 2016-07-13 @741 int max_tu_symbol;
7caff0fc4296eba Andrey Gusakov 2016-07-13 742
7caff0fc4296eba Andrey Gusakov 2016-07-13 743 int left_margin = mode->htotal - mode->hsync_end;
7caff0fc4296eba Andrey Gusakov 2016-07-13 744 int right_margin = mode->hsync_start - mode->hdisplay;
7caff0fc4296eba Andrey Gusakov 2016-07-13 745 int hsync_len = mode->hsync_end - mode->hsync_start;
7caff0fc4296eba Andrey Gusakov 2016-07-13 746 int upper_margin = mode->vtotal - mode->vsync_end;
7caff0fc4296eba Andrey Gusakov 2016-07-13 747 int lower_margin = mode->vsync_start - mode->vdisplay;
7caff0fc4296eba Andrey Gusakov 2016-07-13 748 int vsync_len = mode->vsync_end - mode->vsync_start;
fd70c7755bf0172 Tomi Valkeinen 2019-09-24 749 u32 bits_per_pixel = 24;
fd70c7755bf0172 Tomi Valkeinen 2019-09-24 750 u32 in_bw, out_bw;
7caff0fc4296eba Andrey Gusakov 2016-07-13 751
66d1c3b94d5d59e Andrey Gusakov 2017-11-07 752 /*
66d1c3b94d5d59e Andrey Gusakov 2017-11-07 753 * Recommended maximum number of symbols transferred in a transfer unit:
66d1c3b94d5d59e Andrey Gusakov 2017-11-07 754 * DIV_ROUND_UP((input active video bandwidth in bytes) * tu_size,
66d1c3b94d5d59e Andrey Gusakov 2017-11-07 755 * (output active video bandwidth in bytes))
66d1c3b94d5d59e Andrey Gusakov 2017-11-07 756 * Must be less than tu_size.
66d1c3b94d5d59e Andrey Gusakov 2017-11-07 757 */
fd70c7755bf0172 Tomi Valkeinen 2019-09-24 758
fd70c7755bf0172 Tomi Valkeinen 2019-09-24 759 in_bw = mode->clock * bits_per_pixel / 8;
e7dc8d40eea400d Thierry Reding 2019-10-21 760 out_bw = tc->link.num_lanes * tc->link.rate;
fd70c7755bf0172 Tomi Valkeinen 2019-09-24 761 max_tu_symbol = DIV_ROUND_UP(in_bw * TU_SIZE_RECOMMENDED, out_bw);
66d1c3b94d5d59e Andrey Gusakov 2017-11-07 762
7caff0fc4296eba Andrey Gusakov 2016-07-13 763 dev_dbg(tc->dev, "set mode %dx%d\n",
7caff0fc4296eba Andrey Gusakov 2016-07-13 764 mode->hdisplay, mode->vdisplay);
7caff0fc4296eba Andrey Gusakov 2016-07-13 765 dev_dbg(tc->dev, "H margin %d,%d sync %d\n",
7caff0fc4296eba Andrey Gusakov 2016-07-13 766 left_margin, right_margin, hsync_len);
7caff0fc4296eba Andrey Gusakov 2016-07-13 767 dev_dbg(tc->dev, "V margin %d,%d sync %d\n",
7caff0fc4296eba Andrey Gusakov 2016-07-13 768 upper_margin, lower_margin, vsync_len);
7caff0fc4296eba Andrey Gusakov 2016-07-13 769 dev_dbg(tc->dev, "total: %dx%d\n", mode->htotal, mode->vtotal);
7caff0fc4296eba Andrey Gusakov 2016-07-13 770
7caff0fc4296eba Andrey Gusakov 2016-07-13 771
66d1c3b94d5d59e Andrey Gusakov 2017-11-07 772 /*
66d1c3b94d5d59e Andrey Gusakov 2017-11-07 773 * LCD Ctl Frame Size
66d1c3b94d5d59e Andrey Gusakov 2017-11-07 774 * datasheet is not clear of vsdelay in case of DPI
66d1c3b94d5d59e Andrey Gusakov 2017-11-07 775 * assume we do not need any delay when DPI is a source of
66d1c3b94d5d59e Andrey Gusakov 2017-11-07 776 * sync signals
66d1c3b94d5d59e Andrey Gusakov 2017-11-07 777 */
6d0c38315915d49 Andrey Smirnov 2019-06-18 778 ret = regmap_write(tc->regmap, VPCTRL0,
3f072c304c0a7e1 Andrey Smirnov 2019-06-18 779 FIELD_PREP(VSDELAY, 0) |
7caff0fc4296eba Andrey Gusakov 2016-07-13 780 OPXLFMT_RGB888 | FRMSYNC_DISABLED | MSF_DISABLED);
6d0c38315915d49 Andrey Smirnov 2019-06-18 781 if (ret)
6d0c38315915d49 Andrey Smirnov 2019-06-18 782 return ret;
6d0c38315915d49 Andrey Smirnov 2019-06-18 783
6d0c38315915d49 Andrey Smirnov 2019-06-18 784 ret = regmap_write(tc->regmap, HTIM01,
3f072c304c0a7e1 Andrey Smirnov 2019-06-18 785 FIELD_PREP(HBPR, ALIGN(left_margin, 2)) |
3f072c304c0a7e1 Andrey Smirnov 2019-06-18 786 FIELD_PREP(HPW, ALIGN(hsync_len, 2)));
6d0c38315915d49 Andrey Smirnov 2019-06-18 787 if (ret)
6d0c38315915d49 Andrey Smirnov 2019-06-18 788 return ret;
6d0c38315915d49 Andrey Smirnov 2019-06-18 789
6d0c38315915d49 Andrey Smirnov 2019-06-18 790 ret = regmap_write(tc->regmap, HTIM02,
3f072c304c0a7e1 Andrey Smirnov 2019-06-18 791 FIELD_PREP(HDISPR, ALIGN(mode->hdisplay, 2)) |
3f072c304c0a7e1 Andrey Smirnov 2019-06-18 792 FIELD_PREP(HFPR, ALIGN(right_margin, 2)));
6d0c38315915d49 Andrey Smirnov 2019-06-18 793 if (ret)
6d0c38315915d49 Andrey Smirnov 2019-06-18 794 return ret;
6d0c38315915d49 Andrey Smirnov 2019-06-18 795
6d0c38315915d49 Andrey Smirnov 2019-06-18 796 ret = regmap_write(tc->regmap, VTIM01,
3f072c304c0a7e1 Andrey Smirnov 2019-06-18 797 FIELD_PREP(VBPR, upper_margin) |
3f072c304c0a7e1 Andrey Smirnov 2019-06-18 798 FIELD_PREP(VSPR, vsync_len));
6d0c38315915d49 Andrey Smirnov 2019-06-18 799 if (ret)
6d0c38315915d49 Andrey Smirnov 2019-06-18 800 return ret;
6d0c38315915d49 Andrey Smirnov 2019-06-18 801
6d0c38315915d49 Andrey Smirnov 2019-06-18 802 ret = regmap_write(tc->regmap, VTIM02,
3f072c304c0a7e1 Andrey Smirnov 2019-06-18 803 FIELD_PREP(VFPR, lower_margin) |
3f072c304c0a7e1 Andrey Smirnov 2019-06-18 804 FIELD_PREP(VDISPR, mode->vdisplay));
6d0c38315915d49 Andrey Smirnov 2019-06-18 805 if (ret)
6d0c38315915d49 Andrey Smirnov 2019-06-18 806 return ret;
6d0c38315915d49 Andrey Smirnov 2019-06-18 807
6d0c38315915d49 Andrey Smirnov 2019-06-18 808 ret = regmap_write(tc->regmap, VFUEN0, VFUEN); /* update settings */
6d0c38315915d49 Andrey Smirnov 2019-06-18 809 if (ret)
6d0c38315915d49 Andrey Smirnov 2019-06-18 810 return ret;
7caff0fc4296eba Andrey Gusakov 2016-07-13 811
7caff0fc4296eba Andrey Gusakov 2016-07-13 812 /* Test pattern settings */
6d0c38315915d49 Andrey Smirnov 2019-06-18 813 ret = regmap_write(tc->regmap, TSTCTL,
3f072c304c0a7e1 Andrey Smirnov 2019-06-18 814 FIELD_PREP(COLOR_R, 120) |
3f072c304c0a7e1 Andrey Smirnov 2019-06-18 815 FIELD_PREP(COLOR_G, 20) |
3f072c304c0a7e1 Andrey Smirnov 2019-06-18 816 FIELD_PREP(COLOR_B, 99) |
3f072c304c0a7e1 Andrey Smirnov 2019-06-18 817 ENI2CFILTER |
3f072c304c0a7e1 Andrey Smirnov 2019-06-18 818 FIELD_PREP(COLOR_BAR_MODE, COLOR_BAR_MODE_BARS));
294c331d7725397 Marek Vasut 2022-03-28 819
6d0c38315915d49 Andrey Smirnov 2019-06-18 820 return ret;
294c331d7725397 Marek Vasut 2022-03-28 821 }
294c331d7725397 Marek Vasut 2022-03-28 822
--
0-DAY CI Kernel Test Service
https://01.org/lkp
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v4 10/11] drm/bridge: tc358767: Split tc_set_video_mode() into common and (e)DP part
2022-03-28 21:43 ` [PATCH v4 10/11] drm/bridge: tc358767: Split tc_set_video_mode() into common and (e)DP part Marek Vasut
@ 2022-03-29 2:05 ` kernel test robot
2022-03-29 2:05 ` kernel test robot
1 sibling, 0 replies; 19+ messages in thread
From: kernel test robot @ 2022-03-29 2:05 UTC (permalink / raw)
To: Marek Vasut, dri-devel
Cc: Marek Vasut, kbuild-all, Jonas Karlman, llvm, Neil Armstrong,
Laurent Pinchart, Sam Ravnborg, Maxime Ripard
Hi Marek,
I love your patch! Perhaps something to improve:
[auto build test WARNING on drm/drm-next]
[also build test WARNING on drm-intel/for-linux-next drm-tip/drm-tip drm-exynos/exynos-drm-next tegra-drm/drm/tegra/for-next v5.17 next-20220328]
[cannot apply to airlied/drm-next]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/intel-lab-lkp/linux/commits/Marek-Vasut/drm-bridge-tc358767-Add-DSI-to-DPI-mode-support/20220329-054627
base: git://anongit.freedesktop.org/drm/drm drm-next
config: hexagon-randconfig-r045-20220327 (https://download.01.org/0day-ci/archive/20220329/202203290924.MYL8dU7I-lkp@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project 0f6d9501cf49ce02937099350d08f20c4af86f3d)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/intel-lab-lkp/linux/commit/294c331d772539772ae87613ae1990860bbbc6ff
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Marek-Vasut/drm-bridge-tc358767-Add-DSI-to-DPI-mode-support/20220329-054627
git checkout 294c331d772539772ae87613ae1990860bbbc6ff
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash drivers/gpu/drm/bridge/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
>> drivers/gpu/drm/bridge/tc358767.c:741:6: warning: variable 'max_tu_symbol' set but not used [-Wunused-but-set-variable]
int max_tu_symbol;
^
1 warning generated.
vim +/max_tu_symbol +741 drivers/gpu/drm/bridge/tc358767.c
7caff0fc4296eb Andrey Gusakov 2016-07-13 736
294c331d772539 Marek Vasut 2022-03-28 737 static int tc_set_common_video_mode(struct tc_data *tc,
63f8f3badf799c Laurent Pinchart 2018-04-06 738 const struct drm_display_mode *mode)
7caff0fc4296eb Andrey Gusakov 2016-07-13 739 {
7caff0fc4296eb Andrey Gusakov 2016-07-13 740 int ret;
7caff0fc4296eb Andrey Gusakov 2016-07-13 @741 int max_tu_symbol;
7caff0fc4296eb Andrey Gusakov 2016-07-13 742
7caff0fc4296eb Andrey Gusakov 2016-07-13 743 int left_margin = mode->htotal - mode->hsync_end;
7caff0fc4296eb Andrey Gusakov 2016-07-13 744 int right_margin = mode->hsync_start - mode->hdisplay;
7caff0fc4296eb Andrey Gusakov 2016-07-13 745 int hsync_len = mode->hsync_end - mode->hsync_start;
7caff0fc4296eb Andrey Gusakov 2016-07-13 746 int upper_margin = mode->vtotal - mode->vsync_end;
7caff0fc4296eb Andrey Gusakov 2016-07-13 747 int lower_margin = mode->vsync_start - mode->vdisplay;
7caff0fc4296eb Andrey Gusakov 2016-07-13 748 int vsync_len = mode->vsync_end - mode->vsync_start;
fd70c7755bf017 Tomi Valkeinen 2019-09-24 749 u32 bits_per_pixel = 24;
fd70c7755bf017 Tomi Valkeinen 2019-09-24 750 u32 in_bw, out_bw;
7caff0fc4296eb Andrey Gusakov 2016-07-13 751
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 752 /*
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 753 * Recommended maximum number of symbols transferred in a transfer unit:
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 754 * DIV_ROUND_UP((input active video bandwidth in bytes) * tu_size,
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 755 * (output active video bandwidth in bytes))
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 756 * Must be less than tu_size.
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 757 */
fd70c7755bf017 Tomi Valkeinen 2019-09-24 758
fd70c7755bf017 Tomi Valkeinen 2019-09-24 759 in_bw = mode->clock * bits_per_pixel / 8;
e7dc8d40eea400 Thierry Reding 2019-10-21 760 out_bw = tc->link.num_lanes * tc->link.rate;
fd70c7755bf017 Tomi Valkeinen 2019-09-24 761 max_tu_symbol = DIV_ROUND_UP(in_bw * TU_SIZE_RECOMMENDED, out_bw);
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 762
7caff0fc4296eb Andrey Gusakov 2016-07-13 763 dev_dbg(tc->dev, "set mode %dx%d\n",
7caff0fc4296eb Andrey Gusakov 2016-07-13 764 mode->hdisplay, mode->vdisplay);
7caff0fc4296eb Andrey Gusakov 2016-07-13 765 dev_dbg(tc->dev, "H margin %d,%d sync %d\n",
7caff0fc4296eb Andrey Gusakov 2016-07-13 766 left_margin, right_margin, hsync_len);
7caff0fc4296eb Andrey Gusakov 2016-07-13 767 dev_dbg(tc->dev, "V margin %d,%d sync %d\n",
7caff0fc4296eb Andrey Gusakov 2016-07-13 768 upper_margin, lower_margin, vsync_len);
7caff0fc4296eb Andrey Gusakov 2016-07-13 769 dev_dbg(tc->dev, "total: %dx%d\n", mode->htotal, mode->vtotal);
7caff0fc4296eb Andrey Gusakov 2016-07-13 770
7caff0fc4296eb Andrey Gusakov 2016-07-13 771
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 772 /*
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 773 * LCD Ctl Frame Size
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 774 * datasheet is not clear of vsdelay in case of DPI
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 775 * assume we do not need any delay when DPI is a source of
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 776 * sync signals
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 777 */
6d0c38315915d4 Andrey Smirnov 2019-06-18 778 ret = regmap_write(tc->regmap, VPCTRL0,
3f072c304c0a7e Andrey Smirnov 2019-06-18 779 FIELD_PREP(VSDELAY, 0) |
7caff0fc4296eb Andrey Gusakov 2016-07-13 780 OPXLFMT_RGB888 | FRMSYNC_DISABLED | MSF_DISABLED);
6d0c38315915d4 Andrey Smirnov 2019-06-18 781 if (ret)
6d0c38315915d4 Andrey Smirnov 2019-06-18 782 return ret;
6d0c38315915d4 Andrey Smirnov 2019-06-18 783
6d0c38315915d4 Andrey Smirnov 2019-06-18 784 ret = regmap_write(tc->regmap, HTIM01,
3f072c304c0a7e Andrey Smirnov 2019-06-18 785 FIELD_PREP(HBPR, ALIGN(left_margin, 2)) |
3f072c304c0a7e Andrey Smirnov 2019-06-18 786 FIELD_PREP(HPW, ALIGN(hsync_len, 2)));
6d0c38315915d4 Andrey Smirnov 2019-06-18 787 if (ret)
6d0c38315915d4 Andrey Smirnov 2019-06-18 788 return ret;
6d0c38315915d4 Andrey Smirnov 2019-06-18 789
6d0c38315915d4 Andrey Smirnov 2019-06-18 790 ret = regmap_write(tc->regmap, HTIM02,
3f072c304c0a7e Andrey Smirnov 2019-06-18 791 FIELD_PREP(HDISPR, ALIGN(mode->hdisplay, 2)) |
3f072c304c0a7e Andrey Smirnov 2019-06-18 792 FIELD_PREP(HFPR, ALIGN(right_margin, 2)));
6d0c38315915d4 Andrey Smirnov 2019-06-18 793 if (ret)
6d0c38315915d4 Andrey Smirnov 2019-06-18 794 return ret;
6d0c38315915d4 Andrey Smirnov 2019-06-18 795
6d0c38315915d4 Andrey Smirnov 2019-06-18 796 ret = regmap_write(tc->regmap, VTIM01,
3f072c304c0a7e Andrey Smirnov 2019-06-18 797 FIELD_PREP(VBPR, upper_margin) |
3f072c304c0a7e Andrey Smirnov 2019-06-18 798 FIELD_PREP(VSPR, vsync_len));
6d0c38315915d4 Andrey Smirnov 2019-06-18 799 if (ret)
6d0c38315915d4 Andrey Smirnov 2019-06-18 800 return ret;
6d0c38315915d4 Andrey Smirnov 2019-06-18 801
6d0c38315915d4 Andrey Smirnov 2019-06-18 802 ret = regmap_write(tc->regmap, VTIM02,
3f072c304c0a7e Andrey Smirnov 2019-06-18 803 FIELD_PREP(VFPR, lower_margin) |
3f072c304c0a7e Andrey Smirnov 2019-06-18 804 FIELD_PREP(VDISPR, mode->vdisplay));
6d0c38315915d4 Andrey Smirnov 2019-06-18 805 if (ret)
6d0c38315915d4 Andrey Smirnov 2019-06-18 806 return ret;
6d0c38315915d4 Andrey Smirnov 2019-06-18 807
6d0c38315915d4 Andrey Smirnov 2019-06-18 808 ret = regmap_write(tc->regmap, VFUEN0, VFUEN); /* update settings */
6d0c38315915d4 Andrey Smirnov 2019-06-18 809 if (ret)
6d0c38315915d4 Andrey Smirnov 2019-06-18 810 return ret;
7caff0fc4296eb Andrey Gusakov 2016-07-13 811
7caff0fc4296eb Andrey Gusakov 2016-07-13 812 /* Test pattern settings */
6d0c38315915d4 Andrey Smirnov 2019-06-18 813 ret = regmap_write(tc->regmap, TSTCTL,
3f072c304c0a7e Andrey Smirnov 2019-06-18 814 FIELD_PREP(COLOR_R, 120) |
3f072c304c0a7e Andrey Smirnov 2019-06-18 815 FIELD_PREP(COLOR_G, 20) |
3f072c304c0a7e Andrey Smirnov 2019-06-18 816 FIELD_PREP(COLOR_B, 99) |
3f072c304c0a7e Andrey Smirnov 2019-06-18 817 ENI2CFILTER |
3f072c304c0a7e Andrey Smirnov 2019-06-18 818 FIELD_PREP(COLOR_BAR_MODE, COLOR_BAR_MODE_BARS));
294c331d772539 Marek Vasut 2022-03-28 819
6d0c38315915d4 Andrey Smirnov 2019-06-18 820 return ret;
294c331d772539 Marek Vasut 2022-03-28 821 }
294c331d772539 Marek Vasut 2022-03-28 822
--
0-DAY CI Kernel Test Service
https://01.org/lkp
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v4 10/11] drm/bridge: tc358767: Split tc_set_video_mode() into common and (e)DP part
@ 2022-03-29 2:05 ` kernel test robot
0 siblings, 0 replies; 19+ messages in thread
From: kernel test robot @ 2022-03-29 2:05 UTC (permalink / raw)
To: Marek Vasut, dri-devel
Cc: llvm, kbuild-all, Marek Vasut, Neil Armstrong, Jonas Karlman,
Laurent Pinchart, Sam Ravnborg, Maxime Ripard
Hi Marek,
I love your patch! Perhaps something to improve:
[auto build test WARNING on drm/drm-next]
[also build test WARNING on drm-intel/for-linux-next drm-tip/drm-tip drm-exynos/exynos-drm-next tegra-drm/drm/tegra/for-next v5.17 next-20220328]
[cannot apply to airlied/drm-next]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/intel-lab-lkp/linux/commits/Marek-Vasut/drm-bridge-tc358767-Add-DSI-to-DPI-mode-support/20220329-054627
base: git://anongit.freedesktop.org/drm/drm drm-next
config: hexagon-randconfig-r045-20220327 (https://download.01.org/0day-ci/archive/20220329/202203290924.MYL8dU7I-lkp@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project 0f6d9501cf49ce02937099350d08f20c4af86f3d)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/intel-lab-lkp/linux/commit/294c331d772539772ae87613ae1990860bbbc6ff
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Marek-Vasut/drm-bridge-tc358767-Add-DSI-to-DPI-mode-support/20220329-054627
git checkout 294c331d772539772ae87613ae1990860bbbc6ff
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash drivers/gpu/drm/bridge/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
>> drivers/gpu/drm/bridge/tc358767.c:741:6: warning: variable 'max_tu_symbol' set but not used [-Wunused-but-set-variable]
int max_tu_symbol;
^
1 warning generated.
vim +/max_tu_symbol +741 drivers/gpu/drm/bridge/tc358767.c
7caff0fc4296eb Andrey Gusakov 2016-07-13 736
294c331d772539 Marek Vasut 2022-03-28 737 static int tc_set_common_video_mode(struct tc_data *tc,
63f8f3badf799c Laurent Pinchart 2018-04-06 738 const struct drm_display_mode *mode)
7caff0fc4296eb Andrey Gusakov 2016-07-13 739 {
7caff0fc4296eb Andrey Gusakov 2016-07-13 740 int ret;
7caff0fc4296eb Andrey Gusakov 2016-07-13 @741 int max_tu_symbol;
7caff0fc4296eb Andrey Gusakov 2016-07-13 742
7caff0fc4296eb Andrey Gusakov 2016-07-13 743 int left_margin = mode->htotal - mode->hsync_end;
7caff0fc4296eb Andrey Gusakov 2016-07-13 744 int right_margin = mode->hsync_start - mode->hdisplay;
7caff0fc4296eb Andrey Gusakov 2016-07-13 745 int hsync_len = mode->hsync_end - mode->hsync_start;
7caff0fc4296eb Andrey Gusakov 2016-07-13 746 int upper_margin = mode->vtotal - mode->vsync_end;
7caff0fc4296eb Andrey Gusakov 2016-07-13 747 int lower_margin = mode->vsync_start - mode->vdisplay;
7caff0fc4296eb Andrey Gusakov 2016-07-13 748 int vsync_len = mode->vsync_end - mode->vsync_start;
fd70c7755bf017 Tomi Valkeinen 2019-09-24 749 u32 bits_per_pixel = 24;
fd70c7755bf017 Tomi Valkeinen 2019-09-24 750 u32 in_bw, out_bw;
7caff0fc4296eb Andrey Gusakov 2016-07-13 751
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 752 /*
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 753 * Recommended maximum number of symbols transferred in a transfer unit:
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 754 * DIV_ROUND_UP((input active video bandwidth in bytes) * tu_size,
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 755 * (output active video bandwidth in bytes))
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 756 * Must be less than tu_size.
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 757 */
fd70c7755bf017 Tomi Valkeinen 2019-09-24 758
fd70c7755bf017 Tomi Valkeinen 2019-09-24 759 in_bw = mode->clock * bits_per_pixel / 8;
e7dc8d40eea400 Thierry Reding 2019-10-21 760 out_bw = tc->link.num_lanes * tc->link.rate;
fd70c7755bf017 Tomi Valkeinen 2019-09-24 761 max_tu_symbol = DIV_ROUND_UP(in_bw * TU_SIZE_RECOMMENDED, out_bw);
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 762
7caff0fc4296eb Andrey Gusakov 2016-07-13 763 dev_dbg(tc->dev, "set mode %dx%d\n",
7caff0fc4296eb Andrey Gusakov 2016-07-13 764 mode->hdisplay, mode->vdisplay);
7caff0fc4296eb Andrey Gusakov 2016-07-13 765 dev_dbg(tc->dev, "H margin %d,%d sync %d\n",
7caff0fc4296eb Andrey Gusakov 2016-07-13 766 left_margin, right_margin, hsync_len);
7caff0fc4296eb Andrey Gusakov 2016-07-13 767 dev_dbg(tc->dev, "V margin %d,%d sync %d\n",
7caff0fc4296eb Andrey Gusakov 2016-07-13 768 upper_margin, lower_margin, vsync_len);
7caff0fc4296eb Andrey Gusakov 2016-07-13 769 dev_dbg(tc->dev, "total: %dx%d\n", mode->htotal, mode->vtotal);
7caff0fc4296eb Andrey Gusakov 2016-07-13 770
7caff0fc4296eb Andrey Gusakov 2016-07-13 771
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 772 /*
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 773 * LCD Ctl Frame Size
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 774 * datasheet is not clear of vsdelay in case of DPI
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 775 * assume we do not need any delay when DPI is a source of
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 776 * sync signals
66d1c3b94d5d59 Andrey Gusakov 2017-11-07 777 */
6d0c38315915d4 Andrey Smirnov 2019-06-18 778 ret = regmap_write(tc->regmap, VPCTRL0,
3f072c304c0a7e Andrey Smirnov 2019-06-18 779 FIELD_PREP(VSDELAY, 0) |
7caff0fc4296eb Andrey Gusakov 2016-07-13 780 OPXLFMT_RGB888 | FRMSYNC_DISABLED | MSF_DISABLED);
6d0c38315915d4 Andrey Smirnov 2019-06-18 781 if (ret)
6d0c38315915d4 Andrey Smirnov 2019-06-18 782 return ret;
6d0c38315915d4 Andrey Smirnov 2019-06-18 783
6d0c38315915d4 Andrey Smirnov 2019-06-18 784 ret = regmap_write(tc->regmap, HTIM01,
3f072c304c0a7e Andrey Smirnov 2019-06-18 785 FIELD_PREP(HBPR, ALIGN(left_margin, 2)) |
3f072c304c0a7e Andrey Smirnov 2019-06-18 786 FIELD_PREP(HPW, ALIGN(hsync_len, 2)));
6d0c38315915d4 Andrey Smirnov 2019-06-18 787 if (ret)
6d0c38315915d4 Andrey Smirnov 2019-06-18 788 return ret;
6d0c38315915d4 Andrey Smirnov 2019-06-18 789
6d0c38315915d4 Andrey Smirnov 2019-06-18 790 ret = regmap_write(tc->regmap, HTIM02,
3f072c304c0a7e Andrey Smirnov 2019-06-18 791 FIELD_PREP(HDISPR, ALIGN(mode->hdisplay, 2)) |
3f072c304c0a7e Andrey Smirnov 2019-06-18 792 FIELD_PREP(HFPR, ALIGN(right_margin, 2)));
6d0c38315915d4 Andrey Smirnov 2019-06-18 793 if (ret)
6d0c38315915d4 Andrey Smirnov 2019-06-18 794 return ret;
6d0c38315915d4 Andrey Smirnov 2019-06-18 795
6d0c38315915d4 Andrey Smirnov 2019-06-18 796 ret = regmap_write(tc->regmap, VTIM01,
3f072c304c0a7e Andrey Smirnov 2019-06-18 797 FIELD_PREP(VBPR, upper_margin) |
3f072c304c0a7e Andrey Smirnov 2019-06-18 798 FIELD_PREP(VSPR, vsync_len));
6d0c38315915d4 Andrey Smirnov 2019-06-18 799 if (ret)
6d0c38315915d4 Andrey Smirnov 2019-06-18 800 return ret;
6d0c38315915d4 Andrey Smirnov 2019-06-18 801
6d0c38315915d4 Andrey Smirnov 2019-06-18 802 ret = regmap_write(tc->regmap, VTIM02,
3f072c304c0a7e Andrey Smirnov 2019-06-18 803 FIELD_PREP(VFPR, lower_margin) |
3f072c304c0a7e Andrey Smirnov 2019-06-18 804 FIELD_PREP(VDISPR, mode->vdisplay));
6d0c38315915d4 Andrey Smirnov 2019-06-18 805 if (ret)
6d0c38315915d4 Andrey Smirnov 2019-06-18 806 return ret;
6d0c38315915d4 Andrey Smirnov 2019-06-18 807
6d0c38315915d4 Andrey Smirnov 2019-06-18 808 ret = regmap_write(tc->regmap, VFUEN0, VFUEN); /* update settings */
6d0c38315915d4 Andrey Smirnov 2019-06-18 809 if (ret)
6d0c38315915d4 Andrey Smirnov 2019-06-18 810 return ret;
7caff0fc4296eb Andrey Gusakov 2016-07-13 811
7caff0fc4296eb Andrey Gusakov 2016-07-13 812 /* Test pattern settings */
6d0c38315915d4 Andrey Smirnov 2019-06-18 813 ret = regmap_write(tc->regmap, TSTCTL,
3f072c304c0a7e Andrey Smirnov 2019-06-18 814 FIELD_PREP(COLOR_R, 120) |
3f072c304c0a7e Andrey Smirnov 2019-06-18 815 FIELD_PREP(COLOR_G, 20) |
3f072c304c0a7e Andrey Smirnov 2019-06-18 816 FIELD_PREP(COLOR_B, 99) |
3f072c304c0a7e Andrey Smirnov 2019-06-18 817 ENI2CFILTER |
3f072c304c0a7e Andrey Smirnov 2019-06-18 818 FIELD_PREP(COLOR_BAR_MODE, COLOR_BAR_MODE_BARS));
294c331d772539 Marek Vasut 2022-03-28 819
6d0c38315915d4 Andrey Smirnov 2019-06-18 820 return ret;
294c331d772539 Marek Vasut 2022-03-28 821 }
294c331d772539 Marek Vasut 2022-03-28 822
--
0-DAY CI Kernel Test Service
https://01.org/lkp
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v4 00/11] drm/bridge: tc358767: Add DSI-to-DPI mode support
2022-03-28 21:43 [PATCH v4 00/11] drm/bridge: tc358767: Add DSI-to-DPI mode support Marek Vasut
` (10 preceding siblings ...)
2022-03-28 21:43 ` [PATCH v4 11/11] drm/bridge: tc358767: Add DSI-to-DPI mode support Marek Vasut
@ 2022-03-29 7:47 ` Lucas Stach
2022-03-29 7:58 ` Marek Vasut
11 siblings, 1 reply; 19+ messages in thread
From: Lucas Stach @ 2022-03-29 7:47 UTC (permalink / raw)
To: Marek Vasut, dri-devel
Cc: Neil Armstrong, Sam Ravnborg, Laurent Pinchart, Maxime Ripard,
Jonas Karlman
Hi Marek,
Am Montag, dem 28.03.2022 um 23:43 +0200 schrieb Marek Vasut:
> The TC358767/TC358867/TC9595 are all capable of operating in multiple
> modes, DPI-to-(e)DP, DSI-to-(e)DP, DSI-to-DPI. Clean up the driver,
> switch to atomic ops, and add support for the DSI-to-DPI mode in
> addition to already supported DPI-to-(e)DP mode.
>
Not sure if anyone else has any more comments. For what it is worth, I
just skimmed through the series again and it looks good to me now.
Regards,
Lucas
> Cc: Jonas Karlman <jonas@kwiboo.se>
> Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
> Cc: Maxime Ripard <maxime@cerno.tech>
> Cc: Neil Armstrong <narmstrong@baylibre.com>
> Cc: Sam Ravnborg <sam@ravnborg.org>
> To: dri-devel@lists.freedesktop.org
>
> Marek Vasut (11):
> dt-bindings: display: bridge: tc358867: Document DPI output support
> dt-bindings: display: bridge: tc358867: Document DSI data-lanes
> property
> drm/bridge: tc358767: Change tc_ prefix to tc_edp_ for (e)DP specific
> functions
> drm/bridge: tc358767: Convert to atomic ops
> drm/bridge: tc358767: Implement atomic_check callback
> drm/bridge: tc358767: Move (e)DP bridge endpoint parsing into
> dedicated function
> drm/bridge: tc358767: Wrap (e)DP aux I2C registration into
> tc_aux_link_setup()
> drm/bridge: tc358767: Move bridge ops setup into
> tc_probe_edp_bridge_endpoint()
> drm/bridge: tc358767: Detect bridge mode from connected endpoints in
> DT
> drm/bridge: tc358767: Split tc_set_video_mode() into common and (e)DP
> part
> drm/bridge: tc358767: Add DSI-to-DPI mode support
>
> .../display/bridge/toshiba,tc358767.yaml | 22 +-
> drivers/gpu/drm/bridge/Kconfig | 1 +
> drivers/gpu/drm/bridge/tc358767.c | 566 ++++++++++++++++--
> 3 files changed, 529 insertions(+), 60 deletions(-)
>
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v4 00/11] drm/bridge: tc358767: Add DSI-to-DPI mode support
2022-03-29 7:47 ` [PATCH v4 00/11] " Lucas Stach
@ 2022-03-29 7:58 ` Marek Vasut
0 siblings, 0 replies; 19+ messages in thread
From: Marek Vasut @ 2022-03-29 7:58 UTC (permalink / raw)
To: Lucas Stach, dri-devel
Cc: Neil Armstrong, Sam Ravnborg, Laurent Pinchart, Maxime Ripard,
Jonas Karlman
On 3/29/22 09:47, Lucas Stach wrote:
> Hi Marek,
Hi,
> Am Montag, dem 28.03.2022 um 23:43 +0200 schrieb Marek Vasut:
>> The TC358767/TC358867/TC9595 are all capable of operating in multiple
>> modes, DPI-to-(e)DP, DSI-to-(e)DP, DSI-to-DPI. Clean up the driver,
>> switch to atomic ops, and add support for the DSI-to-DPI mode in
>> addition to already supported DPI-to-(e)DP mode.
>>
> Not sure if anyone else has any more comments. For what it is worth, I
> just skimmed through the series again and it looks good to me now.
The robot does, but that should be easy to fix.
^ permalink raw reply [flat|nested] 19+ messages in thread
end of thread, other threads:[~2022-03-29 7:59 UTC | newest]
Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-28 21:43 [PATCH v4 00/11] drm/bridge: tc358767: Add DSI-to-DPI mode support Marek Vasut
2022-03-28 21:43 ` [PATCH v4 01/11] dt-bindings: display: bridge: tc358867: Document DPI output support Marek Vasut
2022-03-28 21:43 ` Marek Vasut
2022-03-28 21:43 ` [PATCH v4 02/11] dt-bindings: display: bridge: tc358867: Document DSI data-lanes property Marek Vasut
2022-03-28 21:43 ` Marek Vasut
2022-03-28 21:43 ` [PATCH v4 03/11] drm/bridge: tc358767: Change tc_ prefix to tc_edp_ for (e)DP specific functions Marek Vasut
2022-03-28 21:43 ` [PATCH v4 04/11] drm/bridge: tc358767: Convert to atomic ops Marek Vasut
2022-03-28 21:43 ` [PATCH v4 05/11] drm/bridge: tc358767: Implement atomic_check callback Marek Vasut
2022-03-28 21:43 ` [PATCH v4 06/11] drm/bridge: tc358767: Move (e)DP bridge endpoint parsing into dedicated function Marek Vasut
2022-03-28 21:43 ` [PATCH v4 07/11] drm/bridge: tc358767: Wrap (e)DP aux I2C registration into tc_aux_link_setup() Marek Vasut
2022-03-28 21:43 ` [PATCH v4 08/11] drm/bridge: tc358767: Move bridge ops setup into tc_probe_edp_bridge_endpoint() Marek Vasut
2022-03-28 21:43 ` [PATCH v4 09/11] drm/bridge: tc358767: Detect bridge mode from connected endpoints in DT Marek Vasut
2022-03-28 21:43 ` [PATCH v4 10/11] drm/bridge: tc358767: Split tc_set_video_mode() into common and (e)DP part Marek Vasut
2022-03-29 1:54 ` kernel test robot
2022-03-29 2:05 ` kernel test robot
2022-03-29 2:05 ` kernel test robot
2022-03-28 21:43 ` [PATCH v4 11/11] drm/bridge: tc358767: Add DSI-to-DPI mode support Marek Vasut
2022-03-29 7:47 ` [PATCH v4 00/11] " Lucas Stach
2022-03-29 7:58 ` Marek Vasut
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.