Age | Commit message (Collapse) | Author | Files | Lines |
|
There is warning about ignoring return value of 'asprintf'. Fix to
check return value of asprintf().
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com>
[sw0312.kim: cherry-pick the commit 6736ad45d88c from libdrm mainline to remove build warning]
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
Change-Id: I3c445ee2237f1dfcfb3e5413d66a406cd3c0d680
|
|
Change GPL license of Exynos related code to X11/MIT.
I'd like to keep a consistent license across all Exynos code
because License checker notices two more licenses exist
in libdrm.
For the license change I need to get your agree - all committers.
So please give me Acked-by if you agree with me.
Signed-off-by: Inki Dae <inki.dae@samsung.com>
Acked-by: Hyungwon Hwang <human.hwang@samsung.com>
Acked-by: SooChan Lim <sc1.lim@samsung.com>
Acked-by: Sangjin LEE <lsj119@samsung.com>
Acked-by: Boram Park <boram1288.park@samsung.com>
Acked-by: Seung-Woo Kim <sw0312.kim@samsung.com>
Acked-by: Joonyoung Shim <jy0922.shim@samsung.com>
Acked-by: Emil Velikov <emil.l.velikov@gmail.com>
Acked-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
Acked-by: Jan Vesely <jan.vesely@rutgers.edu>
[sw0312.kim: cherry-pick from mainline commit be40710f063a to handle license]
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
Change-Id: Ib9e1f59a50e1f461ced91885c6a9fb4dec1efabd
|
|
Add gem buffer type to sync with kernel header.
libtbm nexell backend refers to gem buffer type for cache control.
So this patch adds the buffer type to identify if cache operation
is required or not.
Change-Id: I7760a54496a1a9e7c2f17f0a77e640e346d9b455
Signed-off-by: Inki Dae <inki.dae@samsung.com>
|
|
Fix build warnings to exynos_fimg2d module.
This patch fixes below two build warnings,
exynos_fimg2d.c: In function 'g2d_scale_and_blend':
exynos_fimg2d.c:978:3: warning: 'scale_y' may be used uninitialized in this function [-Wmaybe-uninitialized]
g2d_add_cmd(ctx, SRC_YSCALE_REG, scale_y);
^
exynos_fimg2d.c:977:3: warning: 'scale_x' may be used uninitialized in this function [-Wmaybe-uninitialized]
g2d_add_cmd(ctx, SRC_XSCALE_REG, scale_x);
Change-Id: I70908088c61b85570aec6e3ec19b405df4820d37
Signed-off-by: Inki Dae <inki.dae@samsung.com>
|
|
Change-Id: Ic7fcb69be2a2fcc58363647278c0be62ac48766e
Signed-off-by: Vladislav Andresov <v.andresov@partner.samsung.com>
|
|
Add nexell drm to test modules for the artik devices.
Change-Id: I5f15f6bf28064ebc6ad178ba40ada0c6e666ad49
Signed-off-by: SooChan Lim <sc1.lim@samsung.com>
|
|
There is a problem that can't test various source resolutions. So
support for the source resolution use to various. The default source
resolution is 720x1280.
The source resolution can be used various as below.
- For example:
: ipptest -s 26#320x320:1440x2560 -d 1
(The source resolution used is 320x320.)
Change-Id: If452fb4524ecffe65cec5ccbe8f3956944bd7ecb
Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
|
|
There is a problem using the IPP_GET_RPOPERTY ioctl because the
structure exynos_ipp_prop_list is not synchronized. Synchronize with
exynos_ipp_prop_list structure of tizen kernel v4.1.36
Refer below commit of tizen kernel:
- Commit id: 60f8b78 (drm/exynos/gsc: Add rotation hardware limits of gscaler)
- https://review.tizen.org/git/?p=platform/kernel/linux-exynos.git;a=commit;h=60f8b78ad8b75c5c3a52f2ad66897da628153eec
Change-Id: Id1989f571c989afb2c73bd4d654f560c3be4067c
Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
|
|
It does not check memory allocation result even though it can be
failed. So, add null check and if failed, handle errors.
Change-Id: Ia0f8daed3be118e27ac6795dc41f52d8780b3568
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
|
|
It can not be verified 90 degrees, 270 due to the rotation hardware
limits on the ipp of kernel.
Therefore, modify the source image size to 720x1280 to avoid exceeding
the rotation hardware limits.
Change-Id: I177a979950d5ddf96b761348790bed259baaa416
Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
|
|
There is a problem for destination image is broken when outputting to
the display for 90 degrees, 270 rotation. So should not swap the width
and height when outputting to the display.
Change-Id: I6a9a9c478b45de0e98563fc48740053bf271891a
Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
|
|
Fix the sizeof operator incorrect usage.
Change-Id: I9d293c00bc20a2c1ae048c4d0564b75a98bcd860
Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
|
|
It is not required to call two functions, so instead, this patch
fixes to use calloc.
Change-Id: Ib7ebee24152c7b41a0b936cb3085aa675de82c0b
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
|
|
To remove warning of static analysis tool, this removes strerror().
Change-Id: I132cf0271ac9a1641c1c88611aca5c3f311240a7
Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
|
|
fimc.c:506:2: warning: implicit declaration of function 'ioctl' [-Wimplicit-function-declaration]
ret = ioctl(fd, DRM_IOCTL_EXYNOS_IPP_SET_PROPERTY, property);
rotator.c:72:2: warning: implicit declaration of function 'ioctl' [-Wimplicit-function-declaration]
ret = ioctl(fd, DRM_IOCTL_EXYNOS_IPP_SET_PROPERTY, property);
Change-Id: I98d220d02c4771f61fbd827064bba94edba9f62e
Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
|
|
util.c:185:8: warning: variable 'planes' set but not used [-Wunused-but-set-variable]
void *planes[3] = { 0, };
Change-Id: Ia05c1b6fffa14fa53358b69fa083e7202514f1d0
Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
|
|
fimc.c:780:2: warning: enumeration value 'IPP_CMD_M2M_NONE' not handled in switch [-Wswitch]
switch (display) {
Change-Id: I57707ce3131685313eb7422a0cfa7c9462dab9ca
Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
|
|
fimc.c:669:23: warning: unused variable 'args' [-Wunused-variable]
struct drm_gem_close args;
fimc.c:957:24: warning: unused variable 'def_pos' [-Wunused-variable]
struct drm_exynos_pos def_pos = {0, 0, 720, 1280};
Change-Id: I8ade8f109d2c1ed58306911f151f1ec92f8ab42f
Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
|
|
This fixes success message for get drm_connector without error value.
Change-Id: Ie0bb0ff44ea296ab74c604ee0a0e5970effcfd56
Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
|
|
This fixes to use unmap and destroy together from util.c to match with
util_kms_gem_create_mmap().
Change-Id: Ibf17d70949a8ceb2edde224e8fbbc870f6ffbf38
Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
|
|
There are unnecessary array variables in the fimc_m2m_set_mode(), so
this removes them.
Change-Id: I7c2bdc989c36f5540d4c80701123280d204771c3
Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
|
|
From fill_smpte_rgb32() function, there is possible path for
division by zero. So this patch removes it.
Change-Id: I3762b129b260e466495468a1faba994b7c8f4425
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
|
|
There are issues including memory leak and invalid error handling,
and use after NULL comparing in connector_find_mode(). So this patch
fixex for refactoring the function.
Change-Id: I333cfde93f1ff88b1d2d664a4856ddd1f4ea259c
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
|
|
The nexell gem alloc api has int parameter, so it should be casted
to uint64_t for kernel gem alloc interface.
Change-Id: Icc98b42645fd8ec234b99193304b5f91bcdd3389
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
|
|
This patch fixes invalid code of error path including NULL
deference and leak in g2d test.
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
Updated to mainline merged version for the commit 9b8ed9dc4736
("tests/exynos: fix memory issues in g2d test").
This and previous commit should be skipped for rebase.
Change-Id: I6a3edb87bce3e3a1db86b6abc9b74ee17083ab1c
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
|
|
The function g2d_blend_test() is blocked to call because of
feature unsafety. This patch blocks with proper feature name
and also blocks the function itself to remove build warning.
Change-Id: Ibf85e67ea2b7ff06fd3f330e887b976b3486dd71
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
|
|
This patch removes unused variable set and wrong type build
warnings.
Change-Id: Ia8b18b2df8e96c0d48d1ac8481817a8b1b72e628
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
|
|
These tests are only valid for exynos drm module, so it is not
necessary to load other modules.
Change-Id: If5297681a8adebc0246eb1efcb4f882c2bede844
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
|
|
The function g2d_reset() is not anymore used after the commit
e3c97d1a2473 ("exynos/fimg2d: add g2d_validate_xyz() functions"),
so it should be removed.
Change-Id: I0a8a9477c0d8b24ea5b1a7bf02489a63ef19b2c9
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
|
|
As like the commit ecc2a097294d ("xf86drm: Fix type-punned pointer
build warning"), this fixes following build warning.
exynos_drm.c: In function 'exynos_handle_event':
exynos_drm.c:420:15: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
e = (struct drm_event *) &buffer[i];
^
Change-Id: I83c7816d946b2d9234b7c5a01a415912418dde2c
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
|
|
In error path of drmGetBusid() and drmGetReservedContextList(),
there are memory leaks for error path. So this removes them.
Change-Id: Iaedaab323a6f4771fe25ca032e2f822f8b5c4004
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
|
|
In rotator test of ipptest, allocated connector is not freed in
dump_connectors, so it should be freed to remove memory leak.
Change-Id: I8ee32e85238901bcc516c855be07bb2cfb5e1d9c
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
|
|
There is missed variable type, so it causes following warning.
fimc.c: In function 'fimc_event_handler':
fimc.c:585:9: warning: type defaults to 'int' in declaration of 'bmp_idx'
static bmp_idx = 0;
^
Change-Id: Id775abce078e4866e25760adcfe4302d9d9f5346
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
|
|
This patch copies license file to /usr/share/licenses/libdrm/.
The licenses of all packages should be noticed when released.
Change-Id: I0224046d19ff73b44ec825c2b2327836e55b44c2
Signed-off-by: Inki Dae <inki.dae@samsung.com>
|
|
This patch removes possible NULL deference in error path.
Change-Id: I9f4c018adea3789e8d1f7e0e9b753a56b81ca88d
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
|
|
This patch fixes memory issues including NULL deference and leak
in g2d test.
Change-Id: I2b554cae51f2a14b88100c8b9fac3239ac737bae
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
|
|
Added the modeSetCrtc func for display mode test of ipptest.
Change-Id: If2d51e0be6b6302dc0c4931ee5b73dd4dbe483c6
Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
|
|
Change-Id: I54ef37f9be4c03a7502b2e0cc8b93e46efe87a8a
Signed-off-by: Joonyoung Shim <jy0922.shim@samsung.com>
|
|
There is no reason to remove kmstest from package.
Change-Id: I772c21ca005f7edb23bb832a2ae3f8b3259d7a08
Signed-off-by: Joonyoung Shim <jy0922.shim@samsung.com>
|
|
After rebase to 2.4.75 version, GBS building errors occur, so fit them.
Change-Id: I0ab0cc6569af20b6f9d157081498926420d7aaf5
Signed-off-by: Joonyoung Shim <jy0922.shim@samsung.com>
|
|
This patch fixes the issue that we cannot identify whether the image displayed on Panel is correct or not after rotation.
For this, it uses fill_smpte_rgb32 function instead of util_draw_buffer_youu.
Change-Id: I764f3b17356c474e0afa628f5dd45e447cd596a8
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
|
|
There was a size error when rotated 270 degrees. Added when
rotated 270 degree.
Change-Id: I744676e1af7e6f2916fc5f6a18ec1f1ccc6fbbe9
Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
|
|
This patch fixes to pass through for get property error to prevent
null pointer deference.
Change-Id: I3b46d9f6fecda79af7f7811acb4be828a2a2d506
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
|
|
For avgerage calculation, this patch adds to check 0 value for
division.
Change-Id: Ie4f0cd6d447dec1acf707226e9e63eef7f74d3c3
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
|
|
This patch imports the nexell api of the libdrm
Change-Id: If8009fc30f46edcf4fdcc04679523e590e165003
Signed-off-by: jino.cho <jino.cho@samsung.com>
|
|
This patch added for the function outputing the M2M to a screen.
So, you can check for Image Post processing dirver to screen.
Also you can select the degree about (0, 90, 180, 270).
I will briefly explain the process of the function:
- Memory(YUV422) ---> IPP ---> Memory(RGB888) ---> Screen
- Create four DMA memory.
- Draw the image of the YUV422 format on three DMA memories.
- Rotate the relevant image by using IPP driver.
- The rotated image is stored in one DMA memory as the RGB888 format.
- The one DMA memory output screen.
I referred to the modetest code.
Change-Id: I2bf5738d98a457ced274dc8237df8ef108153e61
Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
|
|
Without config.h off_t type is 32bit, what causes problem with mmap() on
GEM buffers while running on 64bit kernel. libdrm configure script already
sets _FILE_OFFSET_BITS to 64, so all libdrm tools can use 64bit file system
interface functions.
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I45e5931b76ebb65b3d854c7609821351ead09f92
|
|
Change-Id: I9253557242bb43ec49e4f45eed5d21269bbbb3ac
|
|
In the COPYING file, x86free project is not proper MIT license,
so this patch replaces it as proper MIT license copying.
Change-Id: Ib498509c698d4584a6ec208c198b42ffb8308445
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
|
|
Change-Id: I95abd7e8cc8d0485a7cc06dca54bfa46775eb8aa
|