forked from Minki/linux
f3e9632cb6
drm_format_horz_chroma_subsampling and drm_format_vert_chroma_subsampling are basically a lookup in the drm_format_info table plus an access to the hsub and vsub fields of the appropriate entry. Most drivers are using this function while having access to the entry already, which means that we will perform an unnecessary lookup. Removing the call to these functions is therefore more efficient. Some drivers will not have access to that entry in the function, but in this case the overhead is minimal (we just have to call drm_format_info() to perform the lookup) and we can even avoid multiple, inefficient lookups in some places that need multiple fields from the drm_format_info structure. This is amplified by the fact that most of the time the callers will have to retrieve both the vsub and hsub fields, meaning that they would perform twice the lookup. Reviewed-by: Emil Velikov <emil.velikov@collabora.com> Reviewed-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> Reviewed-by: Philipp Zabel <p.zabel@pengutronix.de> Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> Link: https://patchwork.freedesktop.org/patch/msgid/6b3cceb8161e2c1d40c2681de99202328b0a8abc.1558002671.git-series.maxime.ripard@bootlin.com |
||
|---|---|---|
| .. | ||
| Kconfig | ||
| Makefile | ||
| vc4_bo.c | ||
| vc4_crtc.c | ||
| vc4_debugfs.c | ||
| vc4_dpi.c | ||
| vc4_drv.c | ||
| vc4_drv.h | ||
| vc4_dsi.c | ||
| vc4_fence.c | ||
| vc4_gem.c | ||
| vc4_hdmi.c | ||
| vc4_hvs.c | ||
| vc4_irq.c | ||
| vc4_kms.c | ||
| vc4_packet.h | ||
| vc4_perfmon.c | ||
| vc4_plane.c | ||
| vc4_qpu_defines.h | ||
| vc4_regs.h | ||
| vc4_render_cl.c | ||
| vc4_trace_points.c | ||
| vc4_trace.h | ||
| vc4_txp.c | ||
| vc4_v3d.c | ||
| vc4_validate_shaders.c | ||
| vc4_validate.c | ||
| vc4_vec.c | ||