On 7/23/20 7:49 AM, Sagar Kadam wrote: > Hello Sean, > >> -----Original Message----- >> From: U-Boot <u-boot-boun...@lists.denx.de> On Behalf Of Sean Anderson >> Sent: Wednesday, July 22, 2020 9:21 PM >> To: u-boot@lists.denx.de >> Cc: Bin Meng <bmeng...@gmail.com>; Rick Chen <rickche...@gmail.com>; >> Sean Anderson <sean...@gmail.com> >> Subject: [PATCH 5/6] riscv: Update Kendryte device tree for new CLINT driver >> >> [External Email] Do not click links or attachments unless you recognize the >> sender and know the content is safe >> >> AFAIK because the K210 clock driver does not come up until after >> relocation, the clint will always use the clock-frequency parameter. >> Ideally, it should update itself after relocation to take into account the >> actual CPU frequency. >> >> Signed-off-by: Sean Anderson <sean...@gmail.com> >> --- >> >> arch/riscv/dts/k210.dtsi | 10 ++++++---- >> drivers/clk/kendryte/clk.c | 4 ++++ >> include/dt-bindings/clock/k210-sysctl.h | 1 + > > Can you please consider splitting the dt-bindings include into separate patch > so as to avoid checkpatch warning.
If you'd like. AFAIK this is mostly a kernel thing since dt-bindings often have separate maintainers than the rest of the series. Can anyone comment on whether this applies to U-Boot as well? --Sean
