[MEM/IP] Support dma-buf sharing for I/O buffers

This commit enables dma-buf shring for I/O buffers.
New APIs, buffer_get_dmabuf and bufferget_dmabuf, are added.
It also changes some codes in IP plugin to use these APIs.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[README] Add README.md for user APIs

This commit adds `README.md` which describes how users can build their application with NPU Engine library (i.e., `libhosthost.h`) and the API's detail usage.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[Handler] Minor fix on ne-handler.c

This commit applies some minor fix such as removing uncessary lines and adding error checking.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[Mem] Refactor mem_alloc()

This patch refactors the mem_alloc function to improve readability.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[AppTest] add AppTest to test dma-buf sharing for model data

This PR adds new AppTest to test dma-buf sharing for model data.
Also, there are minor changes in other unittests due to the updated common API.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[NPU] Fix ioctl() warnings in NPU driver API

This commit fixes ioctl() warnings in NPU driver API.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[MEM] Revise .get_dmabuf() to support KERNEL_CMA

This commit revise .get_dmabuf() to support KERNEL_CMA mode.
Currently, get_dmabuf() returns the global dmabuf fd for internal memory pool.
As KERNEL_CMA mode handles multiple CMA buffers, it should be revised to support this.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[MEM] Fix error return statements

This commit fixes error return statements in some functions to keep the same format.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[CONF] Replace sscanf() to strtoull() for parsing 'resv_mem_size'

This commit replaces sscanf() to strtoull() for parsing `resv_mem_size` to avoid unexpected errors.
It resolves #215.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[IP] Change IP-plugin to use dmabuf_register() for BUFFER_DMABUF

This commit changes IP-plugin to use `dmabuf_register()` for BUFFER_DMABUF type.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[CONF] Enable KERNEL_CMA as default in FastModel

This commit enables KERNEL_CMA as default in FastModel.
A zero-sized reserved memory indicates that the memory allocator used Kernel's CMA allocation.
In order to accept the zero size, ne-conf.c should be changed.

Also, some minor bugs are fixed

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[MEM] Add new API, register_dmabuf, to register pre-allocated dmabuf as a hwmem

This commit adds new API in memory allocator to register a pre-allocated dmabuf as a hwmem for user-provided mem.

A user has its ownership to allocate and free, and memory allocator may not manage such memory explicitly.
As long as it can be exported as dmabuf fd, any memory source will be fine because it's physically contiguous.
Users can allocate such memory from NPU engine or other third-party libaries.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[BUG] Fix the bug caused by a non-terminated thread in destructor

This commit fixes the bug in `ne-host-input-service.c`.

Sometimes, `cb_thread` is not terminiated until the destructor is called.
In this case, `pthread_cond_destroy()` generates unexpected bugs because cb_thread is still using this cond (e.g., `pthread_cond_broadcast()` at exit).
So, the destructor needs to make sure that `cb_thread` is terminated or not.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[Handler] Revise APIs in ne-handler

This commit revises APIs in ne-handler such as

1) remove getAvailableModelMemory()
--> it does not provide useful information because we cannot guarantee the available memory size due to compaction. Also, in Kernel-CMA mode, it's impossible to provide this API.

2) add getSizeModels/getSizeBuffers()
--> it provides memory usage information of model and buffers to users.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[ASR] Add device-type for ASR

1. Rename "npu_type" to "dev_type" support ASR IP.
2. Add ASR types.

This is the prerequisite of #199

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>
Suggested-by: Dongju Chae <dongju.chae@samsung.com>

[MEM] Revise N71 to support KERNEL_CMA

This commit revises the impl. of memory allocator to support kernel CMA-based memory allocation.

It adds COMPACT_MODE_KERNEL_CMA selected when the passed memory pool size is zero (in init()).

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[README] add the unittest badge into README.md

This badge shows the number of unittests (and apptests) which are successfully executed.
To generate the badge, it requires gtest output files (.xml).

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[README] Change the url of coverage badge (http --> https)

This commit changes the url of coverage badge to use https instead of http.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[Bug/Test] Bug fix in ne-handler unittest when obtaining data pointers

This commit fixes bugs in ne-handler unittest, which obtained data pointers without activating their hwmems.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[README] Add the coverage badge in README.md

This commit adds the coverage badge in README.md

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[AppTest] Make AppTest, async_callbacks, use ne_test_utils.h

This commit changes AppTest, async_callbacks, to use ne_test_utils.h.
Also, alloc_generic_buffer now has an extra argument for a buffer type.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[GEM] Revise GEM APIs to support multiple malloc-ed addrs in test mode

