+Tero, On 10/06/21 8:55 pm, Tom Rini wrote: > On Thu, Jun 10, 2021 at 12:16:50PM +0530, Lokesh Vutla wrote: > >> Hi Tom, >> Please find the PR for master branch targeted for v2021.10-next branch. >> Details about the PR are updated in the tag message. >> >> Gitlab CI report: >> https://source.denx.de/u-boot/custodians/u-boot-ti/-/pipelines/7780 >> >> >> The following changes since commit e8f720ee1707b43a0e14ade87b40a1f84baeb2f3: >> >> Merge branch '2021-06-08-kconfig-migrations' into next (2021-06-09 >> 08:19:13 -0400) >> >> are available in the Git repository at: >> >> https://source.denx.de/u-boot/custodians/u-boot-ti.git >> tags/ti-v2021.10-next >> >> for you to fetch changes up to 47a10af8f8a90b3d9e83fafb51372800171344a9: >> >> dma: ti: k3-udma: Add support for native configuration of chan/flow >> (2021-06-09 22:23:45 +0530) >> > > NAK: > ERROR: Avoid including common.h and dm.h in header files > #437: FILE: include/k3-clk.h:10: > +#include <common.h>
Interesting, I could not see this issue > > And while you're in there, there's a number of: > WARNING: Use 'if (IS_ENABLED(CONFIG...))' instead of '#if or #ifdef' where > possible > #53: FILE: drivers/clk/clk_fixed_rate.c:62: > +#if defined(CONFIG_CLK_CCF) || defined(CONFIG_SPL_CLK_CCF) > > Where checkpatch is warning about the wrong thing, that should be > CONFIG_IS_ENABLED(CLK_CCF) at minimum and then see if we can use > if (...) instead. > Tero, Can you fix it and re-post? Thanks and regards, Lokesh
