[master/thud][PATCH] jailhouse: update to ti2019.03-rc2

[master/thud][PATCH] jailhouse: update to ti2019.03-rc2

[Jacob Stiffler]

* extend to am65xx family
* and correct am65xx cell and inmate filenames.
* remove earlycon bootargs for am65xx

Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
---
 recipes-kernel/jailhouse/jailhouse_git.bb | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/recipes-kernel/jailhouse/jailhouse_git.bb b/recipes-kernel/jailhouse/jailhouse_git.bb
index 5e2c71b..e1d5263 100644
--- a/recipes-kernel/jailhouse/jailhouse_git.bb
+++ b/recipes-kernel/jailhouse/jailhouse_git.bb
@@ -11,7 +11,7 @@ LIC_FILES_CHKSUM = " \
 "
 
 PV = "0.10+git${SRCPV}"
-SRCREV = "e71fd2dc7e8494a44cc3c762a24824532a412807"
+SRCREV = "96329f098e655cd42d7d0dbb3cd2cc7c3492d633"
 BRANCH = "ti-jailhouse-0.10"
 
 SRC_URI = " \
@@ -47,19 +47,20 @@ JH_CELL_FILES ?= "*.cell"
 JH_CELL_FILES_k3 ?= "k3-*.cell"
 
 JH_INMATE_DTB ?= ""
-JH_INMATE_DTB_am65xx-evm ?= "inmate-k3-am654.dtb"
+JH_INMATE_DTB_am65xx ?= "inmate-k3-am654-idk.dtb"
 JH_INMATE_DTB_j7-evm ?= "inmate-k3-j721e-evm.dtb"
 
 JH_LINUX_DEMO_CELL ?= ""
-JH_LINUX_DEMO_CELL_am65xx-evm ?= "k3-am654-linux-demo.cell"
+JH_LINUX_DEMO_CELL_am65xx ?= "k3-am654-idk-linux-demo.cell"
 JH_LINUX_DEMO_CELL_j7-evm ?= "k3-j721e-evm-linux-demo.cell"
 
 JH_SYSCONFIG_CELL ?= ""
-JH_SYSCONFIG_CELL_am65xx-evm ?= "k3-am654.cell"
+JH_SYSCONFIG_CELL_am65xx ?= "k3-am654-idk.cell"
 JH_SYSCONFIG_CELL_j7-evm ?= "k3-j721e-evm.cell"
 
 JH_CMDLINE ?= ""
