czw., 18 paź 2018, 12:21 użytkownik Linus Walleij <[email protected]> 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 <[email protected]> > Signed-off-by: Linus Walleij <[email protected]> > --- > 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 <[email protected]> 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 > [email protected] > https://lists.freedesktop.org/mailman/listinfo/dri-devel >
_______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
