[PATCH 16/19] drm/tegra: Pass along the format info from .fb_create() to drm_helper_mode_fill_fb_struct()

[Ville Syrjala]

From: Ville Syrjälä <ville.syrj...@linux.intel.com>

Plumb the format info from .fb_create() all the way to
drm_helper_mode_fill_fb_struct() to avoid the redundant
lookup.

For the fbdev case a manual drm_get_format_info() lookup
is needed.

Cc: Thierry Reding <thierry.red...@gmail.com>
Cc: Mikko Perttunen <mperttu...@nvidia.com>
Cc: linux-te...@vger.kernel.org
Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
---
 drivers/gpu/drm/tegra/drm.h   | 1 +
 drivers/gpu/drm/tegra/fb.c    | 5 +++--
 drivers/gpu/drm/tegra/fbdev.c | 4 +++-
 3 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/tegra/drm.h b/drivers/gpu/drm/tegra/drm.h
index 77e520c43f72..1dd3670f37db 100644
--- a/drivers/gpu/drm/tegra/drm.h
+++ b/drivers/gpu/drm/tegra/drm.h
@@ -185,6 +185,7 @@ bool tegra_fb_is_bottom_up(struct drm_framebuffer 
*framebuffer);
 int tegra_fb_get_tiling(struct drm_framebuffer *framebuffer,
                        struct tegra_bo_tiling *tiling);
 struct drm_framebuffer *tegra_fb_alloc(struct drm_device *drm,
+                                      const struct drm_format_info *info,
                                       const struct drm_mode_fb_cmd2 *mode_cmd,
                                       struct tegra_bo **planes,
                                       unsigned int num_planes);
diff --git a/drivers/gpu/drm/tegra/fb.c b/drivers/gpu/drm/tegra/fb.c
index d359683f5ce6..dd041089f797 100644
--- a/drivers/gpu/drm/tegra/fb.c
+++ b/drivers/gpu/drm/tegra/fb.c
@@ -102,6 +102,7 @@ static const struct drm_framebuffer_funcs tegra_fb_funcs = {
 };
 
 struct drm_framebuffer *tegra_fb_alloc(struct drm_device *drm,
+                                      const struct drm_format_info *info,
                                       const struct drm_mode_fb_cmd2 *mode_cmd,
                                       struct tegra_bo **planes,
                                       unsigned int num_planes)
@@ -114,7 +115,7 @@ struct drm_framebuffer *tegra_fb_alloc(struct drm_device 
*drm,
        if (!fb)
                return ERR_PTR(-ENOMEM);
 
-       drm_helper_mode_fill_fb_struct(drm, fb, NULL, mode_cmd);
+       drm_helper_mode_fill_fb_struct(drm, fb, info, mode_cmd);
 
        for (i = 0; i < fb->format->num_planes; i++)
                fb->obj[i] = &planes[i]->gem;
@@ -166,7 +167,7 @@ struct drm_framebuffer *tegra_fb_create(struct drm_device 
*drm,
                planes[i] = to_tegra_bo(gem);
        }
 
-       fb = tegra_fb_alloc(drm, cmd, planes, i);
+       fb = tegra_fb_alloc(drm, info, cmd, planes, i);
        if (IS_ERR(fb)) {
                err = PTR_ERR(fb);
                goto unreference;
diff --git a/drivers/gpu/drm/tegra/fbdev.c b/drivers/gpu/drm/tegra/fbdev.c
index cd9d798f8870..1b70f5e164af 100644
--- a/drivers/gpu/drm/tegra/fbdev.c
+++ b/drivers/gpu/drm/tegra/fbdev.c
@@ -106,7 +106,9 @@ int tegra_fbdev_driver_fbdev_probe(struct drm_fb_helper 
*helper,
                return PTR_ERR(info);
        }
 
-       fb = tegra_fb_alloc(drm, &cmd, &bo, 1);
+       fb = tegra_fb_alloc(drm,
+                           drm_get_format_info(drm, cmd.pixel_format, 
cmd.modifier[0]),
+                           &cmd, &bo, 1);
        if (IS_ERR(fb)) {
                err = PTR_ERR(fb);
                dev_err(drm->dev, "failed to allocate DRM framebuffer: %d\n",
-- 
2.49.0