czw., 18 paź 2018, 12:21 użytkownik Linus Walleij <linus.walleij@xxxxxxxxxx> napisał:
The DSI devices have a maximum operating frequency specified
in their data sheet per the MIPI specification, and DSI hosts
that can scale their frequency need this information to set
their clock dividers right.
As current panel drivers often lack this information, specify
that setting it to zero will make the DSI host use some
reasonable default.
Cc: Andrzej Hajda <a.hajda@xxxxxxxxxxx>
Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
---
ChangeLog v1->v2:
- s/*_rate_hz/*_rate/g
- s/operation/mode/g
- Clarify that zero is only allowed for legacy drivers
---
include/drm/drm_mipi_dsi.h | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/include/drm/drm_mipi_dsi.h b/include/drm/drm_mipi_dsi.h
index 4fef19064b0f..da3499de2dc2 100644
--- a/include/drm/drm_mipi_dsi.h
+++ b/include/drm/drm_mipi_dsi.h
@@ -168,6 +168,12 @@ struct mipi_dsi_device_info {
* @format: pixel format for video mode
* @lanes: number of active data lanes
* @mode_flags: DSI operation mode related flags
+ * @hs_rate: Maximum lane frequency for high speed mode, this should
+ * be set to the real limits of the hardware, zero is only accepted for
+ * legacy drivers
+ * @lp_rate: Maximum lane frequency for low power mode, this should
+ * be set to the real limits of the hardware, zero is only accepted for
+ * legacy drivers
The convention here is no-capital-letters, moreover it would be good to put units in the description.
Regarding zero value, as I looked into some random panel data sheets they do not always have specified rates, so maybe calculation of hs_rate from specified video mode wouldn't be so bad, but in such case it should be described here, up to you, can be adjusted in the future when we will have more input.
So with minor changes from previous sentence:
Reviewed-by: Andrzej Hajda <a.hajda@xxxxxxxxxxx>
I have no access to my corp mail atm, so if there is policy to send tags from the same email, I can do it on Friday.
Regards
Andrzej
*/
struct mipi_dsi_device {
struct mipi_dsi_host *host;
@@ -178,6 +184,8 @@ struct mipi_dsi_device {
unsigned int lanes;
enum mipi_dsi_pixel_format format;
unsigned long mode_flags;
+ unsigned long hs_rate;
+ unsigned long lp_rate;
};
#define MIPI_DSI_MODULE_PREFIX "mipi-dsi:"
--
2.17.2
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
