Re: [PATCH] drm/exynos: ipp: Remove Exynos DRM IPP subsystem

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Dec 14, 2017 at 04:10:15PM +0100, Marek Szyprowski wrote:
> Exynos DRM IPP subsystem is in fact non-functional and frankly speaking
> dead-code. This patch clearly marks that Exynos DRM IPP subsystem is
> broken and never really functional. It will be replaced by a completely
> rewritten API.
> 
> Exynos DRM IPP user-space API can be obsoleted for the following
> reasons:
> 
> 1. Exynos DRM IPP user-space API can be optional in Exynos DRM, so
> userspace should not rely that it is always available and should have
> a software fallback in case it is not there.
> 
> 2. The only mode which was initially semi-working was memory-to-memory
> image processing. The remaining modes (LCD-"writeback" and "output")
> were never operational due to missing code (both in mainline and even
> vendor kernels).
> 
> 3. Exynos DRM IPP mainline user-space API compatibility for
> memory-to-memory got broken very early by commit 083500baefd5 ("drm:
> remove DRM_FORMAT_NV12MT", which removed the support for tiled formats,
> the main feature which made this API somehow useful on Exynos platforms
> (video codec that time produced only tiled frames, to implement xvideo
> or any other video overlay, one has to de-tile them for proper
> display).
> 
> 4. Broken drivers. Especially once support for IOMMU has been added,
> it revealed that drivers don't configure DMA operations properly and in
> many cases operate outside the provided buffers trashing memory around.
> 
> 5. Need for external patches. Although IPP user-space API has been used
> in some vendor kernels, but in such cases there were additional patches
> applied (like reverting mentioned 083500baefd5 patch) what means that
> those userspace apps which might use it, still won't work with the
> mainline kernel version.
> 
> We don't have time machines, so we cannot change it, but Exynos DRM IPP
> extension should never have been merged to mainline in that form.
> 
> Exynos IPP subsystem and user-space API will be rewritten, so remove
> current IPP core code and mark existing drivers as BROKEN.
> 
> Signed-off-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx>
> ---
> This is a follow-up of the discussion in the following thread:
> https://lists.freedesktop.org/archives/dri-devel/2017-November/158989.html
> 
> This patch was a part of the "Exynos DRM: rewrite IPP subsystem and
> userspace API" patchset:
> https://lists.freedesktop.org/archives/dri-devel/2017-November/157130.html
> ---
>  drivers/gpu/drm/exynos/Kconfig          |   11 +-
>  drivers/gpu/drm/exynos/Makefile         |    1 -
>  drivers/gpu/drm/exynos/exynos_drm_drv.c |   12 -
>  drivers/gpu/drm/exynos/exynos_drm_drv.h |    2 -
>  drivers/gpu/drm/exynos/exynos_drm_ipp.c | 1806 -------------------------------
>  drivers/gpu/drm/exynos/exynos_drm_ipp.h |  252 -----
>  include/uapi/drm/exynos_drm.h           |  192 +---
>  7 files changed, 4 insertions(+), 2272 deletions(-)
>  delete mode 100644 drivers/gpu/drm/exynos/exynos_drm_ipp.c
>  delete mode 100644 drivers/gpu/drm/exynos/exynos_drm_ipp.h
> 

Acked-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx>

Best regards,
Krzysztof

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux