Thanks for your review.
1) Yes, zeus should be added to layer.conf.
I tested the patch with master and linux-yocto-dev.
2) intel_qat.ko depends on dh_generic.ko. So CONFIG_CRYPTO_DH must be enabled in
kernel first.
3) I will try ${nonarch_base_libdir}.
After fixing 1) and 2), I will send V2.
--Yongxin
> -----Original Message-----
> From: Mittal, Anuj [mailto:[email protected]]
> Sent: Thursday, January 9, 2020 14:17
> To: Liu, Yongxin; [email protected]; Tan, Raymond
> Subject: Re: [meta-intel][meta-intel-qat][PATCH] qat: Upgrade to
> qat17_4.7.0-00006
>
> Hello,
>
> Thank you for the patch. Can I know if this has been tested with master
> or zeus?
>
> Right now, the layer.conf only has warrior and thud as supported
> versions. Should we change that too if you are using this layer with
> master/zeus?
>
> I'm getting this error when compiling with master:
>
> ERROR: "crypto_dh_decode_key" [/poky/build/tmp/work/intel_corei7_64-
> poky-linux/qat17/4.7.0-00006-
> r0/qat17/quickassist/qat/drivers/crypto/qat/qat_common/intel_qat.ko]
> undefined!
>
> On Wed, 2020-01-08 at 19:54 +0800, Yongxin Liu wrote:
> > Rebased patches:
> > qat16_2.3.0-34-qat-remove-local-path-from-makefile.patch
> > qat17_0.9.0-4-qat-add-install-target-and-add-folder.patch
> >
> > Added patch:
> > qat17_4.7.0-00006-Link-driver-with-object-files.patch
> >
> > Remove duplicate firmware installation, since linux-firmware.bb
> > in OE-Core will do it.
> >
> > Remove qat17-src from PACKAGES due to duplicate packaging.
> >
> > Use /lib instead of ${base_libdir} in recipe file, since
> > ${base_libdir} is parsed as /lib64.
>
> This should be ${nonarch_base_libdir}.
>
> Thanks,
>
> Anuj
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#6320):
https://lists.yoctoproject.org/g/meta-intel/message/6320
Mute This Topic: https://lists.yoctoproject.org/mt/69526180/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/meta-intel/unsub
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-
