On Wed, Dec 14, 2016 at 11:37:46PM +0200, Ville Syrjälä wrote: > On Fri, Nov 18, 2016 at 09:52:36PM +0200, ville.syrjala@xxxxxxxxxxxxxxx wrote: > > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > > > Second installment of my effort to remove the duplicated > > depth/bpp/pixel_format from drm_framebuffer and just use > > struct drm_format_info instead. > > > > I tried to address all of the review feedback, and collect > > up all the r-bs I already got. Thanks for the review, guys. > > > > Changes since the last version are roughly: > > * drm_framebuffer_init() now fails if the fb isn't properly prepared > > * Applied mode cocciry all over to use fb->format more extensively > > * Dropped a few i915 specific patches that were taken care of the > > previous item > > * Took up Laurent's idea that we can just compare the fb->format > > pointers instead of comparing the fb->format->format values > > * Added a new .get_format_info() hooks for drivers to provide custom > > format information + an quick example patch how we'd hook it up > > for i915 render compression support > > > > Link to the previous version: > > https://lists.freedesktop.org/archives/dri-devel/2016-November/124135.html > > > > Entire series is available here: > > git://github.com/vsyrjala/linux.git fb_format_dedup_2 > > > > Cc: Alex Deucher <alexander.deucher@xxxxxxx> > > Cc: Alexey Brodkin <abrodkin@xxxxxxxxxxxx> > > Cc: Ben Skeggs <bskeggs@xxxxxxxxxx> > > Cc: Ben Widawsky <ben@xxxxxxxxxxxx> > > Cc: Brian Starkey <brian.starkey@xxxxxxx> > > Cc: "Christian König" <christian.koenig@xxxxxxx> > > Cc: Dave Airlie <airlied@xxxxxxxxxx> > > Cc: Gerd Hoffmann <kraxel@xxxxxxxxxx> > > Cc: intel-gfx@xxxxxxxxxxxxxxxxxxxxx > > Cc: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > > Cc: linux-graphics-maintainer@xxxxxxxxxx > > Cc: Liviu Dudau <liviu.dudau@xxxxxxx> > > Cc: Mali DP Maintainers <malidp@xxxxxxxxxxxx> > > Cc: Patrik Jakobsson <patrik.r.jakobsson@xxxxxxxxx> > > Cc: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx> > > Cc: Sinclair Yeh <syeh@xxxxxxxxxx> > > Cc: Thomas Hellstrom <thellstrom@xxxxxxxxxx> > > > > Ville Syrjälä (37): > > drm/i915: Add local 'fb' variables > > drm/radeon: Add local 'fb' variables > > drm/radeon: Use DIV_ROUND_UP() > > drm/mgag200: Add local 'fb' variable > > drm/ast: Add local 'fb' variables > > drm/gma500: Add some local 'fb' variables > > drm/cirrus: Add some local 'fb' variables > > drm/arcpgu: Add local 'fb' variables > > drm/arm: Add local 'fb' variables > > drm/nouveau: Fix crtc->primary->fb vs. drm_fb fail > > drm/nouveau: Add local 'fb' variables > > I've pushed up to here to drm-misc-next. Thanks for the reviews. > > I re-ran spatch to regenerate some of the later patches as there had > been a bit of churn in the code. I've reposted the changed patches, > and if no one screams I'll be pushing the rest tomorrowish. > > > drm/vmwgfx: Populate fb->dev before drm_framebuffer_init() > > drm: Pass 'dev' to drm_helper_mode_fill_fb_struct() > > drm/vmwgfx: Populate fb->pixel_format > > drm/qxl: Call drm_helper_mode_fill_fb_struct() before > > drm_framebuffer_init() > > drm/virtio: Call drm_helper_mode_fill_fb_struct() before > > drm_framebuffer_init() > > drm/i915: Set fb->dev early on for inherited fbs > > drm: Populate fb->dev from drm_helper_mode_fill_fb_struct() > > drm: Store a pointer to drm_format_info under drm_framebuffer > > drm/vmwgfx: Populate fb->format correctly > > drm/i915: Populate fb->format early for inherited fbs > > drm: Reject fbs w/o format info in drm_framebuffer_init() > > drm: Replace drm_format_num_planes() with fb->format->num_planes > > drm/i915: Eliminate the ugly 'fb?:' constructs from the ilk/skl wm > > code > > drm: Replace drm_format_plane_cpp() with fb->format->cpp[] > > drm/fb_cma_helper: Replace drm_format_info() with fb->format > > drm/nouveau: Use fb->format rather than drm_format_info() > > drm/i915: Store a pointer to the pixel format info for fbc > > drm: Add drm_framebuffer_plane_{width,height}() > > drm/i915: Use drm_framebuffer_plane_{width,height}() where possible > > drm: Nuke fb->depth > > drm: Nuke fb->bits_per_pixel > > drm: Nuke fb->pixel_format > > drm: Replace 'format->format' comparisons to just 'format' comparisons > > drm: Eliminate the useless "non-RGB fb" debug message And I've just pushed up to here (minus the vmvgfx patches which dropped out due to Daniel's earlier refactorin). > > drm: Add mode_config .get_format_info() hook > > drm/i915: Implement .get_format_info() hook for CCS I'll hang on to these until we get the i915 CCS thing into shape. Thanks for the reviews everyone. > > > > drivers/gpu/drm/amd/amdgpu/amdgpu_display.c | 2 +- > > drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c | 4 +- > > drivers/gpu/drm/amd/amdgpu/dce_v10_0.c | 6 +- > > drivers/gpu/drm/amd/amdgpu/dce_v11_0.c | 6 +- > > drivers/gpu/drm/amd/amdgpu/dce_v6_0.c | 6 +- > > drivers/gpu/drm/amd/amdgpu/dce_v8_0.c | 6 +- > > drivers/gpu/drm/arc/arcpgu_crtc.c | 3 +- > > drivers/gpu/drm/arm/hdlcd_crtc.c | 18 ++-- > > drivers/gpu/drm/arm/malidp_planes.c | 10 +-- > > drivers/gpu/drm/armada/armada_crtc.c | 6 +- > > drivers/gpu/drm/armada/armada_fb.c | 2 +- > > drivers/gpu/drm/armada/armada_fbdev.c | 5 +- > > drivers/gpu/drm/armada/armada_overlay.c | 6 +- > > drivers/gpu/drm/ast/ast_fb.c | 4 +- > > drivers/gpu/drm/ast/ast_main.c | 2 +- > > drivers/gpu/drm/ast/ast_mode.c | 16 ++-- > > drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_layer.c | 2 +- > > drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 22 ++--- > > drivers/gpu/drm/bochs/bochs_fbdev.c | 2 +- > > drivers/gpu/drm/bochs/bochs_mm.c | 2 +- > > drivers/gpu/drm/cirrus/cirrus_fbdev.c | 6 +- > > drivers/gpu/drm/cirrus/cirrus_main.c | 2 +- > > drivers/gpu/drm/cirrus/cirrus_mode.c | 9 +- > > drivers/gpu/drm/drm_atomic.c | 8 +- > > drivers/gpu/drm/drm_crtc.c | 4 +- > > drivers/gpu/drm/drm_crtc_helper.c | 3 +- > > drivers/gpu/drm/drm_fb_cma_helper.c | 13 ++- > > drivers/gpu/drm/drm_fb_helper.c | 10 +-- > > drivers/gpu/drm/drm_fourcc.c | 25 ++++++ > > drivers/gpu/drm/drm_framebuffer.c | 62 +++++++++++-- > > drivers/gpu/drm/drm_modeset_helper.c | 22 +---- > > drivers/gpu/drm/drm_plane.c | 6 +- > > drivers/gpu/drm/exynos/exynos5433_drm_decon.c | 6 +- > > drivers/gpu/drm/exynos/exynos7_drm_decon.c | 8 +- > > drivers/gpu/drm/exynos/exynos_drm_fb.c | 2 +- > > drivers/gpu/drm/exynos/exynos_drm_fbdev.c | 6 +- > > drivers/gpu/drm/exynos/exynos_drm_fimd.c | 4 +- > > drivers/gpu/drm/exynos/exynos_mixer.c | 12 +-- > > drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_plane.c | 4 +- > > drivers/gpu/drm/gma500/accel_2d.c | 2 +- > > drivers/gpu/drm/gma500/framebuffer.c | 6 +- > > drivers/gpu/drm/gma500/gma_display.c | 13 +-- > > drivers/gpu/drm/gma500/mdfld_intel_display.c | 17 ++-- > > drivers/gpu/drm/gma500/oaktrail_crtc.c | 13 +-- > > drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c | 6 +- > > drivers/gpu/drm/i915/i915_debugfs.c | 11 +-- > > drivers/gpu/drm/i915/i915_drv.h | 4 +- > > drivers/gpu/drm/i915/intel_atomic_plane.c | 4 +- > > drivers/gpu/drm/i915/intel_display.c | 115 ++++++++++++++++-------- > > drivers/gpu/drm/i915/intel_fbc.c | 14 +-- > > drivers/gpu/drm/i915/intel_fbdev.c | 10 +-- > > drivers/gpu/drm/i915/intel_overlay.c | 26 +++--- > > drivers/gpu/drm/i915/intel_pm.c | 67 +++++++------- > > drivers/gpu/drm/i915/intel_sprite.c | 14 +-- > > drivers/gpu/drm/imx/ipuv3-plane.c | 40 ++++----- > > drivers/gpu/drm/mediatek/mtk_drm_fb.c | 2 +- > > drivers/gpu/drm/mediatek/mtk_drm_plane.c | 4 +- > > drivers/gpu/drm/mgag200/mgag200_fb.c | 4 +- > > drivers/gpu/drm/mgag200/mgag200_main.c | 2 +- > > drivers/gpu/drm/mgag200/mgag200_mode.c | 23 ++--- > > drivers/gpu/drm/msm/mdp/mdp4/mdp4_plane.c | 2 +- > > drivers/gpu/drm/msm/mdp/mdp5/mdp5_plane.c | 4 +- > > drivers/gpu/drm/msm/msm_fb.c | 12 +-- > > drivers/gpu/drm/msm/msm_fbdev.c | 2 +- > > drivers/gpu/drm/nouveau/dispnv04/crtc.c | 17 ++-- > > drivers/gpu/drm/nouveau/dispnv04/dfp.c | 3 +- > > drivers/gpu/drm/nouveau/dispnv04/overlay.c | 8 +- > > drivers/gpu/drm/nouveau/nouveau_display.c | 4 +- > > drivers/gpu/drm/nouveau/nouveau_fbcon.c | 3 +- > > drivers/gpu/drm/nouveau/nv50_display.c | 14 ++- > > drivers/gpu/drm/omapdrm/omap_fb.c | 12 +-- > > drivers/gpu/drm/omapdrm/omap_fbdev.c | 2 +- > > drivers/gpu/drm/qxl/qxl_display.c | 2 +- > > drivers/gpu/drm/qxl/qxl_draw.c | 2 +- > > drivers/gpu/drm/qxl/qxl_fb.c | 5 +- > > drivers/gpu/drm/radeon/atombios_crtc.c | 19 ++-- > > drivers/gpu/drm/radeon/r100.c | 10 ++- > > drivers/gpu/drm/radeon/radeon_display.c | 8 +- > > drivers/gpu/drm/radeon/radeon_fb.c | 4 +- > > drivers/gpu/drm/radeon/radeon_legacy_crtc.c | 16 ++-- > > drivers/gpu/drm/rcar-du/rcar_du_plane.c | 4 +- > > drivers/gpu/drm/rcar-du/rcar_du_vsp.c | 4 +- > > drivers/gpu/drm/rockchip/rockchip_drm_fb.c | 2 +- > > drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c | 5 +- > > drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 22 ++--- > > drivers/gpu/drm/shmobile/shmob_drm_crtc.c | 6 +- > > drivers/gpu/drm/shmobile/shmob_drm_plane.c | 4 +- > > drivers/gpu/drm/sti/sti_gdp.c | 10 +-- > > drivers/gpu/drm/sti/sti_hqvdp.c | 2 +- > > drivers/gpu/drm/sun4i/sun4i_backend.c | 5 +- > > drivers/gpu/drm/tegra/dc.c | 8 +- > > drivers/gpu/drm/tegra/drm.c | 5 +- > > drivers/gpu/drm/tegra/fb.c | 6 +- > > drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 4 +- > > drivers/gpu/drm/tilcdc/tilcdc_plane.c | 4 +- > > drivers/gpu/drm/udl/udl_fb.c | 6 +- > > drivers/gpu/drm/vc4/vc4_plane.c | 8 +- > > drivers/gpu/drm/virtio/virtgpu_display.c | 3 +- > > drivers/gpu/drm/virtio/virtgpu_fb.c | 4 +- > > drivers/gpu/drm/vmwgfx/vmwgfx_fb.c | 13 +-- > > drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 29 ++++-- > > drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c | 5 +- > > drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c | 4 +- > > drivers/gpu/drm/vmwgfx/vmwgfx_stdu.c | 2 +- > > drivers/gpu/drm/zte/zx_plane.c | 4 +- > > include/drm/drm_fourcc.h | 6 ++ > > include/drm/drm_framebuffer.h | 27 +++--- > > include/drm/drm_mode_config.h | 15 ++++ > > include/drm/drm_modeset_helper.h | 3 +- > > include/uapi/drm/drm_fourcc.h | 3 + > > 110 files changed, 641 insertions(+), 470 deletions(-) > > > > -- > > 2.7.4 > > -- > Ville Syrjälä > Intel OTC > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Ville Syrjälä Intel OTC _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel