Re: [Intel-gfx] [PATCH 11/21] drm/i915: Use drm_fb_helper_fill_info
On Wed, Mar 27, 2019 at 12:10 PM Joonas Lahtinen wrote: > > Quoting Daniel Vetter (2019-03-26 15:19:58) > > This changes the fb name from "inteldrmfb" to "i915drmfb". > > I'm fairly sure I already commented that is is Acked-by in the > case the other drivers are doing the same consolidation. > > Just have to make sure we don't break any IGTs that might depend > on the module name. Must have missed it, but either way CI is happy. There's also not a big chance we do have an igt which looks at this, since this is only the legacy fbdev driver name, not the native drm driver name. I'm not even sure we have a single testcase for that. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH 11/21] drm/i915: Use drm_fb_helper_fill_info
Quoting Daniel Vetter (2019-03-26 15:19:58) > This changes the fb name from "inteldrmfb" to "i915drmfb". I'm fairly sure I already commented that is is Acked-by in the case the other drivers are doing the same consolidation. Just have to make sure we don't break any IGTs that might depend on the module name. Regards, Joonas ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH 11/21] drm/i915: Use drm_fb_helper_fill_info
Den 26.03.2019 14.19, skrev Daniel Vetter: > This changes the fb name from "inteldrmfb" to "i915drmfb". > > v2: Rebase > > Signed-off-by: Daniel Vetter > Cc: Jani Nikula > Cc: Joonas Lahtinen > Cc: Rodrigo Vivi > Cc: intel-gfx@lists.freedesktop.org > --- Reviewed-by: Noralf Trønnes ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
[Intel-gfx] [PATCH 11/21] drm/i915: Use drm_fb_helper_fill_info
This changes the fb name from "inteldrmfb" to "i915drmfb". v2: Rebase Signed-off-by: Daniel Vetter Cc: Jani Nikula Cc: Joonas Lahtinen Cc: Rodrigo Vivi Cc: intel-gfx@lists.freedesktop.org --- drivers/gpu/drm/i915/intel_fbdev.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_fbdev.c b/drivers/gpu/drm/i915/intel_fbdev.c index f9ab94b99a3c..ef93c27e60b4 100644 --- a/drivers/gpu/drm/i915/intel_fbdev.c +++ b/drivers/gpu/drm/i915/intel_fbdev.c @@ -235,12 +235,8 @@ static int intelfb_create(struct drm_fb_helper *helper, goto out_unpin; } - info->par = helper; - ifbdev->helper.fb = fb; - strcpy(info->fix.id, "inteldrmfb"); - info->fbops = &intelfb_ops; /* setup aperture base/size for vesafb takeover */ @@ -259,8 +255,7 @@ static int intelfb_create(struct drm_fb_helper *helper, info->screen_base = vaddr; info->screen_size = vma->node.size; - drm_fb_helper_fill_fix(info, fb->pitches[0], fb->format->depth); - drm_fb_helper_fill_var(info, &ifbdev->helper, sizes->fb_width, sizes->fb_height); + drm_fb_helper_fill_info(info, &ifbdev->helper, sizes); /* If the object is shmemfs backed, it will have given us zeroed pages. * If the object is stolen however, it will be full of whatever -- 2.20.1 ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx