lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <cover.92acdec88ee4c280cb74e08ea22f0075e5fa055c.1553032382.git-series.maxime.ripard@bootlin.com>
Date:   Tue, 19 Mar 2019 22:57:05 +0100
From:   Maxime Ripard <maxime.ripard@...tlin.com>
To:     Daniel Vetter <daniel.vetter@...el.com>,
        David Airlie <airlied@...ux.ie>,
        Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
        Sean Paul <seanpaul@...omium.org>,
        Maxime Ripard <maxime.ripard@...tlin.com>,
        Mauro Carvalho Chehab <mchehab@...nel.org>
Cc:     Sakari Ailus <sakari.ailus@...ux.intel.com>,
        Hans Verkuil <hans.verkuil@...co.com>,
        Laurent Pinchart <laurent.pinchart@...asonboard.com>,
        Thomas Petazzoni <thomas.petazzoni@...tlin.com>,
        Paul Kocialkowski <paul.kocialkowski@...tlin.com>,
        dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
        linux-media@...r.kernel.org
Subject: [RFC PATCH 00/20] drm: Split out the formats API and move it to a common place

Hi,

DRM comes with an extensive format support to retrieve the various
parameters associated with a given format (such as the subsampling, or the
bits per pixel), as well as some helpers and utilities to ease the driver
development.

v4l2, on the other side, doesn't provide such facilities, leaving each
driver reimplement a subset of the formats parameters for the one supported
by that particular driver. This leads to a lot of duplication and
boilerplate code in the v4l2 drivers.

This series tries to address this by moving the DRM format API into lib and
turning it into a more generic API. In order to do this, we've needed to do
some preliminary changes on the DRM drivers, then moved the API and finally
converted a v4l2 driver to give an example of how such library could be
used.

Let me know what you think,
Maxime

