| Message ID | 20240705045030.1141934-8-c-vankar@ti.com |
|---|---|
| State | Changes Requested |
| Delegated to: | Tom Rini |
| Headers | show |
| Series | Add support for Ethernet Boot on SK-AM62 | expand |
On Fri, Jul 05, 2024 at 10:20:27AM +0530, Chintan Vankar wrote: > From: Kishon Vijay Abraham I <kishon@ti.com> > > Add configs for enabling ETHBOOT in R5SPL. > > Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com> > Signed-off-by: Andreas Dannenberg <dannenberg@ti.com> > Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com> > Signed-off-by: Chintan Vankar <c-vankar@ti.com> > --- > > Link to v2: > https://lore.kernel.org/r/20240425120822.2048012-8-c-vankar@ti.com/ > > Changes from v2 to v3: > - No changes. > > configs/am62x_evm_r5_ethboot_defconfig | 20 ++++++++++++++++++++ > 1 file changed, 20 insertions(+) > create mode 100644 configs/am62x_evm_r5_ethboot_defconfig > > diff --git a/configs/am62x_evm_r5_ethboot_defconfig b/configs/am62x_evm_r5_ethboot_defconfig > new file mode 100644 > index 0000000000..7b1932b215 > --- /dev/null > +++ b/configs/am62x_evm_r5_ethboot_defconfig > @@ -0,0 +1,20 @@ > +#include<configs/am62x_evm_r5_defconfig> > + > +CONFIG_SPL_GPIO=y You need a few other options set in order for buildman to work in this case, please look at the other #include users.
On 05/07/24 21:07, Tom Rini wrote: > On Fri, Jul 05, 2024 at 10:20:27AM +0530, Chintan Vankar wrote: > >> From: Kishon Vijay Abraham I <kishon@ti.com> >> >> Add configs for enabling ETHBOOT in R5SPL. >> >> Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com> >> Signed-off-by: Andreas Dannenberg <dannenberg@ti.com> >> Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com> >> Signed-off-by: Chintan Vankar <c-vankar@ti.com> >> --- >> >> Link to v2: >> https://lore.kernel.org/r/20240425120822.2048012-8-c-vankar@ti.com/ >> >> Changes from v2 to v3: >> - No changes. >> >> configs/am62x_evm_r5_ethboot_defconfig | 20 ++++++++++++++++++++ >> 1 file changed, 20 insertions(+) >> create mode 100644 configs/am62x_evm_r5_ethboot_defconfig >> >> diff --git a/configs/am62x_evm_r5_ethboot_defconfig b/configs/am62x_evm_r5_ethboot_defconfig >> new file mode 100644 >> index 0000000000..7b1932b215 >> --- /dev/null >> +++ b/configs/am62x_evm_r5_ethboot_defconfig >> @@ -0,0 +1,20 @@ >> +#include<configs/am62x_evm_r5_defconfig> >> + >> +CONFIG_SPL_GPIO=y > > You need a few other options set in order for buildman to work in this > case, please look at the other #include users. > Are you referring to include "configs/k3_efi_capsule.config" ?
On Tue, Jul 09, 2024 at 12:29:15PM +0530, Chintan Vankar wrote: > > > On 05/07/24 21:07, Tom Rini wrote: > > On Fri, Jul 05, 2024 at 10:20:27AM +0530, Chintan Vankar wrote: > > > > > From: Kishon Vijay Abraham I <kishon@ti.com> > > > > > > Add configs for enabling ETHBOOT in R5SPL. > > > > > > Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com> > > > Signed-off-by: Andreas Dannenberg <dannenberg@ti.com> > > > Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com> > > > Signed-off-by: Chintan Vankar <c-vankar@ti.com> > > > --- > > > > > > Link to v2: > > > https://lore.kernel.org/r/20240425120822.2048012-8-c-vankar@ti.com/ > > > > > > Changes from v2 to v3: > > > - No changes. > > > > > > configs/am62x_evm_r5_ethboot_defconfig | 20 ++++++++++++++++++++ > > > 1 file changed, 20 insertions(+) > > > create mode 100644 configs/am62x_evm_r5_ethboot_defconfig > > > > > > diff --git a/configs/am62x_evm_r5_ethboot_defconfig b/configs/am62x_evm_r5_ethboot_defconfig > > > new file mode 100644 > > > index 0000000000..7b1932b215 > > > --- /dev/null > > > +++ b/configs/am62x_evm_r5_ethboot_defconfig > > > @@ -0,0 +1,20 @@ > > > +#include<configs/am62x_evm_r5_defconfig> > > > + > > > +CONFIG_SPL_GPIO=y > > > > You need a few other options set in order for buildman to work in this > > case, please look at the other #include users. > > > > Are you referring to include "configs/k3_efi_capsule.config" ? No, I'm referring to: $ grep -l '#include' configs configs/am62px_evm_a53_defconfig configs/am62x_beagleplay_a53_defconfig configs/am62x_evm_a53_defconfig configs/am62x_lpsk_a53_defconfig configs/am62x_lpsk_r5_defconfig configs/am64x_evm_a53_defconfig configs/am68_sk_a72_defconfig configs/am68_sk_r5_defconfig configs/am69_sk_a72_defconfig configs/am69_sk_r5_defconfig configs/j721e_beagleboneai64_a72_defconfig configs/j721e_evm_a72_defconfig configs/j721e_sk_a72_defconfig configs/j721e_sk_r5_defconfig configs/j784s4_evm_a72_defconfig
diff --git a/configs/am62x_evm_r5_ethboot_defconfig b/configs/am62x_evm_r5_ethboot_defconfig new file mode 100644 index 0000000000..7b1932b215 --- /dev/null +++ b/configs/am62x_evm_r5_ethboot_defconfig @@ -0,0 +1,20 @@ +#include<configs/am62x_evm_r5_defconfig> + +CONFIG_SPL_GPIO=y +CONFIG_SPL_MMC=n +CONFIG_SPL_BOARD_INIT=y +CONFIG_SPL_STACK_R_MALLOC_SIMPLE_LEN=0x200000 +CONFIG_SPL_BSS_MAX_SIZE=0X3100 +CONFIG_SPL_DMA=y +CONFIG_SPL_ENV_SUPPORT=y +CONFIG_SPL_ETH=y +CONFIG_SPL_I2C=y +CONFIG_SPL_NET=y +CONFIG_SPL_NET_VCI_STRING="AM62X U-Boot R5 SPL" +CONFIG_CMD_DHCP=y +CONFIG_SPL_SYSCON=y +CONFIG_DMA_CHANNELS=y +CONFIG_TI_K3_NAVSS_UDMA=y +CONFIG_DM_I2C=y +CONFIG_PHY_TI_DP83867=y +CONFIG_TI_AM65_CPSW_NUSS=y