| Message ID | 20240708060054.1985156-1-s-vadapalli@ti.com |
|---|---|
| State | Superseded |
| Delegated to: | Ryan Eatmon |
| Headers | show |
| Series | [meta-ti,scarthgap,v2] conf: machine: am62*-evm-k3r5.conf: apply fragment for enabling USB DFU | expand |
Hi Siddharth, I believe this patch is applicable to master branch as well. So, an update is needed in patch title. s/[scarthgap]/[master/scarthgap] On 08/07/24 11:30, Siddharth Vadapalli wrote: > The config fragment "am62x_r5_usbdfu.config" is applicable to AM62x, > AM62x-LP-EVM, AM62Ax and AM62Px devices and enables USB DFU boot support at > the R5 stage of boot. We need similar update in am62xxsip-evm-k3r5.conf [1] as well to enable USB DFU support on AM62SIP by default. [1]: https://git.ti.com/cgit/arago-project/meta-ti/tree/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf?h=scarthgap > > Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com> > --- > > Hello, > > This patch is based on commit > ff9bf7b9 CI/CD Auto-Merger: cicd.scarthgap.202407071800 > of scarthgap-next branch of meta-ti tree. > > v1: > https://lore.kernel.org/r/20240708052457.1974547-1-s-vadapalli@ti.com/ > Changes since v1: > - As pointed out by Vignesh Raghavendra <vigneshr@ti.com> at: > https://lore.kernel.org/r/829dc5b5-e87c-4679-9e2d-7417c744a9c9@ti.com/ > the A53 defconfigs already include the "am62x_a53_usbdfu.config" config > fragment. Hence, they are not required in the am62*-evm.conf files and > have been dropped in this patch. > - Commit message has been updated accordingly. > > Regards, > Siddharth. > > meta-ti-bsp/conf/machine/am62axx-evm-k3r5.conf | 1 + > meta-ti-bsp/conf/machine/am62pxx-evm-k3r5.conf | 1 + > meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf | 1 + > meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf | 1 + > 4 files changed, 4 insertions(+) > > diff --git a/meta-ti-bsp/conf/machine/am62axx-evm-k3r5.conf b/meta-ti-bsp/conf/machine/am62axx-evm-k3r5.conf > index 2af3317e..5ba8c27e 100644 > --- a/meta-ti-bsp/conf/machine/am62axx-evm-k3r5.conf > +++ b/meta-ti-bsp/conf/machine/am62axx-evm-k3r5.conf > @@ -9,3 +9,4 @@ SYSFW_CONFIG = "evm" > SYSFW_SUFFIX = "hs-fs" > > UBOOT_MACHINE = "am62ax_evm_r5_defconfig" > +UBOOT_CONFIG_FRAGMENTS = "am62x_r5_usbdfu.config" > diff --git a/meta-ti-bsp/conf/machine/am62pxx-evm-k3r5.conf b/meta-ti-bsp/conf/machine/am62pxx-evm-k3r5.conf > index 36915381..9b68e626 100644 > --- a/meta-ti-bsp/conf/machine/am62pxx-evm-k3r5.conf > +++ b/meta-ti-bsp/conf/machine/am62pxx-evm-k3r5.conf > @@ -10,3 +10,4 @@ SYSFW_CONFIG = "evm" > SYSFW_SUFFIX = "hs-fs" > > UBOOT_MACHINE = "am62px_evm_r5_defconfig" > +UBOOT_CONFIG_FRAGMENTS = "am62x_r5_usbdfu.config" > diff --git a/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf b/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf > index 548369ca..85c48334 100644 > --- a/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf > +++ b/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf > @@ -10,3 +10,4 @@ SYSFW_CONFIG = "evm" > SYSFW_SUFFIX = "hs-fs" > > UBOOT_MACHINE = "am62x_evm_r5_defconfig" > +UBOOT_CONFIG_FRAGMENTS = "am62x_r5_usbdfu.config" > diff --git a/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf b/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf > index 52b69a72..24f96858 100644 > --- a/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf > +++ b/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf > @@ -10,3 +10,4 @@ SYSFW_CONFIG = "evm" > SYSFW_SUFFIX = "hs-fs" > > UBOOT_MACHINE = "am62x_lpsk_r5_defconfig" > +UBOOT_CONFIG_FRAGMENTS = "am62x_r5_usbdfu.config"
On Mon, Jul 08, 2024 at 01:57:23PM +0530, Chirag Shilwant wrote: > Hi Siddharth, > > > I believe this patch is applicable to master branch as well. > So, an update is needed in patch title. > > > s/[scarthgap]/[master/scarthgap] Sure, I will update this in the v3 patch. > > On 08/07/24 11:30, Siddharth Vadapalli wrote: > > The config fragment "am62x_r5_usbdfu.config" is applicable to AM62x, > > AM62x-LP-EVM, AM62Ax and AM62Px devices and enables USB DFU boot support at > > the R5 stage of boot. > > > > > We need similar update in am62xxsip-evm-k3r5.conf [1] as well to enable USB > DFU support > on AM62SIP by default. > > [1]: https://git.ti.com/cgit/arago-project/meta-ti/tree/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf?h=scarthgap Thank you for pointing this out. I will include AM62 SIP as well in the v3 patch. [...] Regards, Siddharth.
diff --git a/meta-ti-bsp/conf/machine/am62axx-evm-k3r5.conf b/meta-ti-bsp/conf/machine/am62axx-evm-k3r5.conf index 2af3317e..5ba8c27e 100644 --- a/meta-ti-bsp/conf/machine/am62axx-evm-k3r5.conf +++ b/meta-ti-bsp/conf/machine/am62axx-evm-k3r5.conf @@ -9,3 +9,4 @@ SYSFW_CONFIG = "evm" SYSFW_SUFFIX = "hs-fs" UBOOT_MACHINE = "am62ax_evm_r5_defconfig" +UBOOT_CONFIG_FRAGMENTS = "am62x_r5_usbdfu.config" diff --git a/meta-ti-bsp/conf/machine/am62pxx-evm-k3r5.conf b/meta-ti-bsp/conf/machine/am62pxx-evm-k3r5.conf index 36915381..9b68e626 100644 --- a/meta-ti-bsp/conf/machine/am62pxx-evm-k3r5.conf +++ b/meta-ti-bsp/conf/machine/am62pxx-evm-k3r5.conf @@ -10,3 +10,4 @@ SYSFW_CONFIG = "evm" SYSFW_SUFFIX = "hs-fs" UBOOT_MACHINE = "am62px_evm_r5_defconfig" +UBOOT_CONFIG_FRAGMENTS = "am62x_r5_usbdfu.config" diff --git a/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf b/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf index 548369ca..85c48334 100644 --- a/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf +++ b/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf @@ -10,3 +10,4 @@ SYSFW_CONFIG = "evm" SYSFW_SUFFIX = "hs-fs" UBOOT_MACHINE = "am62x_evm_r5_defconfig" +UBOOT_CONFIG_FRAGMENTS = "am62x_r5_usbdfu.config" diff --git a/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf b/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf index 52b69a72..24f96858 100644 --- a/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf +++ b/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf @@ -10,3 +10,4 @@ SYSFW_CONFIG = "evm" SYSFW_SUFFIX = "hs-fs" UBOOT_MACHINE = "am62x_lpsk_r5_defconfig" +UBOOT_CONFIG_FRAGMENTS = "am62x_r5_usbdfu.config"
The config fragment "am62x_r5_usbdfu.config" is applicable to AM62x, AM62x-LP-EVM, AM62Ax and AM62Px devices and enables USB DFU boot support at the R5 stage of boot. Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com> --- Hello, This patch is based on commit ff9bf7b9 CI/CD Auto-Merger: cicd.scarthgap.202407071800 of scarthgap-next branch of meta-ti tree. v1: https://lore.kernel.org/r/20240708052457.1974547-1-s-vadapalli@ti.com/ Changes since v1: - As pointed out by Vignesh Raghavendra <vigneshr@ti.com> at: https://lore.kernel.org/r/829dc5b5-e87c-4679-9e2d-7417c744a9c9@ti.com/ the A53 defconfigs already include the "am62x_a53_usbdfu.config" config fragment. Hence, they are not required in the am62*-evm.conf files and have been dropped in this patch. - Commit message has been updated accordingly. Regards, Siddharth. meta-ti-bsp/conf/machine/am62axx-evm-k3r5.conf | 1 + meta-ti-bsp/conf/machine/am62pxx-evm-k3r5.conf | 1 + meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf | 1 + meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf | 1 + 4 files changed, 4 insertions(+)