[Buildroot] [PATCH 1/1] package/freescale-imx/imx-gpu-g2d: fix the file name

[Buildroot] [PATCH 1/1] package/freescale-imx/imx-gpu-g2d: fix the file name

[Timo Ketola]

NXP apparently changed file naming convention for this particular file.

Signed-off-by: Timo Ketola <timo.ketola@exertus.fi>
---
 package/freescale-imx/imx-gpu-g2d/imx-gpu-g2d.hash | 2 +-
 package/freescale-imx/imx-gpu-g2d/imx-gpu-g2d.mk   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/freescale-imx/imx-gpu-g2d/imx-gpu-g2d.hash b/package/freescale-imx/imx-gpu-g2d/imx-gpu-g2d.hash
index 8ecfbfe..98ca7fb 100644
--- a/package/freescale-imx/imx-gpu-g2d/imx-gpu-g2d.hash
+++ b/package/freescale-imx/imx-gpu-g2d/imx-gpu-g2d.hash
@@ -1,5 +1,5 @@
 # Locally calculated
-sha256	6d6d02d035d988cbdb2014794f2efe970a3f5ac277202882bbf553bb7fd789d2  imx-gpu-g2d-6.4.0.p1.0-aarch32.bin
+sha256	6d6d02d035d988cbdb2014794f2efe970a3f5ac277202882bbf553bb7fd789d2  imx-gpu-g2d-6.4.0.p1.0-arm.bin
 sha256	35885f435dbe951d2c0f439a376ed8a59a50b67a1b7e1e14fa2ab03995abb1ad  imx-gpu-g2d-6.4.0.p1.0-aarch64.bin
 sha256	d55f024af2bfff714b90de596f6d0399124b999e8c18a86b13a3b507bae6f561  COPYING
 sha256	9665930f69c0b6f4a4c055d7fe2b8ee563e771efbc83892abb1955e61492cdf7  EULA
diff --git a/package/freescale-imx/imx-gpu-g2d/imx-gpu-g2d.mk b/package/freescale-imx/imx-gpu-g2d/imx-gpu-g2d.mk
index 6598e4c..7700170 100644
--- a/package/freescale-imx/imx-gpu-g2d/imx-gpu-g2d.mk
+++ b/package/freescale-imx/imx-gpu-g2d/imx-gpu-g2d.mk
@@ -7,7 +7,7 @@
 ifeq ($(BR2_aarch64),y)
 IMX_GPU_G2D_VERSION = 6.4.0.p1.0-aarch64
 else
-IMX_GPU_G2D_VERSION = 6.4.0.p1.0-aarch32
+IMX_GPU_G2D_VERSION = 6.4.0.p1.0-arm
 endif
 IMX_GPU_G2D_SITE = $(FREESCALE_IMX_SITE)
 IMX_GPU_G2D_SOURCE = imx-gpu-g2d-$(IMX_GPU_G2D_VERSION).bin
-- 
2.7.4

[Buildroot] [PATCH 1/1] package/freescale-imx/imx-gpu-g2d: fix the file name

[Gary Bisson]

Hi,

Thanks for your contribution.

On Thu, Apr 23, 2020 at 07:51:49AM +0300, Timo Ketola wrote:
> NXP apparently changed file naming convention for this particular file.

Yes apparently they started naming it this way with 6.4.0.p1.0 so that
it simplifies the recipe [1]:
FSLBIN_NAME_arm = "${PN}-${PV}-${TARGET_ARCH}"

So IMO this commit log should mention it fixes last commit so that it's
picked up for "LTS" branches:
Fixes:
1d0ea8d433 (package/freescale-imx/imx-gpu-g2d: bump to version
6.4.0.p1.0)

> Signed-off-by: Timo Ketola <timo.ketola@exertus.fi>

Otherwise:
Reviewed-by: Gary Bisson <gary.bisson@boundarydevices.com>

Regards,
Gary

[1] https://source.codeaurora.org/external/imx/meta-fsl-bsp-release/tree/imx/meta-bsp/recipes-graphics/imx-gpu-g2d/imx-gpu-g2d_6.4.0.p1.0.bb?h=warrior-4.19.35-1.1.0

[Buildroot] [PATCH v2 1/1] package/freescale-imx/imx-gpu-g2d: fix the file name

