diff mbox series

[meta-arago,kirkstone/master] kms++: update to the latest and bump version

Message ID 20240430094003.196209-1-j-choudhary@ti.com
State Changes Requested
Delegated to: Ryan Eatmon
Headers show
Series [meta-arago,kirkstone/master] kms++: update to the latest and bump version | expand

Commit Message

Jayesh Choudhary April 30, 2024, 9:40 a.m. UTC
ltp-ddt scripts are using latest dmabuf utilities but the current
source code does not have that support in kms++.
Bump up the SRCREV to take into account the latest dmabuf changes.

Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
---
 meta-arago-extras/recipes-graphics/kms++/kms++_git.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Ryan Eatmon April 30, 2024, 10:54 p.m. UTC | #1
Is this supposed to be for kirkstone?  Or did you mean scarthgap?


On 4/30/2024 4:40 AM, Jayesh Choudhary wrote:
> ltp-ddt scripts are using latest dmabuf utilities but the current
> source code does not have that support in kms++.
> Bump up the SRCREV to take into account the latest dmabuf changes.
> 
> Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
> ---
>   meta-arago-extras/recipes-graphics/kms++/kms++_git.bb | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta-arago-extras/recipes-graphics/kms++/kms++_git.bb b/meta-arago-extras/recipes-graphics/kms++/kms++_git.bb
> index 2c0abaa7..bae03683 100644
> --- a/meta-arago-extras/recipes-graphics/kms++/kms++_git.bb
> +++ b/meta-arago-extras/recipes-graphics/kms++/kms++_git.bb
> @@ -3,12 +3,12 @@ HOMEPAGE = "https://github.com/tomba/kmsxx"
>   LICENSE = "MPL-2.0"
>   LIC_FILES_CHKSUM = "file://LICENSE;md5=815ca599c9df247a0c7f619bab123dad"
>   
> -PV = "2.1"
> +PV = "2.2"
>   PR = "r1"
>   
>   BRANCH = "master"
>   SRC_URI = "git://github.com/tomba/kmsxx.git;protocol=https;branch=${BRANCH}"
> -SRCREV = "5afc8d918f2c084acd65027604868dfde43395cf"
> +SRCREV = "9ae90ce75478e49844cf984562db0dc1a074462f"
>   
>   DEPENDS = "drm fmt"
>
Jayesh Choudhary May 2, 2024, 5:11 a.m. UTC | #2
On 01/05/24 04:24, Ryan Eatmon wrote:
> 
> Is this supposed to be for kirkstone?  Or did you mean scarthgap?
> 

Are we not maintaining kirstone anymore?

This patch applies to all three branches. It applies on scarthgap too.

-Jayesh

> 
> On 4/30/2024 4:40 AM, Jayesh Choudhary wrote:
>> ltp-ddt scripts are using latest dmabuf utilities but the current
>> source code does not have that support in kms++.
>> Bump up the SRCREV to take into account the latest dmabuf changes.
>>
>> Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
>> ---
>>   meta-arago-extras/recipes-graphics/kms++/kms++_git.bb | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta-arago-extras/recipes-graphics/kms++/kms++_git.bb 
>> b/meta-arago-extras/recipes-graphics/kms++/kms++_git.bb
>> index 2c0abaa7..bae03683 100644
>> --- a/meta-arago-extras/recipes-graphics/kms++/kms++_git.bb
>> +++ b/meta-arago-extras/recipes-graphics/kms++/kms++_git.bb
>> @@ -3,12 +3,12 @@ HOMEPAGE = "https://github.com/tomba/kmsxx"
>>   LICENSE = "MPL-2.0"
>>   LIC_FILES_CHKSUM = 
>> "file://LICENSE;md5=815ca599c9df247a0c7f619bab123dad"
>> -PV = "2.1"
>> +PV = "2.2"
>>   PR = "r1"
>>   BRANCH = "master"
>>   SRC_URI = 
>> "git://github.com/tomba/kmsxx.git;protocol=https;branch=${BRANCH}"
>> -SRCREV = "5afc8d918f2c084acd65027604868dfde43395cf"
>> +SRCREV = "9ae90ce75478e49844cf984562db0dc1a074462f"
>>   DEPENDS = "drm fmt"
>
Ryan Eatmon May 3, 2024, 1:05 a.m. UTC | #3
On 4/30/2024 4:40 AM, Jayesh Choudhary wrote:
> ltp-ddt scripts are using latest dmabuf utilities but the current
> source code does not have that support in kms++.
> Bump up the SRCREV to take into account the latest dmabuf changes.
> 
> Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
> ---
>   meta-arago-extras/recipes-graphics/kms++/kms++_git.bb | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta-arago-extras/recipes-graphics/kms++/kms++_git.bb b/meta-arago-extras/recipes-graphics/kms++/kms++_git.bb
> index 2c0abaa7..bae03683 100644
> --- a/meta-arago-extras/recipes-graphics/kms++/kms++_git.bb
> +++ b/meta-arago-extras/recipes-graphics/kms++/kms++_git.bb
> @@ -3,12 +3,12 @@ HOMEPAGE = "https://github.com/tomba/kmsxx"
>   LICENSE = "MPL-2.0"
>   LIC_FILES_CHKSUM = "file://LICENSE;md5=815ca599c9df247a0c7f619bab123dad"
>   
> -PV = "2.1"
> +PV = "2.2"
>   PR = "r1"
>   
>   BRANCH = "master"
>   SRC_URI = "git://github.com/tomba/kmsxx.git;protocol=https;branch=${BRANCH}"
> -SRCREV = "5afc8d918f2c084acd65027604868dfde43395cf"
> +SRCREV = "9ae90ce75478e49844cf984562db0dc1a074462f"

Did you do any test builds with this?  I'm getting build errors related 
to the line in the recipe:

EXTRA_OEMESON = "-Dsystem-pybind11=enabled"

Which needs to be removed according to this commit:

https://github.com/tomba/kmsxx/commit/77eedf7574809ce5edf4d4ea6aa55542b3045c02

So I'm not sure how you did a test build that worked...  Can you send a v2?



>   DEPENDS = "drm fmt"
>
diff mbox series

Patch

diff --git a/meta-arago-extras/recipes-graphics/kms++/kms++_git.bb b/meta-arago-extras/recipes-graphics/kms++/kms++_git.bb
index 2c0abaa7..bae03683 100644
--- a/meta-arago-extras/recipes-graphics/kms++/kms++_git.bb
+++ b/meta-arago-extras/recipes-graphics/kms++/kms++_git.bb
@@ -3,12 +3,12 @@  HOMEPAGE = "https://github.com/tomba/kmsxx"
 LICENSE = "MPL-2.0"
 LIC_FILES_CHKSUM = "file://LICENSE;md5=815ca599c9df247a0c7f619bab123dad"
 
-PV = "2.1"
+PV = "2.2"
 PR = "r1"
 
 BRANCH = "master"
 SRC_URI = "git://github.com/tomba/kmsxx.git;protocol=https;branch=${BRANCH}"
-SRCREV = "5afc8d918f2c084acd65027604868dfde43395cf"
+SRCREV = "9ae90ce75478e49844cf984562db0dc1a074462f"
 
 DEPENDS = "drm fmt"