Re: [OE-core] [PATCH 2/2] linux-firmware: package nvidia firmware

["Quentin Schulz" <quentin.schulz@streamunlimited.com>]

Hi Max,

On Mon, Oct 19, 2020 at 11:38:01AM +0200, Max Krummenacher wrote:
> Add packages for the firmware required by the Nvidia chipsets.
> Split it in Tegra K1, all other Tegras and desktop GPU packages.
> 
> Signed-off-by: Max Krummenacher <max.krummenacher@toradex.com>
> ---
>  .../linux-firmware/linux-firmware_20200817.bb | 24 +++++++++++++++++++
>  1 file changed, 24 insertions(+)
> 
> diff --git a/meta/recipes-kernel/linux-firmware/linux-firmware_20200817.bb b/meta/recipes-kernel/linux-firmware/linux-firmware_20200817.bb
> index 3d6644e504..5bb27c51c7 100644
> --- a/meta/recipes-kernel/linux-firmware/linux-firmware_20200817.bb
> +++ b/meta/recipes-kernel/linux-firmware/linux-firmware_20200817.bb
> @@ -288,6 +288,9 @@ PACKAGES =+ "${PN}-ralink-license ${PN}-ralink \
>               ${PN}-adsp-sst-license ${PN}-adsp-sst \
>               ${PN}-bnx2-mips \
>               ${PN}-liquidio \
> +             ${PN}-nvidia-license \
> +             ${PN}-nvidia-tegra-k1 ${PN}-nvidia-tegra \
> +             ${PN}-nvidia-gpu \
>               ${PN}-netronome-license ${PN}-netronome \
>               ${PN}-qat ${PN}-qat-license \
>               ${PN}-qcom-license \
> @@ -478,6 +481,27 @@ FILES_${PN}-netronome = " \
>  
>  RDEPENDS_${PN}-netronome += "${PN}-netronome-license"
>  
> +# For Nvidia
> +LICENSE_${PN}-nvidia-gpu = "Firmware-nvidia"
> +LICENSE_${PN}-nvidia-tegra = "Firmware-nvidia"
> +LICENSE_${PN}-nvidia-tegra-k1 = "Firmware-nvidia"
> +LICENSE_${PN}-nvidia-license = "Firmware-nvidia"
> +
> +FILES_${PN}-nvidia-gpu = "/lib/firmware/nvidia"
                             ^^^^ nonarch_base_libdir instead?
> +FILES_${PN}-nvidia-tegra = " \
> +  ${nonarch_base_libdir}/firmware/nvidia/tegra* \
> +  ${nonarch_base_libdir}/firmware/nvidia/gm20b \
> +  ${nonarch_base_libdir}/firmware/nvidia/gp10b \
> +"
> +FILES_${PN}-nvidia-tegra-k1 = " \
> +  ${nonarch_base_libdir}/firmware/nvidia/tegra124 \
> +  ${nonarch_base_libdir}/firmware/nvidia/gk20a \
> +"
> +FILES_${PN}-nvidia-license = "/lib/firmware/LICENCE.nvidia"
                                 ^^^^ nonarch_base_libdir instead?

Thanks,
Quentin