diff mbox series

[meta-ti,dunfell,V2] conf: machine: j721s2-evm: Add support for J721S2 HS-FS

Message ID 20230210152641.14513-1-sinthu.raja@ti.com
State Superseded
Delegated to: Ryan Eatmon
Headers show
Series [meta-ti,dunfell,V2] conf: machine: j721s2-evm: Add support for J721S2 HS-FS | expand

Commit Message

Sinthu Raja M Feb. 10, 2023, 3:26 p.m. UTC
From: Sinthu Raja <sinthu.raja@ti.com>

Add the basic J721S2 HS-FS configuration.
    - Add configurations to support both HS-FS and GP and keep GP
      as the default.
    - Add ti-sci-firmware overrides for HS-FS.

Signed-off-by: Aniket Limaye <a-limaye@ti.com>
Signed-off-by: Sinthu Raja <sinthu.raja@ti.com>
---

Changes in V2:
=============
Address review comment:
- Keep GP as the default.

V1: https://lists.yoctoproject.org/g/meta-ti/message/15771

 conf/machine/j721s2-evm-k3r5-gp.conf        | 20 ++++++++++++++++++++
 conf/machine/j721s2-evm-k3r5-sr1-hs-fs.conf | 15 +++++++++++++++
 conf/machine/j721s2-evm.conf                | 12 ++++++++++--
 conf/multiconfig/k3r5-sr1-hs-fs.conf        |  3 +++
 conf/multiconfig/k3r5-sr1.conf              |  3 +++
 recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb      |  3 ++-
 6 files changed, 53 insertions(+), 3 deletions(-)
 create mode 100644 conf/machine/j721s2-evm-k3r5-gp.conf
 create mode 100644 conf/machine/j721s2-evm-k3r5-sr1-hs-fs.conf
 create mode 100644 conf/multiconfig/k3r5-sr1-hs-fs.conf
 create mode 100644 conf/multiconfig/k3r5-sr1.conf

Comments

Ryan Eatmon Feb. 10, 2023, 8:23 p.m. UTC | #1
On 2/10/2023 9:26, Sinthu Raja wrote:
> From: Sinthu Raja <sinthu.raja@ti.com>
> 
> Add the basic J721S2 HS-FS configuration.
>      - Add configurations to support both HS-FS and GP and keep GP
>        as the default.
>      - Add ti-sci-firmware overrides for HS-FS.
> 
> Signed-off-by: Aniket Limaye <a-limaye@ti.com>
> Signed-off-by: Sinthu Raja <sinthu.raja@ti.com>

This patch does not even build.  Both j721s2-evm and j721s2-hs-evm are 
broken.

What the reasoning for trying to get this into dunfell given that the 
transition to kirkstone is right around the corner and the plan is to 
redo all of HS stuff for all platforms at that time?