[Timo Ketola]

NXP apparently changed file naming convention for this particular file.

Fixes:
1d0ea8d433 (package/freescale-imx/imx-gpu-g2d: bump to version 6.4.0.p1.0)

Signed-off-by: Timo Ketola <timo.ketola@exertus.fi>
---
Changes v1 -> v2:

 - Add 'Fixes:' line as suggested by Gary

 package/freescale-imx/imx-gpu-g2d/imx-gpu-g2d.hash | 2 +-
 package/freescale-imx/imx-gpu-g2d/imx-gpu-g2d.mk   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/freescale-imx/imx-gpu-g2d/imx-gpu-g2d.hash b/package/freescale-imx/imx-gpu-g2d/imx-gpu-g2d.hash
index 8ecfbfe..98ca7fb 100644
--- a/package/freescale-imx/imx-gpu-g2d/imx-gpu-g2d.hash
+++ b/package/freescale-imx/imx-gpu-g2d/imx-gpu-g2d.hash
@@ -1,5 +1,5 @@
 # Locally calculated
-sha256	6d6d02d035d988cbdb2014794f2efe970a3f5ac277202882bbf553bb7fd789d2  imx-gpu-g2d-6.4.0.p1.0-aarch32.bin
+sha256	6d6d02d035d988cbdb2014794f2efe970a3f5ac277202882bbf553bb7fd789d2  imx-gpu-g2d-6.4.0.p1.0-arm.bin
 sha256	35885f435dbe951d2c0f439a376ed8a59a50b67a1b7e1e14fa2ab03995abb1ad  imx-gpu-g2d-6.4.0.p1.0-aarch64.bin
 sha256	d55f024af2bfff714b90de596f6d0399124b999e8c18a86b13a3b507bae6f561  COPYING
 sha256	9665930f69c0b6f4a4c055d7fe2b8ee563e771efbc83892abb1955e61492cdf7  EULA
diff --git a/package/freescale-imx/imx-gpu-g2d/imx-gpu-g2d.mk b/package/freescale-imx/imx-gpu-g2d/imx-gpu-g2d.mk
index 6598e4c..7700170 100644
--- a/package/freescale-imx/imx-gpu-g2d/imx-gpu-g2d.mk
+++ b/package/freescale-imx/imx-gpu-g2d/imx-gpu-g2d.mk
@@ -7,7 +7,7 @@
 ifeq ($(BR2_aarch64),y)
 IMX_GPU_G2D_VERSION = 6.4.0.p1.0-aarch64
 else
-IMX_GPU_G2D_VERSION = 6.4.0.p1.0-aarch32
+IMX_GPU_G2D_VERSION = 6.4.0.p1.0-arm
 endif
 IMX_GPU_G2D_SITE = $(FREESCALE_IMX_SITE)
 IMX_GPU_G2D_SOURCE = imx-gpu-g2d-$(IMX_GPU_G2D_VERSION).bin
-- 
2.7.4

[Buildroot] [PATCH v2 1/1] package/freescale-imx/imx-gpu-g2d: fix the file name

[Thomas Petazzoni]

On Thu, 23 Apr 2020 14:06:45 +0300
Timo Ketola <timo.ketola@exertus.fi> wrote:

> NXP apparently changed file naming convention for this particular file.
> 
> Fixes:
> 1d0ea8d433 (package/freescale-imx/imx-gpu-g2d: bump to version 6.4.0.p1.0)
> 
> Signed-off-by: Timo Ketola <timo.ketola@exertus.fi>
> ---
> Changes v1 -> v2:

Applied to master, thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

[Buildroot] [PATCH v2 1/1] package/freescale-imx/imx-gpu-g2d: fix the file name

[Peter Korsgaard]

>>>>> "Timo" == Timo Ketola <timo.ketola@exertus.fi> writes:

 > NXP apparently changed file naming convention for this particular file.
 > Fixes:
 > 1d0ea8d433 (package/freescale-imx/imx-gpu-g2d: bump to version 6.4.0.p1.0)

 > Signed-off-by: Timo Ketola <timo.ketola@exertus.fi>
 > ---
 > Changes v1 -> v2:

 >  - Add 'Fixes:' line as suggested by Gary

Committed to 2020.02.x, thanks.

-- 
Bye, Peter Korsgaard