We now can provide a map and have the standard fdtfile variable set from
code itself. This allows for bootstd to "just work".
While at this, replace findfdt in environment with a warning as it is no
longer needed.
Signed-off-by: Nishanth Menon
---
Changes from V1: None.
I hav
| 19 ++
> 5 files changed, 51 insertions(+), 6 deletions(-)
Please update am64x documentation as well. People will need to know
how to actually build, flash and use it.
>
>
> base-commit: c2c598e87cfe56f5991730762c00733c5aa9a994
> prerequisite-patch-id: e04
gt; +CONFIG_SPL_NAND_IDENT=y
> +CONFIG_SPL_NAND_ECC=y
> +CONFIG_SYS_NAND_MAX_CHIPS=1
> +CONFIG_SYS_MAX_NAND_DEVICE=1
> +# CONFIG_SPL_NAND_AM33XX_BCH is not set
> +CONFIG_SPL_MTD_SUPPORT=y
> +CONFIG_SYS_NAND_U_BOOT_LOCATIONS=y
> +CONFIG_SYS_NAND_U_BOOT_OFFS=0x60
> --
> 2.34.1
>
Why not a config fragment?
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
On 18:20-20240109, Roger Quadros wrote:
>
>
> On 08/01/2024 19:32, Nishanth Menon wrote:
> > We now can provide a map and have the standard fdtfile variable set from
> > code itself. This allows for bootstd to "just work".
> >
> > While at this, re
I have mentioned previously for
TFA as well - I am looking for some common sequence to be defined
between am62x and J7200 family rather than each go completely
tangentially, until that happens, please consider my standing NAK.
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
t; + */
> > +struct ti_fdt_map {
> > + const char *name_fdt;
>
> Can we call this board_name? as name_fdt corresponds to device tree name.
>
Done.
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
sleading in the board name as the boards are now shipped with
> HS-FS chip and are no longer GP.
Intent of the series was to get rid of findfdt - it was not meant to
cleanup existing name usage in the files. If there is a desire to do so,
please - patches are welcome.
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
gt; .phys = 0x5UL,
> - .size = 0x4UL,
> + .size = 0x38000UL,
> .attrs = PTE_BLOCK_MEMTYPE(MT_DEVICE_NGNRNE) |
> PTE_BLOCK_NON_SHARE |
> PTE_BLOCK_PXN | PTE_BLOCK_UX
it just returns a hard coded value, which could be based on
> the CONFIG_DEFAULT_DEVICE_TREE config.
Ends up being superflous - there is already a push to reduce the config
options.
[..]
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
On 20:24-20240108, Jon Humphreys wrote:
> Nishanth Menon writes:
>
> > Stop using the findfdt script and switch to setting the fdtfile from C
> > code.
> >
> > While at this, replace findfdt in environment with a warning as it is
> > no longer needed
&
ame as described by kernel
> > + */
> > +struct ti_fdt_map {
> > + const char *name_fdt;
> > + char *fdt_file_name;
> > +};
> > +
> > +/**
> > + * ti_set_fdt_env - Find the correct device tree file name and set
> > 'fdtfile'
>
> "Find the correct device tree file name based on the board name and "...
OK.
>
> > + * env variable with correct folder structure appropriate to the
> > architecture
> > + * and kernel conventions. This function is invoked typically as part of
> > + * board_late_init
> > + *
> > + * fdt name is picked by:
> > + * a) If a match is found, use the match
>
> "a) If a board name match is found, use the match"
OK.
[...]
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
t" and adds warning for
> "findfdt" like your commit message says you do..).
Will fix up the commit message in the next revision
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
On 12:50-20240108, Andrew Davis wrote:
> On 1/8/24 11:32 AM, Nishanth Menon wrote:
> > Introduce a common fdt operations library for basic device tree
> > operations that are common between various boards.
> >
> > The first library to introduce here is the capab
Stop using the findfdt script and switch to setting the fdtfile from
C code.
While at this, replace findfdt in environment with a warning as it is
no longer needed
Signed-off-by: Nishanth Menon
---
board/ti/am62ax/am62ax.env | 1 -
board/ti/am62ax/evm.c| 10
Stop using the findfdt script and switch to setting the fdtfile from C
code.
While at this, replace findfdt in environment with a warning as it is
no longer needed.
Signed-off-by: Nishanth Menon
---
board/beagle/beagleplay/beagleplay.c | 14 ++
board/beagle/beagleplay
We now can provide a map and have the standard fdtfile variable set from
code itself. This allows for bootstd to "just work".
While at this, replace findfdt in environment with a warning as it is no
longer needed.
Signed-off-by: Nishanth Menon
---
board/ti/am64x/am64x.env | 9 --
We now can provide a map and have the standard fdtfile variable set from
code itself. This allows for bootstd to "just work".
While at this, replace findfdt in environment with a warning as it is no
longer needed.
Signed-off-by: Nishanth Menon
---
board/ti/am65x/am65x.env | 3 ---
on next branch at: c2c598e87cfe Merge branch 'staging' of
https://source.denx.de/u-boot/custodians/u-boot-tegra into next
Nishanth Menon (10):
board: ti: common: Introduce a common fdt ops library
board: ti: am62ax: Set fdtfile from C code instead of findfdt script
board: ti: am62x: Set
We now can provide a map and have the standard fdtfile variable set from
code itself. This allows for bootstd to "just work".
While at this, replace findfdt in environment with a warning as it is no
longer needed.
Signed-off-by: Nishanth Menon
---
board/ti/j721e/evm.c | 8 +
Stop using the findfdt script and switch to setting the fdtfile from
C code.
While at this, replace findfdt in environment with a warning as it is
no longer needed
Signed-off-by: Nishanth Menon
---
board/ti/am62x/am62x.env| 1 -
board/ti/am62x/evm.c| 8
configs
-by: Nishanth Menon
---
board/ti/common/Kconfig | 12
board/ti/common/Makefile | 1 +
board/ti/common/fdt_ops.c | 65 +++
board/ti/common/fdt_ops.h | 41
4 files changed, 119 insertions(+)
create mode 100644 board/ti/common
We shouldn't need finfdt anymore. Drop the env script.
Signed-off-by: Nishanth Menon
---
include/env/ti/default_findfdt.env | 12
1 file changed, 12 deletions(-)
delete mode 100644 include/env/ti/default_findfdt.env
diff --git a/include/env/ti/default_findfdt.env
b/include/env
We now can provide a map and have the standard fdtfile variable set from
code itself. This allows for bootstd to "just work".
While at this, replace findfdt in environment with a warning as it is no
longer needed.
Signed-off-by: Nishanth Menon
---
board/ti/j721s2/evm.c | 8 +
Stop using the findfdt script and switch to setting the fdtfile from C
code.
While at this, replace findfdt in environment with a warning as it is
no longer needed
Signed-off-by: Nishanth Menon
---
board/beagle/beagleboneai64/beagleboneai64.c | 14 ++
board/beagle/beagleboneai64
On 13:23-20240104, Neha Malcom Francis wrote:
> Use yamllint for checking whether YAML configuration files are adhering
> to default yamllint rules.
>
> Signed-off-by: Neha Malcom Francis
> Suggested-by: Nishanth Menon
> ---
> tools/binman/etype/ti_board_config.py
> +F: arch/arm/dts/k3-j784s4-ddr.dtsi
> +F: arch/arm/dts/k3-j784s4-ddr-evm-lp4-4266.dtsi
> +F: doc/board/ti/j784s4_evm.rst
> +
> +AM69 BOARD
AM69 SK BOARD please
> +F: arch/arm/dts/k3-am69-sk.dts
> +F: arch/arm/dts/k3-am69-sk-u-boot.dtsi
> +F: arch/arm/dts/k3-am69-
gt; +.. include:: k3.rst
> +:start-after: .. k3_rst_include_start_openocd_connect_XDS110
> +:end-before: .. k3_rst_include_end_openocd_connect_XDS110
> +
> +To start OpenOCD and connect to J784S4-EVM or AM69-SK board, use the
> +following.
> +
> +.. code-block:: bash
>
gs/am69_sk_r5.config
> new file mode 100644
> index 00..9194694393
> --- /dev/null
> +++ b/configs/am69_sk_r5.config
> @@ -0,0 +1,3 @@
> +# Defconfig fragment to apply on top of j784s4_evm_r5_defconfig
> +
> +CONFIG_DEFAULT_DEVICE_TREE="k3-am69-r5-sk"
See my
+CONFIG_SYS_MAX_FLASH_BANKS_DETECT=y
> +CONFIG_DM_SPI_FLASH=y
> +CONFIG_SPI_FLASH_SFDP_SUPPORT=y
> +CONFIG_SPI_FLASH_SOFT_RESET=y
> +CONFIG_SPI_FLASH_SOFT_RESET_ON_BOOT=y
> +CONFIG_SPI_FLASH_SPANSION=y
> +CONFIG_SPI_FLASH_S28HX_T=y
> +CONFIG_SPI_FLASH_STMICRO=y
> +CONFIG_SPI_FLASH_MT35XU=y
> +CONFIG_PINCTRL=y
> +# CONFIG_PINCTRL_GENERIC is not set
> +CONFIG_SPL_PINCTRL=y
> +# CONFIG_SPL_PINCTRL_GENERIC is not set
> +CONFIG_PINCTRL_SINGLE=y
> +CONFIG_POWER_DOMAIN=y
> +CONFIG_TI_POWER_DOMAIN=y
> +CONFIG_K3_SYSTEM_CONTROLLER=y
> +CONFIG_REMOTEPROC_TI_K3_ARM64=y
> +CONFIG_RESET_TI_SCI=y
> +CONFIG_DM_SERIAL=y
> +CONFIG_SOC_DEVICE=y
> +CONFIG_SOC_DEVICE_TI_K3=y
> +CONFIG_SOC_TI=y
> +CONFIG_SPI=y
> +CONFIG_DM_SPI=y
> +CONFIG_CADENCE_QSPI=y
> +CONFIG_SYSRESET=y
> +CONFIG_SPL_SYSRESET=y
> +CONFIG_SYSRESET_TI_SCI=y
> +CONFIG_DM_THERMAL=y
> +CONFIG_TIMER=y
> +CONFIG_SPL_TIMER=y
> +CONFIG_OMAP_TIMER=y
> +CONFIG_FS_EXT4=y
> +CONFIG_FS_FAT_MAX_CLUSTSIZE=16384
> +CONFIG_PANIC_HANG=y
> +CONFIG_LIB_RATIONAL=y
> +CONFIG_SPL_LIB_RATIONAL=y
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
4000>,
Is there a patch submitted for upstream kernel?
This should be part of your patch for board.dts instead of a
patch that fixes multiple board.dts after introducing the dts in the
same series.
So squash this change to relevant patches please and please refer to
upstream kernel patch so that we can drop this later on.
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
m69-sk.dtb"
as I said in the previous patch - this approach is broken. we have two
different builds, we dont need to do this multi-conf stuff.
We should be just using templates for j784s4 and it should just work
with the same board-u-boot.dtsi and r5-board.dts files.
[...]
--
Regards,
Nishan
gt; + ${mtdparts}
> +run_kern=booti ${loadaddr} ${rd_spec} ${fdtaddr}
> +
> +boot_targets=mmc1 mmc0 usb pxe dhcp
> +boot=mmc
> +mmcdev=1
> +bootpart=1:2
> +bootdir=/boot
> +rd_spec=-
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
a bunch of warnings, but it does the same for other yamls as
well. Not sure it was supposed to be yamlinted - Neha? Might be good
idea than manual review to ensure some basic checks are done?
Either way,
Reviewed-by: Nishanth Menon
Ref - I used yamllint 1.26.3
--
Regards,
Nishanth Menon
Key
env_set("fdtfile", fdtfile);
See board/xilinx/zynqmp/zynqmp.c With that fdtfile will be available by
default removing the need for findfdt and script mumbo jumbo.
> +}
> diff --git a/include/configs/j784s4_evm.h b/include/configs/j784s4_evm.h
> new file mode 100644
> index 00..a4e0521198
> --- /dev/null
> +++ b/include/configs/j784s4_evm.h
> @@ -0,0 +1,15 @@
> +/* SPDX-License-Identifier: GPL-2.0-or-later */
> +/*
> + * Configuration header file for K3 J784S4 EVM
> + *
> + * Copyright (C) 2023 Texas Instruments Incorporated - https://www.ti.com/
btw, for next edition - 2023-2024 - here and elsewhere.
> + * Hari Nagalla
> + */
> +
> +#ifndef __CONFIG_J784S4_EVM_H
> +#define __CONFIG_J784S4_EVM_H
> +
> +/* Now for the remaining common defines */
> +#include
> +
> +#endif /* __CONFIG_J784S4_EVM_H */
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
On 00:45-20231220, Apurva Nandan wrote:
> Add support for DMA in J784S4 SoC.
>
> Signed-off-by: Jayesh Choudhary
> Signed-off-by: Hari Nagalla
> Signed-off-by: Apurva Nandan
> ---
Cursory review (have not looked at actual data):
Reviewed-by: Nishanth Menon
--
Regards,
ual data), looks fine..
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
t;AM62AX";
> break;
> + case JTAG_ID_PARTNO_J784S4:
> + family = "J784S4";
> + break;
> default:
> family = "Unknown Silicon";
> };
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
MMC2;
> + }
> + case BACKUP_BOOT_DEVICE_SPI:
> + return BOOT_DEVICE_SPI;
> + case BACKUP_BOOT_DEVICE_I2C:
> + return BOOT_DEVICE_I2C;
> + }
> +
> + return BOOT_DEVICE_RAM;
> +}
> +
> +static u32 __get_primary_bootmedia(u32 main_de
y don't care to have two patches introduced - you should probably
squash it.. as long as the kernel dts files match up, we should be good.
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
I suggest
changing the order of the patches: Introduce the base C files then
introduce the dts files.
> dtb-$(CONFIG_SOC_K3_AM642) += k3-am642-evm.dtb \
> k3-am642-r5-evm.dtb \
> k3-am642-sk.dtb \
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
On 09:37-20240103, Nishanth Menon wrote:
> On 00:45-20231220, Apurva Nandan wrote:
> > Hello Everyone!
> >
> > This series will introduce basic support (SD and UART) support for Texas
> > Instruments J784S4 EVM.
> >
> > The J784S4 SoC device tree patches
inal branch and stop patching, run "git am --abort".
You will probably have to rebase this series once any review comments
are addressed.
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
er which dtb build is happening.
>
> Tested-by: Tom Rini
> Signed-off-by: Bryan Brattlof
> ---
[...]
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
On 11:47-20231229, Bryan Brattlof wrote:
> With the Linux and U-Boot board dtb files unified, we now have a
> duplicate mcu secure proxy node. Remove it
>
> Tested-by: Tom Rini
> Signed-off-by: Bryan Brattlof
> ---
[...]
Reviewed-by: Nishanth Menon
--
Regards,
from cbass_mcu to the root node to avoid any
> confusion.
>
> Tested-by: Tom Rini
> Signed-off-by: Bryan Brattlof
> ---
[...]
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
On 11:47-20231229, Bryan Brattlof wrote:
> These aliases are not needed in U-Boot. Remove them
>
> Tested-by: Tom Rini
> Signed-off-by: Bryan Brattlof
> ---
[...]
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232
On 11:47-20231229, Bryan Brattlof wrote:
> With the Linux and U-Boot board dtb files unified, we have duplicate
> properties in the root node. Remove them
>
> Tested-by: Tom Rini
> Signed-off-by: Bryan Brattlof
> ---
[...]
Reviewed-by: Nishanth Menon
--
Regards,
On 11:47-20231229, Bryan Brattlof wrote:
> With the Linux and U-Boot board dtb files unified, we now have a
> duplicate vtt_pinmux node. Remove it
>
> Tested-by: Tom Rini
> Signed-off-by: Bryan Brattlof
> ---
[...]
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (
On 11:47-20231229, Bryan Brattlof wrote:
> With the Linux and U-Boot board dtb files unified, we now have a
> duplicate mdio node. Remove it
>
> Tested-by: Tom Rini
> Signed-off-by: Bryan Brattlof
[...]
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (0xD
On 11:47-20231229, Bryan Brattlof wrote:
> The pinmux for usb0 is missing from the Linux board dtb file. Remove it
> until we can introduce it in Linux
>
> Tested-by: Tom Rini
> Signed-off-by: Bryan Brattlof
> ---
[...]
Reviewed-by: Nishanth Menon
--
Regards,
On 11:47-20231229, Bryan Brattlof wrote:
> With the Linux and U-Boot board dtb files unified, we now have a
> duplicate ospi0 node. Remove it
>
> Tested-by: Tom Rini
> Signed-off-by: Bryan Brattlof
> ---
[...]
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (
On 11:47-20231229, Bryan Brattlof wrote:
> With the Linux and U-Boot board dtb files unified, we now have a
> duplicate wkup_i2c0 node. Remove it
>
> Tested-by: Tom Rini
> Signed-off-by: Bryan Brattlof
> ---
[...]
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (
On 11:47-20231229, Bryan Brattlof wrote:
> With the Linux and U-Boot board dtb files unified, we now have a
> duplicate sdhci1 pinmux node. Remove it
>
> Tested-by: Tom Rini
> Signed-off-by: Bryan Brattlof
> ---
[...]
Reviewed-by: Nishanth Menon
--
Regards,
On 11:47-20231229, Bryan Brattlof wrote:
> With the Linux and U-Boot board dtb files unified, we now have
> a duplicate sdhci0 pinmux node. Remove it
>
> Tested-by: Tom Rini
> Signed-off-by: Bryan Brattlof
> ---
[...]
Reviewed-by: Nishanth Menon
--
Regards,
On 11:47-20231229, Bryan Brattlof wrote:
> With the Linux and U-Boot board dtb files unified, we now have a
> duplicate main_uart0 node. Remove it
>
> Tested-by: Tom Rini
> Signed-off-by: Bryan Brattlof
> ---
[...]
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (
On 11:47-20231229, Bryan Brattlof wrote:
> With the Linux and U-Boot board dtb files unified we now have a
> duplicate mcu_uart0 node. Remove it
>
> Tested-by: Tom Rini
> Signed-off-by: Bryan Brattlof
> ---
[...]
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (
tag is published
> + */
> +_udmap {
> + reg = <0x0 0x285c 0x0 0x100>,
> + <0x0 0x2a80 0x0 0x4>,
> + <0x0 0x2aa0 0x0 0x4>,
> + <0x0 0x284a 0x0 0x4000>,
> + <0x0 0x284c 0x0 0x4000>,
> +
> 1 file changed, 13 deletions(-)
>
[...]
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
> 1 file changed, 10 deletions(-)
>
[...]
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
by: Bryan Brattlof
> ---
[...]
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
> 1 file changed, 12 deletions(-)
[...]
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
ile changed, 9 deletions(-)
[...]
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
ts-v1/;
>
> -#include "k3-am654.dtsi"
> +#include "k3-am654-base-board.dts"
> +#include "k3-am654-base-board-u-boot.dtsi"
> #include "k3-am654-base-board-ddr4-1600MTs.dtsi"
> #include "k3-am654-ddr.dtsi"
[...]
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
ph-pre-ram;
> +};
> +
> +_main {
> + bootph-pre-ram;
> +};
I understand this in an intermediate state and the final dts looks fine.
So,
[...]
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
, 209 deletions(-)
>
[...]
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
the Linux board file with this one.
>
> Tested-by: Tom Rini
> Signed-off-by: Bryan Brattlof
> ---
[...]
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
> ---
> configs/am65x_evm_a53_defconfig | 1 -
> 1 file changed, 1 deletion(-)
>
[...]
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
le changed, 2 insertions(+)
>
[...]
Reviewed-by: Nishanth Menon
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
On 07:08-20231227, Nishanth Menon wrote:
> Distroboot and bootstd both mandate a findfdt variable pointing to the
> correct device tree blob. Current mechanism calls a find_fdt function
> to set this variable. We do not need a find_fdt command to set the
> environment variable to
-by: Nishanth Menon
---
include/env/ti/default_findfdt.env | 13 -
1 file changed, 4 insertions(+), 9 deletions(-)
diff --git a/include/env/ti/default_findfdt.env
b/include/env/ti/default_findfdt.env
index a2b51dd923bb..1a1ab8406c9e 100644
--- a/include/env/ti/default_findfdt.env
iewed-by: Mattijs Korpershoek
>
> Applied to u-boot/next, thanks!
Shouldn't main_cpsw0_qsgmii_phyinit be part of the driver?? why are we
scripting up driver initialization?
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
- could you look at syncing those once an update to this series is
posted? We could probably have a smoother v6.8-rc1 from then on.
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
wer-domain properties to avoid making calls to
> + * SYSFW before it is loaded. Public ROM has already
> + * set it up for us anyway.
> + */
Same - please fix the comment style here as well.
> {
> clock-names = "clk_xin";
> clocks = <_200mhz>;
s point. Delete the
> + * power-domain properties to avoid making calls to
> + * SYSFW before it is loaded. Public ROM has already
> + * set it up for us anyway.
> + */
Please fix your comment style here:
/*
* blah blah
*/
> {
> clock-names = "clk_xin";
>
er moving this to r5 dts - there is no use in having this in
u-boot. should be pre-ram?
[...]
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
synced after the v6.8-rc1 linux
> + * tag is published
> + */
and here.
> +_udmap {
> + reg = <0x0 0x285c 0x0 0x100>,
> + <0x0 0x2a80 0x0 0x4>,
> + <0x0 0x2aa0 0x0 0x4>,
> + <0x0 0x284a 0x0 0x4000&
>
> All of the DT patches are already part of upstream linux kernel except
> for patch 08/16. This patch is currently in linux-next and will be in
> linux in 6.8-rc1.
>
NAK.
Send a sync at 6.8-rc1 and then start working - DONOT SEND patches
modifying SoC dtsi or evm files directly
at all firmware are supposed to
implement. unfortunately the defconfig option for that API wasn't
properly implemented for various firmware components correctly. -
impacted chips are: j721e, j7200, j721s2/am68, j784s4/am69. Other
K3 processor firmware have it implemented correctly. The latest
9.1.8--v09.01.08 has the bug as well - but hopefully should be fixed
and released soon-ish.
[1]
https://software-dl.ti.com/tisci/esd/latest/2_tisci_msgs/general/core.html#tisci-msg-query-fw-caps
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
then
> > try refactoring with config fragments for a second step?
> >
>
> I agree. It's on my TODO list to try to clean up some of the messes
> we've made so I thought I chime in
>
> Don't let me stop you from adding features
I prefer we do the fragments with this series. We
On 08:46-20231205, Andrew Davis wrote:
> On 12/4/23 1:29 PM, Nishanth Menon wrote:
> > On 15:59-20231201, Dhruva Gole wrote:
> > > Delete the flag CFG_WITH_SOFTWARE_PRNG as it's not necessary/ boot
> > > requirement for this SoC
> > >
> > > Signed-off
G Flag for OPTEE
>
> doc/board/beagle/am62x_beagleplay.rst | 1 -
> doc/board/ti/am62x_sk.rst | 1 -
> 2 files changed, 2 deletions(-)
>
NAK to the series. RNG seed is needed for KASLR. either enable h/w RNG
or at least enable S/W RNG in OPTEE.
--
Regards,
Nishanth Menon
gt; --
> 2.34.1
>
NAK. RNG is needed to seed standard distros.
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
d?
Is there absolutely no way for people to use the new firmware and keep
their old u-boot? Rather disappointing to see backward compatible
breakage occurring this stage of the device. This is not appreciated.
Please see if you can fix your firmware to maintain compatibility (as
has been the case for yea
R seperately? I
suppose OPTEE will come in the way?
Looks like the no-map is not honored well.. as you were mentioning, a
zephyr style generating the map from dts might be much better, but then
we do support multiple dtbs as well..
OK - I guess we need to leave this cleanup for now.
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
lib/firmware/j721s2-c71_0-fw 7
> /lib/firmware/j721s2-c71_1-fw
> +rproc_fw_binaries= 2 /lib/firmware/j721s2-main-r5f0_0-fw 4
> /lib/firmware/j721s2-main-r5f1_0-fw 6 /lib/firmware/j721s2-c71_0-fw 7
> /lib/firmware/j721s2-c71_1-fw
I really would prefer us to work towards getting stdboot remote
is (2):
> arm: mach-k3: Remove non-cached memory map areas
> arm: mach-k3: Merge initial memory maps
>
> arch/arm/mach-k3/arm64-mmu.c | 253 +--
> 1 file changed, 2 insertions(+), 251 deletions(-)
>
> --
> 2.39.2
>
Could you rebas
e same change for plats now in -next
>
Reviewed-by: Nishanth Menon
Tested-by: Nishanth Menon
j721e-idk-gw
am68_sk-fs
j721s2-evm
j721e-sk
j7200-hsevm
j7200-evm
am64xx-hsevm
am64xx_sk-fs
beagleplay-gp-uart
am62xx_sk-fs
am62xx-sk
am62axx_sk-fs
Though, I'd prefer this along wit
BOOT_MODE_B_SHIFT;
>
> - if (bootmode == BOOT_DEVICE_OSPI || bootmode == BOOT_DEVICE_QSPI)
> + if (bootmode == BOOT_DEVICE_OSPI || bootmode == BOOT_DEVICE_QSPI ||
> + bootmode == BOOT_DEVICE_XSPI)
> bootmode = BOOT_DEVICE_SPI;
On 22:28-20231114, Roger Quadros wrote:
> BCDMA can be used at SPL for OSPI boot and mem-to-mem DMA
> so add "bootph-all" to BCDMA node.
>
> Suggested-by: Nishanth Menon
> Fixes: 9a3f2b6798b0 ("arm: dts: k3-am625-verdin-wifi-dev-u-boot.dtsi: Fix
> DMA/Ethernet
Kernel dts import now provides bootph-all and bootph-pre-ram properties
for the properties we have been overriding so far. Drop the same.
Reviewed-by: Dhruva Gole
Tested-by: Dhruva Gole
Signed-off-by: Nishanth Menon
---
Changes since v1:
- tested and reviewed tags
V1: https://lore.kernel.org
now.
Acked-by: Francesco Dolcini
Tested-by: Dhruva Gole
Signed-off-by: Nishanth Menon
---
Changes since v1:
- just picked up acks and tested tags
V1: https://lore.kernel.org/r/20231113145919.1928812-3...@ti.com
arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi | 54 --
arch/arm/dts
Signed-off-by: Nishanth Menon
---
Changes:
- capitalize TIFS and UART.
- pick up reviewed and tested tags
V1: https://lore.kernel.org/r/20231113145919.1928812-5...@ti.com
arch/arm/dts/k3-am625-r5-sk.dts | 12 +---
arch/arm/dts/k3-am625-sk-u-boot.dtsi | 92
2
Sync with kernel v6.7-rc1 and sync up the u-boot dts files accordingly.
Tested-by: Dhruva Gole
Signed-off-by: Nishanth Menon
---
Changes since v1:
- just tested-by tag.
V1: https://lore.kernel.org/r/20231113145919.1928812-2...@ti.com
arch/arm/dts/k3-am62-main.dtsi| 12 ++-
arch/arm
://lore.kernel.org/u-boot/20231104080137.9628-1...@ti.com/
I haven't had a chance to test this out on verdin (only build tested)
- going to depend on any checks folks can provide.
V1: https://lore.kernel.org/all/20231113145919.1928812-1...@ti.com/
Nishanth Menon (4):
arm: dts: k3-am625*: Sync with kernel v6.7
On 09:03-20231114, Dhruva Gole wrote:
[...]
> > #ifdef CONFIG_TARGET_AM625_A53_EVM
> >
> > #define SPL_AM625_BEAGLEPLAY_DTB "spl/dts/k3-am625-beagleplay.dtb"
>
> You didn't talk about this in the commit message?
There is nothing changing in this patch here.
--
Regards,
Nishanth Menonh
ps://lore.kernel.org/all/20231109140958.1093235-1-joao.goncal...@toradex.com/
> Signed-off-by: Joao Paulo Goncalves
> ---
Tested-by: Nishanth Menon
https://gist.github.com/nmenon/ee47a34d6b581916d6fba164617ea93b
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
isc/ti,j721e-esm.yaml| 53 +++
What is the rule here?
https://tgit.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/misc/ti,j721e-esm.yaml
we have the binding in upstream kernel.
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / F
haven't had a chance to test this out on verdin (only build tested)
- so will be nice to verify.
Nishanth Menon (4):
arm: dts: k3-am625*: Sync with kernel v6.7-rc1
arm: dts: k3-am625: Drop SoC provided bootph params from board
u-boot/r5 dtsi
arm: dts: k3-am625-beagleplay-u-boot: drop
now.
Signed-off-by: Nishanth Menon
---
arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi | 54 --
arch/arm/dts/k3-am625-r5-beagleplay.dts | 9 ---
arch/arm/dts/k3-am625-r5-sk.dts | 9 ---
arch/arm/dts/k3-am625-sk-u-boot.dtsi | 53 --
arch
Kernel dts import now provides bootph-all and bootph-pre-ram properties
for the properties we have been overriding so far. Drop the same.
Signed-off-by: Nishanth Menon
---
arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi | 56
1 file changed, 56 deletions(-)
diff --git a/arch
101 - 200 of 1401 matches
Mail list logo