Hi Tero, On 8/7/19 5:56 PM, Anna, Suman wrote: > Hi Tero, > > On 8/7/19 8:04 AM, Tero Kristo wrote: >> This is needed by the TI DM timer driver. > > Again can do with some better patch descriptions. Similar to the > previous patch, missing the equivalent patches for OMAP4 and OMAP5. > You can use my downstream patches for these - [1][2][3] that has all the > needed Fixes by details. Only difference is that you used a single line > change on DRA7, and this should suffice since all the sources are same, > but OMAP4 and OMAP5 needed different ones. > > [1] OMAP4: > http://git.ti.com/gitweb/?p=rpmsg/remoteproc.git;a=commit;h=9d45dc42fbed8395d733366dbf6c0fd5ec171e2f > [2] OMAP5: > http://git.ti.com/gitweb/?p=rpmsg/remoteproc.git;a=commit;h=34f4682a91173386307b310d7f4955d46dcaaea2 > [3] DRA7: > http://git.ti.com/gitweb/?p=rpmsg/remoteproc.git;a=commit;h=2a662694437ae7192b5ef759ec40abe796d2a058 > > Technically, this data need to be added back for all OMAP2+ SoCs which > support dmtimer with any other drivers wanting to use the timers. So, I checked and these aliases already are defined on OMAP2, OMAP3, AM33xx, AM43xx, DM814x and DM816x SoCs. So, just include the OMAP4 and OMAP5 ones along with the DRA7x one. regards Suman > > regards > Suman > >> >> Signed-off-by: Tero Kristo <t-kristo@xxxxxx> >> --- >> drivers/clk/ti/clk-7xx.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/drivers/clk/ti/clk-7xx.c b/drivers/clk/ti/clk-7xx.c >> index 5208eb8..64507b8 100644 >> --- a/drivers/clk/ti/clk-7xx.c >> +++ b/drivers/clk/ti/clk-7xx.c >> @@ -792,6 +792,7 @@ >> static struct ti_dt_clk dra7xx_clks[] = { >> DT_CLK(NULL, "timer_32k_ck", "sys_32k_ck"), >> DT_CLK(NULL, "sys_clkin_ck", "timer_sys_clk_div"), >> + DT_CLK(NULL, "timer_sys_ck", "timer_sys_clk_div"), >> DT_CLK(NULL, "sys_clkin", "sys_clkin1"), >> DT_CLK(NULL, "atl_dpll_clk_mux", "atl-clkctrl:0000:24"), >> DT_CLK(NULL, "atl_gfclk_mux", "atl-clkctrl:0000:26"), >> >
