diff mbox series

[meta-arago,master/kirkstone,v2] branding-ltsprep: LTS migration brand

Message ID 20230317161550.17946-1-reatmon@ti.com
State Superseded
Delegated to: Ryan Eatmon
Headers show
Series [meta-arago,master/kirkstone,v2] branding-ltsprep: LTS migration brand | expand

Commit Message

Ryan Eatmon March 17, 2023, 4:15 p.m. UTC
Add a new brand to control picking the next LTS version of kernel and
u-boot while they are under development.

Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
v2: Make the versions wildcarded so that they actually match...

 .../conf/distro/include/branding-ltsprep.inc          | 11 +++++++++++
 1 file changed, 11 insertions(+)
 create mode 100644 meta-arago-distro/conf/distro/include/branding-ltsprep.inc

Comments

Bajjuri, Praneeth March 17, 2023, 4:26 p.m. UTC | #1
On 3/17/23 11:15, Ryan Eatmon wrote:
> Add a new brand to control picking the next LTS version of kernel and
> u-boot while they are under development.
>
> Signed-off-by: Ryan Eatmon <reatmon@ti.com>


Reviewed-by: Praneeth Bajjuri <praneeth@ti.com>


> ---
> v2: Make the versions wildcarded so that they actually match...
>
>   .../conf/distro/include/branding-ltsprep.inc          | 11 +++++++++++
>   1 file changed, 11 insertions(+)
>   create mode 100644 meta-arago-distro/conf/distro/include/branding-ltsprep.inc
>
> diff --git a/meta-arago-distro/conf/distro/include/branding-ltsprep.inc b/meta-arago-distro/conf/distro/include/branding-ltsprep.inc
> new file mode 100644
> index 00000000..0238327c
> --- /dev/null
> +++ b/meta-arago-distro/conf/distro/include/branding-ltsprep.inc
> @@ -0,0 +1,11 @@
> +
> +PREFERRED_VERSION_linux-ti-staging = "6.1%"
> +PREFERRED_VERSION_linux-ti-staging-rt = "6.1%"
> +PREFERRED_VERSION_linux-ti-staging-systest = "6.1%"
> +PREFERRED_VERSION_u-boot-ti-staging = "2023.01%"
> +
> +# GPU support requires out-of-tree SGX and RGX drivers not available in until
> +# LTS prep is complete.  fall back to software rendering
> +MACHINE_FEATURES:remove = "gpu"
> +PREFERRED_PROVIDER_virtual/gpudriver = ""
> +
Andrew Davis March 17, 2023, 5:15 p.m. UTC | #2
On 3/17/23 11:15 AM, Ryan Eatmon via lists.yoctoproject.org wrote:
> Add a new brand to control picking the next LTS version of kernel and
> u-boot while they are under development.
> 
> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> ---
> v2: Make the versions wildcarded so that they actually match...
> 
>   .../conf/distro/include/branding-ltsprep.inc          | 11 +++++++++++
>   1 file changed, 11 insertions(+)
>   create mode 100644 meta-arago-distro/conf/distro/include/branding-ltsprep.inc
> 
> diff --git a/meta-arago-distro/conf/distro/include/branding-ltsprep.inc b/meta-arago-distro/conf/distro/include/branding-ltsprep.inc
> new file mode 100644
> index 00000000..0238327c
> --- /dev/null
> +++ b/meta-arago-distro/conf/distro/include/branding-ltsprep.inc
> @@ -0,0 +1,11 @@
> +
> +PREFERRED_VERSION_linux-ti-staging = "6.1%"
> +PREFERRED_VERSION_linux-ti-staging-rt = "6.1%"
> +PREFERRED_VERSION_linux-ti-staging-systest = "6.1%"
> +PREFERRED_VERSION_u-boot-ti-staging = "2023.01%"
> +
> +# GPU support requires out-of-tree SGX and RGX drivers not available in until
> +# LTS prep is complete.  fall back to software rendering
> +MACHINE_FEATURES:remove = "gpu"

We can leave the MACHINE_FEATURES as is, that way we still build the GUI bits,
only the below line is needed to switch to software rendering.

Otherwise, I would have preferred this patch the other way, have a new branding
to use the old unsupported kernel. Either way this seems reasonable for now.

Acked-by: Andrew Davis <afd@ti.com>

> +PREFERRED_PROVIDER_virtual/gpudriver = ""
> +
Denys Dmytriyenko March 17, 2023, 6:12 p.m. UTC | #3
On Fri, Mar 17, 2023 at 12:15:01PM -0500, Andrew Davis via lists.yoctoproject.org wrote:
> On 3/17/23 11:15 AM, Ryan Eatmon via lists.yoctoproject.org wrote:
> >Add a new brand to control picking the next LTS version of kernel and
> >u-boot while they are under development.
> >
> >Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> >---
> >v2: Make the versions wildcarded so that they actually match...
> >
> >  .../conf/distro/include/branding-ltsprep.inc          | 11 +++++++++++
> >  1 file changed, 11 insertions(+)
> >  create mode 100644 meta-arago-distro/conf/distro/include/branding-ltsprep.inc
> >
> >diff --git a/meta-arago-distro/conf/distro/include/branding-ltsprep.inc b/meta-arago-distro/conf/distro/include/branding-ltsprep.inc
> >new file mode 100644
> >index 00000000..0238327c
> >--- /dev/null
> >+++ b/meta-arago-distro/conf/distro/include/branding-ltsprep.inc
> >@@ -0,0 +1,11 @@
> >+
> >+PREFERRED_VERSION_linux-ti-staging = "6.1%"
> >+PREFERRED_VERSION_linux-ti-staging-rt = "6.1%"
> >+PREFERRED_VERSION_linux-ti-staging-systest = "6.1%"
> >+PREFERRED_VERSION_u-boot-ti-staging = "2023.01%"
> >+
> >+# GPU support requires out-of-tree SGX and RGX drivers not available in until
> >+# LTS prep is complete.  fall back to software rendering
> >+MACHINE_FEATURES:remove = "gpu"
> 
> We can leave the MACHINE_FEATURES as is, that way we still build the GUI bits,
> only the below line is needed to switch to software rendering.

There's at least one demo in Arago that requires HW accelerated graphics and 
doesn't make sense with software rendering, hence it still checks for "gpu" 
machine feature.


> Otherwise, I would have preferred this patch the other way, have a new branding
> to use the old unsupported kernel. Either way this seems reasonable for now.

I guess we could switch Arago to default to 6.1, as long as we keep meta-ti on 
5.10 for the migration phase.


> Acked-by: Andrew Davis <afd@ti.com>
> 
> >+PREFERRED_PROVIDER_virtual/gpudriver = ""
diff mbox series

Patch

diff --git a/meta-arago-distro/conf/distro/include/branding-ltsprep.inc b/meta-arago-distro/conf/distro/include/branding-ltsprep.inc
new file mode 100644
index 00000000..0238327c
--- /dev/null
+++ b/meta-arago-distro/conf/distro/include/branding-ltsprep.inc
@@ -0,0 +1,11 @@ 
+
+PREFERRED_VERSION_linux-ti-staging = "6.1%"
+PREFERRED_VERSION_linux-ti-staging-rt = "6.1%"
+PREFERRED_VERSION_linux-ti-staging-systest = "6.1%"
+PREFERRED_VERSION_u-boot-ti-staging = "2023.01%"
+
+# GPU support requires out-of-tree SGX and RGX drivers not available in until
+# LTS prep is complete.  fall back to software rendering
+MACHINE_FEATURES:remove = "gpu"
+PREFERRED_PROVIDER_virtual/gpudriver = ""
+