Hi Chintan On 25/02/25 17:18, Chintan Vankar wrote: > Update dev-data and clk-data generated using ksswtool-autogen to include
The tool is not publicly available as of now so you can remove the name and mention that it is auto-generated instead. > CPSW device which is required for Ethernet Boot, also use ARRAY_SIZE() > MACRO to avoid hard-coding array size. > > Reviewed-by: Bryan Brattlof <b...@ti.com> > Signed-off-by: Chintan Vankar <c-van...@ti.com> > --- > > Link to v2: > https://lore.kernel.org/r/20250219104831.2315464-5-c-van...@ti.com/ > > Changes from v2 to v3: > - No changes. > > arch/arm/mach-k3/r5/j721s2/clk-data.c | 58 +++++++++++++++++++++++++-- > arch/arm/mach-k3/r5/j721s2/dev-data.c | 3 +- > 2 files changed, 57 insertions(+), 4 deletions(-) > > diff --git a/arch/arm/mach-k3/r5/j721s2/clk-data.c > b/arch/arm/mach-k3/r5/j721s2/clk-data.c > index 0c5c321c1eb..0130c9c4b86 100644 > --- a/arch/arm/mach-k3/r5/j721s2/clk-data.c > +++ b/arch/arm/mach-k3/r5/j721s2/clk-data.c > @@ -5,7 +5,7 @@ > * This file is auto generated. Please do not hand edit and report any issues > * to Dave Gerlach <d-gerl...@ti.com>. > * > - * Copyright (C) 2020-2021 Texas Instruments Incorporated - > https://www.ti.com/ > + * Copyright (C) 2020-2025 Texas Instruments Incorporated - > https://www.ti.com/ > */ > > #include <linux/clk-provider.h> > @@ -55,6 +55,32 @@ static const char * const > mcu_ospi_ref_clk_sel_out1_parents[] = { > "hsdiv4_16fft_mcu_2_hsdivout4_clk", > }; > > +static const char * const wkup_gpio0_clksel_out0_parents[] = { > + "k3_pll_ctrl_wrap_wkup_0_chip_div1_clk_clk", > + "k3_pll_ctrl_wrap_wkup_0_chip_div1_clk_clk", > + "j7am_wakeup_16ff_wkup_0_wkup_rcosc_32k_clk", > + "j7am_wakeup_16ff_wkup_0_wkup_rcosc_12p5m_clk", > +}; > + > +static const char * const cpsw2g_cpts_rclk_sel_out0_parents[] = { > + "hsdiv4_16fft_main_3_hsdivout1_clk", > + "postdiv3_16fft_main_0_hsdivout6_clk", > + "board_0_mcu_cpts0_rft_clk_out", > + "board_0_cpts0_rft_clk_out", > + "board_0_mcu_ext_refclk0_out", > + "board_0_ext_refclk1_out", > + NULL, > + NULL, > + NULL, > + NULL, > + NULL, > + NULL, > + NULL, > + NULL, > + "hsdiv4_16fft_mcu_2_hsdivout1_clk", > + "k3_pll_ctrl_wrap_wkup_0_chip_div1_clk_clk", > +}; > + > static const char * const mcu_usart_clksel_out0_parents[] = { > "hsdiv4_16fft_mcu_1_hsdivout3_clk", > "postdiv3_16fft_main_1_hsdivout5_clk", > @@ -174,7 +200,11 @@ static const struct clk_data clk_list[] = { > CLK_FIXED_RATE("board_0_hfosc1_clk_out", 0, 0), > CLK_FIXED_RATE("board_0_mcu_ospi0_dqs_out", 0, 0), > CLK_FIXED_RATE("board_0_mcu_ospi1_dqs_out", 0, 0), > + CLK_FIXED_RATE("board_0_mcu_rgmii1_rxc_out", 0, 0), > + CLK_FIXED_RATE("board_0_mcu_rmii1_ref_clk_out", 0, 0), > CLK_FIXED_RATE("board_0_wkup_i2c0_scl_out", 0, 0), > + CLK_FIXED_RATE("cpsw_2guss_mcu_0_mdio_mdclk_o", 0, 0), > + CLK_FIXED_RATE("cpsw_2guss_mcu_0_rgmii1_txc_o", 0, 0), > CLK_FIXED_RATE("fss_mcu_0_hyperbus1p0_0_hpb_out_clk_n", 0, 0), > CLK_FIXED_RATE("fss_mcu_0_hyperbus1p0_0_hpb_out_clk_p", 0, 0), > CLK_FIXED_RATE("fss_mcu_0_ospi_0_ospi_oclk_clk", 0, 0), > @@ -199,6 +229,8 @@ static const struct clk_data clk_list[] = { > CLK_DIV("k3_pll_ctrl_wrap_wkup_0_chip_div1_clk_clk", > "k3_pll_ctrl_wrap_wkup_0_sysclkout_clk", 0x42010118, 0, 5, 0, 0), > CLK_MUX("mcu_ospi_ref_clk_sel_out0", mcu_ospi_ref_clk_sel_out0_parents, > 2, 0x40f08030, 0, 1, 0), > CLK_MUX("mcu_ospi_ref_clk_sel_out1", mcu_ospi_ref_clk_sel_out1_parents, > 2, 0x40f08034, 0, 1, 0), > + CLK_MUX("wkup_gpio0_clksel_out0", wkup_gpio0_clksel_out0_parents, 4, > 0x43008070, 0, 2, 0), > + CLK_MUX("cpsw2g_cpts_rclk_sel_out0", cpsw2g_cpts_rclk_sel_out0_parents, > 16, 0x40f08050, 8, 4, 0), > CLK_MUX("mcu_usart_clksel_out0", mcu_usart_clksel_out0_parents, 2, > 0x40f081c0, 0, 1, 0), > CLK_MUX("wkup_i2c_mcupll_bypass_out0", > wkup_i2c_mcupll_bypass_out0_parents, 2, 0x43008060, 0, 1, 0), > CLK_MUX("main_pll_hfosc_sel_out0", main_pll_hfosc_sel_out0_parents, 2, > 0x43008080, 0, 1, 0), > @@ -275,6 +307,24 @@ static const struct dev_clk soc_dev_clk_data[] = { > DEV_CLK(4, 0, "hsdiv0_16fft_main_8_hsdivout0_clk"), > DEV_CLK(4, 1, "hsdiv0_16fft_main_7_hsdivout0_clk"), > DEV_CLK(4, 2, "k3_pll_ctrl_wrap_main_0_chip_div1_clk_clk"), > + DEV_CLK(29, 3, "cpsw2g_cpts_rclk_sel_out0"), > + DEV_CLK(29, 4, "hsdiv4_16fft_main_3_hsdivout1_clk"), > + DEV_CLK(29, 5, "postdiv3_16fft_main_0_hsdivout6_clk"), > + DEV_CLK(29, 6, "board_0_mcu_cpts0_rft_clk_out"), > + DEV_CLK(29, 7, "board_0_cpts0_rft_clk_out"), > + DEV_CLK(29, 8, "board_0_mcu_ext_refclk0_out"), > + DEV_CLK(29, 9, "board_0_ext_refclk1_out"), > + DEV_CLK(29, 18, "hsdiv4_16fft_mcu_2_hsdivout1_clk"), > + DEV_CLK(29, 19, "k3_pll_ctrl_wrap_wkup_0_chip_div1_clk_clk"), > + DEV_CLK(29, 20, "hsdiv4_16fft_mcu_2_hsdivout0_clk"), > + DEV_CLK(29, 21, "hsdiv4_16fft_mcu_2_hsdivout0_clk"), > + DEV_CLK(29, 22, "board_0_mcu_rgmii1_rxc_out"), > + DEV_CLK(29, 26, "board_0_mcu_rmii1_ref_clk_out"), > + DEV_CLK(29, 28, "k3_pll_ctrl_wrap_wkup_0_chip_div1_clk_clk"), > + DEV_CLK(29, 29, "hsdiv4_16fft_mcu_2_hsdivout0_clk"), > + DEV_CLK(29, 30, "hsdiv4_16fft_mcu_2_hsdivout0_clk"), > + DEV_CLK(29, 32, "hsdiv4_16fft_mcu_2_hsdivout0_clk"), > + DEV_CLK(29, 33, "hsdiv4_16fft_mcu_2_hsdivout0_clk"), > DEV_CLK(43, 0, "postdiv3_16fft_main_0_hsdivout8_clk"), > DEV_CLK(43, 1, "hsdiv4_16fft_main_0_hsdivout3_clk"), > DEV_CLK(43, 2, "gluelogic_hfosc0_clkout"), > @@ -367,6 +417,7 @@ static const struct dev_clk soc_dev_clk_data[] = { > DEV_CLK(157, 187, "fss_mcu_0_ospi_1_ospi_oclk_clk"), > DEV_CLK(157, 194, "emmcsd4ss_main_0_emmcsdss_io_clk_o"), > DEV_CLK(157, 197, "j7am_ddr_ew_wrap_dv_wrap_main_0_ddrss_io_ck_n"), > + DEV_CLK(157, 207, "cpsw_2guss_mcu_0_mdio_mdclk_o"), > DEV_CLK(157, 208, "j7am_ddr_ew_wrap_dv_wrap_main_1_ddrss_io_ck_n"), > DEV_CLK(157, 214, "fss_mcu_0_hyperbus1p0_0_hpb_out_clk_p"), > DEV_CLK(157, 221, "mcu_clkout_mux_out0"), > @@ -374,6 +425,7 @@ static const struct dev_clk soc_dev_clk_data[] = { > DEV_CLK(157, 223, "hsdiv4_16fft_mcu_2_hsdivout0_clk"), > DEV_CLK(157, 225, "emmc8ss_16ffc_main_0_emmcss_io_clk"), > DEV_CLK(157, 231, "fss_mcu_0_hyperbus1p0_0_hpb_out_clk_n"), > + DEV_CLK(157, 244, "cpsw_2guss_mcu_0_rgmii1_txc_o"), > DEV_CLK(157, 352, "dpi0_ext_clksel_out0"), > DEV_CLK(180, 0, "gluelogic_hfosc0_clkout"), > DEV_CLK(180, 2, "k3_pll_ctrl_wrap_wkup_0_chip_div1_clk_clk"), > @@ -400,7 +452,7 @@ static const struct dev_clk soc_dev_clk_data[] = { > > const struct ti_k3_clk_platdata j721s2_clk_platdata = { > .clk_list = clk_list, > - .clk_list_cnt = 105, > + .clk_list_cnt = ARRAY_SIZE(clk_list), > .soc_dev_clk_data = soc_dev_clk_data, > - .soc_dev_clk_data_cnt = 124, > + .soc_dev_clk_data_cnt = ARRAY_SIZE(soc_dev_clk_data), > }; > diff --git a/arch/arm/mach-k3/r5/j721s2/dev-data.c > b/arch/arm/mach-k3/r5/j721s2/dev-data.c > index df70c5e5d7c..b78550707c5 100644 > --- a/arch/arm/mach-k3/r5/j721s2/dev-data.c > +++ b/arch/arm/mach-k3/r5/j721s2/dev-data.c > @@ -5,7 +5,7 @@ > * This file is auto generated. Please do not hand edit and report any issues > * to Dave Gerlach <d-gerl...@ti.com>. > * > - * Copyright (C) 2020-2021 Texas Instruments Incorporated - > https://www.ti.com/ > + * Copyright (C) 2020-2025 Texas Instruments Incorporated - > https://www.ti.com/ > */ > > #include "k3-dev.h" > @@ -47,6 +47,7 @@ static struct ti_lpsc soc_lpsc_list[] = { > }; > > static struct ti_dev soc_dev_list[] = { > + PSC_DEV(29, &soc_lpsc_list[0]), > PSC_DEV(35, &soc_lpsc_list[0]), > PSC_DEV(108, &soc_lpsc_list[0]), > PSC_DEV(109, &soc_lpsc_list[0]), -- Thanking You Neha Malcom Francis