Maxime Ripard (20):
  drm: Remove users of drm_format_num_planes
  drm: Remove users of drm_format_(horz|vert)_chroma_subsampling
  drm/fourcc: Pass the format_info pointer to drm_format_plane_cpp
  drm/fourcc: Pass the format_info pointer to drm_format_plane_width/height
  drm: Replace instances of drm_format_info by drm_get_format_info
  lib: Add video format information library
  drm/fb: Move from drm_format_info to image_format_info
  drm/malidp: Convert to generic image format library
  drm/client: Convert to generic image format library
  drm/exynos: Convert to generic image format library
  drm/i915: Convert to generic image format library
  drm/ipuv3: Convert to generic image format library
  drm/msm: Convert to generic image format library
  drm/omap: Convert to generic image format library
  drm/rockchip: Convert to generic image format library
  drm/tegra: Convert to generic image format library
  drm/fourcc: Remove old DRM format API
  lib: image-formats: Add v4l2 formats support
  lib: image-formats: Add more functions
  media: sun6i: Convert to the image format API

 drivers/gpu/drm/Kconfig                             |   2 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c              |   4 +-
 drivers/gpu/drm/arm/malidp_drv.c                    |   5 +-
 drivers/gpu/drm/arm/malidp_hw.c                     |   4 +-
 drivers/gpu/drm/arm/malidp_mw.c                     |   2 +-
 drivers/gpu/drm/arm/malidp_planes.c                 |   8 +-
 drivers/gpu/drm/armada/armada_fb.c                  |   3 +-
 drivers/gpu/drm/armada/armada_overlay.c             |   3 +-
 drivers/gpu/drm/armada/armada_plane.c               |   3 +-
 drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c     |  13 +-
 drivers/gpu/drm/bochs/bochs.h                       |   4 +-
 drivers/gpu/drm/bochs/bochs_hw.c                    |   3 +-
 drivers/gpu/drm/cirrus/cirrus_fbdev.c               |   4 +-
 drivers/gpu/drm/cirrus/cirrus_main.c                |   4 +-
 drivers/gpu/drm/drm_atomic.c                        |   1 +-
 drivers/gpu/drm/drm_client.c                        |   8 +-
 drivers/gpu/drm/drm_crtc.c                          |   1 +-
 drivers/gpu/drm/drm_fb_cma_helper.c                 |   5 +-
 drivers/gpu/drm/drm_fb_helper.c                     |  15 +-
 drivers/gpu/drm/drm_fourcc.c                        | 318 +-----
 drivers/gpu/drm/drm_framebuffer.c                   |  11 +-
 drivers/gpu/drm/drm_gem_framebuffer_helper.c        |   5 +-
 drivers/gpu/drm/drm_plane.c                         |   1 +-
 drivers/gpu/drm/exynos/exynos_drm_fb.c              |   3 +-
 drivers/gpu/drm/exynos/exynos_drm_ipp.c             |   2 +-
 drivers/gpu/drm/exynos/exynos_drm_ipp.h             |   4 +-
 drivers/gpu/drm/exynos/exynos_drm_scaler.c          |   3 +-
 drivers/gpu/drm/gma500/framebuffer.c                |   4 +-
 drivers/gpu/drm/i915/i915_drv.h                     |   6 +-
 drivers/gpu/drm/i915/intel_display.c                |  15 +-
 drivers/gpu/drm/i915/intel_sprite.c                 |   3 +-
 drivers/gpu/drm/imx/ipuv3-plane.c                   |  20 +-
 drivers/gpu/drm/mediatek/mtk_drm_fb.c               |   8 +-
 drivers/gpu/drm/meson/meson_overlay.c               |  14 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_formats.c         |  11 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c           |  10 +-
 drivers/gpu/drm/msm/disp/mdp5/mdp5_crtc.c           |   4 +-
 drivers/gpu/drm/msm/disp/mdp5/mdp5_plane.c          |  25 +-
 drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c            |   8 +-
 drivers/gpu/drm/msm/msm_fb.c                        |  18 +-
 drivers/gpu/drm/omapdrm/dss/dispc.c                 |   9 +-
 drivers/gpu/drm/omapdrm/omap_fb.c                   |  15 +-
 drivers/gpu/drm/radeon/radeon_fb.c                  |   4 +-
 drivers/gpu/drm/rockchip/rockchip_drm_fb.c          |  17 +-
 drivers/gpu/drm/rockchip/rockchip_drm_vop.c         |  14 +-
 drivers/gpu/drm/selftests/Makefile                  |   3 +-
 drivers/gpu/drm/selftests/drm_modeset_selftests.h   |   3 +-
 drivers/gpu/drm/selftests/test-drm_format.c         | 280 +----
 drivers/gpu/drm/selftests/test-drm_modeset_common.h |   3 +-
 drivers/gpu/drm/stm/ltdc.c                          |   2 +-
 drivers/gpu/drm/sun4i/sun4i_backend.c               |   7 +-
 drivers/gpu/drm/sun4i/sun4i_frontend.c              |  19 +-
 drivers/gpu/drm/sun4i/sun8i_ui_layer.c              |   2 +-
 drivers/gpu/drm/sun4i/sun8i_vi_layer.c              |   6 +-
 drivers/gpu/drm/sun4i/sun8i_vi_scaler.c             |   6 +-
 drivers/gpu/drm/sun4i/sun8i_vi_scaler.h             |   5 +-
 drivers/gpu/drm/tegra/fb.c                          |  14 +-
 drivers/gpu/drm/tegra/plane.c                       |   4 +-
 drivers/gpu/drm/tinydrm/core/tinydrm-helpers.c      |   2 +-
 drivers/gpu/drm/vc4/vc4_plane.c                     |  15 +-
 drivers/gpu/drm/zte/zx_plane.c                      |   6 +-
 drivers/gpu/ipu-v3/ipu-pre.c                        |   3 +-
 drivers/gpu/ipu-v3/ipu-prg.c                        |   3 +-
 drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.c  |  88 +-
 drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.h  |  46 +-
 include/drm/drm_fourcc.h                            | 219 +---
 include/drm/drm_framebuffer.h                       |   3 +-
 include/drm/drm_mode_config.h                       |   4 +-
 include/linux/image-formats.h                       | 253 ++++-
 lib/Kconfig                                         |   7 +-
 lib/Makefile                                        |   3 +-
 lib/image-formats-selftests.c                       | 326 +++++-
 lib/image-formats.c                                 | 869 +++++++++++++-
 73 files changed, 1719 insertions(+), 1126 deletions(-)
 delete mode 100644 drivers/gpu/drm/selftests/test-drm_format.c
 create mode 100644 include/linux/image-formats.h
 create mode 100644 lib/image-formats-selftests.c
 create mode 100644 lib/image-formats.c

base-commit: 98f41dc3b3eeabfc80d5d5eb1c1a6294ff59b4ec
-- 
git-series 0.9.1

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