drm/i914: s/l_info/dimm_l/ etc.
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Wed, 6 Mar 2019 20:35:48 +0000 (22:35 +0200)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Thu, 7 Mar 2019 21:19:37 +0000 (23:19 +0200)
Rename the dimm info structs for clarity.

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

index 08914761303db376f08555fddfa334b4b27eaedf..54b40c9ea1f72beda01e61f91697d0629a9ff6ab 100644 (file)
@@ -1130,24 +1130,24 @@ static int
 skl_dram_get_channel_info(struct dram_channel_info *ch,
                          int channel, u32 val)
 {
-       skl_dram_get_dimm_info(&ch->l_info, channel, 'L', val & 0xffff);
-       skl_dram_get_dimm_info(&ch->s_info, channel, 'S', val >> 16);
+       skl_dram_get_dimm_info(&ch->dimm_l, channel, 'L', val & 0xffff);
+       skl_dram_get_dimm_info(&ch->dimm_s, channel, 'S', val >> 16);
 
-       if (ch->l_info.size == 0 && ch->s_info.size == 0) {
+       if (ch->dimm_l.size == 0 && ch->dimm_s.size == 0) {
                DRM_DEBUG_KMS("CH%u not populated\n", channel);
                return -EINVAL;
        }
 
-       if (ch->l_info.ranks == 2 || ch->s_info.ranks == 2)
+       if (ch->dimm_l.ranks == 2 || ch->dimm_s.ranks == 2)
                ch->ranks = 2;
-       else if (ch->l_info.ranks == 1 && ch->s_info.ranks == 1)
+       else if (ch->dimm_l.ranks == 1 && ch->dimm_s.ranks == 1)
                ch->ranks = 2;
        else
                ch->ranks = 1;
 
        ch->is_16gb_dimm =
-               skl_is_16gb_dimm(&ch->l_info) ||
-               skl_is_16gb_dimm(&ch->s_info);
+               skl_is_16gb_dimm(&ch->dimm_l) ||
+               skl_is_16gb_dimm(&ch->dimm_s);
 
        DRM_DEBUG_KMS("CH%u ranks: %u, 16Gb DIMMs: %s\n",
                      channel, ch->ranks, yesno(ch->is_16gb_dimm));
@@ -1160,8 +1160,8 @@ intel_is_dram_symmetric(const struct dram_channel_info *ch0,
                        const struct dram_channel_info *ch1)
 {
        return !memcmp(ch0, ch1, sizeof(*ch0)) &&
-               (ch0->s_info.size == 0 ||
-                !memcmp(&ch0->l_info, &ch0->s_info, sizeof(ch0->l_info)));
+               (ch0->dimm_s.size == 0 ||
+                !memcmp(&ch0->dimm_l, &ch0->dimm_s, sizeof(ch0->dimm_l)));
 }
 
 static int
index 84e3e3a0bd1e95663c8a6f682955f13e7e3f7610..41b7530b3d05e79055a81b443d09f28459f5fe9a 100644 (file)
@@ -2057,7 +2057,7 @@ struct dram_dimm_info {
 };
 
 struct dram_channel_info {
-       struct dram_dimm_info l_info, s_info;
+       struct dram_dimm_info dimm_l, dimm_s;
        u8 ranks;
        bool is_16gb_dimm;
 };