This commit revises GEM APIs such as
- support multiple virtual malloc-ed addresses used in test mode
- remove uncessary argument in mmap()

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[IP-plugin] Reuse the codes in runNPU_async() for runNPU_sync()

This commit changes the code flow of runNPU_sync() to be the same with runNPU_async().
runNPU_sync() uses the internal output callback to reuse existing codes in runNPU_async().

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[Bug] Fix deadlock bug in ip plugin to handle async callbacks

This commit fixes the deadlock in ip plugin when handling async callbacks

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[Bug] Replace 'npuengine' in some files to 'npu-engine'

This commit replaces 'npuengine' in some files to 'npu-engine'.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[Bug] bug fix in output callback handling

This commit fixes minor bugs in output callback handling.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[API] Revise buffer types defined in libnpuhost.h

This commit revises the buffer types defined in libnpuhost.h.

1) BUFFER_CONT/BUFFER_NON_CONT --> BUFFER_MAPPED

`BUFFER_CONT` cannot be used for user-provided buffers to avoid memory copies, because there is no way to find out their physical address.
For the purpose of avoiding memory copy, dma-buf sharing would be enough.
Also, `BUFFER_MAPPED` will indicate that the buffer already has a mapped buffer and users just can access this.

2) BUFFER_FD --> BUFFER_DMABUF

`BUFFER_FD` is somewhat ambiguous and confused with `BUFFER_FILE`.
So it's changed to `BUFFER_DMABUF`.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[NPUdrvAPI/test] Bug fix for npu open name

Added bug fix for opening npu device name
Added check for npu test cases based on testmode or not
This fix makes npu dev_open_close work with npu fast model simulator

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Core/Utils] Make logging utility functions be thread-safe

This patch makes logging utility functions be thread-safe.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[Core/Utils] Replace malloc() for logging

This patch replaces a malloc function for logging with char array.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[Tests/IP] Add test cases for registerNPUmodel() with default conf

This patch adds unit test cases for the functions related to
getNPUdevice() and registerNPUmodel() when the default conf is used.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[Core/IP] bugfix: Make sure to return negative value for the error case

In the 'registerNPUmodel' function, there is a case that returns 0
(which means NO ERROR) even if the 'registerNPUmodelmem' function is
failed. This patch fixes this bug.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[Core/IP] Refactor the registerNPUmodelmem function

This patch refactors the registerNPUmodelmem function including addition
of missing free().

Signed-off-by: Wook Song <wook16.song@samsung.com>

[Tests] Separate utilities for creation of dummy model and buffer

In order to make unit tests for several modules such as plugin-comm-ip
or app tests, it is required to create dummy models to be provided.
Fortunately, those procedure is already included in the
'dummy_inference' app test. Therefore, this patch separates those utility
functions for creation of dummy model and generic buffer from the
'dummy_inference' apptest. According to this change, dummy_inference
is also modified to use those separated functions.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[Core/IP] bugfix: Add a missing parameter to loginfo

This patch fixes a bug which is that a required parameter is omitted
from loginfo in runNPU_sync().

Signed-off-by: Wook Song <wook16.song@samsung.com>

[test/handler] Adding unittest for handler

Adding unittest for handler and corresponding fixes

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[GEM] Change the base path of GEM driver

This commit changes the base path of GEM driver
: /dev/dri/cardXX --> /dev/cardXX

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[AppTest] Add an example AppTest to test async output callbacks

This commit adds an AppTest to test async output callbacks.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[ip] Added bug fixes for IP

Added bug fixes for IP for the apptest to pass

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[test/inf] Adding unittest for ne-inf

Adding unittest for ne-inf and corresponding bug fixes
Merged functionality of switch_buffers into get_next_buffer

Code coverage -
ne-inf 4.0 -> 84.8
ne-host-input-service 6.6 -> 70.0

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[NE] Add order in constructor and destructor

Add order in the calling of constructor and destructor
The order is -
IP->handler->mem,scheduler->n4
inputservices->NPUdrvAPI->conf

The order of IP and inputservices (ip->inputservices) is currently
maintained using priority of constructor and destructor

The order of constructor and destructor is important as the
constructor/destructor of the modules are dependent on each other
and without the order being set, the behavior is randomized

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Packaging] Append apptest packaging and analyze its test coverage

This commit changes .spec file to append scripts for apptest packaging and its test coverage.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[NE] Bug fix on ne-handler.c and plugin-comm-ip.c

This commit fixes minor bugs detected during AppTest execution.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[AppTest] Add example AppTest: dummy inference

This commit adds an example AppTest program, named dummy inference.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[GEM] Increase test coverage in GEM APIs

This commit increases the test coverage in GEM APIs.

Test coverage: 34% --> 75.5%.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[N71] Add extra test cases to increase coverage and fix some bugs

