diff mbox

[RFC,06/12] drm/i915/dsi: add some constness to vbt panel driver

Message ID 83eb14fae46779cd3b2daaa9387c39ac38ed0f2e.1421410274.git.jani.nikula@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Jani Nikula Jan. 16, 2015, 12:27 p.m. UTC 
Const is good for you. No functional changes.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/intel_dsi_panel_vbt.c | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

Comments

Shobhit Kumar Jan. 22, 2015, 11:25 a.m. UTC | #1 
On 01/16/2015 05:57 PM, Jani Nikula wrote:
> Const is good for you. No functional changes.
>
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Reviewed-By: Shobhit Kumar <shobhit.kumar@intel.com>

> ---
>   drivers/gpu/drm/i915/intel_dsi_panel_vbt.c | 17 +++++++++--------
>   1 file changed, 9 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c b/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c
> index b0e7327a485f..561ec2981dfd 100644
> --- a/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c
> +++ b/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c
> @@ -99,7 +99,8 @@ static inline enum port intel_dsi_seq_port_to_port(u8 port)
>   	return port ? PORT_C : PORT_A;
>   }
>
> -static u8 *mipi_exec_send_packet(struct intel_dsi *intel_dsi, u8 *data)
> +static const u8 *mipi_exec_send_packet(struct intel_dsi *intel_dsi,
> +				       const u8 *data)
>   {
>   	u8 type, byte, mode, vc, seq_port;
>   	u16 len;
> @@ -165,9 +166,9 @@ static u8 *mipi_exec_send_packet(struct intel_dsi *intel_dsi, u8 *data)
>   	return data;
>   }
>
> -static u8 *mipi_exec_delay(struct intel_dsi *intel_dsi, u8 *data)
> +static const u8 *mipi_exec_delay(struct intel_dsi *intel_dsi, const u8 *data)
>   {
> -	u32 delay = *((u32 *) data);
> +	u32 delay = *((const u32 *) data);
>
>   	usleep_range(delay, delay + 10);
>   	data += 4;
> @@ -175,7 +176,7 @@ static u8 *mipi_exec_delay(struct intel_dsi *intel_dsi, u8 *data)
>   	return data;
>   }
>
> -static u8 *mipi_exec_gpio(struct intel_dsi *intel_dsi, u8 *data)
> +static const u8 *mipi_exec_gpio(struct intel_dsi *intel_dsi, const u8 *data)
>   {
>   	u8 gpio, action;
>   	u16 function, pad;
> @@ -208,7 +209,8 @@ static u8 *mipi_exec_gpio(struct intel_dsi *intel_dsi, u8 *data)
>   	return data;
>   }
>
> -typedef u8 * (*fn_mipi_elem_exec)(struct intel_dsi *intel_dsi, u8 *data);
> +typedef const u8 * (*fn_mipi_elem_exec)(struct intel_dsi *intel_dsi,
> +					const u8 *data);
>   static const fn_mipi_elem_exec exec_elem[] = {
>   	NULL, /* reserved */
>   	mipi_exec_send_packet,
> @@ -232,13 +234,12 @@ static const char * const seq_name[] = {
>   	"MIPI_SEQ_DEASSERT_RESET"
>   };
>
> -static void generic_exec_sequence(struct intel_dsi *intel_dsi, char *sequence)
> +static void generic_exec_sequence(struct intel_dsi *intel_dsi, const u8 *data)
>   {
> -	u8 *data = sequence;
>   	fn_mipi_elem_exec mipi_elem_exec;
>   	int index;
>
> -	if (!sequence)
> +	if (!data)
>   		return;
>
>   	DRM_DEBUG_DRIVER("Starting MIPI sequence - %s\n", seq_name[*data]);
>
diff mbox

Patch

diff --git a/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c b/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c
index b0e7327a485f..561ec2981dfd 100644
--- a/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c
+++ b/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c
@@ -99,7 +99,8 @@  static inline enum port intel_dsi_seq_port_to_port(u8 port)
 	return port ? PORT_C : PORT_A;
 }
 
-static u8 *mipi_exec_send_packet(struct intel_dsi *intel_dsi, u8 *data)
+static const u8 *mipi_exec_send_packet(struct intel_dsi *intel_dsi,
+				       const u8 *data)
 {
 	u8 type, byte, mode, vc, seq_port;
 	u16 len;
@@ -165,9 +166,9 @@  static u8 *mipi_exec_send_packet(struct intel_dsi *intel_dsi, u8 *data)
 	return data;
 }
 
-static u8 *mipi_exec_delay(struct intel_dsi *intel_dsi, u8 *data)
+static const u8 *mipi_exec_delay(struct intel_dsi *intel_dsi, const u8 *data)
 {
-	u32 delay = *((u32 *) data);
+	u32 delay = *((const u32 *) data);
 
 	usleep_range(delay, delay + 10);
 	data += 4;
@@ -175,7 +176,7 @@  static u8 *mipi_exec_delay(struct intel_dsi *intel_dsi, u8 *data)
 	return data;
 }
 
-static u8 *mipi_exec_gpio(struct intel_dsi *intel_dsi, u8 *data)
+static const u8 *mipi_exec_gpio(struct intel_dsi *intel_dsi, const u8 *data)
 {
 	u8 gpio, action;
 	u16 function, pad;
@@ -208,7 +209,8 @@  static u8 *mipi_exec_gpio(struct intel_dsi *intel_dsi, u8 *data)
 	return data;
 }
 
-typedef u8 * (*fn_mipi_elem_exec)(struct intel_dsi *intel_dsi, u8 *data);
+typedef const u8 * (*fn_mipi_elem_exec)(struct intel_dsi *intel_dsi,
+					const u8 *data);
 static const fn_mipi_elem_exec exec_elem[] = {
 	NULL, /* reserved */
 	mipi_exec_send_packet,
@@ -232,13 +234,12 @@  static const char * const seq_name[] = {
 	"MIPI_SEQ_DEASSERT_RESET"
 };
 
-static void generic_exec_sequence(struct intel_dsi *intel_dsi, char *sequence)
+static void generic_exec_sequence(struct intel_dsi *intel_dsi, const u8 *data)
 {
-	u8 *data = sequence;
 	fn_mipi_elem_exec mipi_elem_exec;
 	int index;
 
-	if (!sequence)
+	if (!data)
 		return;
 
 	DRM_DEBUG_DRIVER("Starting MIPI sequence - %s\n", seq_name[*data]);