All of lore.kernel.org
 help / color / mirror / Atom feed
* [master/scarthgap][PATCH] meta-ti-bsp: linux-ti-staging-rt_6.12.bb: Add AM57xx RT branch
@ 2025-07-10  9:39 Beleswar Padhi
  2025-07-10 14:32 ` Ryan Eatmon
  0 siblings, 1 reply; 6+ messages in thread
From: Beleswar Padhi @ 2025-07-10  9:39 UTC (permalink / raw)
  To: reatmon, denys; +Cc: a-limaye, afd, u-kumar1, meta-ti

While all other devices use the same branch for RT and non-RT builds,
the AM57xx ARM32 devices have a separate branch for RT Kernel which is
"ti-rt-linux-6.12.y-arm32". Update the BRANCH and SRCREV for am57xx
device family for RT builds.

Signed-off-by: Beleswar Padhi <b-padhi@ti.com>
---
 meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb b/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb
index 9e5c4ee4..e78f046d 100644
--- a/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb
+++ b/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb
@@ -6,4 +6,7 @@ KERNEL_LOCALVERSION:append = "-rt"
 # This will have priority over generic non-rt path
 FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}-6.12:"
 
+BRANCH:am57xx = "ti-rt-linux-6.12.y-arm32"
+SRCREV:am57xx = "f0e4f5ca0905956c70779b31663f594c08c6a3bc"
+
 include ${@ 'recipes-kernel/linux/ti-extras-rt.inc' if d.getVar('TI_EXTRAS') else ''}
-- 
2.34.1



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [master/scarthgap][PATCH] meta-ti-bsp: linux-ti-staging-rt_6.12.bb: Add AM57xx RT branch
  2025-07-10  9:39 [master/scarthgap][PATCH] meta-ti-bsp: linux-ti-staging-rt_6.12.bb: Add AM57xx RT branch Beleswar Padhi
@ 2025-07-10 14:32 ` Ryan Eatmon
  2025-07-10 15:06   ` [meta-ti] " Denys Dmytriyenko
  0 siblings, 1 reply; 6+ messages in thread
From: Ryan Eatmon @ 2025-07-10 14:32 UTC (permalink / raw)
  To: Beleswar Padhi, denys; +Cc: a-limaye, afd, u-kumar1, meta-ti



On 7/10/2025 4:39 AM, Beleswar Padhi wrote:
> While all other devices use the same branch for RT and non-RT builds,
> the AM57xx ARM32 devices have a separate branch for RT Kernel which is
> "ti-rt-linux-6.12.y-arm32". Update the BRANCH and SRCREV for am57xx
> device family for RT builds.
> 
> Signed-off-by: Beleswar Padhi <b-padhi@ti.com>
> ---
>   meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb b/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb
> index 9e5c4ee4..e78f046d 100644
> --- a/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb
> +++ b/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb
> @@ -6,4 +6,7 @@ KERNEL_LOCALVERSION:append = "-rt"
>   # This will have priority over generic non-rt path
>   FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}-6.12:"
>   
> +BRANCH:am57xx = "ti-rt-linux-6.12.y-arm32"
> +SRCREV:am57xx = "f0e4f5ca0905956c70779b31663f594c08c6a3bc"

NAK.

This is applies to all 32bit platforms, not just am57xx.  So the 
mechanism needs to be done in a way to support that.

BRANCH_32BIT = "ti-rt-linux-6.12.y-arm32"
SRCREV_32BIT = "f0e4f5ca0905956c70779b31663f594c08c6a3bc"

BRANCH:ti33x = "${BRANCH_32BIT}"
BRANCH:ti43x = "${BRANCH_32BIT}"
BRANCH:am57xx = "${BRANCH_32BIT}"

SRCREV:ti33x = "${SRCREV_32BIT}"
SRCREV:ti43x = "${SRCREV_32BIT}"
SRCREV:am57xx = "${SRCREV_32BIT}"


Also, update the commit message to not be am57 specific.

And in the subject, we don't need the meta-ti-bsp or the .bb:

[master/scarthgap][PATCH] linux-ti-staging-rt_6.12: Add 32bit RT branch

Keep the subject shorter when possible

>   include ${@ 'recipes-kernel/linux/ti-extras-rt.inc' if d.getVar('TI_EXTRAS') else ''}

-- 
Ryan Eatmon                reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [meta-ti] [master/scarthgap][PATCH] meta-ti-bsp: linux-ti-staging-rt_6.12.bb: Add AM57xx RT branch
  2025-07-10 14:32 ` Ryan Eatmon
