mirror of https://github.com/torvalds/linux.git
451921e7bb
The functions drm_framebuffer_plane_{width,height} and
fb_plane_{width,height} do exactly the same job of its
equivalents drm_format_info_plane_{width,height} from drm_fourcc.
The only reason to have these functions on drm_framebuffer
would be if they would added a abstraction layer to call it just
passing a drm_framebuffer pointer and the desired plane index,
which is not the case, where these functions actually implements
just part of it. In the actual implementation, every call to both
drm_framebuffer_plane_{width,height} and fb_plane_{width,height} should
pass some drm_framebuffer attribute, which is the same as calling the
drm_format_info_plane_{width,height} functions.
The drm_format_info_pane_{width,height} functions are much more
consistent in both its implementation and its location on code. The
kind of calculation that they do is intrinsically derivated from the
drm_format_info struct and has not to do with drm_framebuffer, except
by the potential motivation described above, which is still not a good
justification to have drm_framebuffer functions to calculate it.
So, replace each drm_framebuffer_plane_{width,height} and
fb_plane_{width,height} call to drm_format_info_plane_{width,height}
and remove them.
Signed-off-by: Carlos Eduardo Gallo Filho <gcarlos@disroot.org>
Reviewed-by: André Almeida <andrealmeid@igalia.com>
Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
Link: https://patchwork.freedesktop.org/patch/msgid/20230926141519.9315-3-gcarlos@disroot.org
|
||
|---|---|---|
| .. | ||
| bridge | ||
| display | ||
| i2c | ||
| ttm | ||
| amd_asic_type.h | ||
| drm_accel.h | ||
| drm_aperture.h | ||
| drm_atomic.h | ||
| drm_atomic_helper.h | ||
| drm_atomic_state_helper.h | ||
| drm_atomic_uapi.h | ||
| drm_audio_component.h | ||
| drm_auth.h | ||
| drm_blend.h | ||
| drm_bridge.h | ||
| drm_bridge_connector.h | ||
| drm_buddy.h | ||
| drm_cache.h | ||
| drm_client.h | ||
| drm_color_mgmt.h | ||
| drm_connector.h | ||
| drm_crtc.h | ||
| drm_crtc_helper.h | ||
| drm_damage_helper.h | ||
| drm_debugfs.h | ||
| drm_debugfs_crc.h | ||
| drm_device.h | ||
| drm_displayid.h | ||
| drm_drv.h | ||
| drm_edid.h | ||
| drm_encoder.h | ||
| drm_encoder_slave.h | ||
| drm_exec.h | ||
| drm_fb_dma_helper.h | ||
| drm_fb_helper.h | ||
| drm_fbdev_dma.h | ||
| drm_fbdev_generic.h | ||
| drm_file.h | ||
| drm_fixed.h | ||
| drm_flip_work.h | ||
| drm_format_helper.h | ||
| drm_fourcc.h | ||
| drm_framebuffer.h | ||
| drm_gem.h | ||
| drm_gem_atomic_helper.h | ||
| drm_gem_dma_helper.h | ||
| drm_gem_framebuffer_helper.h | ||
| drm_gem_shmem_helper.h | ||
| drm_gem_ttm_helper.h | ||
| drm_gem_vram_helper.h | ||
| drm_gpuvm.h | ||
| drm_ioctl.h | ||
| drm_kunit_helpers.h | ||
| drm_lease.h | ||
| drm_legacy.h | ||
| drm_managed.h | ||
| drm_mipi_dbi.h | ||
| drm_mipi_dsi.h | ||
| drm_mm.h | ||
| drm_mode_config.h | ||
| drm_mode_object.h | ||
| drm_modes.h | ||
| drm_modeset_helper.h | ||
| drm_modeset_helper_vtables.h | ||
| drm_modeset_lock.h | ||
| drm_module.h | ||
| drm_of.h | ||
| drm_panel.h | ||
| drm_pciids.h | ||
| drm_plane.h | ||
| drm_plane_helper.h | ||
| drm_prime.h | ||
| drm_print.h | ||
| drm_privacy_screen_consumer.h | ||
| drm_privacy_screen_driver.h | ||
| drm_privacy_screen_machine.h | ||
| drm_probe_helper.h | ||
| drm_property.h | ||
| drm_rect.h | ||
| drm_self_refresh_helper.h | ||
| drm_simple_kms_helper.h | ||
| drm_suballoc.h | ||
| drm_syncobj.h | ||
| drm_sysfs.h | ||
| drm_util.h | ||
| drm_utils.h | ||
| drm_vblank.h | ||
| drm_vblank_work.h | ||
| drm_vma_manager.h | ||
| drm_writeback.h | ||
| gma_drm.h | ||
| gpu_scheduler.h | ||
| gud.h | ||
| i915_component.h | ||
| i915_drm.h | ||
| i915_gsc_proxy_mei_interface.h | ||
| i915_hdcp_interface.h | ||
| i915_pciids.h | ||
| i915_pxp_tee_interface.h | ||
| intel-gtt.h | ||
| intel_lpe_audio.h | ||
| spsc_queue.h | ||
| task_barrier.h | ||