[master/scarthgap] jailhouse: revert changes for am62lxx

From: Denys Dmytriyenko <denys@konsulko.com>

This reverts commit 31b423275aa2bc685a1348218bda16894a35ad8b.

This uses an unpublished/uncommitten change - more details:
https://lists.yoctoproject.org/g/meta-ti/topic/111318869

Cc: Paresh Bhagat <p-bhagat@ti.com>
Signed-off-by: Denys Dmytriyenko <denys@konsulko.com>
---
 meta-ti-bsp/conf/machine/am62lxx-evm.conf            |  1 -
 meta-ti-bsp/recipes-bsp/u-boot/ti-extras.inc         | 10 +++++-----
 meta-ti-bsp/recipes-kernel/linux/ti-extras-rt.inc    | 10 +++++-----
 meta-ti-bsp/recipes-kernel/linux/ti-extras.inc       | 10 +++++-----
 meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb |  4 ----
 meta-ti-extras/recipes-ti/jailhouse/ti-jailhouse.inc |  5 ++---
 6 files changed, 17 insertions(+), 23 deletions(-)

Hi Paresh & Denys,

On 15/03/25 07:17, Denys Dmytriyenko wrote:
> From: Denys Dmytriyenko <denys@konsulko.com>
>
> This reverts commit 31b423275aa2bc685a1348218bda16894a35ad8b.
>
> This uses an unpublished/uncommitten change - more details:
> https://lists.yoctoproject.org/g/meta-ti/topic/111318869



Instead of reverting the entire commit which enabled
Jailhouse support in AM62L, can't we just update
the SRCREV to point to a valid commit under ti-jailhouse?

Something that Jared tried to suggest at [0]


[0]: https://lists.yoctoproject.org/g/meta-ti/message/18333


--
Chirag



> Cc: Paresh Bhagat <p-bhagat@ti.com>
> Signed-off-by: Denys Dmytriyenko <denys@konsulko.com>
> ---
>   meta-ti-bsp/conf/machine/am62lxx-evm.conf            |  1 -
>   meta-ti-bsp/recipes-bsp/u-boot/ti-extras.inc         | 10 +++++-----
>   meta-ti-bsp/recipes-kernel/linux/ti-extras-rt.inc    | 10 +++++-----
>   meta-ti-bsp/recipes-kernel/linux/ti-extras.inc       | 10 +++++-----
>   meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb |  4 ----
>   meta-ti-extras/recipes-ti/jailhouse/ti-jailhouse.inc |  5 ++---
>   6 files changed, 17 insertions(+), 23 deletions(-)

Hi Denys and Chirag,

On 15/03/25 19:20, Chirag Shilwant wrote:
> Hi Paresh & Denys,
>
> On 15/03/25 07:17, Denys Dmytriyenko wrote:
>> From: Denys Dmytriyenko <denys@konsulko.com>
>>
>> This reverts commit 31b423275aa2bc685a1348218bda16894a35ad8b.
>>
>> This uses an unpublished/uncommitten change - more details:
>> https://lists.yoctoproject.org/g/meta-ti/topic/111318869
>
>
>
> Instead of reverting the entire commit which enabled
> Jailhouse support in AM62L, can't we just update
> the SRCREV to point to a valid commit under ti-jailhouse?
>
> Something that Jared tried to suggest at [0]
>
>
> [0]: https://lists.yoctoproject.org/g/meta-ti/message/18333
>
>

Due to some last min changes, I had to force push which caused the 
commit to change. Additionally, the commits for Jailhouse branches in 
meta-ti are usually an older version since Jailhouse updates are done 
after a RC tag (in meta-tisdk). I can send a patch to correct the commit 
instead of reverting current changes.


Thanks

> -- 
> Chirag
>
>
>
>> Cc: Paresh Bhagat <p-bhagat@ti.com>
>> Signed-off-by: Denys Dmytriyenko <denys@konsulko.com>
>> ---
>>   meta-ti-bsp/conf/machine/am62lxx-evm.conf            |  1 -
>>   meta-ti-bsp/recipes-bsp/u-boot/ti-extras.inc         | 10 +++++-----
>>   meta-ti-bsp/recipes-kernel/linux/ti-extras-rt.inc    | 10 +++++-----
>>   meta-ti-bsp/recipes-kernel/linux/ti-extras.inc       | 10 +++++-----
>>   meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb |  4 ----
>>   meta-ti-extras/recipes-ti/jailhouse/ti-jailhouse.inc |  5 ++---
>>   6 files changed, 17 insertions(+), 23 deletions(-)

On 3/17/2025 1:09 AM, Paresh Bhagat via lists.yoctoproject.org wrote:
> Hi Denys and Chirag,
> 
> On 15/03/25 19:20, Chirag Shilwant wrote:
>> Hi Paresh & Denys,
>>
>> On 15/03/25 07:17, Denys Dmytriyenko wrote:
>>> From: Denys Dmytriyenko <denys@konsulko.com>
>>>
>>> This reverts commit 31b423275aa2bc685a1348218bda16894a35ad8b.
>>>
>>> This uses an unpublished/uncommitten change - more details:
>>> https://lists.yoctoproject.org/g/meta-ti/topic/111318869
>>
>>
>>
>> Instead of reverting the entire commit which enabled
>> Jailhouse support in AM62L, can't we just update
>> the SRCREV to point to a valid commit under ti-jailhouse?
>>
>> Something that Jared tried to suggest at [0]
>>
>>
>> [0]: https://lists.yoctoproject.org/g/meta-ti/message/18333
>>
>>
> 
> Due to some last min changes, I had to force push which caused the 
> commit to change. Additionally, the commits for Jailhouse branches in 
> meta-ti are usually an older version since Jailhouse updates are done 
> after a RC tag (in meta-tisdk). I can send a patch to correct the commit 
> instead of reverting current changes.

It is generally almost always a bad idea to force push a "prod" branch 
for any repo.  We should get out of the habit of doing that.  Just 
commit new changes to fix whatever issue you are trying to fix.  You 
never know what all is pointing at a commit that will be torpedoed by a 
force push.


> 
> Thanks
> 
>> -- 
>> Chirag
>>
>>
>>
>>> Cc: Paresh Bhagat <p-bhagat@ti.com>
>>> Signed-off-by: Denys Dmytriyenko <denys@konsulko.com>
>>> ---
>>>   meta-ti-bsp/conf/machine/am62lxx-evm.conf            |  1 -
>>>   meta-ti-bsp/recipes-bsp/u-boot/ti-extras.inc         | 10 +++++-----
>>>   meta-ti-bsp/recipes-kernel/linux/ti-extras-rt.inc    | 10 +++++-----
>>>   meta-ti-bsp/recipes-kernel/linux/ti-extras.inc       | 10 +++++-----
>>>   meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb |  4 ----
>>>   meta-ti-extras/recipes-ti/jailhouse/ti-jailhouse.inc |  5 ++---
>>>   6 files changed, 17 insertions(+), 23 deletions(-)
> 
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#18392): https://lists.yoctoproject.org/g/meta-ti/message/18392
> Mute This Topic: https://lists.yoctoproject.org/mt/111711058/6551054
> Group Owner: meta-ti+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [reatmon@ti.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>