drm/i915/bios: remove an unnecessary temp variable
authorJani Nikula <jani.nikula@intel.com>
Thu, 28 Sep 2017 08:22:00 +0000 (11:22 +0300)
committerJani Nikula <jani.nikula@intel.com>
Tue, 10 Oct 2017 06:05:54 +0000 (09:05 +0300)
Prepare for merging parse_device_mapping() into
parse_general_definitions(). No functional changes.

Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/f1c3621e2622f4debdfb4a2f5c1959845754ac04.1506586821.git.jani.nikula@intel.com
drivers/gpu/drm/i915/intel_bios.c

index c91cdf1..2b72a7a 100644 (file)
@@ -1265,7 +1265,6 @@ parse_device_mapping(struct drm_i915_private *dev_priv,
 {
        const struct bdb_general_definitions *defs;
        const struct child_device_config *child;
-       struct child_device_config *child_dev_ptr;
        int i, child_device_num, count;
        u8 expected_size;
        u16 block_size;
@@ -1337,16 +1336,14 @@ parse_device_mapping(struct drm_i915_private *dev_priv,
                        continue;
                }
 
-               child_dev_ptr = dev_priv->vbt.child_dev + count;
-               count++;
-
                /*
                 * Copy as much as we know (sizeof) and is available
                 * (child_dev_size) of the child device. Accessing the data must
                 * depend on VBT version.
                 */
-               memcpy(child_dev_ptr, child,
+               memcpy(dev_priv->vbt.child_dev + count, child,
                       min_t(size_t, defs->child_dev_size, sizeof(*child)));
+               count++;
        }
        return;
 }