> ---
> 
> Changes in V2:
> =============
> Address review comment:
> - Keep GP as the default.
> 
> V1: https://lists.yoctoproject.org/g/meta-ti/message/15771
> 
>   conf/machine/j721s2-evm-k3r5-gp.conf        | 20 ++++++++++++++++++++
>   conf/machine/j721s2-evm-k3r5-sr1-hs-fs.conf | 15 +++++++++++++++
>   conf/machine/j721s2-evm.conf                | 12 ++++++++++--
>   conf/multiconfig/k3r5-sr1-hs-fs.conf        |  3 +++
>   conf/multiconfig/k3r5-sr1.conf              |  3 +++
>   recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb      |  3 ++-
>   6 files changed, 53 insertions(+), 3 deletions(-)
>   create mode 100644 conf/machine/j721s2-evm-k3r5-gp.conf
>   create mode 100644 conf/machine/j721s2-evm-k3r5-sr1-hs-fs.conf
>   create mode 100644 conf/multiconfig/k3r5-sr1-hs-fs.conf
>   create mode 100644 conf/multiconfig/k3r5-sr1.conf
> 
> diff --git a/conf/machine/j721s2-evm-k3r5-gp.conf b/conf/machine/j721s2-evm-k3r5-gp.conf
> new file mode 100644
> index 00000000..51362b85
> --- /dev/null
> +++ b/conf/machine/j721s2-evm-k3r5-gp.conf
> @@ -0,0 +1,20 @@
> +#@TYPE: Machine
> +#@NAME: J721S2 GP EVM (R5F)
> +#@DESCRIPTION: Machine configuration for the TI J721S2 GP EVM (R5F core)
> +
> +# Booting GP requires different SYSFW, the rest is handled at runtime
> +
> +require conf/machine/include/k3r5.inc
> +SOC_FAMILY_append = ":k3r5-gp"
> +
> +SYSFW_SOC = "j721s2"
> +SYSFW_CONFIG = "evm"
> +SYSFW_SUFFIX = "gp"
> +SYSFW_TIBOOT3_SYMLINK = ""
> +
> +UBOOT_MACHINE = "j721s2_evm_r5_defconfig"
> +
> +SPL_BINARY = ""
> +UBOOT_BINARY = "u-boot-spl.${UBOOT_SUFFIX}"
> +UBOOT_IMAGE = "u-boot-r5spl-gp-${MAINMACHINE}-${PV}-${PR}.${UBOOT_SUFFIX}"
> +UBOOT_SYMLINK = "u-boot-r5spl-gp.${UBOOT_SUFFIX}"
> diff --git a/conf/machine/j721s2-evm-k3r5-sr1-hs-fs.conf b/conf/machine/j721s2-evm-k3r5-sr1-hs-fs.conf
> new file mode 100644
> index 00000000..786338a9
> --- /dev/null
> +++ b/conf/machine/j721s2-evm-k3r5-sr1-hs-fs.conf
> @@ -0,0 +1,15 @@
> +#@TYPE: Machine
> +#@NAME: J721S2 HS-FS EVM (R5F)
> +#@DESCRIPTION: Machine configuration for the TI J721S2 HS-FS EVM (R5F core)
> +
> +# Booting HS-FS requires different SYSFW, the rest is handled at runtime
> +
> +require conf/machine/include/k3r5.inc
> +SOC_FAMILY_append = ":k3r5-sr1-hs-fs"
> +
> +SYSFW_SOC = "j721s2"
> +SYSFW_CONFIG = "evm"
> +SYSFW_SUFFIX = "hs-fs"
> +SYSFW_TIBOOT3_SYMLINK = ""
> +
> +UBOOT_MACHINE = "j721s2_evm_r5_defconfig"
> diff --git a/conf/machine/j721s2-evm.conf b/conf/machine/j721s2-evm.conf
> index 7c120dcc..7d2597b5 100644
> --- a/conf/machine/j721s2-evm.conf
> +++ b/conf/machine/j721s2-evm.conf
> @@ -40,5 +40,13 @@ KERNEL_DEVICETREE = " \
>   
>   UBOOT_MACHINE = "j721s2_evm_a72_defconfig"
>   
> -do_image_wic[mcdepends] = "mc::k3r5:ti-sci-fw:do_deploy"
> -do_image_tar[mcdepends] = "mc::k3r5:ti-sci-fw:do_deploy"
> +# Default tiboot3.bin on J721S2 is for GP
> +BBMULTICONFIG = "k3r5-gp"
> +do_image_wic[mcdepends] = "mc::k3r5-gp:ti-sci-fw:do_deploy"
> +do_image_tar[mcdepends] = "mc::k3r5-gp:ti-sci-fw:do_deploy"
> +
> +# Since default tiboot3.bin on J721S2 is for GP, add a version for SR1.0 HS-FS
> +BBMULTICONFIG += "k3r5-sr1-hs-fs"
> +IMAGE_BOOT_FILES += " tiboot3-j721s2-hs-fs-evm.bin"
> +do_image_wic[mcdepends] += "mc::k3r5-sr1-hs-fs:ti-sci-fw:do_deploy"
> +do_image_tar[mcdepends] += "mc::k3r5-sr1-hs-fs:ti-sci-fw:do_deploy"
> diff --git a/conf/multiconfig/k3r5-sr1-hs-fs.conf b/conf/multiconfig/k3r5-sr1-hs-fs.conf
> new file mode 100644
> index 00000000..2fa28c95
> --- /dev/null
> +++ b/conf/multiconfig/k3r5-sr1-hs-fs.conf
> @@ -0,0 +1,3 @@
> +require k3r5-sr1.conf
> +
> +MACHINE_append = "-hs-fs"
> diff --git a/conf/multiconfig/k3r5-sr1.conf b/conf/multiconfig/k3r5-sr1.conf
> new file mode 100644
> index 00000000..2b87de8a
> --- /dev/null
> +++ b/conf/multiconfig/k3r5-sr1.conf
> @@ -0,0 +1,3 @@
> +require k3r5.conf
> +
> +MACHINE_append = "-sr1"
> diff --git a/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb b/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
> index 7896c366..cb197954 100644
> --- a/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
> +++ b/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
> @@ -33,7 +33,8 @@ SYSFW_PREFIX_j7-hs-evm-k3r5 = "fs"
>   SYSFW_PREFIX_j7-hs-evm-k3r5-sr1-1 = "fs"
>   SYSFW_PREFIX_j7200-evm-k3r5 = "fs"
>   SYSFW_PREFIX_j7200-hs-evm-k3r5 = "fs"
> -SYSFW_PREFIX_j721s2-evm-k3r5 = "fs"
> +SYSFW_PREFIX_j721s2-evm-k3r5-gp = "fs"
> +SYSFW_PREFIX_j721s2-evm-k3r5-sr1-hs-fs = "fs"
>   SYSFW_PREFIX_j721s2-hs-evm-k3r5 = "fs"
>   SYSFW_PREFIX_j784s4-evm-k3r5 = "fs"
>   SYSFW_PREFIX_j784s4-hs-evm-k3r5 = "fs"
Sinthu Raja M Feb. 13, 2023, 6:45 a.m. UTC | #2
On Sat, Feb 11, 2023 at 1:53 AM Ryan Eatmon <reatmon@ti.com> wrote:
>
>
>
> On 2/10/2023 9:26, Sinthu Raja wrote:
> > From: Sinthu Raja <sinthu.raja@ti.com>
> >
> > Add the basic J721S2 HS-FS configuration.
> >      - Add configurations to support both HS-FS and GP and keep GP
> >        as the default.
> >      - Add ti-sci-firmware overrides for HS-FS.
> >
> > Signed-off-by: Aniket Limaye <a-limaye@ti.com>
> > Signed-off-by: Sinthu Raja <sinthu.raja@ti.com>
>
> This patch does not even build.  Both j721s2-evm and j721s2-hs-evm are
> broken.
>

