Sorry this has taken so long.  The change is currently in review, hopefully
it'll only be a few more days and I'll get it merged.

--Mark

On 4/22/21 11:04 AM, Ricardo Salveti wrote:
> On Thu, Apr 1, 2021 at 2:49 PM Mark Hatle
> <[email protected]> wrote:
>>
>> Sorry, didn't see it before.  I'll take a look at it in the next few days and
>> likely merge it or commit it early next week.
> 
> Gentle ping :-)
> 
>>
>> --Mark
>>
>> On 4/1/21 12:33 PM, Ricardo Salveti wrote:
>>> Any comments/reviews?
>>>
>>> Tried sending a similar patch via github pr but also got no response.
>>>
>>> Thanks,
>>>
>>> Ricardo
>>>
>>> On Wed, Mar 3, 2021 at 12:28 PM Ricardo Salveti <[email protected]> 
>>> wrote:
>>>>
>>>> Fixes a packaging error when using 'usrmerge' in DISTRO_FEATURES
>>>> (nonarch_base_libdir is set to /usr/lib in this case, and to /lib when
>>>>  usrmerge is not used).
>>>>
>>>> Signed-off-by: Ricardo Salveti <[email protected]>
>>>> ---
>>>>  classes/fpgamanager_custom.bbclass                   | 10 +++++-----
>>>>  classes/fpgamanager_dtg.bbclass                      | 10 +++++-----
>>>>  .../fpga-manager-util/fpga-manager-util_1.0.bb       | 12 ++++++------
>>>>  3 files changed, 16 insertions(+), 16 deletions(-)
>>>>
>>>> diff --git a/classes/fpgamanager_custom.bbclass 
>>>> b/classes/fpgamanager_custom.bbclass
>>>> index 733f403..ba1f79c 100644
>>>> --- a/classes/fpgamanager_custom.bbclass
>>>> +++ b/classes/fpgamanager_custom.bbclass
>>>> @@ -63,14 +63,14 @@ python devicetree_do_compile_append() {
>>>>  }
>>>>
>>>>  do_install() {
>>>> -    install -d ${D}/lib/firmware/xilinx/${PN}/
>>>> -    install -Dm 0644 *.dtbo ${D}/lib/firmware/xilinx/${PN}/${PN}.dtbo
>>>> -    install -Dm 0644 ${PN}.bit.bin 
>>>> ${D}/lib/firmware/xilinx/${PN}/${PN}.bit.bin
>>>> +    install -d ${D}${nonarch_base_libdir}/firmware/xilinx/${PN}/
>>>> +    install -Dm 0644 *.dtbo 
>>>> ${D}${nonarch_base_libdir}/firmware/xilinx/${PN}/${PN}.dtbo
>>>> +    install -Dm 0644 ${PN}.bit.bin 
>>>> ${D}${nonarch_base_libdir}/firmware/xilinx/${PN}/${PN}.bit.bin
>>>>      if ls ${WORKDIR}/${XCL_PATH}/*.xclbin >/dev/null 2>&1; then
>>>> -        install -Dm 0644 ${WORKDIR}/${XCL_PATH}/*.xclbin 
>>>> ${D}/lib/firmware/xilinx/${PN}/${PN}.xclbin
>>>> +        install -Dm 0644 ${WORKDIR}/${XCL_PATH}/*.xclbin 
>>>> ${D}${nonarch_base_libdir}/firmware/xilinx/${PN}/${PN}.xclbin
>>>>      fi
>>>>  }
>>>>
>>>>  do_deploy[noexec] = "1"
>>>>
>>>> -FILES_${PN} += "/lib/firmware/xilinx/{PN}"
>>>> +FILES_${PN} += "${nonarch_base_libdir}/firmware/xilinx/{PN}"
>>>> diff --git a/classes/fpgamanager_dtg.bbclass 
>>>> b/classes/fpgamanager_dtg.bbclass
>>>> index 048bde0..61f57ba 100644
>>>> --- a/classes/fpgamanager_dtg.bbclass
>>>> +++ b/classes/fpgamanager_dtg.bbclass
>>>> @@ -84,14 +84,14 @@ python devicetree_do_compile_append() {
>>>>  }
>>>>
>>>>  do_install() {
>>>> -    install -d ${D}/lib/firmware/xilinx/${PN}/
>>>> -    install -Dm 0644 pl-final.dtbo 
>>>> ${D}/lib/firmware/xilinx/${PN}/${PN}.dtbo
>>>> -    install -Dm 0644 ${PN}.bit.bin 
>>>> ${D}/lib/firmware/xilinx/${PN}/${PN}.bit.bin
>>>> +    install -d ${D}${nonarch_base_libdir}/firmware/xilinx/${PN}/
>>>> +    install -Dm 0644 pl-final.dtbo 
>>>> ${D}${nonarch_base_libdir}/firmware/xilinx/${PN}/${PN}.dtbo
>>>> +    install -Dm 0644 ${PN}.bit.bin 
>>>> ${D}${nonarch_base_libdir}/firmware/xilinx/${PN}/${PN}.bit.bin
>>>>      if ls ${WORKDIR}/${XCL_PATH}/*.xclbin >/dev/null 2>&1; then
>>>> -        install -Dm 0644 ${WORKDIR}/${XCL_PATH}/*.xclbin 
>>>> ${D}/lib/firmware/xilinx/${PN}/${PN}.xclbin
>>>> +        install -Dm 0644 ${WORKDIR}/${XCL_PATH}/*.xclbin 
>>>> ${D}${nonarch_base_libdir}/firmware/xilinx/${PN}/${PN}.xclbin
>>>>      fi
>>>>  }
>>>>
>>>>  do_deploy[noexec] = "1"
>>>>
>>>> -FILES_${PN} += "/lib/firmware/xilinx/${PN}"
>>>> +FILES_${PN} += "${nonarch_base_libdir}/firmware/xilinx/${PN}"
>>>> diff --git a/recipes-bsp/fpga-manager-util/fpga-manager-util_1.0.bb 
>>>> b/recipes-bsp/fpga-manager-util/fpga-manager-util_1.0.bb
>>>> index 53a041c..6c2a062 100644
>>>> --- a/recipes-bsp/fpga-manager-util/fpga-manager-util_1.0.bb
>>>> +++ b/recipes-bsp/fpga-manager-util/fpga-manager-util_1.0.bb
>>>> @@ -130,23 +130,23 @@ do_compile() {
>>>>  }
>>>>
>>>>  do_install() {
>>>> -        install -d ${D}/lib/firmware/xilinx/base
>>>> +        install -d ${D}${nonarch_base_libdir}/firmware/xilinx/base
>>>>          if [ -e "base.dtbo" ]; then
>>>>              #install base hdf artifacts
>>>> -            install -Dm 0644 base.dtbo 
>>>> ${D}/lib/firmware/xilinx/base/base.dtbo
>>>> +            install -Dm 0644 base.dtbo 
>>>> ${D}${nonarch_base_libdir}/firmware/xilinx/base/base.dtbo
>>>>              if [ "${SOC_FAMILY}" != "versal" ]; then
>>>>                  newname=`basename *.bit.bin_base | awk -F '.bit.bin_' 
>>>> '{print $1}'`
>>>> -                install -Dm 0644 *.bit.bin_base 
>>>> ${D}/lib/firmware/xilinx/base/${newname}.bit.bin
>>>> +                install -Dm 0644 *.bit.bin_base 
>>>> ${D}${nonarch_base_libdir}/firmware/xilinx/base/${newname}.bit.bin
>>>>              else
>>>>                  #partial pdi in xsa is not yet supported, will need to 
>>>> modify this part once supported
>>>>                  echo "TODO"
>>>>              fi
>>>>          fi
>>>>          for hdf in ${HDF_LIST}; do
>>>> -                install -Dm 0644 ${hdf}.dtbo 
>>>> ${D}/lib/firmware/xilinx/${hdf}/${hdf}.dtbo
>>>> +                install -Dm 0644 ${hdf}.dtbo 
>>>> ${D}${nonarch_base_libdir}/firmware/xilinx/${hdf}/${hdf}.dtbo
>>>>                  if [ "${SOC_FAMILY}" != "versal" ]; then
>>>>                      newname=`basename *.bit.bin_${hdf} | awk -F 
>>>> '.bit.bin_' '{print $1}'`
>>>> -                    install -Dm 0644 *.bit.bin_${hdf} 
>>>> ${D}/lib/firmware/xilinx/${hdf}/${newname}.bit.bin
>>>> +                    install -Dm 0644 *.bit.bin_${hdf} 
>>>> ${D}${nonarch_base_libdir}/firmware/xilinx/${hdf}/${newname}.bit.bin
>>>>                  else
>>>>                      #partial pdi in xsa is not yet supported, will need 
>>>> to modify this part once supported
>>>>                      echo "TODO"
>>>> @@ -160,7 +160,7 @@ python () {
>>>>          if d.getVar('FPGA_MNGR_RECONFIG_ENABLE') == '1':
>>>>                  extra = d.getVar('EXTRA_HDF')
>>>>                  pn = d.getVar('PN')
>>>> -                baselib = d.getVar('base_libdir')
>>>> +                baselib = d.getVar('nonarch_base_libdir')
>>>>                  packages = d.getVar('PACKAGES').split()
>>>>
>>>>                  #package base hdf
>>>> --
>>>> 2.30.1
>>>>
>>>
>>>
>>>
>>>
>>>
>>>
> 
> 
> 
> 
> 
> 
> 
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#4890): 
https://lists.yoctoproject.org/g/meta-xilinx/message/4890
Mute This Topic: https://lists.yoctoproject.org/mt/81053937/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/meta-xilinx/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to