@ 2025-07-10 15:06   ` Denys Dmytriyenko
  2025-07-10 15:09     ` Ryan Eatmon
  0 siblings, 1 reply; 6+ messages in thread
From: Denys Dmytriyenko @ 2025-07-10 15:06 UTC (permalink / raw)
  To: reatmon; +Cc: Beleswar Padhi, denys, a-limaye, afd, u-kumar1, meta-ti

On Thu, Jul 10, 2025 at 09:32:51AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
> 
> 
> On 7/10/2025 4:39 AM, Beleswar Padhi wrote:
> >While all other devices use the same branch for RT and non-RT builds,
> >the AM57xx ARM32 devices have a separate branch for RT Kernel which is
> >"ti-rt-linux-6.12.y-arm32". Update the BRANCH and SRCREV for am57xx
> >device family for RT builds.
> >
> >Signed-off-by: Beleswar Padhi <b-padhi@ti.com>
> >---
> >  meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb | 3 +++
> >  1 file changed, 3 insertions(+)
> >
> >diff --git a/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb b/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb
> >index 9e5c4ee4..e78f046d 100644
> >--- a/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb
> >+++ b/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb
> >@@ -6,4 +6,7 @@ KERNEL_LOCALVERSION:append = "-rt"
> >  # This will have priority over generic non-rt path
> >  FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}-6.12:"
> >+BRANCH:am57xx = "ti-rt-linux-6.12.y-arm32"
> >+SRCREV:am57xx = "f0e4f5ca0905956c70779b31663f594c08c6a3bc"
> 
> NAK.
> 
> This is applies to all 32bit platforms, not just am57xx.  So the
> mechanism needs to be done in a way to support that.
> 
> BRANCH_32BIT = "ti-rt-linux-6.12.y-arm32"
> SRCREV_32BIT = "f0e4f5ca0905956c70779b31663f594c08c6a3bc"
> 
> BRANCH:ti33x = "${BRANCH_32BIT}"
> BRANCH:ti43x = "${BRANCH_32BIT}"
> BRANCH:am57xx = "${BRANCH_32BIT}"
> 
> SRCREV:ti33x = "${SRCREV_32BIT}"
> SRCREV:ti43x = "${SRCREV_32BIT}"
> SRCREV:am57xx = "${SRCREV_32BIT}"

What about dra7xx? Why not simply use armv7a override?

Also, not sure about RT on omapl128, but that's armv5.


