[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