Hi,

When I check the tf-static-checks for the FIP/FCONF series I pushed for STM32MP1, I have a failure: 1d204ee4a:plat/st/common/bl2_io_storage.c: ['tools_share/firmware_image_package.h should be in project group, after system group']

First I didn't understand the issue, as the file is in project group. But the issue is due to previous include: #include <plat/common/platform.h>

As it is inside include/plat directory, it is seen as a platform include.

For me the files in this include/plat/common directory are more project, but what is your point of view?

If they are platform files, I should modify my series to reflect that.

If they are project files, then the check-include-order.py script should be updated. Here is what could be the correction:

diff --git a/script/static-checks/check-include-order.py b/script/static-checks/check-include-order.py index aaf84f1..53d355b 100755 --- a/script/static-checks/check-include-order.py +++ b/script/static-checks/check-include-order.py @@ -87,6 +87,8 @@ def inc_order_is_correct(inc_list, path, commit_hash=""): incs = collections.defaultdict(list) error_msgs = [] plat_incs = dir_include_paths("plat") | dir_include_paths("include/plat") + plat_common_incs = dir_include_paths("include/plat/common") + plat_incs.difference_update(plat_common_incs) libc_incs = dir_include_paths("include/lib/libc")

Best regards, Yann