> What the reasoning for trying to get this into dunfell given that the
> transition to kirkstone is right around the corner and the plan is to
> redo all of HS stuff for all platforms at that time?
>
Ryan,
AM68 SK and J721S2 EVM use the same MACHINE name which is j721s2-evm.
This patch should be there in dunfell to support the AM68 SK HS-FS
which is critical for APL release.
I am sending the fixed version of this patch which I have tested locally.

Regards
Sinthu Raja.
> > ---
> >
> > Changes in V2:
> > =============
> > Address review comment:
> > - Keep GP as the default.
> >
> > V1: https://lists.yoctoproject.org/g/meta-ti/message/15771
> >
> >   conf/machine/j721s2-evm-k3r5-gp.conf        | 20 ++++++++++++++++++++
> >   conf/machine/j721s2-evm-k3r5-sr1-hs-fs.conf | 15 +++++++++++++++
> >   conf/machine/j721s2-evm.conf                | 12 ++++++++++--
> >   conf/multiconfig/k3r5-sr1-hs-fs.conf        |  3 +++
> >   conf/multiconfig/k3r5-sr1.conf              |  3 +++
> >   recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb      |  3 ++-
> >   6 files changed, 53 insertions(+), 3 deletions(-)
> >   create mode 100644 conf/machine/j721s2-evm-k3r5-gp.conf
> >   create mode 100644 conf/machine/j721s2-evm-k3r5-sr1-hs-fs.conf
> >   create mode 100644 conf/multiconfig/k3r5-sr1-hs-fs.conf
> >   create mode 100644 conf/multiconfig/k3r5-sr1.conf
> >
> > diff --git a/conf/machine/j721s2-evm-k3r5-gp.conf b/conf/machine/j721s2-evm-k3r5-gp.conf
> > new file mode 100644
> > index 00000000..51362b85
> > --- /dev/null
> > +++ b/conf/machine/j721s2-evm-k3r5-gp.conf
> > @@ -0,0 +1,20 @@
> > +#@TYPE: Machine
> > +#@NAME: J721S2 GP EVM (R5F)
> > +#@DESCRIPTION: Machine configuration for the TI J721S2 GP EVM (R5F core)
> > +
> > +# Booting GP requires different SYSFW, the rest is handled at runtime
> > +
> > +require conf/machine/include/k3r5.inc
> > +SOC_FAMILY_append = ":k3r5-gp"
> > +
> > +SYSFW_SOC = "j721s2"
> > +SYSFW_CONFIG = "evm"
> > +SYSFW_SUFFIX = "gp"
> > +SYSFW_TIBOOT3_SYMLINK = ""
> > +
> > +UBOOT_MACHINE = "j721s2_evm_r5_defconfig"
> > +
> > +SPL_BINARY = ""
> > +UBOOT_BINARY = "u-boot-spl.${UBOOT_SUFFIX}"
> > +UBOOT_IMAGE = "u-boot-r5spl-gp-${MAINMACHINE}-${PV}-${PR}.${UBOOT_SUFFIX}"
> > +UBOOT_SYMLINK = "u-boot-r5spl-gp.${UBOOT_SUFFIX}"
> > diff --git a/conf/machine/j721s2-evm-k3r5-sr1-hs-fs.conf b/conf/machine/j721s2-evm-k3r5-sr1-hs-fs.conf
> > new file mode 100644
> > index 00000000..786338a9
> > --- /dev/null
> > +++ b/conf/machine/j721s2-evm-k3r5-sr1-hs-fs.conf
> > @@ -0,0 +1,15 @@
> > +#@TYPE: Machine
> > +#@NAME: J721S2 HS-FS EVM (R5F)
> > +#@DESCRIPTION: Machine configuration for the TI J721S2 HS-FS EVM (R5F core)
> > +
> > +# Booting HS-FS requires different SYSFW, the rest is handled at runtime
> > +
> > +require conf/machine/include/k3r5.inc
> > +SOC_FAMILY_append = ":k3r5-sr1-hs-fs"
> > +
> > +SYSFW_SOC = "j721s2"
> > +SYSFW_CONFIG = "evm"
> > +SYSFW_SUFFIX = "hs-fs"
> > +SYSFW_TIBOOT3_SYMLINK = ""
> > +
> > +UBOOT_MACHINE = "j721s2_evm_r5_defconfig"
> > diff --git a/conf/machine/j721s2-evm.conf b/conf/machine/j721s2-evm.conf
> > index 7c120dcc..7d2597b5 100644
> > --- a/conf/machine/j721s2-evm.conf
> > +++ b/conf/machine/j721s2-evm.conf
> > @@ -40,5 +40,13 @@ KERNEL_DEVICETREE = " \
> >
> >   UBOOT_MACHINE = "j721s2_evm_a72_defconfig"
> >
> > -do_image_wic[mcdepends] = "mc::k3r5:ti-sci-fw:do_deploy"
> > -do_image_tar[mcdepends] = "mc::k3r5:ti-sci-fw:do_deploy"
> > +# Default tiboot3.bin on J721S2 is for GP
> > +BBMULTICONFIG = "k3r5-gp"
> > +do_image_wic[mcdepends] = "mc::k3r5-gp:ti-sci-fw:do_deploy"
> > +do_image_tar[mcdepends] = "mc::k3r5-gp:ti-sci-fw:do_deploy"
> > +
> > +# Since default tiboot3.bin on J721S2 is for GP, add a version for SR1.0 HS-FS
> > +BBMULTICONFIG += "k3r5-sr1-hs-fs"
> > +IMAGE_BOOT_FILES += " tiboot3-j721s2-hs-fs-evm.bin"
> > +do_image_wic[mcdepends] += "mc::k3r5-sr1-hs-fs:ti-sci-fw:do_deploy"
> > +do_image_tar[mcdepends] += "mc::k3r5-sr1-hs-fs:ti-sci-fw:do_deploy"
> > diff --git a/conf/multiconfig/k3r5-sr1-hs-fs.conf b/conf/multiconfig/k3r5-sr1-hs-fs.conf
> > new file mode 100644
> > index 00000000..2fa28c95
> > --- /dev/null
> > +++ b/conf/multiconfig/k3r5-sr1-hs-fs.conf
> > @@ -0,0 +1,3 @@
> > +require k3r5-sr1.conf
> > +
> > +MACHINE_append = "-hs-fs"
> > diff --git a/conf/multiconfig/k3r5-sr1.conf b/conf/multiconfig/k3r5-sr1.conf
> > new file mode 100644
> > index 00000000..2b87de8a
> > --- /dev/null
> > +++ b/conf/multiconfig/k3r5-sr1.conf
> > @@ -0,0 +1,3 @@
> > +require k3r5.conf
> > +
> > +MACHINE_append = "-sr1"
> > diff --git a/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb b/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
> > index 7896c366..cb197954 100644
> > --- a/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
> > +++ b/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
> > @@ -33,7 +33,8 @@ SYSFW_PREFIX_j7-hs-evm-k3r5 = "fs"
> >   SYSFW_PREFIX_j7-hs-evm-k3r5-sr1-1 = "fs"
> >   SYSFW_PREFIX_j7200-evm-k3r5 = "fs"
> >   SYSFW_PREFIX_j7200-hs-evm-k3r5 = "fs"
> > -SYSFW_PREFIX_j721s2-evm-k3r5 = "fs"
> > +SYSFW_PREFIX_j721s2-evm-k3r5-gp = "fs"
> > +SYSFW_PREFIX_j721s2-evm-k3r5-sr1-hs-fs = "fs"
> >   SYSFW_PREFIX_j721s2-hs-evm-k3r5 = "fs"
> >   SYSFW_PREFIX_j784s4-evm-k3r5 = "fs"
> >   SYSFW_PREFIX_j784s4-hs-evm-k3r5 = "fs"
>
> --
> Ryan Eatmon                reatmon@ti.com
> -----------------------------------------
> Texas Instruments, Inc.  -  LCPD  -  MGTS
Andrew Davis Feb. 13, 2023, 3:32 p.m. UTC | #3
On 2/13/23 12:45 AM, Sinthu Raja M via lists.yoctoproject.org wrote:
> On Sat, Feb 11, 2023 at 1:53 AM Ryan Eatmon <reatmon@ti.com> wrote:
>>
>>
>>
>> On 2/10/2023 9:26, Sinthu Raja wrote:
>>> From: Sinthu Raja <sinthu.raja@ti.com>
>>>
>>> Add the basic J721S2 HS-FS configuration.
>>>       - Add configurations to support both HS-FS and GP and keep GP
>>>         as the default.
>>>       - Add ti-sci-firmware overrides for HS-FS.
>>>
>>> Signed-off-by: Aniket Limaye <a-limaye@ti.com>
>>> Signed-off-by: Sinthu Raja <sinthu.raja@ti.com>
>>
>> This patch does not even build.  Both j721s2-evm and j721s2-hs-evm are
>> broken.
>>
> 
>> What the reasoning for trying to get this into dunfell given that the
>> transition to kirkstone is right around the corner and the plan is to
>> redo all of HS stuff for all platforms at that time?
>>
> Ryan,
> AM68 SK and J721S2 EVM use the same MACHINE name which is j721s2-evm.
> This patch should be there in dunfell to support the AM68 SK HS-FS
> which is critical for APL release.
> I am sending the fixed version of this patch which I have tested locally.