> Also, update the commit message to not be am57 specific.
> 
> And in the subject, we don't need the meta-ti-bsp or the .bb:
> 
> [master/scarthgap][PATCH] linux-ti-staging-rt_6.12: Add 32bit RT branch
> 
> Keep the subject shorter when possible
> 
> >  include ${@ 'recipes-kernel/linux/ti-extras-rt.inc' if d.getVar('TI_EXTRAS') else ''}


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [meta-ti] [master/scarthgap][PATCH] meta-ti-bsp: linux-ti-staging-rt_6.12.bb: Add AM57xx RT branch
  2025-07-10 15:06   ` [meta-ti] " Denys Dmytriyenko
@ 2025-07-10 15:09     ` Ryan Eatmon
  2025-07-10 18:19       ` Denys Dmytriyenko
  0 siblings, 1 reply; 6+ messages in thread
From: Ryan Eatmon @ 2025-07-10 15:09 UTC (permalink / raw)
  To: Denys Dmytriyenko; +Cc: Beleswar Padhi, denys, a-limaye, afd, u-kumar1, meta-ti



On 7/10/2025 10:06 AM, Denys Dmytriyenko wrote:
> On Thu, Jul 10, 2025 at 09:32:51AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
>>
>>
>> On 7/10/2025 4:39 AM, Beleswar Padhi wrote:
>>> While all other devices use the same branch for RT and non-RT builds,
>>> the AM57xx ARM32 devices have a separate branch for RT Kernel which is
>>> "ti-rt-linux-6.12.y-arm32". Update the BRANCH and SRCREV for am57xx
>>> device family for RT builds.
>>>
>>> Signed-off-by: Beleswar Padhi <b-padhi@ti.com>
>>> ---
>>>   meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb | 3 +++
>>>   1 file changed, 3 insertions(+)
>>>
>>> diff --git a/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb b/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb
>>> index 9e5c4ee4..e78f046d 100644
>>> --- a/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb
>>> +++ b/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb
>>> @@ -6,4 +6,7 @@ KERNEL_LOCALVERSION:append = "-rt"
>>>   # This will have priority over generic non-rt path
>>>   FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}-6.12:"
>>> +BRANCH:am57xx = "ti-rt-linux-6.12.y-arm32"
>>> +SRCREV:am57xx = "f0e4f5ca0905956c70779b31663f594c08c6a3bc"
>>
>> NAK.
>>
>> This is applies to all 32bit platforms, not just am57xx.  So the
>> mechanism needs to be done in a way to support that.
>>
>> BRANCH_32BIT = "ti-rt-linux-6.12.y-arm32"
>> SRCREV_32BIT = "f0e4f5ca0905956c70779b31663f594c08c6a3bc"
>>
>> BRANCH:ti33x = "${BRANCH_32BIT}"
>> BRANCH:ti43x = "${BRANCH_32BIT}"
>> BRANCH:am57xx = "${BRANCH_32BIT}"
>>
>> SRCREV:ti33x = "${SRCREV_32BIT}"
>> SRCREV:ti43x = "${SRCREV_32BIT}"
>> SRCREV:am57xx = "${SRCREV_32BIT}"
> 
> What about dra7xx? Why not simply use armv7a override?

We have "dropped" support for dra7xx, we do not build it or test it.

> Also, not sure about RT on omapl128, but that's armv5.

Same for omapl128.  We are not really supporting or testing it anymore.


So the list is just those three, which is why I suggested this path 
rather than the more generic one.

> 
>> Also, update the commit message to not be am57 specific.
>>
>> And in the subject, we don't need the meta-ti-bsp or the .bb:
>>
>> [master/scarthgap][PATCH] linux-ti-staging-rt_6.12: Add 32bit RT branch
>>
>> Keep the subject shorter when possible
>>
>>>   include ${@ 'recipes-kernel/linux/ti-extras-rt.inc' if d.getVar('TI_EXTRAS') else ''}

-- 
Ryan Eatmon                reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [meta-ti] [master/scarthgap][PATCH] meta-ti-bsp: linux-ti-staging-rt_6.12.bb: Add AM57xx RT branch
  2025-07-10 15:09     ` Ryan Eatmon
@ 2025-07-10 18:19       ` Denys Dmytriyenko
  2025-07-10 18:43         ` Ryan Eatmon
  0 siblings, 1 reply; 6+ messages in thread
From: Denys Dmytriyenko @ 2025-07-10 18:19 UTC (permalink / raw)
  To: reatmon; +Cc: Beleswar Padhi, denys, a-limaye, afd, u-kumar1, meta-ti

