On 10/06/2021 18:33, Lokesh Vutla wrote:
+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?
Yeah, I can take a look at these tomorrow.
-Tero
Thanks and regards,
Lokesh
