* [PATCH v7 0/2] *** IT6505 driver read dt properties ***
@ 2022-11-03 9:12 allen
2022-11-03 9:12 ` [PATCH v7 1/2] dt-bindings: it6505: add properties to restrict output bandwidth allen
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: allen @ 2022-11-03 9:12 UTC (permalink / raw)
Cc: Allen Chen, Jau-Chih Tseng, Hermes Wu, Kenneth Hung, Pin-yen Lin,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
open list:DRM DRIVERS, Jernej Skrabec, Jonas Karlman,
Laurent Pinchart, open list
This series let driver can read properties from dt to restrict dp output
bandwidth.
Changes in v3:
-Rename property name.
Changes in v4:
-Use data-lanes and link-frequencies instead of "ite,dp-output-data-lane-count" and "ite,dp-output-max-pixel-clock-mhz".
Changes in v5:
-Add a port and a endpoint.
-Move data-lanes property to the output endpoint.
Changes in v6:
-Modified data-lanes description by suggestion.
Changes in v7:
-Add commit messages to explain reason for breaking users.
allen chen (2):
dt-bindings: it6505: add properties to restrict output bandwidth
drm/bridge: add it6505 driver to read data-lanes and link-frequencies
from dt
.../bindings/display/bridge/ite,it6505.yaml | 68 ++++++++++++++--
drivers/gpu/drm/bridge/ite-it6505.c | 80 ++++++++++++++++++-
2 files changed, 139 insertions(+), 9 deletions(-)
--
2.25.1
^ permalink raw reply [flat|nested] 7+ messages in thread* [PATCH v7 1/2] dt-bindings: it6505: add properties to restrict output bandwidth 2022-11-03 9:12 [PATCH v7 0/2] *** IT6505 driver read dt properties *** allen @ 2022-11-03 9:12 ` allen 2022-11-03 16:03 ` Krzysztof Kozlowski 2022-11-03 9:12 ` [PATCH v7 2/2] drm/bridge: add it6505 driver to read data-lanes and link-frequencies from dt allen 2022-12-15 13:36 ` [PATCH v7 0/2] *** IT6505 driver read dt properties *** Robert Foss 2 siblings, 1 reply; 7+ messages in thread From: allen @ 2022-11-03 9:12 UTC (permalink / raw) Cc: Allen Chen, Jau-Chih Tseng, Hermes Wu, Kenneth Hung, Pin-yen Lin, Andrzej Hajda, Neil Armstrong, Robert Foss, Laurent Pinchart, Jonas Karlman, Jernej Skrabec, David Airlie, Daniel Vetter, Rob Herring, Krzysztof Kozlowski, open list:DRM DRIVERS, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, open list From: allen chen <allen.chen@ite.com.tw> Currently there are no "upstream" users. So, no existing users to break. Add properties to restrict dp output data-lanes and clock. Signed-off-by: Pin-Yen Lin <treapking@chromium.org> Signed-off-by: Allen Chen <allen.chen@ite.com.tw> --- .../bindings/display/bridge/ite,it6505.yaml | 68 +++++++++++++++++-- 1 file changed, 62 insertions(+), 6 deletions(-) diff --git a/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml b/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml index 833d11b2303a7..b16a9d9127ddb 100644 --- a/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml +++ b/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml @@ -52,9 +52,49 @@ properties: maxItems: 1 description: extcon specifier for the Power Delivery - port: - $ref: /schemas/graph.yaml#/properties/port - description: A port node pointing to DPI host port node + ports: + $ref: /schemas/graph.yaml#/properties/ports + + properties: + port@0: + $ref: /schemas/graph.yaml#/$defs/port-base + unevaluatedProperties: false + description: A port node pointing to DPI host port node + + properties: + endpoint: + $ref: /schemas/graph.yaml#/$defs/endpoint-base + unevaluatedProperties: false + + properties: + link-frequencies: + minItems: 1 + maxItems: 1 + description: Allowed max link frequencies in Hz + + port@1: + $ref: /schemas/graph.yaml#/$defs/port-base + unevaluatedProperties: false + description: Video port for DP output + + properties: + endpoint: + $ref: /schemas/graph.yaml#/$defs/endpoint-base + unevaluatedProperties: false + + properties: + data-lanes: + minItems: 1 + uniqueItems: true + items: + - enum: [ 0, 1 ] + - const: 1 + - const: 2 + - const: 3 + + required: + - port@0 + - port@1 required: - compatible @@ -63,6 +103,7 @@ required: - interrupts - reset-gpios - extcon + - ports additionalProperties: false @@ -85,9 +126,24 @@ examples: reset-gpios = <&pio 179 1>; extcon = <&usbc_extcon>; - port { - it6505_in: endpoint { - remote-endpoint = <&dpi_out>; + ports { + #address-cells = <1>; + #size-cells = <0>; + + port@0 { + reg = <0>; + it6505_in: endpoint { + remote-endpoint = <&dpi_out>; + link-frequencies = /bits/ 64 <150000000>; + }; + }; + + port@1 { + reg = <1>; + it6505_out: endpoint { + remote-endpoint = <&dp_in>; + data-lanes = <0 1>; + }; }; }; }; -- 2.25.1 ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v7 1/2] dt-bindings: it6505: add properties to restrict output bandwidth 2022-11-03 9:12 ` [PATCH v7 1/2] dt-bindings: it6505: add properties to restrict output bandwidth allen @ 2022-11-03 16:03 ` Krzysztof Kozlowski 0 siblings, 0 replies; 7+ messages in thread From: Krzysztof Kozlowski @ 2022-11-03 16:03 UTC (permalink / raw) To: allen Cc: Jau-Chih Tseng, Hermes Wu, Kenneth Hung, Pin-yen Lin, Andrzej Hajda, Neil Armstrong, Robert Foss, Laurent Pinchart, Jonas Karlman, Jernej Skrabec, David Airlie, Daniel Vetter, Rob Herring, Krzysztof Kozlowski, open list:DRM DRIVERS, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, open list On 03/11/2022 05:12, allen wrote: > From: allen chen <allen.chen@ite.com.tw> > > Currently there are no "upstream" users. So, no existing users to break. > Add properties to restrict dp output data-lanes and clock. > > Signed-off-by: Pin-Yen Lin <treapking@chromium.org> > Signed-off-by: Allen Chen <allen.chen@ite.com.tw> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v7 2/2] drm/bridge: add it6505 driver to read data-lanes and link-frequencies from dt 2022-11-03 9:12 [PATCH v7 0/2] *** IT6505 driver read dt properties *** allen 2022-11-03 9:12 ` [PATCH v7 1/2] dt-bindings: it6505: add properties to restrict output bandwidth allen @ 2022-11-03 9:12 ` allen 2022-11-21 5:39 ` Pin-yen Lin 2022-12-15 13:36 ` [PATCH v7 0/2] *** IT6505 driver read dt properties *** Robert Foss 2 siblings, 1 reply; 7+ messages in thread From: allen @ 2022-11-03 9:12 UTC (permalink / raw) Cc: Allen Chen, Jau-Chih Tseng, Hermes Wu, Kenneth Hung, Pin-yen Lin, Andrzej Hajda, Neil Armstrong, Robert Foss, Laurent Pinchart, Jonas Karlman, Jernej Skrabec, David Airlie, Daniel Vetter, open list:DRM DRIVERS, open list From: allen chen <allen.chen@ite.com.tw> Add driver to read data-lanes and link-frequencies from dt property to restrict output bandwidth. Signed-off-by: Allen chen <allen.chen@ite.com.tw> Signed-off-by: Pin-yen Lin <treapking@chromium.org> --- drivers/gpu/drm/bridge/ite-it6505.c | 80 +++++++++++++++++++++++++++-- 1 file changed, 77 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/bridge/ite-it6505.c b/drivers/gpu/drm/bridge/ite-it6505.c index a4302492cf8df..ed4536cde3140 100644 --- a/drivers/gpu/drm/bridge/ite-it6505.c +++ b/drivers/gpu/drm/bridge/ite-it6505.c @@ -437,6 +437,8 @@ struct it6505 { bool powered; bool hpd_state; u32 afe_setting; + u32 max_dpi_pixel_clock; + u32 max_lane_count; enum hdcp_state hdcp_status; struct delayed_work hdcp_work; struct work_struct hdcp_wait_ksv_list; @@ -1476,7 +1478,8 @@ static void it6505_parse_link_capabilities(struct it6505 *it6505) it6505->lane_count = link->num_lanes; DRM_DEV_DEBUG_DRIVER(dev, "Sink support %d lanes training", it6505->lane_count); - it6505->lane_count = min_t(int, it6505->lane_count, MAX_LANE_COUNT); + it6505->lane_count = min_t(int, it6505->lane_count, + it6505->max_lane_count); it6505->branch_device = drm_dp_is_branch(it6505->dpcd); DRM_DEV_DEBUG_DRIVER(dev, "Sink %sbranch device", @@ -2912,7 +2915,7 @@ it6505_bridge_mode_valid(struct drm_bridge *bridge, if (mode->flags & DRM_MODE_FLAG_INTERLACE) return MODE_NO_INTERLACE; - if (mode->clock > DPI_PIXEL_CLK_MAX) + if (mode->clock > it6505->max_dpi_pixel_clock) return MODE_CLOCK_HIGH; it6505->video_info.clock = mode->clock; @@ -3099,10 +3102,32 @@ static int it6505_init_pdata(struct it6505 *it6505) return 0; } +static int it6505_get_data_lanes_count(const struct device_node *endpoint, + const unsigned int min, + const unsigned int max) +{ + int ret; + + ret = of_property_count_u32_elems(endpoint, "data-lanes"); + if (ret < 0) + return ret; + + if (ret < min || ret > max) + return -EINVAL; + + return ret; +} + static void it6505_parse_dt(struct it6505 *it6505) { struct device *dev = &it6505->client->dev; + struct device_node *np = dev->of_node, *ep = NULL; + int len; + u64 link_frequencies; + u32 data_lanes[4]; u32 *afe_setting = &it6505->afe_setting; + u32 *max_lane_count = &it6505->max_lane_count; + u32 *max_dpi_pixel_clock = &it6505->max_dpi_pixel_clock; it6505->lane_swap_disabled = device_property_read_bool(dev, "no-laneswap"); @@ -3118,7 +3143,56 @@ static void it6505_parse_dt(struct it6505 *it6505) } else { *afe_setting = 0; } - DRM_DEV_DEBUG_DRIVER(dev, "using afe_setting: %d", *afe_setting); + + ep = of_graph_get_endpoint_by_regs(np, 1, 0); + of_node_put(ep); + + if (ep) { + len = it6505_get_data_lanes_count(ep, 1, 4); + + if (len > 0 && len != 3) { + of_property_read_u32_array(ep, "data-lanes", + data_lanes, len); + *max_lane_count = len; + } else { + *max_lane_count = MAX_LANE_COUNT; + dev_err(dev, "error data-lanes, use default"); + } + } else { + *max_lane_count = MAX_LANE_COUNT; + dev_err(dev, "error endpoint, use default"); + } + + ep = of_graph_get_endpoint_by_regs(np, 0, 0); + of_node_put(ep); + + if (ep) { + len = of_property_read_variable_u64_array(ep, + "link-frequencies", + &link_frequencies, 0, + 1); + if (len >= 0) { + do_div(link_frequencies, 1000); + if (link_frequencies > 297000) { + dev_err(dev, + "max pixel clock error, use default"); + *max_dpi_pixel_clock = DPI_PIXEL_CLK_MAX; + } else { + *max_dpi_pixel_clock = link_frequencies; + } + } else { + dev_err(dev, "error link frequencies, use default"); + *max_dpi_pixel_clock = DPI_PIXEL_CLK_MAX; + } + } else { + dev_err(dev, "error endpoint, use default"); + *max_dpi_pixel_clock = DPI_PIXEL_CLK_MAX; + } + + DRM_DEV_DEBUG_DRIVER(dev, "using afe_setting: %u, max_lane_count: %u", + it6505->afe_setting, it6505->max_lane_count); + DRM_DEV_DEBUG_DRIVER(dev, "using max_dpi_pixel_clock: %u kHz", + it6505->max_dpi_pixel_clock); } static ssize_t receive_timing_debugfs_show(struct file *file, char __user *buf, -- 2.25.1 ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v7 2/2] drm/bridge: add it6505 driver to read data-lanes and link-frequencies from dt 2022-11-03 9:12 ` [PATCH v7 2/2] drm/bridge: add it6505 driver to read data-lanes and link-frequencies from dt allen @ 2022-11-21 5:39 ` Pin-yen Lin 2022-12-14 3:29 ` Pin-yen Lin 0 siblings, 1 reply; 7+ messages in thread From: Pin-yen Lin @ 2022-11-21 5:39 UTC (permalink / raw) To: allen Cc: Jau-Chih Tseng, Hermes Wu, Kenneth Hung, Andrzej Hajda, Neil Armstrong, Robert Foss, Laurent Pinchart, Jonas Karlman, Jernej Skrabec, David Airlie, Daniel Vetter, open list:DRM DRIVERS, open list Hi all, Friendly ping on this patch. Regards, Pin-yen On Thu, Nov 3, 2022 at 5:13 PM allen <allen.chen@ite.com.tw> wrote: > > From: allen chen <allen.chen@ite.com.tw> > > Add driver to read data-lanes and link-frequencies from dt property to > restrict output bandwidth. > > Signed-off-by: Allen chen <allen.chen@ite.com.tw> > Signed-off-by: Pin-yen Lin <treapking@chromium.org> > --- > drivers/gpu/drm/bridge/ite-it6505.c | 80 +++++++++++++++++++++++++++-- > 1 file changed, 77 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/bridge/ite-it6505.c b/drivers/gpu/drm/bridge/ite-it6505.c > index a4302492cf8df..ed4536cde3140 100644 > --- a/drivers/gpu/drm/bridge/ite-it6505.c > +++ b/drivers/gpu/drm/bridge/ite-it6505.c > @@ -437,6 +437,8 @@ struct it6505 { > bool powered; > bool hpd_state; > u32 afe_setting; > + u32 max_dpi_pixel_clock; > + u32 max_lane_count; > enum hdcp_state hdcp_status; > struct delayed_work hdcp_work; > struct work_struct hdcp_wait_ksv_list; > @@ -1476,7 +1478,8 @@ static void it6505_parse_link_capabilities(struct it6505 *it6505) > it6505->lane_count = link->num_lanes; > DRM_DEV_DEBUG_DRIVER(dev, "Sink support %d lanes training", > it6505->lane_count); > - it6505->lane_count = min_t(int, it6505->lane_count, MAX_LANE_COUNT); > + it6505->lane_count = min_t(int, it6505->lane_count, > + it6505->max_lane_count); > > it6505->branch_device = drm_dp_is_branch(it6505->dpcd); > DRM_DEV_DEBUG_DRIVER(dev, "Sink %sbranch device", > @@ -2912,7 +2915,7 @@ it6505_bridge_mode_valid(struct drm_bridge *bridge, > if (mode->flags & DRM_MODE_FLAG_INTERLACE) > return MODE_NO_INTERLACE; > > - if (mode->clock > DPI_PIXEL_CLK_MAX) > + if (mode->clock > it6505->max_dpi_pixel_clock) > return MODE_CLOCK_HIGH; > > it6505->video_info.clock = mode->clock; > @@ -3099,10 +3102,32 @@ static int it6505_init_pdata(struct it6505 *it6505) > return 0; > } > > +static int it6505_get_data_lanes_count(const struct device_node *endpoint, > + const unsigned int min, > + const unsigned int max) > +{ > + int ret; > + > + ret = of_property_count_u32_elems(endpoint, "data-lanes"); > + if (ret < 0) > + return ret; > + > + if (ret < min || ret > max) > + return -EINVAL; > + > + return ret; > +} > + > static void it6505_parse_dt(struct it6505 *it6505) > { > struct device *dev = &it6505->client->dev; > + struct device_node *np = dev->of_node, *ep = NULL; > + int len; > + u64 link_frequencies; > + u32 data_lanes[4]; > u32 *afe_setting = &it6505->afe_setting; > + u32 *max_lane_count = &it6505->max_lane_count; > + u32 *max_dpi_pixel_clock = &it6505->max_dpi_pixel_clock; > > it6505->lane_swap_disabled = > device_property_read_bool(dev, "no-laneswap"); > @@ -3118,7 +3143,56 @@ static void it6505_parse_dt(struct it6505 *it6505) > } else { > *afe_setting = 0; > } > - DRM_DEV_DEBUG_DRIVER(dev, "using afe_setting: %d", *afe_setting); > + > + ep = of_graph_get_endpoint_by_regs(np, 1, 0); > + of_node_put(ep); > + > + if (ep) { > + len = it6505_get_data_lanes_count(ep, 1, 4); > + > + if (len > 0 && len != 3) { > + of_property_read_u32_array(ep, "data-lanes", > + data_lanes, len); > + *max_lane_count = len; > + } else { > + *max_lane_count = MAX_LANE_COUNT; > + dev_err(dev, "error data-lanes, use default"); > + } > + } else { > + *max_lane_count = MAX_LANE_COUNT; > + dev_err(dev, "error endpoint, use default"); > + } > + > + ep = of_graph_get_endpoint_by_regs(np, 0, 0); > + of_node_put(ep); > + > + if (ep) { > + len = of_property_read_variable_u64_array(ep, > + "link-frequencies", > + &link_frequencies, 0, > + 1); > + if (len >= 0) { > + do_div(link_frequencies, 1000); > + if (link_frequencies > 297000) { > + dev_err(dev, > + "max pixel clock error, use default"); > + *max_dpi_pixel_clock = DPI_PIXEL_CLK_MAX; > + } else { > + *max_dpi_pixel_clock = link_frequencies; > + } > + } else { > + dev_err(dev, "error link frequencies, use default"); > + *max_dpi_pixel_clock = DPI_PIXEL_CLK_MAX; > + } > + } else { > + dev_err(dev, "error endpoint, use default"); > + *max_dpi_pixel_clock = DPI_PIXEL_CLK_MAX; > + } > + > + DRM_DEV_DEBUG_DRIVER(dev, "using afe_setting: %u, max_lane_count: %u", > + it6505->afe_setting, it6505->max_lane_count); > + DRM_DEV_DEBUG_DRIVER(dev, "using max_dpi_pixel_clock: %u kHz", > + it6505->max_dpi_pixel_clock); > } > > static ssize_t receive_timing_debugfs_show(struct file *file, char __user *buf, > -- > 2.25.1 > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v7 2/2] drm/bridge: add it6505 driver to read data-lanes and link-frequencies from dt 2022-11-21 5:39 ` Pin-yen Lin @ 2022-12-14 3:29 ` Pin-yen Lin 0 siblings, 0 replies; 7+ messages in thread From: Pin-yen Lin @ 2022-12-14 3:29 UTC (permalink / raw) To: allen Cc: Jau-Chih Tseng, Hermes Wu, Kenneth Hung, Andrzej Hajda, Neil Armstrong, Robert Foss, Laurent Pinchart, Jonas Karlman, Jernej Skrabec, David Airlie, Daniel Vetter, open list:DRM DRIVERS, open list Friendly ping. Pin-yen Pin-yen On Mon, Nov 21, 2022 at 1:39 PM Pin-yen Lin <treapking@chromium.org> wrote: > > Hi all, > Friendly ping on this patch. > > Regards, > Pin-yen > > On Thu, Nov 3, 2022 at 5:13 PM allen <allen.chen@ite.com.tw> wrote: > > > > From: allen chen <allen.chen@ite.com.tw> > > > > Add driver to read data-lanes and link-frequencies from dt property to > > restrict output bandwidth. > > > > Signed-off-by: Allen chen <allen.chen@ite.com.tw> > > Signed-off-by: Pin-yen Lin <treapking@chromium.org> > > --- > > drivers/gpu/drm/bridge/ite-it6505.c | 80 +++++++++++++++++++++++++++-- > > 1 file changed, 77 insertions(+), 3 deletions(-) > > > > diff --git a/drivers/gpu/drm/bridge/ite-it6505.c b/drivers/gpu/drm/bridge/ite-it6505.c > > index a4302492cf8df..ed4536cde3140 100644 > > --- a/drivers/gpu/drm/bridge/ite-it6505.c > > +++ b/drivers/gpu/drm/bridge/ite-it6505.c > > @@ -437,6 +437,8 @@ struct it6505 { > > bool powered; > > bool hpd_state; > > u32 afe_setting; > > + u32 max_dpi_pixel_clock; > > + u32 max_lane_count; > > enum hdcp_state hdcp_status; > > struct delayed_work hdcp_work; > > struct work_struct hdcp_wait_ksv_list; > > @@ -1476,7 +1478,8 @@ static void it6505_parse_link_capabilities(struct it6505 *it6505) > > it6505->lane_count = link->num_lanes; > > DRM_DEV_DEBUG_DRIVER(dev, "Sink support %d lanes training", > > it6505->lane_count); > > - it6505->lane_count = min_t(int, it6505->lane_count, MAX_LANE_COUNT); > > + it6505->lane_count = min_t(int, it6505->lane_count, > > + it6505->max_lane_count); > > > > it6505->branch_device = drm_dp_is_branch(it6505->dpcd); > > DRM_DEV_DEBUG_DRIVER(dev, "Sink %sbranch device", > > @@ -2912,7 +2915,7 @@ it6505_bridge_mode_valid(struct drm_bridge *bridge, > > if (mode->flags & DRM_MODE_FLAG_INTERLACE) > > return MODE_NO_INTERLACE; > > > > - if (mode->clock > DPI_PIXEL_CLK_MAX) > > + if (mode->clock > it6505->max_dpi_pixel_clock) > > return MODE_CLOCK_HIGH; > > > > it6505->video_info.clock = mode->clock; > > @@ -3099,10 +3102,32 @@ static int it6505_init_pdata(struct it6505 *it6505) > > return 0; > > } > > > > +static int it6505_get_data_lanes_count(const struct device_node *endpoint, > > + const unsigned int min, > > + const unsigned int max) > > +{ > > + int ret; > > + > > + ret = of_property_count_u32_elems(endpoint, "data-lanes"); > > + if (ret < 0) > > + return ret; > > + > > + if (ret < min || ret > max) > > + return -EINVAL; > > + > > + return ret; > > +} > > + > > static void it6505_parse_dt(struct it6505 *it6505) > > { > > struct device *dev = &it6505->client->dev; > > + struct device_node *np = dev->of_node, *ep = NULL; > > + int len; > > + u64 link_frequencies; > > + u32 data_lanes[4]; > > u32 *afe_setting = &it6505->afe_setting; > > + u32 *max_lane_count = &it6505->max_lane_count; > > + u32 *max_dpi_pixel_clock = &it6505->max_dpi_pixel_clock; > > > > it6505->lane_swap_disabled = > > device_property_read_bool(dev, "no-laneswap"); > > @@ -3118,7 +3143,56 @@ static void it6505_parse_dt(struct it6505 *it6505) > > } else { > > *afe_setting = 0; > > } > > - DRM_DEV_DEBUG_DRIVER(dev, "using afe_setting: %d", *afe_setting); > > + > > + ep = of_graph_get_endpoint_by_regs(np, 1, 0); > > + of_node_put(ep); > > + > > + if (ep) { > > + len = it6505_get_data_lanes_count(ep, 1, 4); > > + > > + if (len > 0 && len != 3) { > > + of_property_read_u32_array(ep, "data-lanes", > > + data_lanes, len); > > + *max_lane_count = len; > > + } else { > > + *max_lane_count = MAX_LANE_COUNT; > > + dev_err(dev, "error data-lanes, use default"); > > + } > > + } else { > > + *max_lane_count = MAX_LANE_COUNT; > > + dev_err(dev, "error endpoint, use default"); > > + } > > + > > + ep = of_graph_get_endpoint_by_regs(np, 0, 0); > > + of_node_put(ep); > > + > > + if (ep) { > > + len = of_property_read_variable_u64_array(ep, > > + "link-frequencies", > > + &link_frequencies, 0, > > + 1); > > + if (len >= 0) { > > + do_div(link_frequencies, 1000); > > + if (link_frequencies > 297000) { > > + dev_err(dev, > > + "max pixel clock error, use default"); > > + *max_dpi_pixel_clock = DPI_PIXEL_CLK_MAX; > > + } else { > > + *max_dpi_pixel_clock = link_frequencies; > > + } > > + } else { > > + dev_err(dev, "error link frequencies, use default"); > > + *max_dpi_pixel_clock = DPI_PIXEL_CLK_MAX; > > + } > > + } else { > > + dev_err(dev, "error endpoint, use default"); > > + *max_dpi_pixel_clock = DPI_PIXEL_CLK_MAX; > > + } > > + > > + DRM_DEV_DEBUG_DRIVER(dev, "using afe_setting: %u, max_lane_count: %u", > > + it6505->afe_setting, it6505->max_lane_count); > > + DRM_DEV_DEBUG_DRIVER(dev, "using max_dpi_pixel_clock: %u kHz", > > + it6505->max_dpi_pixel_clock); > > } > > > > static ssize_t receive_timing_debugfs_show(struct file *file, char __user *buf, > > -- > > 2.25.1 > > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v7 0/2] *** IT6505 driver read dt properties *** 2022-11-03 9:12 [PATCH v7 0/2] *** IT6505 driver read dt properties *** allen 2022-11-03 9:12 ` [PATCH v7 1/2] dt-bindings: it6505: add properties to restrict output bandwidth allen 2022-11-03 9:12 ` [PATCH v7 2/2] drm/bridge: add it6505 driver to read data-lanes and link-frequencies from dt allen @ 2022-12-15 13:36 ` Robert Foss 2 siblings, 0 replies; 7+ messages in thread From: Robert Foss @ 2022-12-15 13:36 UTC (permalink / raw) To: allen Cc: Robert Foss, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, Jernej Skrabec, Hermes Wu, open list, Pin-yen Lin, Kenneth Hung, Jau-Chih Tseng, open list:DRM DRIVERS, Jonas Karlman, Laurent Pinchart On Thu, 3 Nov 2022 17:12:41 +0800, allen wrote: > This series let driver can read properties from dt to restrict dp output > bandwidth. > > Changes in v3: > -Rename property name. > > Changes in v4: > -Use data-lanes and link-frequencies instead of "ite,dp-output-data-lane-count" and "ite,dp-output-max-pixel-clock-mhz". > > [...] Allen: The email name you use must exactly match Signed-off-by string, or there will be issues like the below when applying the patch. -:139: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email name mismatch: 'From: allen chen <allen.chen@ite.com.tw>' != 'Signed-off-by: Allen chen <allen.chen@ite.com.tw>' With that fixed, applied! Repo: https://cgit.freedesktop.org/drm/drm-misc/ [1/2] dt-bindings: it6505: add properties to restrict output bandwidth commit: 380d920b582d0f83852ac6885af868d93c38095b [2/2] drm/bridge: add it6505 driver to read data-lanes and link-frequencies from dt commit: 380d920b582d0f83852ac6885af868d93c38095b Rob ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-12-15 13:36 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-11-03 9:12 [PATCH v7 0/2] *** IT6505 driver read dt properties *** allen 2022-11-03 9:12 ` [PATCH v7 1/2] dt-bindings: it6505: add properties to restrict output bandwidth allen 2022-11-03 16:03 ` Krzysztof Kozlowski 2022-11-03 9:12 ` [PATCH v7 2/2] drm/bridge: add it6505 driver to read data-lanes and link-frequencies from dt allen 2022-11-21 5:39 ` Pin-yen Lin 2022-12-14 3:29 ` Pin-yen Lin 2022-12-15 13:36 ` [PATCH v7 0/2] *** IT6505 driver read dt properties *** Robert Foss
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®