On Thu, Jul 10, 2025 at 10:09:52AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
> 
> 
> On 7/10/2025 10:06 AM, Denys Dmytriyenko wrote:
> >On Thu, Jul 10, 2025 at 09:32:51AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
> >>
> >>
> >>On 7/10/2025 4:39 AM, Beleswar Padhi wrote:
> >>>While all other devices use the same branch for RT and non-RT builds,
> >>>the AM57xx ARM32 devices have a separate branch for RT Kernel which is
> >>>"ti-rt-linux-6.12.y-arm32". Update the BRANCH and SRCREV for am57xx
> >>>device family for RT builds.
> >>>
> >>>Signed-off-by: Beleswar Padhi <b-padhi@ti.com>
> >>>---
> >>>  meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb | 3 +++
> >>>  1 file changed, 3 insertions(+)
> >>>
> >>>diff --git a/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb b/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb
> >>>index 9e5c4ee4..e78f046d 100644
> >>>--- a/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb
> >>>+++ b/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb
> >>>@@ -6,4 +6,7 @@ KERNEL_LOCALVERSION:append = "-rt"
> >>>  # This will have priority over generic non-rt path
> >>>  FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}-6.12:"
> >>>+BRANCH:am57xx = "ti-rt-linux-6.12.y-arm32"
> >>>+SRCREV:am57xx = "f0e4f5ca0905956c70779b31663f594c08c6a3bc"
> >>
> >>NAK.
> >>
> >>This is applies to all 32bit platforms, not just am57xx.  So the
> >>mechanism needs to be done in a way to support that.
> >>
> >>BRANCH_32BIT = "ti-rt-linux-6.12.y-arm32"
> >>SRCREV_32BIT = "f0e4f5ca0905956c70779b31663f594c08c6a3bc"
> >>
> >>BRANCH:ti33x = "${BRANCH_32BIT}"
> >>BRANCH:ti43x = "${BRANCH_32BIT}"
> >>BRANCH:am57xx = "${BRANCH_32BIT}"
> >>
> >>SRCREV:ti33x = "${SRCREV_32BIT}"
> >>SRCREV:ti43x = "${SRCREV_32BIT}"
> >>SRCREV:am57xx = "${SRCREV_32BIT}"
> >
> >What about dra7xx? Why not simply use armv7a override?
> 
> We have "dropped" support for dra7xx, we do not build it or test it.
> 
> >Also, not sure about RT on omapl128, but that's armv5.
> 
> Same for omapl128.  We are not really supporting or testing it anymore.
> 
> 
> So the list is just those three, which is why I suggested this path
> rather than the more generic one.

It's probably time to start removing unsupported machine configs, at least 
from the master, to avoid confusion.


> >>Also, update the commit message to not be am57 specific.
> >>
> >>And in the subject, we don't need the meta-ti-bsp or the .bb:
> >>
> >>[master/scarthgap][PATCH] linux-ti-staging-rt_6.12: Add 32bit RT branch
> >>
> >>Keep the subject shorter when possible
> >>
> >>>  include ${@ 'recipes-kernel/linux/ti-extras-rt.inc' if d.getVar('TI_EXTRAS') else ''}


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [meta-ti] [master/scarthgap][PATCH] meta-ti-bsp: linux-ti-staging-rt_6.12.bb: Add AM57xx RT branch
  2025-07-10 18:19       ` Denys Dmytriyenko
@ 2025-07-10 18:43         ` Ryan Eatmon
  0 siblings, 0 replies; 6+ messages in thread
From: Ryan Eatmon @ 2025-07-10 18:43 UTC (permalink / raw)
  To: Denys Dmytriyenko; +Cc: Beleswar Padhi, denys, a-limaye, afd, u-kumar1, meta-ti



