drm/i915/DMC/KBL: Load DMC on KBL using the no_stepping_info array
authorAnusha Srivatsa <anusha.srivatsa@intel.com>
Tue, 25 Oct 2016 00:28:21 +0000 (17:28 -0700)
committerRodrigo Vivi <rodrigo.vivi@intel.com>
Wed, 26 Oct 2016 21:20:53 +0000 (14:20 -0700)
Currently, for display there is only one DMC image for KBL.
Remove the stepping_info table for KBL and use the no_stepping_info
array for loading the firmware.

v2: Removed the block of code as pointed out by Rodrigo to make the
loads as generic as possible.

Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Anusha Srivatsa <anusha.srivatsa@intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Acked-by: Imre Deak <imre.deak@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1477355301-7035-1-git-send-email-anusha.srivatsa@intel.com
drivers/gpu/drm/i915/intel_csr.c

index 1ea0e1f433971ea201debcff43e36d6cd75a35f4..d7a04bca8c28f8c9e583e0ebd09d558c61523cd6 100644 (file)
@@ -168,12 +168,6 @@ struct stepping_info {
        char substepping;
 };
 
-static const struct stepping_info kbl_stepping_info[] = {
-       {'A', '0'}, {'B', '0'}, {'C', '0'},
-       {'D', '0'}, {'E', '0'}, {'F', '0'},
-       {'G', '0'}, {'H', '0'}, {'I', '0'},
-};
-
 static const struct stepping_info skl_stepping_info[] = {
        {'A', '0'}, {'B', '0'}, {'C', '0'},
        {'D', '0'}, {'E', '0'}, {'F', '0'},
@@ -194,10 +188,7 @@ intel_get_stepping_info(struct drm_i915_private *dev_priv)
        const struct stepping_info *si;
        unsigned int size;
 
-       if (IS_KABYLAKE(dev_priv)) {
-               size = ARRAY_SIZE(kbl_stepping_info);
-               si = kbl_stepping_info;
-       } else if (IS_SKYLAKE(dev_priv)) {
+       if (IS_SKYLAKE(dev_priv)) {
                size = ARRAY_SIZE(skl_stepping_info);
                si = skl_stepping_info;
        } else if (IS_BROXTON(dev_priv)) {