Commit fd620bf9 authored by Jani Nikula's avatar Jani Nikula

drm/i915/backlight: switch to kernel unsigned int types

We have fairly mixed uintN_t vs. uN usage throughout the driver, but try
to stick to kernel types at least where it's more prevalent.
Reviewed-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/e68c3f16738eb3ab9f276d797f20326ed6d15848.1528794959.git.jani.nikula@intel.com
parent accb1eb5
...@@ -26,7 +26,7 @@ ...@@ -26,7 +26,7 @@
static void set_aux_backlight_enable(struct intel_dp *intel_dp, bool enable) static void set_aux_backlight_enable(struct intel_dp *intel_dp, bool enable)
{ {
uint8_t reg_val = 0; u8 reg_val = 0;
/* Early return when display use other mechanism to enable backlight. */ /* Early return when display use other mechanism to enable backlight. */
if (!(intel_dp->edp_dpcd[1] & DP_EDP_BACKLIGHT_AUX_ENABLE_CAP)) if (!(intel_dp->edp_dpcd[1] & DP_EDP_BACKLIGHT_AUX_ENABLE_CAP))
...@@ -54,11 +54,11 @@ static void set_aux_backlight_enable(struct intel_dp *intel_dp, bool enable) ...@@ -54,11 +54,11 @@ static void set_aux_backlight_enable(struct intel_dp *intel_dp, bool enable)
* Read the current backlight value from DPCD register(s) based * Read the current backlight value from DPCD register(s) based
* on if 8-bit(MSB) or 16-bit(MSB and LSB) values are supported * on if 8-bit(MSB) or 16-bit(MSB and LSB) values are supported
*/ */
static uint32_t intel_dp_aux_get_backlight(struct intel_connector *connector) static u32 intel_dp_aux_get_backlight(struct intel_connector *connector)
{ {
struct intel_dp *intel_dp = enc_to_intel_dp(&connector->encoder->base); struct intel_dp *intel_dp = enc_to_intel_dp(&connector->encoder->base);
uint8_t read_val[2] = { 0x0 }; u8 read_val[2] = { 0x0 };
uint16_t level = 0; u16 level = 0;
if (drm_dp_dpcd_read(&intel_dp->aux, DP_EDP_BACKLIGHT_BRIGHTNESS_MSB, if (drm_dp_dpcd_read(&intel_dp->aux, DP_EDP_BACKLIGHT_BRIGHTNESS_MSB,
&read_val, sizeof(read_val)) < 0) { &read_val, sizeof(read_val)) < 0) {
...@@ -82,7 +82,7 @@ intel_dp_aux_set_backlight(const struct drm_connector_state *conn_state, u32 lev ...@@ -82,7 +82,7 @@ intel_dp_aux_set_backlight(const struct drm_connector_state *conn_state, u32 lev
{ {
struct intel_connector *connector = to_intel_connector(conn_state->connector); struct intel_connector *connector = to_intel_connector(conn_state->connector);
struct intel_dp *intel_dp = enc_to_intel_dp(&connector->encoder->base); struct intel_dp *intel_dp = enc_to_intel_dp(&connector->encoder->base);
uint8_t vals[2] = { 0x0 }; u8 vals[2] = { 0x0 };
vals[0] = level; vals[0] = level;
...@@ -178,7 +178,7 @@ static void intel_dp_aux_enable_backlight(const struct intel_crtc_state *crtc_st ...@@ -178,7 +178,7 @@ static void intel_dp_aux_enable_backlight(const struct intel_crtc_state *crtc_st
{ {
struct intel_connector *connector = to_intel_connector(conn_state->connector); struct intel_connector *connector = to_intel_connector(conn_state->connector);
struct intel_dp *intel_dp = enc_to_intel_dp(&connector->encoder->base); struct intel_dp *intel_dp = enc_to_intel_dp(&connector->encoder->base);
uint8_t dpcd_buf, new_dpcd_buf, edp_backlight_mode; u8 dpcd_buf, new_dpcd_buf, edp_backlight_mode;
if (drm_dp_dpcd_readb(&intel_dp->aux, if (drm_dp_dpcd_readb(&intel_dp->aux,
DP_EDP_BACKLIGHT_MODE_SET_REGISTER, &dpcd_buf) != 1) { DP_EDP_BACKLIGHT_MODE_SET_REGISTER, &dpcd_buf) != 1) {
......
...@@ -406,11 +406,11 @@ intel_panel_detect(struct drm_i915_private *dev_priv) ...@@ -406,11 +406,11 @@ intel_panel_detect(struct drm_i915_private *dev_priv)
* Return @source_val in range [@source_min..@source_max] scaled to range * Return @source_val in range [@source_min..@source_max] scaled to range
* [@target_min..@target_max]. * [@target_min..@target_max].
*/ */
static uint32_t scale(uint32_t source_val, static u32 scale(u32 source_val,
uint32_t source_min, uint32_t source_max, u32 source_min, u32 source_max,
uint32_t target_min, uint32_t target_max) u32 target_min, u32 target_max)
{ {
uint64_t target_val; u64 target_val;
WARN_ON(source_min > source_max); WARN_ON(source_min > source_max);
WARN_ON(target_min > target_max); WARN_ON(target_min > target_max);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment