[meta-intel] [PATCH] dldt-inference-engine: add PACKAGECONFIG for myriad plugin

Mittal, Anuj anuj.mittal at intel.com
Fri Oct 25 19:12:18 PDT 2019


On Fri, 2019-10-25 at 19:59 +0200, Martin Jansa wrote:
> I think MYRIAD is already enabled by default:
> https://github.com/opencv/dldt/blob/2019/inference-engine/cmake/features.cmake#L48
> when not disabled together with VPU:
> https://github.com/opencv/dldt/blob/2019/inference-engine/cmake/check_features.cmake#L42
> 
> so I'm not sure if we need 2 PACKAGECONFIGs for that.

Thanks, I agree. I somehow missed that it's ON by default. Will send a
separate patch with just the installation of udev rules for myriad
based on vpu PACKAGECONFIG.

> 
> Also using 8 spaces for indentation is inconsistent with everything
> else, oe-core is using tabs for shell tasks and 4 spaces for python
> task, most other layers are using 4 spaces everywhere (worst cases
> combine both in the same task - e.g. do_install_ptest_base_prepend in
> this recipe).

I have been doing it this way in meta-intel and might also have done it
for some oe-core recipes as well in the past. I don't have any
preference though and will change.

Thanks,

Anuj


More information about the meta-intel mailing list