On 6/10/2026 5:58 PM, Antonios Christidis wrote:
Dear Ryan Denys,


Let me know your thoughts on the overall patch. I am particularly interested on your opinions on the following change:

On 6/10/26 5:51 PM, Antonios Christidis via lists.yoctoproject.org wrote:
BSP_MESA_PVR_VERSION:bsp-ti-6_18: = "2%"

So... I think what this does is establish the pattern for which version it will match to. And then from that pattern it will pick the highest version.

So for ALL 6_18 builds it will probably choose 25 and never 24. Which is not what you want.

... I think ...

We have never tried to mix versions like this in the past, so I'm not 100% sure what the best course is.

Likely a mixture of the override based on machine needs to be in there. And for that I'm thinking you might need an extra variable.

BSP_MESA_PVR_VERSION_6_18 ?= "25%"
BSP_MESA_PVR_VERSION_6_18:am57 = "24%"
BSP_MESA_PVR_VERSION_6_18:ti33 = "24%"
etc...

BSP_MESA_PVR_VERSION:bsp-ti-6_18 = "${BSP_MESA_PVR_VERSION_6_18}"


Something along those lines? That would be the most clear for people to follow as well.



I wish there was a way to use syntax like "BSP_MESA_PVR_VERSION:bsp-ti-6_18 = "24%|25%" or even "24.0.1|25.2.8". Is there a better way of enabling this logic ?

Also already aware of the extra ":" post-pended to the variable, I can send a v3 if that's all the feedback.


Kind Regards,

Antonios


--
Ryan Eatmon                [email protected]
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#20018): 
https://lists.yoctoproject.org/g/meta-ti/message/20018
Mute This Topic: https://lists.yoctoproject.org/mt/119748763/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to