It should be assumed that *all* patches one sends are tested locally first ;)

Andrew

> 
> Regards
> Sinthu Raja.
>>> ---
>>>
>>> Changes in V2:
>>> =============
>>> Address review comment:
>>> - Keep GP as the default.
>>>
>>> V1: https://lists.yoctoproject.org/g/meta-ti/message/15771
>>>
>>>    conf/machine/j721s2-evm-k3r5-gp.conf        | 20 ++++++++++++++++++++
>>>    conf/machine/j721s2-evm-k3r5-sr1-hs-fs.conf | 15 +++++++++++++++
>>>    conf/machine/j721s2-evm.conf                | 12 ++++++++++--
>>>    conf/multiconfig/k3r5-sr1-hs-fs.conf        |  3 +++
>>>    conf/multiconfig/k3r5-sr1.conf              |  3 +++
>>>    recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb      |  3 ++-
>>>    6 files changed, 53 insertions(+), 3 deletions(-)
>>>    create mode 100644 conf/machine/j721s2-evm-k3r5-gp.conf
>>>    create mode 100644 conf/machine/j721s2-evm-k3r5-sr1-hs-fs.conf
>>>    create mode 100644 conf/multiconfig/k3r5-sr1-hs-fs.conf
>>>    create mode 100644 conf/multiconfig/k3r5-sr1.conf
>>>
>>> diff --git a/conf/machine/j721s2-evm-k3r5-gp.conf b/conf/machine/j721s2-evm-k3r5-gp.conf
>>> new file mode 100644
>>> index 00000000..51362b85
>>> --- /dev/null
>>> +++ b/conf/machine/j721s2-evm-k3r5-gp.conf
>>> @@ -0,0 +1,20 @@
>>> +#@TYPE: Machine
>>> +#@NAME: J721S2 GP EVM (R5F)
>>> +#@DESCRIPTION: Machine configuration for the TI J721S2 GP EVM (R5F core)
>>> +
>>> +# Booting GP requires different SYSFW, the rest is handled at runtime
>>> +
>>> +require conf/machine/include/k3r5.inc
>>> +SOC_FAMILY_append = ":k3r5-gp"
>>> +
>>> +SYSFW_SOC = "j721s2"
>>> +SYSFW_CONFIG = "evm"
>>> +SYSFW_SUFFIX = "gp"
>>> +SYSFW_TIBOOT3_SYMLINK = ""
>>> +
>>> +UBOOT_MACHINE = "j721s2_evm_r5_defconfig"
>>> +
>>> +SPL_BINARY = ""
>>> +UBOOT_BINARY = "u-boot-spl.${UBOOT_SUFFIX}"
>>> +UBOOT_IMAGE = "u-boot-r5spl-gp-${MAINMACHINE}-${PV}-${PR}.${UBOOT_SUFFIX}"
>>> +UBOOT_SYMLINK = "u-boot-r5spl-gp.${UBOOT_SUFFIX}"
>>> diff --git a/conf/machine/j721s2-evm-k3r5-sr1-hs-fs.conf b/conf/machine/j721s2-evm-k3r5-sr1-hs-fs.conf
>>> new file mode 100644
>>> index 00000000..786338a9
>>> --- /dev/null
>>> +++ b/conf/machine/j721s2-evm-k3r5-sr1-hs-fs.conf
>>> @@ -0,0 +1,15 @@
>>> +#@TYPE: Machine
>>> +#@NAME: J721S2 HS-FS EVM (R5F)
>>> +#@DESCRIPTION: Machine configuration for the TI J721S2 HS-FS EVM (R5F core)
>>> +
>>> +# Booting HS-FS requires different SYSFW, the rest is handled at runtime
>>> +
>>> +require conf/machine/include/k3r5.inc
>>> +SOC_FAMILY_append = ":k3r5-sr1-hs-fs"
>>> +
>>> +SYSFW_SOC = "j721s2"
>>> +SYSFW_CONFIG = "evm"
>>> +SYSFW_SUFFIX = "hs-fs"
>>> +SYSFW_TIBOOT3_SYMLINK = ""
>>> +
>>> +UBOOT_MACHINE = "j721s2_evm_r5_defconfig"
>>> diff --git a/conf/machine/j721s2-evm.conf b/conf/machine/j721s2-evm.conf
>>> index 7c120dcc..7d2597b5 100644
>>> --- a/conf/machine/j721s2-evm.conf
>>> +++ b/conf/machine/j721s2-evm.conf
>>> @@ -40,5 +40,13 @@ KERNEL_DEVICETREE = " \
>>>
>>>    UBOOT_MACHINE = "j721s2_evm_a72_defconfig"
>>>
>>> -do_image_wic[mcdepends] = "mc::k3r5:ti-sci-fw:do_deploy"
>>> -do_image_tar[mcdepends] = "mc::k3r5:ti-sci-fw:do_deploy"
>>> +# Default tiboot3.bin on J721S2 is for GP
>>> +BBMULTICONFIG = "k3r5-gp"
>>> +do_image_wic[mcdepends] = "mc::k3r5-gp:ti-sci-fw:do_deploy"
>>> +do_image_tar[mcdepends] = "mc::k3r5-gp:ti-sci-fw:do_deploy"
>>> +
>>> +# Since default tiboot3.bin on J721S2 is for GP, add a version for SR1.0 HS-FS
>>> +BBMULTICONFIG += "k3r5-sr1-hs-fs"
>>> +IMAGE_BOOT_FILES += " tiboot3-j721s2-hs-fs-evm.bin"
>>> +do_image_wic[mcdepends] += "mc::k3r5-sr1-hs-fs:ti-sci-fw:do_deploy"
>>> +do_image_tar[mcdepends] += "mc::k3r5-sr1-hs-fs:ti-sci-fw:do_deploy"
>>> diff --git a/conf/multiconfig/k3r5-sr1-hs-fs.conf b/conf/multiconfig/k3r5-sr1-hs-fs.conf
>>> new file mode 100644
>>> index 00000000..2fa28c95
>>> --- /dev/null
>>> +++ b/conf/multiconfig/k3r5-sr1-hs-fs.conf
>>> @@ -0,0 +1,3 @@
>>> +require k3r5-sr1.conf
>>> +
>>> +MACHINE_append = "-hs-fs"
>>> diff --git a/conf/multiconfig/k3r5-sr1.conf b/conf/multiconfig/k3r5-sr1.conf
>>> new file mode 100644
>>> index 00000000..2b87de8a
>>> --- /dev/null
>>> +++ b/conf/multiconfig/k3r5-sr1.conf
>>> @@ -0,0 +1,3 @@
>>> +require k3r5.conf
>>> +
>>> +MACHINE_append = "-sr1"
>>> diff --git a/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb b/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
>>> index 7896c366..cb197954 100644
>>> --- a/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
>>> +++ b/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
>>> @@ -33,7 +33,8 @@ SYSFW_PREFIX_j7-hs-evm-k3r5 = "fs"
>>>    SYSFW_PREFIX_j7-hs-evm-k3r5-sr1-1 = "fs"
>>>    SYSFW_PREFIX_j7200-evm-k3r5 = "fs"
>>>    SYSFW_PREFIX_j7200-hs-evm-k3r5 = "fs"
>>> -SYSFW_PREFIX_j721s2-evm-k3r5 = "fs"
>>> +SYSFW_PREFIX_j721s2-evm-k3r5-gp = "fs"
>>> +SYSFW_PREFIX_j721s2-evm-k3r5-sr1-hs-fs = "fs"
>>>    SYSFW_PREFIX_j721s2-hs-evm-k3r5 = "fs"
>>>    SYSFW_PREFIX_j784s4-evm-k3r5 = "fs"
>>>    SYSFW_PREFIX_j784s4-hs-evm-k3r5 = "fs"
>>
>> --
>> Ryan Eatmon                reatmon@ti.com
>> -----------------------------------------
>> Texas Instruments, Inc.  -  LCPD  -  MGTS
> 
> 
> 
> 
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#15817): https://lists.yoctoproject.org/g/meta-ti/message/15817
> Mute This Topic: https://lists.yoctoproject.org/mt/96878140/3619733
> Group Owner: meta-ti+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [afd@ti.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
diff mbox series

Patch

diff --git a/conf/machine/j721s2-evm-k3r5-gp.conf b/conf/machine/j721s2-evm-k3r5-gp.conf
new file mode 100644
index 00000000..51362b85
--- /dev/null
+++ b/conf/machine/j721s2-evm-k3r5-gp.conf
@@ -0,0 +1,20 @@ 
+#@TYPE: Machine
+#@NAME: J721S2 GP EVM (R5F)
+#@DESCRIPTION: Machine configuration for the TI J721S2 GP EVM (R5F core)
+
+# Booting GP requires different SYSFW, the rest is handled at runtime
+
+require conf/machine/include/k3r5.inc
+SOC_FAMILY_append = ":k3r5-gp"
+
+SYSFW_SOC = "j721s2"
+SYSFW_CONFIG = "evm"
+SYSFW_SUFFIX = "gp"
+SYSFW_TIBOOT3_SYMLINK = ""
+
+UBOOT_MACHINE = "j721s2_evm_r5_defconfig"
+
+SPL_BINARY = ""
+UBOOT_BINARY = "u-boot-spl.${UBOOT_SUFFIX}"
+UBOOT_IMAGE = "u-boot-r5spl-gp-${MAINMACHINE}-${PV}-${PR}.${UBOOT_SUFFIX}"
+UBOOT_SYMLINK = "u-boot-r5spl-gp.${UBOOT_SUFFIX}"
diff --git a/conf/machine/j721s2-evm-k3r5-sr1-hs-fs.conf b/conf/machine/j721s2-evm-k3r5-sr1-hs-fs.conf
new file mode 100644
index 00000000..786338a9
--- /dev/null
+++ b/conf/machine/j721s2-evm-k3r5-sr1-hs-fs.conf
@@ -0,0 +1,15 @@ 
+#@TYPE: Machine
+#@NAME: J721S2 HS-FS EVM (R5F)
+#@DESCRIPTION: Machine configuration for the TI J721S2 HS-FS EVM (R5F core)
+
+# Booting HS-FS requires different SYSFW, the rest is handled at runtime
+
+require conf/machine/include/k3r5.inc
+SOC_FAMILY_append = ":k3r5-sr1-hs-fs"
+
+SYSFW_SOC = "j721s2"
+SYSFW_CONFIG = "evm"
+SYSFW_SUFFIX = "hs-fs"
+SYSFW_TIBOOT3_SYMLINK = ""
+
+UBOOT_MACHINE = "j721s2_evm_r5_defconfig"
diff --git a/conf/machine/j721s2-evm.conf b/conf/machine/j721s2-evm.conf
index 7c120dcc..7d2597b5 100644
--- a/conf/machine/j721s2-evm.conf
+++ b/conf/machine/j721s2-evm.conf
@@ -40,5 +40,13 @@  KERNEL_DEVICETREE = " \
 
 UBOOT_MACHINE = "j721s2_evm_a72_defconfig"
 
-do_image_wic[mcdepends] = "mc::k3r5:ti-sci-fw:do_deploy"
-do_image_tar[mcdepends] = "mc::k3r5:ti-sci-fw:do_deploy"
+# Default tiboot3.bin on J721S2 is for GP
+BBMULTICONFIG = "k3r5-gp"
+do_image_wic[mcdepends] = "mc::k3r5-gp:ti-sci-fw:do_deploy"
+do_image_tar[mcdepends] = "mc::k3r5-gp:ti-sci-fw:do_deploy"
+
+# Since default tiboot3.bin on J721S2 is for GP, add a version for SR1.0 HS-FS
+BBMULTICONFIG += "k3r5-sr1-hs-fs"
+IMAGE_BOOT_FILES += " tiboot3-j721s2-hs-fs-evm.bin"
+do_image_wic[mcdepends] += "mc::k3r5-sr1-hs-fs:ti-sci-fw:do_deploy"
+do_image_tar[mcdepends] += "mc::k3r5-sr1-hs-fs:ti-sci-fw:do_deploy"
diff --git a/conf/multiconfig/k3r5-sr1-hs-fs.conf b/conf/multiconfig/k3r5-sr1-hs-fs.conf
new file mode 100644
index 00000000..2fa28c95
--- /dev/null
+++ b/conf/multiconfig/k3r5-sr1-hs-fs.conf
@@ -0,0 +1,3 @@ 
+require k3r5-sr1.conf
+
+MACHINE_append = "-hs-fs"
diff --git a/conf/multiconfig/k3r5-sr1.conf b/conf/multiconfig/k3r5-sr1.conf
new file mode 100644
index 00000000..2b87de8a
--- /dev/null
+++ b/conf/multiconfig/k3r5-sr1.conf
@@ -0,0 +1,3 @@ 
+require k3r5.conf
+
+MACHINE_append = "-sr1"
diff --git a/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb b/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
index 7896c366..cb197954 100644
--- a/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
+++ b/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
@@ -33,7 +33,8 @@  SYSFW_PREFIX_j7-hs-evm-k3r5 = "fs"
 SYSFW_PREFIX_j7-hs-evm-k3r5-sr1-1 = "fs"
 SYSFW_PREFIX_j7200-evm-k3r5 = "fs"
 SYSFW_PREFIX_j7200-hs-evm-k3r5 = "fs"
-SYSFW_PREFIX_j721s2-evm-k3r5 = "fs"
+SYSFW_PREFIX_j721s2-evm-k3r5-gp = "fs"
+SYSFW_PREFIX_j721s2-evm-k3r5-sr1-hs-fs = "fs"
 SYSFW_PREFIX_j721s2-hs-evm-k3r5 = "fs"
 SYSFW_PREFIX_j784s4-evm-k3r5 = "fs"
 SYSFW_PREFIX_j784s4-hs-evm-k3r5 = "fs"