-JH_CMDLINE_k3 ?= "console=ttyS1,115200n8 root=/dev/mmcblk0p2 rw rootfstype=ext4 rootwait earlycon=ns16550a,mmio32,0x2810000"
+JH_CMDLINE_k3 ?= "console=ttyS1,115200n8 root=/dev/mmcblk0p2 rw rootfstype=ext4 rootwait"
+JH_CMDLINE_j7-evm ?= "console=ttyS1,115200n8 root=/dev/mmcblk0p2 rw rootfstype=ext4 rootwait earlycon=ns16550a,mmio32,0x2810000"
 
 do_configure() {
 	if [ -d ${STAGING_DIR_HOST}/${CELLCONF_DIR} ]; 
-- 
2.7.4

Re: [master/thud][PATCH] jailhouse: update to ti2019.03-rc2

[Denys Dmytriyenko]

On Mon, Aug 19, 2019 at 08:58:14AM -0400, Jacob Stiffler wrote:
> * extend to am65xx family
> * and correct am65xx cell and inmate filenames.
> * remove earlycon bootargs for am65xx

IIRC, earlycon was causing some issues and was supposed to be removed by an 
owner... As well as updating commit and fixing DTBs and cells...


> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> ---
>  recipes-kernel/jailhouse/jailhouse_git.bb | 11 ++++++-----
>  1 file changed, 6 insertions(+), 5 deletions(-)
> 
> diff --git a/recipes-kernel/jailhouse/jailhouse_git.bb b/recipes-kernel/jailhouse/jailhouse_git.bb
> index 5e2c71b..e1d5263 100644
> --- a/recipes-kernel/jailhouse/jailhouse_git.bb
> +++ b/recipes-kernel/jailhouse/jailhouse_git.bb
> @@ -11,7 +11,7 @@ LIC_FILES_CHKSUM = " \
>  "
>  
>  PV = "0.10+git${SRCPV}"
> -SRCREV = "e71fd2dc7e8494a44cc3c762a24824532a412807"
> +SRCREV = "96329f098e655cd42d7d0dbb3cd2cc7c3492d633"
>  BRANCH = "ti-jailhouse-0.10"
>  
>  SRC_URI = " \
> @@ -47,19 +47,20 @@ JH_CELL_FILES ?= "*.cell"
>  JH_CELL_FILES_k3 ?= "k3-*.cell"
>  
>  JH_INMATE_DTB ?= ""
> -JH_INMATE_DTB_am65xx-evm ?= "inmate-k3-am654.dtb"
> +JH_INMATE_DTB_am65xx ?= "inmate-k3-am654-idk.dtb"
>  JH_INMATE_DTB_j7-evm ?= "inmate-k3-j721e-evm.dtb"
>  
>  JH_LINUX_DEMO_CELL ?= ""
> -JH_LINUX_DEMO_CELL_am65xx-evm ?= "k3-am654-linux-demo.cell"
> +JH_LINUX_DEMO_CELL_am65xx ?= "k3-am654-idk-linux-demo.cell"
>  JH_LINUX_DEMO_CELL_j7-evm ?= "k3-j721e-evm-linux-demo.cell"
>  
>  JH_SYSCONFIG_CELL ?= ""
> -JH_SYSCONFIG_CELL_am65xx-evm ?= "k3-am654.cell"
> +JH_SYSCONFIG_CELL_am65xx ?= "k3-am654-idk.cell"
>  JH_SYSCONFIG_CELL_j7-evm ?= "k3-j721e-evm.cell"
>  
>  JH_CMDLINE ?= ""
> -JH_CMDLINE_k3 ?= "console=ttyS1,115200n8 root=/dev/mmcblk0p2 rw rootfstype=ext4 rootwait earlycon=ns16550a,mmio32,0x2810000"
> +JH_CMDLINE_k3 ?= "console=ttyS1,115200n8 root=/dev/mmcblk0p2 rw rootfstype=ext4 rootwait"
> +JH_CMDLINE_j7-evm ?= "console=ttyS1,115200n8 root=/dev/mmcblk0p2 rw rootfstype=ext4 rootwait earlycon=ns16550a,mmio32,0x2810000"
>  
>  do_configure() {
>  	if [ -d ${STAGING_DIR_HOST}/${CELLCONF_DIR} ]; 
> -- 
> 2.7.4
> 
> -- 
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti

Re: [master/thud][PATCH] jailhouse: update to ti2019.03-rc2

[Jacob Stiffler]

On 8/19/2019 1:48 PM, Denys Dmytriyenko wrote:
> On Mon, Aug 19, 2019 at 08:58:14AM -0400, Jacob Stiffler wrote:
>> * extend to am65xx family
>> * and correct am65xx cell and inmate filenames.
>> * remove earlycon bootargs for am65xx
> IIRC, earlycon was causing some issues and was supposed to be removed by an
> owner... As well as updating commit and fixing DTBs and cells...


I was not aware. I've been tasked with the OOB experience, so 
integration seemed like a logical first step. :)

I am also looking into packaging the tools and python modules so that 
users can play around with this and change the kernel image and bootargs 
on the target.


>
>> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
>> ---
>>   recipes-kernel/jailhouse/jailhouse_git.bb | 11 ++++++-----
>>   1 file changed, 6 insertions(+), 5 deletions(-)
>>
>> diff --git a/recipes-kernel/jailhouse/jailhouse_git.bb b/recipes-kernel/jailhouse/jailhouse_git.bb
>> index 5e2c71b..e1d5263 100644
>> --- a/recipes-kernel/jailhouse/jailhouse_git.bb
>> +++ b/recipes-kernel/jailhouse/jailhouse_git.bb
>> @@ -11,7 +11,7 @@ LIC_FILES_CHKSUM = " \
>>   "
>>   
>>   PV = "0.10+git${SRCPV}"
>> -SRCREV = "e71fd2dc7e8494a44cc3c762a24824532a412807"
>> +SRCREV = "96329f098e655cd42d7d0dbb3cd2cc7c3492d633"
>>   BRANCH = "ti-jailhouse-0.10"
>>   
>>   SRC_URI = " \
>> @@ -47,19 +47,20 @@ JH_CELL_FILES ?= "*.cell"
>>   JH_CELL_FILES_k3 ?= "k3-*.cell"
>>   
>>   JH_INMATE_DTB ?= ""
>> -JH_INMATE_DTB_am65xx-evm ?= "inmate-k3-am654.dtb"
>> +JH_INMATE_DTB_am65xx ?= "inmate-k3-am654-idk.dtb"
>>   JH_INMATE_DTB_j7-evm ?= "inmate-k3-j721e-evm.dtb"
>>   
>>   JH_LINUX_DEMO_CELL ?= ""
>> -JH_LINUX_DEMO_CELL_am65xx-evm ?= "k3-am654-linux-demo.cell"
>> +JH_LINUX_DEMO_CELL_am65xx ?= "k3-am654-idk-linux-demo.cell"
>>   JH_LINUX_DEMO_CELL_j7-evm ?= "k3-j721e-evm-linux-demo.cell"
>>   
>>   JH_SYSCONFIG_CELL ?= ""
>> -JH_SYSCONFIG_CELL_am65xx-evm ?= "k3-am654.cell"
>> +JH_SYSCONFIG_CELL_am65xx ?= "k3-am654-idk.cell"
>>   JH_SYSCONFIG_CELL_j7-evm ?= "k3-j721e-evm.cell"
>>   
>>   JH_CMDLINE ?= ""
>> -JH_CMDLINE_k3 ?= "console=ttyS1,115200n8 root=/dev/mmcblk0p2 rw rootfstype=ext4 rootwait earlycon=ns16550a,mmio32,0x2810000"
>> +JH_CMDLINE_k3 ?= "console=ttyS1,115200n8 root=/dev/mmcblk0p2 rw rootfstype=ext4 rootwait"
>> +JH_CMDLINE_j7-evm ?= "console=ttyS1,115200n8 root=/dev/mmcblk0p2 rw rootfstype=ext4 rootwait earlycon=ns16550a,mmio32,0x2810000"
>>   
>>   do_configure() {
>>   	if [ -d ${STAGING_DIR_HOST}/${CELLCONF_DIR} ];
>> -- 
>> 2.7.4
>>
>> -- 
>> _______________________________________________
>> meta-ti mailing list
>> meta-ti@yoctoproject.org
>> https://lists.yoctoproject.org/listinfo/meta-ti

Re: [master/thud][PATCH] jailhouse: update to ti2019.03-rc2

[Jacob Stiffler]

On 8/19/2019 1:24 PM, Lokesh Vutla wrote:
> Hi Jacob,
>
> On 19/08/19 6:28 PM, Jacob Stiffler wrote:
>> * extend to am65xx family
>> * and correct am65xx cell and inmate filenames.
>> * remove earlycon bootargs for am65xx
>>
>> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> Thanks for doing this. Some minor comments below.
>
>> ---
>>   recipes-kernel/jailhouse/jailhouse_git.bb | 11 ++++++-----
>>   1 file changed, 6 insertions(+), 5 deletions(-)
>>
>> diff --git a/recipes-kernel/jailhouse/jailhouse_git.bb b/recipes-kernel/jailhouse/jailhouse_git.bb
>> index 5e2c71b..e1d5263 100644
>> --- a/recipes-kernel/jailhouse/jailhouse_git.bb
>> +++ b/recipes-kernel/jailhouse/jailhouse_git.bb
>> @@ -11,7 +11,7 @@ LIC_FILES_CHKSUM = " \
>>   "
>>   
>>   PV = "0.10+git${SRCPV}"
>> -SRCREV = "e71fd2dc7e8494a44cc3c762a24824532a412807"
>> +SRCREV = "96329f098e655cd42d7d0dbb3cd2cc7c3492d633"
>>   BRANCH = "ti-jailhouse-0.10"
>>   
>>   SRC_URI = " \
>> @@ -47,19 +47,20 @@ JH_CELL_FILES ?= "*.cell"
>>   JH_CELL_FILES_k3 ?= "k3-*.cell"
>>   
>>   JH_INMATE_DTB ?= ""
>> -JH_INMATE_DTB_am65xx-evm ?= "inmate-k3-am654.dtb"
>> +JH_INMATE_DTB_am65xx ?= "inmate-k3-am654-idk.dtb"
>>   JH_INMATE_DTB_j7-evm ?= "inmate-k3-j721e-evm.dtb"
>>   
>>   JH_LINUX_DEMO_CELL ?= ""
>> -JH_LINUX_DEMO_CELL_am65xx-evm ?= "k3-am654-linux-demo.cell"
>> +JH_LINUX_DEMO_CELL_am65xx ?= "k3-am654-idk-linux-demo.cell"
>>   JH_LINUX_DEMO_CELL_j7-evm ?= "k3-j721e-evm-linux-demo.cell"
>>   
>>   JH_SYSCONFIG_CELL ?= ""
>> -JH_SYSCONFIG_CELL_am65xx-evm ?= "k3-am654.cell"
>> +JH_SYSCONFIG_CELL_am65xx ?= "k3-am654-idk.cell"
>>   JH_SYSCONFIG_CELL_j7-evm ?= "k3-j721e-evm.cell"
>>   
>>   JH_CMDLINE ?= ""
>> -JH_CMDLINE_k3 ?= "console=ttyS1,115200n8 root=/dev/mmcblk0p2 rw rootfstype=ext4 rootwait earlycon=ns16550a,mmio32,0x2810000"
>> +JH_CMDLINE_k3 ?= "console=ttyS1,115200n8 root=/dev/mmcblk0p2 rw rootfstype=ext4 rootwait"
> Should this be JH_CMDLINE_AM654?
>
> I am not really a fan of keeping filesystem in mmc. Can we use ramdisk for the
> default demo?


I have not tried this yet, but it looks like it should be possible to 
set up with the jailhouse-cell-linux tool using the "--initrd" option. 
Is this correct? What is the expected image format of the ramfs (e.g. 
*.cpio.gz)?

There may be some technical challenges to work through as I am not aware 
of a method to include one rootfs image inside of another rootfs image. 
Denys, do you know if this is possible?


>> +JH_CMDLINE_j7-evm ?= "console=ttyS1,115200n8 root=/dev/mmcblk0p2 rw rootfstype=ext4 rootwait earlycon=ns16550a,mmio32,0x2810000"
> Can you drop earlycon from here as well?


Sure. Hopefully we can get a common set of args between j7-evm and am65xx.


> Thanks and regards,
> Lokesh
>
>
>>   
>>   do_configure() {
>>   	if [ -d ${STAGING_DIR_HOST}/${CELLCONF_DIR} ];
>>