Check aarch for arm-related build option (#4735)

Check aarch32 and aarch64 for BUILD_ARMCOMPUTE and BUILD_PURE_ARM_COMPUTE default

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Add mobiilenet to model benchmark list (#4732)

Add mobiilenet to model benchmark list

Fix gbs build script (#4731)

Use nnfw command instead of run (removed)

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

[neurun] Remove unused header inclusions (#4730)

Remove unused header inclusions which causes unnecessary coupling.

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>

Remove CMake branch for Android build (#4729)

Let's use fine-control over each component later.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Revert same padding calculation (#4727)

Revert same padding calculation because current padding calculation is incorrect

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

[neurun] Add missing generated tests (#4725)

In #4667, I've missed generated tests related that ops because of rebasing.
This commit adds missing generated tests.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>

[neurun] Update same_padding helper (#4723)

In #3629, `same_padding` helper is updated to correct.
This commit updates to the same as `same_padding` of `PACL`.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>

[neurun] Enable L2Pool2D op (#4688)

This commit enables `L2Pool2D` op for `acl_cl`.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>

[neurun] Prepend license to wrapper/execution.cc (#4718)

License block for this file was missing.

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>

Remove testbuild in Makefile (#4693)

Remove deprecated build option testbuild
Now svace and coverity use make all

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Remove `run` script that is not used any more (#4717)

`run` is replaced with `nnfw` script.

Signed-off-by: Sanggyu Lee <sg5.lee@samsung.com>

Do NOT update global link directory setting (#4716)

This commit removes global link_directories in the top-level
CMakeLists.txt.

This command was introduced as a workaround for CI build failure, but is
no longer required.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Coverage report for neurun (#4706)

Change coverage report target to neurun, not pure runtime

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Fix SetIO test (#4715)

- Remove dynamic casting
- Remove unused header
- Remove unused alias

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Make clean and distclean for external handling (#4649)

* Make clean and clean_all for external handling

make clean: clean only workdir
make clean_all: clean workdir and external stamp file

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>
* change command clean_all to distclean

[neurun] Enable Floor, SpaceToDepth ops (#4686)

This commit enables `Floor`, `SpaceToDepth` for `acl_cl`.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>

Set the default value of BUILD_PURE_ARM_COMPUTE (#4710)

Now, the default value of BUILD_PURE_ARM_COMPUTE is decided according to
the target architecture.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Use add_subdirectories for contrib and tools (#4708)

This commit simplifies "contrib/CMakeLists.txt" and
"tools/CMakeLists.txt" using "add_subdirectories" function.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Allow explicit control over CMake with configure command (#4702)

This commit updates "nnfw configure" to pass all of its command-line arguments
to CMake, which allows users to control all the CMake options explicitly.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

[neurun] Extract NNAPI frontend from src (#4705)

This commit extracts NNAPI implementations (e.g. ANeuralNetworksModel_create)
from "src" and moves it to newly introduced frontend/nnapi directory.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Remove duplicated '-Wno-ignore-attributes' FLAG (#4704)

- Remove duplicated '-Wno-ignore-attributes' FLAG from option files
- Add the FLAG into option_linux.cmake

Signed-off-by: Chunseok Lee <chunseok.lee@samsung.com>

Remove unnecessary if for BUILD_ARMCOMPUTE (#4703)

Now, the default value of BUILD_ARMCOMPUTE for x86_64 is OFF.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Disable warning ignore-atrribute for aarch64 (#4701)

- To fix tizen gbs build error on aarch64 ( aarch64 cross build,too)
- related issue : https://github.com/ARM-software/ComputeLibrary/issues/330

Signed-off-by: Chunseok Lee <chunseok.lee@samsung.com>

Add comments for nnfw_find_package (#4700)

* Add comments for nnfw_find_package

This will add some example comments for nnfw_find_package in CMakeLists.txt file

Signed-off-by: SaeHie Park <saehie.park@samsung.com>
* apply comments

[neurun] Enable ResizeBilinear, ReLU1, ReLU6 ops (#4667)

Related : #4259

This commit enables `ResizeBilinear`, `ReLU1`, `ReLU6` for `acl_cl`.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>

Fix coverage report bug (#4666)

Fix coverage report bug setting SRC_PREFIX: use default value
gen-coverage-report command use default variable for command

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Pass GENERATE_NNAPI_TESTS to docker container (#4695)

Pass host's GENERATE_NNAPI_TESTS setting to docker container

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Remove default constructors (#4694)

Remove default consturctor for TypeInfo and TensorInfo
Block implicit using of default constructor

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Do NOT use quote in CMAKE_DEPENDENT_OPTIONS (#4692)

This does not work as expected.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Set BUILD_ARMCOMPUTE according to TARGET_ARCH (#4684)

Now, the default vaule of BUILD_ARMCOMPUTE option will be
set according to TARGET_ARCH.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

[neurun] Enable ReLU, RSQRT ops (#4642)

This commit enables `ReLU`, `RSQRT` for `acl_ac`.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>

[neurun] Add Squeeze Operation (#4678)

Add squeeze operation.
The implementation comes from pure arm compute runtime.

Signed-off-by: Sanggyu Lee <sg5.lee@samsung.com>

Update build toolchain in dockerfile (#4669)

- Remove deprecated build tool: gcc-5 softfp
- Install gcc-6 for x64 build

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

[neurun] Simplify ARM Compute function includes (#4682)

In #4636, `PACL` uses a set of meta headers that includes all the functions.
This commit applies this to neurun.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>

Introduce BUILD_LOGGING option (#4679)

This commit introduces BUILD_LOGGING option which allows users to
enable/disable logging runtime build, and updates CMakeLists.txt to
use it instead of OBS_BUILD.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Remove OBS_BUILD from GTest package (#4663)

This commit removes OBS_BUILD occurences from GTest package, and uses
BUILD_GTEST instead.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Update document to use docker image (#4677)

Update command usage and standalone scripts using docker image

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Build independent with download option (#4648)

* Build independent with download option

If download option is off and there is correct external directory & stamp, try to build.

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>
* ExternalSource_Get use ExternalSource_Download

* Remove variables not used

* Fix typo

Simplify tests/tools/CMakeLists.txt (#4670)

This commit introduces BUILD_TFLITE_RUN option in order to simplify
"tests/tools/CMakeLists.txt".

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

[nuerun] Introduce checking padding of the tensor. (#4647)

This commit introduce checking padding of the tensor.

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

Enable debug build in acl (#4671)

This patch enables acl debug build.
The `asserts` and `logging` are forced to 1 for debug build.

Signed-off-by: Jiyoung Yun <jy910.yun@samsung.com>

Always build generated NNAPI tests (#4664)

With this commit, generated NNAPI tests are always built when NNAPI test
build is enabled.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

[neurun] Rename operand::Context as OperandContext (#4629)

Rename to reduce namespace level. This class was the only class in this
namespace.

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>

Enable logistic_qaunt nnapi tests in neurun and pureacl (#4660)

This commit enables logistic quantize nnapi tests.
It is fixed in acl v19.02.

Signed-off-by: Jiyoung Yun <jy910.yun@samsung.com>

Introduce BUILD_TENSORFLOW_LITE option (#4658)

This commit introduces BUILD_TENSORFLOW_LITE option and removes one
occurences of OBS_BUILD in CMakeLists.txt via using this flag instead.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Introduce BUILD_TOOLS options (#4654)

This commit introduces BUILD_TOOLS option and partially removes the
the occurence of OBS_BUILD in CMakeLists.txt by using this BUILD_TOOLS
option.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

[PACL] Use meta headers to simplify include(s) (#4656)

This commit simplifies includes(s) in compilation.cc via using meta
headers that ARM Compute and ARM Compute Ex libraries provide.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Fix build error : remove logging runtime dep to boost library (#4650)

- remove logging runtime dep to boost library(not required)

Signed-off-by: Chunseok Lee <chunseok.lee@samsung.com>

Introduce ARMComputeEx meta header (#4653)

This commit introduces a meta header for each domain (CL/NEON) that
includes all the functions that each domain provides similarly as
ARMCompute does.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Always enable test (#4634)

This commit moves enable_testing() call from GTestConfig to top-level
CMakeLists.txt.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Remove CLNormalizationLayerEx.h (#4646)

"CLNormalizationLayerExKernel.h" is already removed from the repo, and
thus it is possible to compile this header.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

[neurun] Enable NotEqual,ReduceMax,LogicalAnd ops (#4633)

To avoid frequent conflicts among op PRs(migration is too slow..), This commit enables three ops for `acl_cl` at the same time.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>

Remove unused CMake global variabes (#4644)

This commit removes the following (unused) CMake global variables:
- NNFW_INCLUDE_DIR
- NNFW_EXTERNALS_DIR
- TFLITE_DEPEND_DIR

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Add cmake build support for Boost library (#4572)

* Add cmake build support for Boost library

This patch downloads Boost source code(v1.65.1) from upstream
if DOWNLOAD_BOOST is true, and build and install the boost library
if BUILD_BOOST is true.

For the various compilers support, BoostConfing.cmake file
creates project-config.jam file to setup a proper toolchain
according to toolchain setting, and target libraries to build.

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Remove CROSS_BUILD flag

We have already another flag - BUILD_IS_NATIVE
so remove CROSS_BUILD flag and use BUILD_IS_NATIVE instead.

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Find boost package at each sub module

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Check if b2 binary exists and clean up

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Add some TODO descriptions

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Add /externals/boost to .gitignore

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>
Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Pass arguments to b2 script

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Clean up user jam file support

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Disable Boost build by default

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Remove unnecessary debug messages

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Remove _Boost_Import function

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Remove BOOST_BASE flag

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>
Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Clean up BoostConfig.cmake

- Remove _Boost_Check function.
- Add system and filesystem module build to boost build options.

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Fix typo

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Use QUIET instead of REQUIRED

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Fix invalid argument error

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Update Boost variables at boost build

Signed-off-by: Inki Dae <inki.dae@samsung.com>
* Fix build error

Signed-off-by: Inki Dae <inki.dae@samsung.com>

Remove workaround for pre-built acl (#4639)

Use pre-built acl v19.02 again

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Remove deprecated scripts in scripts/command (#4640)

Remove deprecated scripts in scripts/command
Build scripts are in scripts/standalone

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

[neurun] Correct header inclusions in Executor (#4618)

- `BackendResolver` is not needed
- Plan is required for `Executor`, not `IExecutor`

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>

Introduce nnapi-header CMake target (#4631)

This commit introduces nnapi-header target in CMake, and removes all
the occurences of NNFW_INCLUDE_DIR via linking this target instead.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

acl version up to 19.02 (#4549)

* build ACL unconditionally

- This patch should be reverted after CI infra(e.g .pre-built ACL binary) is updated.

* Compute Library update to v19.02

- Update ACL to v19.02
- Remove unness. files from ARMComputeEx

Signed-off-by: Chunseok Lee <chunseok.lee@samsung.com>

Fix split implementation and test (#4622)

* Fix split implementation and test

Fix plit implementation based on NeuralNetworksEx.h spec
- Fix split in nnapi_delegate.cpp
- Fix pacl implementation
- Fix spli test

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>
* Fix comment

[neurun] ReduceSum op (#4612)

This commit adds `ReduceSumEx` op and enable testcase into neurun.

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

[neurun] Enable Logistic op in acl_cl (#4599)

* [neurun] Enable Logistic op in acl_cl

This commit allows to run logistic op using acl_cl kernel

Signed-off-by: Jiyoung Yun <jy910.yun@samsung.com>
* Remove these tests in skip list of neurun

Do NOT use TFLITE_DEPEND (#4623)

Each source package provides the source path as a variable.
Let's use these dedicated variables instead of TFLITE_DEPEND.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Remove OPTION_TOOLCHAIN variable (#4626)

This commit removes OPTION_TOOLCHAIN variable in Makefile, and uses
OPTION instead in order to simplify Makefile.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Introduce CI infra neurun build and test scripts (#4611)

* Introduce CI infra neurun build and test scripts

Introduce neurun build scripts
Introduce neurun cpu and acl_cl test

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>
* Backend setting for acl_cl

[neurun] Fix build break due to namespace change (#4625)

Remove specifying `compiler::` namespace for `IExecutionBuilder`. Use
`using` at the top of the source code instead.

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>

Always require NEON2SSESource for TFLITE build (#4620)

* Always require NEON2SSESource for TFLITE build

NEON2SSE is a header-only library, and thus there is no need to exclude
it for cross or arm native build.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>
* Check TARGET_ARCH in NEON2SSESource package

[neurun] Enable Exp op in neurun (#4554)

Related : #4259

This commit enables `Exp` op in `neurun` for `acl_cl`, that is from `PACL`.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>

Remove (deprecated) kernel testing project (#4621)

This kernel testing project was introduced as a test at the early stage,
but is not used anymore.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

[neurun] Remove LINEARIZED state (#4617)

Remove LINEARIZED state from Compiler's state. This is not a general
state since `DataflowExecutor` does not have this state.

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>

[neurun] Introduce Config `EXECUTOR` (#4616)

Introduce environment variable `EXECUTOR` to specify which executor to
use.

Now we can run with Dataflow Executor with these variables.

```bash
export CPU_MEMORY_PLANNER=Bump
export EXECUTOR=Dataflow
```

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>

Update variable setting in scripts (#4603)

- Remove export in scripts/standalone

We can remove them because we call other script using source

- Remove setting BENCHMARK_ACL_BUILD

- Use make command more lines

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

[neurun] Remove unused inclusion in Executor.cc (#4615)

Remove unused header inclusion in `Executor.cc`.

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>

Fix packaging related code (#4593)

Packaging scripts in test suite to support test in target device
Call build_test_suite for x64 test

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Do not guess build mode (native/cross) (#4607)

This commit updates identify_platform.cmake to use configuration
variable that user provides.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Introduce CI infra build script for model test (#4613)

Introduce runtime build script for model test

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Do NOT require Flatbuffers (#4614)

It turns out that TensorFlow Lite does not depend on Flatbuffers
runtime.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

[neurun] Fix a compile error (#4609)

This commit fixes an error occurred due to a gap of the source merge.

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

[neurun] Extract IExecutionBuilder to a file (#4602)

Extract IExecutionBuilder to a file in namespace `compiler`.

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>

[neurun] Add a helper function and an assertion (#4598)

This commit adds a helper function and an assertion in `model::operand::Object`.
  - The helper function `hasData()` check whether Object has the data or not.
  - Add an assertion to `model::operand::Object::data()`.

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

[neurun] Fix missing error handling (#4597)

This commit fixes missing error handling.

Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

[neurun] Fixed typo in LogisticNode (#4600)

This commit fixes typo in LogisticNode

Signed-off-by: Jiyoung Yun <jy910.yun@samsung.com>

Generate NNAPI tests during CMake configuration (#4601)

The current implementation invokes test generating script inside
Makefile.

Now, CMake will invoke this script during configuration.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Rename and fix pacl test (#4595)

- Rename pacl build scripts
- Rename pacl test script

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Version update to 0.3 for tizen gbs build (#4596)

- Note that nnfw of tizen 5.5 branch had been updated to v0.3

Signed-off-by: Chunseok Lee <chunseok.lee@samsung.com>

Introduce CI infra coverage scripts (#4582)

- Introduce coverage build script
- Introduce coverage test script
- Introduce coverage report script
- Rename coverage report command

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

[Neurun] Transpose op (#4541)

* [Neurun] Transpose op

This adds `transpose` op and test cases into Neurun.

Signed-off-by: Hyun Sik Yoon <hyunsik.yoon@samsung.com>
* move assert(...) in StageGenerator to OperationValidator

* remove visit() of TransposeNode in cpu/StqageGenerator.h

Introduce CI infra x64 build and test scripts (#4592)

Introduce x64 build and test scripts

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

[neurun] Remove switch stmt from addOperationEx (#4591)

As applied in #4577, This commit removes switch stmt from addOperationEx too.

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>

[neurun] Enable DIV op in neurun (#4551)

* [neurun] Enable DIV op in neurun

This commit enables `DIV` op in `neurun` for `acl_cl`, that is from `PACL`

Signed-off-by: sjsujinkim <sjsujin.kim@samsung.com>
* Fix typo

* Add override

Rename directory for standalone script (#4589)

Rename directory for standalone script: scripts/ci -> scripts/standalone

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Move tizen test script (#4581)

Move tizen test script from scripts/command to scripts/ci

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

[neurun] Extract Backend as a separate file (#4587)

Extract `backend::Backend` as a separate file.

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>

[neurun] Enable compilation for DataflowExecutor (#4586)

* [neurun] Enable compilation for DataflowExecutor

This enables compilation compiles to DataflowExecutor. Currently it uses
only Executor, however the `use_dataflow_executor` will be decided with
a config value.

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>
* Set use_dataflow_executor = false

As default is Executor(Linear).

Signed-off-by: Hanjoung Lee <hanjoung.lee@samsung.com>

Change file access mode in script/ci (#4583)

Change docker_build_cross_arm_ubuntu_release.sh access mode in script/ci

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Remove acl and internal_acl_build targets (#4580)

Now, there is no code/routine that invokes these targets.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Remove benchmark acl script (#4564)

This commit removes ACL benchmark script, and updates related test
driver.

Note that this commit does not remove the option of test driver to
maintain the compatibility with CI infrastructure.

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>

Move CI infra tizen build script (#4578)

Move CI infra tizen cross build and gbs build script from script/command

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

Rename format check command (#4579)

Rename format check command: format-checker -> format_check

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>