This commit adds extra test cases in N71.
And, it fixes the bug for resizing hwmem.

Test coverage is increated from 79.7% to 85.7%.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[NE] Remove submodels heirarchy

Remove the submodel heirarchy (#119)
Updated the related sections to handle model instead of submodel

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[ip] Add test mode for plugin-comm ip

Add test mode for plugin-comm ip to enable testing of npu-engine.so
without device

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[mem/N11] Remove double buffering related code

Remove double buffering related code as it is not possible
Its not possible for user to allocate memory to be reused in npu-engine.
1. If user allocates just memory for input, then there isnt enough space
for output to be accomodated in it based on output offset from compiled
binary.
2. If user wants to allocates memory big enough for input as well as output
, then user will need access to npubinary with proper interfacing.
So, for now, independent of the format the user provides data in, there
will be a buffer allocated in ne-mem and the input will be copied.

In future, add API where user can request for an input buffer, where it
can directly load the data.

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Utils] Increase test coverage in ne-conf/utils.c

This commit adds test cases to increase test coverage for ne-conf/utils.c

- ne-conf.c: 51.6% --> 94%
- ne-utils.c: 85.9% --> 92.3%

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[Build/Dir] Reconstruct core source directory

This commit reconstructs core source directory.

The below is the summary of what's changed.

1. reconstruct src directory (issue #153), like
+-- src/
|   +-- host/
|   +-- core/
|       +-- ne-*.c/ne-*.h
|       +-- ip/
|       +-- gem/
|       +-- npu/
|   +-- test/
|       +-- unittests/
|       +-- apptests/
2. change meson.build scripts to keep the same style.
3. change source file names to have the same prefix.

It resolves #153.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[GEM] Add test_mode in gem driver API.

This commit adds the test mode in gem driver API.
It means that memory allocator can keep the same code semantics regardless the test mode.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[README] Change README.md to add some related links

This commit adds some related links to README.md.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[mem-test] Added unittest for ne-mem for npu async modes

Added unittest for ne-mem for npu async modes and corresponding fixes
Minor bug fixes for unittest of ne-mem

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[N11] Update permission and buffer sequence

Log skipping of output buffers
Check all buffers/processing is complete before changing modes
Using memory given by user directly has been removed, and added memcpy for all cases
Minor bug fixes

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[N11] Setup sequence number

Setup sequence number for each input and buffer

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[async mode] Handle N11 async mode with buffering

Add handling of async mode for input with buffering
Add cb check for NULL in runNPU_async

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[N11] Implement generic buffer for input and output

Implement generic buffer for input output and models

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[N11] Implement generic buffer for input and output

Implement generic buffer for input output and models

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[NPUdrvAPI] Using conf test_mode for testing NPUdrvAPI

Using test_mode from conf for testing NPUdrvAPI
Restructuring - add libutils with conf and util files
This allows sharing conf+util files easily with libnpu and libgem

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[N11] Update interface to support for different kinds of buffers

Updated interface to support different kinds of buffers

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Conf] Fix the bug in parsing 'main.test_mode'

This commit fixes the parsing bug because 'main.test_mode' can be a string like 'False'.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[BUILD] remove debug packages of npu-engine

This commit removes debug packages.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[Packaging] Add example package including UnitTests and AppTests

This commit append the packaging script for npu-engine-example.
It will include unit tests and app tests (TBD).

This package will be uploaded and executed in ARM FastModel emulator.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[TAOS-CI/Test] Fix test coverage bug in TAOS-CI system

This commit fixes the bug to analyze test coverage in CI system.
This bug results from that gcov/lcov couldn't find a right relative path (e.g., ../)

It resolves issue #141.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[N11] Added N11 prototype implementation

Added N11 prototype implementation
Updated interface of ne-handler to return id/version on register
Added minor bug fixes

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Build] Fix type errors to support ARM32 build

This commit fixes type castring to support ARM32 build

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[Test/Bug] Apply valgrind bug report in NPU API test

This commit fixes a bug in NPU API test, reported by valgrind.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[BUILD] Fix typo in rpm spec file

This commit fixes typo in rpm spec file (e.g., core->src)

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[BUILD] Add use_valgrind option in rpm spec file

Add option to enable valgrind

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[Services] Updated N4x services based on update on interface

Updated host input service (NPU) and ARM plugin based on update on interface

Refer #105

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[UnitTest] Enable valgrind for unit tests in x86_64

This commit enables valgrind for unit tests (x86_64 only)

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[BUILD] Revise directories for libnpu-engine