On 7/10/2025 1:19 PM, Denys Dmytriyenko wrote:
> On Thu, Jul 10, 2025 at 10:09:52AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
>>
>>
>> On 7/10/2025 10:06 AM, Denys Dmytriyenko wrote:
>>> On Thu, Jul 10, 2025 at 09:32:51AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
>>>>
>>>>
>>>> On 7/10/2025 4:39 AM, Beleswar Padhi wrote:
>>>>> While all other devices use the same branch for RT and non-RT builds,
>>>>> the AM57xx ARM32 devices have a separate branch for RT Kernel which is
>>>>> "ti-rt-linux-6.12.y-arm32". Update the BRANCH and SRCREV for am57xx
>>>>> device family for RT builds.
>>>>>
>>>>> Signed-off-by: Beleswar Padhi <b-padhi@ti.com>
>>>>> ---
>>>>>   meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb | 3 +++
>>>>>   1 file changed, 3 insertions(+)
>>>>>
>>>>> diff --git a/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb b/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb
>>>>> index 9e5c4ee4..e78f046d 100644
>>>>> --- a/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb
>>>>> +++ b/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging-rt_6.12.bb
>>>>> @@ -6,4 +6,7 @@ KERNEL_LOCALVERSION:append = "-rt"
>>>>>   # This will have priority over generic non-rt path
>>>>>   FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}-6.12:"
>>>>> +BRANCH:am57xx = "ti-rt-linux-6.12.y-arm32"
>>>>> +SRCREV:am57xx = "f0e4f5ca0905956c70779b31663f594c08c6a3bc"
>>>>
>>>> NAK.
>>>>
>>>> This is applies to all 32bit platforms, not just am57xx.  So the
>>>> mechanism needs to be done in a way to support that.
>>>>
>>>> BRANCH_32BIT = "ti-rt-linux-6.12.y-arm32"
>>>> SRCREV_32BIT = "f0e4f5ca0905956c70779b31663f594c08c6a3bc"
>>>>
>>>> BRANCH:ti33x = "${BRANCH_32BIT}"
>>>> BRANCH:ti43x = "${BRANCH_32BIT}"
>>>> BRANCH:am57xx = "${BRANCH_32BIT}"
>>>>
>>>> SRCREV:ti33x = "${SRCREV_32BIT}"
>>>> SRCREV:ti43x = "${SRCREV_32BIT}"
>>>> SRCREV:am57xx = "${SRCREV_32BIT}"
>>>
>>> What about dra7xx? Why not simply use armv7a override?
>>
>> We have "dropped" support for dra7xx, we do not build it or test it.
>>
>>> Also, not sure about RT on omapl128, but that's armv5.
>>
>> Same for omapl128.  We are not really supporting or testing it anymore.
>>
>>
>> So the list is just those three, which is why I suggested this path
>> rather than the more generic one.
> 
> It's probably time to start removing unsupported machine configs, at least
> from the master, to avoid confusion.

Agreed.  I'll start that convo over here.


> 
>>>> Also, update the commit message to not be am57 specific.
>>>>
>>>> And in the subject, we don't need the meta-ti-bsp or the .bb:
>>>>
>>>> [master/scarthgap][PATCH] linux-ti-staging-rt_6.12: Add 32bit RT branch
>>>>
>>>> Keep the subject shorter when possible
>>>>
>>>>>   include ${@ 'recipes-kernel/linux/ti-extras-rt.inc' if d.getVar('TI_EXTRAS') else ''}

-- 
Ryan Eatmon                reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2025-07-10 19:46 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-10  9:39 [master/scarthgap][PATCH] meta-ti-bsp: linux-ti-staging-rt_6.12.bb: Add AM57xx RT branch Beleswar Padhi
2025-07-10 14:32 ` Ryan Eatmon
2025-07-10 15:06   ` [meta-ti] " Denys Dmytriyenko
2025-07-10 15:09     ` Ryan Eatmon
2025-07-10 18:19       ` Denys Dmytriyenko
2025-07-10 18:43         ` Ryan Eatmon

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.