drm/i915: refactor stepping info retrieval
authorJani Nikula <jani.nikula@intel.com>
Tue, 20 Oct 2015 12:38:33 +0000 (15:38 +0300)
committerJani Nikula <jani.nikula@intel.com>
Thu, 12 Nov 2015 14:29:43 +0000 (16:29 +0200)
Have only one if ladder for platforms and only one range check for
size. Makes it easier to handle new platforms. Remove the use of
negative return values in char, which might underflow to be positive for
some negative error codes.

Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1445344713-1407-3-git-send-email-jani.nikula@intel.com
drivers/gpu/drm/i915/intel_csr.c

index e9eb867698d344953e5c87f7c2e4ec00ccfbbc20..ed9d9667736e650539d43164ba6b418774c45c5a 100644 (file)
@@ -177,28 +177,25 @@ static const struct stepping_info bxt_stepping_info[] = {
        {'B', '0'}, {'B', '1'}, {'B', '2'}
 };
 
-static char intel_get_stepping(struct drm_device *dev)
+static const struct stepping_info *intel_get_stepping_info(struct drm_device *dev)
 {
-       if (IS_SKYLAKE(dev) && (dev->pdev->revision <
-                       ARRAY_SIZE(skl_stepping_info)))
-               return skl_stepping_info[dev->pdev->revision].stepping;
-       else if (IS_BROXTON(dev) && (dev->pdev->revision <
-                               ARRAY_SIZE(bxt_stepping_info)))
-               return bxt_stepping_info[dev->pdev->revision].stepping;
-       else
-               return -ENODATA;
-}
+       const struct stepping_info *si;
+       unsigned int size;
+
+       if (IS_SKYLAKE(dev)) {
+               size = ARRAY_SIZE(skl_stepping_info);
+               si = skl_stepping_info;
+       } else if (IS_BROXTON(dev)) {
+               size = ARRAY_SIZE(bxt_stepping_info);
+               si = bxt_stepping_info;
+       } else {
+               return NULL;
+       }
 
-static char intel_get_substepping(struct drm_device *dev)
-{
-       if (IS_SKYLAKE(dev) && (dev->pdev->revision <
-                       ARRAY_SIZE(skl_stepping_info)))
-               return skl_stepping_info[dev->pdev->revision].substepping;
-       else if (IS_BROXTON(dev) && (dev->pdev->revision <
-                       ARRAY_SIZE(bxt_stepping_info)))
-               return bxt_stepping_info[dev->pdev->revision].substepping;
-       else
-               return -ENODATA;
+       if (INTEL_REVID(dev) < size)
+               return si + INTEL_REVID(dev);
+
+       return NULL;
 }
 
 /**
@@ -285,8 +282,8 @@ static void finish_csr_load(const struct firmware *fw, void *context)
        struct intel_package_header *package_header;
        struct intel_dmc_header *dmc_header;
        struct intel_csr *csr = &dev_priv->csr;
-       char stepping = intel_get_stepping(dev);
-       char substepping = intel_get_substepping(dev);
+       const struct stepping_info *stepping_info = intel_get_stepping_info(dev);
+       char stepping, substepping;
        uint32_t dmc_offset = CSR_DEFAULT_FW_OFFSET, readcount = 0, nbytes;
        uint32_t i;
        uint32_t *dmc_payload;
@@ -295,11 +292,14 @@ static void finish_csr_load(const struct firmware *fw, void *context)
        if (!fw)
                goto out;
 
-       if ((stepping == -ENODATA) || (substepping == -ENODATA)) {
+       if (!stepping_info) {
                DRM_ERROR("Unknown stepping info, firmware loading failed\n");
                goto out;
        }
 
+       stepping = stepping_info->stepping;
+       substepping = stepping_info->substepping;
+
        /* Extract CSS Header information*/
        css_header = (struct intel_css_header *)fw->data;
        if (sizeof(struct intel_css_header) !=