| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
| |
it is not used in 1.9.91 version
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
|
|
|
| |
1. Use ivi_layout_interface directly without ivi-extension APIs
2. Use weston_output for IVI Screen APIs.
3. Remove content_observer because IVI Layout API is removed
4. Use new listener APIs instead of notification APIs
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
|
| |
it is copied from ivi-extension.c, because ivi-extension.c
will be removed.
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
|
| |
Check logic should be improved because iviscrn is never null.
Besides, do not iterate over empty list like send_surface_add_event.
Signed-off-by: Wataru Natsume <wataru_natsume@xddp.denso.co.jp>
|
|
|
|
|
|
| |
Check logic should be improved because ivilayer is never null.
Signed-off-by: Wataru Natsume <wataru_natsume@xddp.denso.co.jp>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Iterating over empty list will modify first value passed to
wl_list_for_each, but that won't be valid list entry, which in
case of send_surface_add_even leads to crash.
V2 change:
Use break instead of continue
Signed-off-by: Mateusz Polrola <mateuszx.potrola@intel.com>
Signed-off-by: Wataru Natsume <wataru_natsume@xddp.denso.co.jp>
|
|
|
|
|
|
|
|
| |
The update_layer_prop function only updates on_screen member
of ivilayer struct. But on_screen member is never used.
Therefore, I removed the function and the member.
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
|
|
| |
The update_surface_prop function only updates on_layer member
of ivisurf struct. But on_layer member is never used.
Therefore, I removed the function and the member.
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
|
|
|
|
| |
ivi_layout_get_get_layers_under_surface allocates memory.
It is responsibility of the caller to free this memory,
as it is stated in ivi-layout.c
Therefore, the memory should be freed in the end of the function
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
|
|
|
| |
We have to wait before freeing ivisurf that all clients destroy
their proxies. Otherwise, a client could send an event to the ivisurf
just after we destroyed it. This would cause a race condition and
potentially SEGV.
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We have to wait before freeing ivilayer that all clients destroy
their proxies. Otherwise, a client could send an event to the ivilayer
just after we destroyed it. This would cause a race condition and
potentially SEGV.
Furthermore, the layout_layer pointer of ivilayer set to NULL after
ivilayer freed in the old implementation. Therefore, old implementation
writes 4 bytes to invalid location. This implementation also solves the
invalid write problem by moving the free code.
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
|
|
|
|
| |
With earlier implementation memory for NULL byte of l->filename string
was not being allocated by malloc (strlen is returning length of string
without NULL terminating byte) which may result in some random
characters appended in filename string.
Now strdup is begin used for string copy instead of malloc/strcpy.
Signed-off-by: Mateusz Polrola <mateuszx.potrola at intel.com>
|
|
|
|
|
|
|
|
|
|
| |
this patch will allow to install the modules and libraries in correct library directory
e.g. if we are built for 64 bit architecture
libraries are installed in /lib$LIB_SUFFIX
weston modules are installed in /lib$LIB_SUFFIX/weston
Signed-off-by: Eugen Friedrich <efriedrich@de.adit-jv.com>
|
|
|
|
|
|
|
| |
It is unnecessary to send content removed event to the controller.
It is enough to send destroyed event.
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
| |
createPid should be a pid of process that creates surface.
Signed-off-by: Wataru Natsume <wnatsume@jp.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
|
|
|
|
|
| |
It is wrong to send surface_send_layer event with resources
of every client. Instead, the event should only sent once with
the layer resource of client, which owns also the surface resource.
Otherwise a wayland communication error happens, because a client gets
a surface_send_layer event with a resource which is owned by
another client.
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
| |
It uses libgbm to handle id to share graphic buffer inter processes.
Signed-off-by: Nobuhiko Tanibata <ntanibata@jp.adit-jv.com>
|
|
|
|
|
|
|
|
|
|
|
| |
the shared library.
ivi-input-controller shall work with ivi-controller.so not solely.
So the patch proposes a way to specify it as a ivi-input-module in
weston.ini. The shared library, ivi-input-controller.so, is loaded by
ivi-controller.so.
Signed-off-by: Nobuhiko Tanibata <ntanibata@jp.adit-jv.com>
|
|
|
|
|
|
|
|
|
|
|
| |
With earlier implementation read_pixels was called without
weston_output_schedule_repaint, causing screenshot to have image of
previously triggered screenshot event. & first screenshot was always
blank.
Now we add frame listener, call weston_output_schedule_repaint and
then capture the screenshot in the frame listener.
Signed-off-by: Mitul Chokshi <mitul.chokshi@intel.com>
|
|
|
|
|
|
|
|
|
| |
1. weston_seat interface is changed at version 1.9 and it is not compatible with 1.8
2. Surface clipping works in version 1.9 of weston.
3. Some bugfixes are landed in version 1.9.
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
Signed-off-by: Nobuhiko Tanibata <ntanibata@jp.adit-jv.com>
|
|
|
|
|
|
| |
Because it is not used.
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
|
| |
Previous code used fixed screen 0 only. The new code creates multi
ivi-screens according to the number of screen from output_list.
Signed-off-by: Nobuhiko Tanibata <NOBUHIKO_TANIBATA@xddp.denso.co.jp>
|
|
|
|
|
|
|
|
|
| |
A surface can be on only one layer. Therefore, the struct link_layer is
not required. Instead, struct ivisurface has a pointer to the layer. In
the future, A feature that a surface belongs to several layers would be
supported if somebody requires.
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
|
|
|
| |
A layer can be on only one screen. Therefore, the struct link_screen is
not required. Instead, struct ivilayer has a pointer to the screen. In
the future, a feature to support that a layer belongs to several screens
would be supported again if somebody requires.
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Checking wl_list_empty() on a link offers no information: if it returns
true, wl_list_remove() is safe to do. If it returns false, you still do
not know if wl_list_remove() is safe; the link could be part of a list,
or the link could be "uninitialized" (e.g. just wl_list_remove()'d).
(From Pekka Paalanen's comment at
http://lists.freedesktop.org/archives/wayland-devel/2015-August/023987.html).
Calling wl_list_init just before wl_list_insert is redundant.
Because the links of the list are not read before it is overwritten by
wl_list_insert.
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
Signed-off-by: Nobuhiko Tanibata <nobuhiko_tanibata@xddp.denso.co.jp>
|
|
|
|
|
|
|
| |
image_buffer has a size of image_size. Therefore, image_size should be used in the write operation.
Otherwise, weston crashes.
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
|
| |
surface_configure_event sends the ivi_controller_surface_send_configuration
event with original dimensions of the wayland-buffer of a surface.
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
| |
otherwise ilm_takeSurfaceScreenshot causes SIGSEGV for weston-ivi-shell
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
|
| |
compositor.h is not located in the source directory of wayland-ivi-extension.
Therefore, brackets should be used instead of quotation marks.
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
|
|
|
| |
init_ivi_shell and setup_ivi_controller_server functions are defined at
ivi-controller-impl.c, but called at ivi-extensions.c. They are not declared
at the compilation of ivi-extension.c. I am introducing with this patch a new
header file to avoid -Wimplicit-function-declaration
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
| |
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
| |
ivi-controller does not have a dependency to GLES, EGL or weston libraries.
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|
|
|
|
|
|
| |
GLES is not used since it is removed from controller_surface_screenshot
Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
|