This commit revises the whole directory such as
- merge each src folders into /src/
- merge common/host include files into /include/
- change packaging script to build libnpu-engine

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[Dist/Tizen] Change build depenedency on kernel headers

In order to enable build of npu-engine for x86 and armv7l architectures,
this patch changes build dependency on linux-fvp-headers to that on
linux-npu-headers.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[UnitTest] Add the python script to analyze unittest coverage.

This commit adds the python script to analyze unittest coverage.
Note that it's copied from the nnstreamer repository.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[N71] If a GEM driver is unavailable, fall back to test mode

This commit adds fallback mode for testing (checked using .ini option).

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[Dist/Tizen] Fix gbs build errors when unit tests are enabled

This patch fixes errors during gbs-build when unit tests are enabled.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[Valgrid] Fix memory leak in ne-conf.c and ne-mem.c

This commit fixes memory leak in `ne-conf.c` and `ne-mem.c`.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[N1] change API args to use model id/version instead of model itself

This commit changes function arguments to use model id/param.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[TAOS-CI] Add test coverage package for TAOS-CI

This commit appends scripts for test coverage package to the spec file.
The package is used in TAOS-CI to show the analysis result for test coverage.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[Conf] Load default configurations in constructor

This commit updates to load default configurations in constructor.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[N4C] Added implementations for configure and dataready

Added implementations for configure and dataready
Updated changes in npu-host-service for successful build

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Meson] Enable tests for core libraries

This patch enables tests for core libraries.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[Meson] Clean-up meson build scripts

This patch revises meson build scripts in the whole project.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[NPUdrvAPI/test] Added unit tests for NPU driver api

Added unit test cases for NPU driver api
This only tests the internal working of the api
Syscalls/Interaction with device is ignored in this testing
This test suite should pass with/without device driver/device available
Fixed Doxygen entries

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[Meson] Apply work-around in order to fix gbs-build errors

This patch applies several work-around solutions in order to fix
gbs-build errors. Note that this is not permanent solutions. After CI
system is enabled, we need to take look into whole build scripts.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[Dist/Tizen] Modify the rpm sepc file according to meson revision

Since the meson build scripts are revised, it is required to revise the
rpm spec file as well.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[Engine/Scheduler] Replace 'n4_opmode' with 'submodel_opmode'

There is no 'n4_opmode' anymore. This patch replaces 'n4_opmode' with
'submodel_opmode' to fix build errors.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[Engine/HostInputService] Define a dummy for 'N4_OPS_HOST_INPUT'

Since 'N4_OPS_HOST_INPUT' is omitted from the current source tree, this
patch just defines a dummy for 'N4_OPS_HOST_INPUT' as a work-around.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[Engine/InputService] Fix build error due to incompatible-pointer-types

This patch fixes build errors in the header file for 'inputservice'
due to incompatible-pointer-types.

Signed-off-by: Wook Song <wook16.song@samsung.com>

Engine/HostInputService] Block lines refering to non-existent member

In order to fix build errors, this patch blocks lines that refer to
non-existent member of a struct. Note that this ia a work-around. We
need to fix it.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[Core/HostInputService] Fix build error, 'maybe-uninitialized'

This patch fixes a build error caused by a local variable used without
initialization.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[Core/HostInputService] Include 'srnpu.h' from 'misc'

This patch changes the location of 'srnpu.h' from 'linux' to 'misc'.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[Core] Include srnpu.h from 'misc'

This patch changes the location of 'srnpu.h' from 'linux' to 'misc'.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[Engine/ArmPlugin] Fix build error due to incompatible-pointer-types

This patch fixes a build error due to initialization from incompatible
pointer types between the arm_plugin_configure function and the function
pointer, configure, in the inputservice structure.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[Engine/Handler] Fix undefined reference to un-implemented functions

This patch fixes build errors due to undefined reference to
un-implemented functions, exitNEcomm and initNEcomm.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[DIST/Tizen] Add build dependency on linux-fvp

This patch fixes errors occurring since a necessary kernel header is not
provided.

Signed-off-by: Wook Song <wook16.song@samsung.com>

[N12] The prototype of FastModel comm plugin

This commit adds the prototype of fastmodel comm plugin using TCP/IP.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

[Build] revise meson files to make a single project

merge core and host into a single project
npu-engine final .so with interface of libnpuhost.h in ip comm mode only
libnpuhost.h and npu-engine interface to be done by N11 (plugin-ip-comm.c)
npu-engine exe build only in fast model mode
include common dir for header properly

Working on issue #114

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[ne] Create submodel for input service

Create dummy submodel for input computation service corresponding to the input opmode
Add enum in typedef corresponding to the type of submodel in its meta
n4_opmode replaces with submodel_ops
updated the order of the submodels in the model

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>