On Wed, Jun 12, 2024 at 02:41:22PM +0530, Jayesh Choudhary wrote:

> Introduce the initial configs needed to support the J722S SoC family.
> 
> Signed-off-by: Vaishnav Achath <vaishna...@ti.com>
> Signed-off-by: Jayesh Choudhary <j-choudh...@ti.com>
> ---
>  configs/j722s_evm_a53_defconfig | 172 ++++++++++++++++++++++++++++++++
>  configs/j722s_evm_r5_defconfig  | 131 ++++++++++++++++++++++++
>  include/configs/j722s_evm.h     |  14 +++
>  3 files changed, 317 insertions(+)
>  create mode 100644 configs/j722s_evm_a53_defconfig
>  create mode 100644 configs/j722s_evm_r5_defconfig
>  create mode 100644 include/configs/j722s_evm.h
[snip]
> diff --git a/include/configs/j722s_evm.h b/include/configs/j722s_evm.h
> new file mode 100644
> index 0000000000..10f2e2d3ed
> --- /dev/null
> +++ b/include/configs/j722s_evm.h
> @@ -0,0 +1,14 @@
> +/* SPDX-License-Identifier: GPL-2.0+ */
> +/*
> + * Configuration header file for K3 J722S SoC family
> + *
> + * Copyright (C) 2024 Texas Instruments Incorporated - https://www.ti.com/
> + */
> +
> +#ifndef __CONFIG_J722S_EVM_H
> +#define __CONFIG_J722S_EVM_H
> +
> +/* Now for the remaining common defines */
> +#include <configs/ti_armv7_common.h>
> +
> +#endif /* __CONFIG_J722S_EVM_H */

If there's any other feedback and you need to do a v3, instead of
#include <configs/ti_armv7_common.h> here just define CFG_SYS_SDRAM_BASE
as that's the only thing in the common header that new platforms should
need.

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to