diff options
author | Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> | 2018-04-06 10:39:01 -0400 |
---|---|---|
committer | Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> | 2019-01-13 20:51:14 -0500 |
commit | 63f8f3badf799c8b63ff33a489886bc138ce5d09 (patch) | |
tree | b78e60a8f136b64c8daa624bc17996ff192b088f /drivers/gpu/drm/bridge/synopsys | |
parent | e3d093070eb0b5e3df668d3eb04100ea79343c65 (diff) |
drm: bridge: Constify mode arguments to bridge .mode_set() operation
The mode and ajusted_mode passed to the bridge .mode_set() operation
should never be modified by the bridge (and are not in any of the
existing bridge drivers). Make them const to make this clear.
Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'drivers/gpu/drm/bridge/synopsys')
-rw-r--r-- | drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 4 | ||||
-rw-r--r-- | drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 18 |
2 files changed, 11 insertions, 11 deletions
diff --git a/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c b/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c index 64c3cf027518..4fed3eda6ded 100644 --- a/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c +++ b/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | |||
@@ -1998,8 +1998,8 @@ dw_hdmi_bridge_mode_valid(struct drm_bridge *bridge, | |||
1998 | } | 1998 | } |
1999 | 1999 | ||
2000 | static void dw_hdmi_bridge_mode_set(struct drm_bridge *bridge, | 2000 | static void dw_hdmi_bridge_mode_set(struct drm_bridge *bridge, |
2001 | struct drm_display_mode *orig_mode, | 2001 | const struct drm_display_mode *orig_mode, |
2002 | struct drm_display_mode *mode) | 2002 | const struct drm_display_mode *mode) |
2003 | { | 2003 | { |
2004 | struct dw_hdmi *hdmi = bridge->driver_private; | 2004 | struct dw_hdmi *hdmi = bridge->driver_private; |
2005 | 2005 | ||
diff --git a/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c b/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c index 2f4b145b73af..23a5977a3b0a 100644 --- a/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c +++ b/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | |||
@@ -248,7 +248,7 @@ static inline bool dw_mipi_is_dual_mode(struct dw_mipi_dsi *dsi) | |||
248 | * The controller should generate 2 frames before | 248 | * The controller should generate 2 frames before |
249 | * preparing the peripheral. | 249 | * preparing the peripheral. |
250 | */ | 250 | */ |
251 | static void dw_mipi_dsi_wait_for_two_frames(struct drm_display_mode *mode) | 251 | static void dw_mipi_dsi_wait_for_two_frames(const struct drm_display_mode *mode) |
252 | { | 252 | { |
253 | int refresh, two_frames; | 253 | int refresh, two_frames; |
254 | 254 | ||
@@ -564,7 +564,7 @@ static void dw_mipi_dsi_init(struct dw_mipi_dsi *dsi) | |||
564 | } | 564 | } |
565 | 565 | ||
566 | static void dw_mipi_dsi_dpi_config(struct dw_mipi_dsi *dsi, | 566 | static void dw_mipi_dsi_dpi_config(struct dw_mipi_dsi *dsi, |
567 | struct drm_display_mode *mode) | 567 | const struct drm_display_mode *mode) |
568 | { | 568 | { |
569 | u32 val = 0, color = 0; | 569 | u32 val = 0, color = 0; |
570 | 570 | ||
@@ -607,7 +607,7 @@ static void dw_mipi_dsi_packet_handler_config(struct dw_mipi_dsi *dsi) | |||
607 | } | 607 | } |
608 | 608 | ||
609 | static void dw_mipi_dsi_video_packet_config(struct dw_mipi_dsi *dsi, | 609 | static void dw_mipi_dsi_video_packet_config(struct dw_mipi_dsi *dsi, |
610 | struct drm_display_mode *mode) | 610 | const struct drm_display_mode *mode) |
611 | { | 611 | { |
612 | /* | 612 | /* |
613 | * TODO dw drv improvements | 613 | * TODO dw drv improvements |
@@ -642,7 +642,7 @@ static void dw_mipi_dsi_command_mode_config(struct dw_mipi_dsi *dsi) | |||
642 | 642 | ||
643 | /* Get lane byte clock cycles. */ | 643 | /* Get lane byte clock cycles. */ |
644 | static u32 dw_mipi_dsi_get_hcomponent_lbcc(struct dw_mipi_dsi *dsi, | 644 | static u32 dw_mipi_dsi_get_hcomponent_lbcc(struct dw_mipi_dsi *dsi, |
645 | struct drm_display_mode *mode, | 645 | const struct drm_display_mode *mode, |
646 | u32 hcomponent) | 646 | u32 hcomponent) |
647 | { | 647 | { |
648 | u32 frac, lbcc; | 648 | u32 frac, lbcc; |
@@ -658,7 +658,7 @@ static u32 dw_mipi_dsi_get_hcomponent_lbcc(struct dw_mipi_dsi *dsi, | |||
658 | } | 658 | } |
659 | 659 | ||
660 | static void dw_mipi_dsi_line_timer_config(struct dw_mipi_dsi *dsi, | 660 | static void dw_mipi_dsi_line_timer_config(struct dw_mipi_dsi *dsi, |
661 | struct drm_display_mode *mode) | 661 | const struct drm_display_mode *mode) |
662 | { | 662 | { |
663 | u32 htotal, hsa, hbp, lbcc; | 663 | u32 htotal, hsa, hbp, lbcc; |
664 | 664 | ||
@@ -681,7 +681,7 @@ static void dw_mipi_dsi_line_timer_config(struct dw_mipi_dsi *dsi, | |||
681 | } | 681 | } |
682 | 682 | ||
683 | static void dw_mipi_dsi_vertical_timing_config(struct dw_mipi_dsi *dsi, | 683 | static void dw_mipi_dsi_vertical_timing_config(struct dw_mipi_dsi *dsi, |
684 | struct drm_display_mode *mode) | 684 | const struct drm_display_mode *mode) |
685 | { | 685 | { |
686 | u32 vactive, vsa, vfp, vbp; | 686 | u32 vactive, vsa, vfp, vbp; |
687 | 687 | ||
@@ -818,7 +818,7 @@ static unsigned int dw_mipi_dsi_get_lanes(struct dw_mipi_dsi *dsi) | |||
818 | } | 818 | } |
819 | 819 | ||
820 | static void dw_mipi_dsi_mode_set(struct dw_mipi_dsi *dsi, | 820 | static void dw_mipi_dsi_mode_set(struct dw_mipi_dsi *dsi, |
821 | struct drm_display_mode *adjusted_mode) | 821 | const struct drm_display_mode *adjusted_mode) |
822 | { | 822 | { |
823 | const struct dw_mipi_dsi_phy_ops *phy_ops = dsi->plat_data->phy_ops; | 823 | const struct dw_mipi_dsi_phy_ops *phy_ops = dsi->plat_data->phy_ops; |
824 | void *priv_data = dsi->plat_data->priv_data; | 824 | void *priv_data = dsi->plat_data->priv_data; |
@@ -861,8 +861,8 @@ static void dw_mipi_dsi_mode_set(struct dw_mipi_dsi *dsi, | |||
861 | } | 861 | } |
862 | 862 | ||
863 | static void dw_mipi_dsi_bridge_mode_set(struct drm_bridge *bridge, | 863 | static void dw_mipi_dsi_bridge_mode_set(struct drm_bridge *bridge, |
864 | struct drm_display_mode *mode, | 864 | const struct drm_display_mode *mode, |
865 | struct drm_display_mode *adjusted_mode) | 865 | const struct drm_display_mode *adjusted_mode) |
866 | { | 866 | { |
867 | struct dw_mipi_dsi *dsi = bridge_to_dsi(bridge); | 867 | struct dw_mipi_dsi *dsi = bridge_to_dsi(bridge); |
868 | 868 | ||