[1/7] mgmt: Add support for Mesh in the kernel

These commands and events allow User space apps to test for Mesh
support, and request incoming mesh packets be delivered and request
outbound mesh packets to be sent. This is the basis for sharing
one controller between the legacy bluetoothd daemon and the mesh
bluetooth-meshd daemon.
---
 doc/mgmt-api.txt | 180 +++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 180 insertions(+)

This is automated email and please do not reply to this email!

Dear submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=617384

---Test result---

Test Summary:
CheckPatch                    FAIL      4.85 seconds
GitLint                       PASS      2.61 seconds
Prep - Setup ELL              PASS      48.28 seconds
Build - Prep                  PASS      0.44 seconds
Build - Configure             PASS      9.45 seconds
Build - Make                  PASS      1347.83 seconds
Make Check                    PASS      12.25 seconds
Make Check w/Valgrind         PASS      465.19 seconds
Make Distcheck                PASS      243.36 seconds
Build w/ext ELL - Configure   PASS      8.66 seconds
Build w/ext ELL - Make        PASS      1383.39 seconds
Incremental Build with patchesFAIL      4627.24 seconds

Details
##############################
Test: CheckPatch - FAIL
Desc: Run checkpatch.pl script with rule in .checkpatch.conf
Output:
[1/7] mgmt: Add support for Mesh in the kernel
WARNING:REPEATED_WORD: Possible repeated word: 'command'
#252: FILE: doc/mgmt-api.txt:5143:
+	the Set Mesh Receiver command command, specifying which AD Types to

/github/workspace/src/12757782.patch total: 0 errors, 1 warnings, 189 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

/github/workspace/src/12757782.patch has style problems, please review.

NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.

[3/7] mesh: Add common MGMT command accessors
WARNING:TYPO_SPELLING: 'accross' may be misspelled - perhaps 'across'?
#69: 
a single socket, shared accross all consumers.
                        ^^^^^^^

ERROR:FUNCTION_WITHOUT_ARGS: Bad function definition - void mesh_mgmt_destroy() should probably be void mesh_mgmt_destroy(void)
#285: FILE: mesh/mesh-mgmt.c:226:
+void mesh_mgmt_destroy()

/github/workspace/src/12757783.patch total: 1 errors, 1 warnings, 243 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

/github/workspace/src/12757783.patch has style problems, please review.

NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.

[4/7] mesh: rework Mesh-IO for multiple transports
WARNING:LINE_SPACING: Missing a blank line after declarations
#334: FILE: mesh/mesh-io-mgmt.c:108:
+	const struct dup_filter *filter = a;
+	return !memcmp(filter->addr, b, 6);

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#337: FILE: mesh/mesh-io-mgmt.c:111:
+static void filter_timeout (struct l_timeout *timeout, void *user_data)

ERROR:CODE_INDENT: code indent should use tabs where possible
#367: FILE: mesh/mesh-io-mgmt.c:141:
+                                                       uint32_t instant)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#367: FILE: mesh/mesh-io-mgmt.c:141:
+                                                       uint32_t instant)$

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 16)
#465: FILE: mesh/mesh-io-mgmt.c:239:
+		if (adv[1] >= 0x29 && adv[1] <= 0x2B)
+		process_rx(io->pvt, ev->rssi, instant, addr, adv + 1, adv[0]);

WARNING:BRACES: braces {} are not necessary for single statement blocks
#1102: FILE: mesh/mesh-io.c:51:
+		if (table[i].type == type) {
+			return table[i].api;
+		}

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#1173: FILE: mesh/mesh-io.c:97:
+	if (mesh && type != MESH_IO_TYPE_GENERIC) {
[...]
-	if (l_queue_push_head(io_list, io))
[...]

WARNING:BRACES: braces {} are not necessary for single statement blocks
#1179: FILE: mesh/mesh-io.c:101:
+	else if (!pwr) {
+		api = io_api(MESH_IO_TYPE_GENERIC);
+	}

/github/workspace/src/12757784.patch total: 1 errors, 7 warnings, 1227 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

NOTE: Whitespace errors detected.
      You may wish to use scripts/cleanpatch or scripts/cleanfile

/github/workspace/src/12757784.patch has style problems, please review.

NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.


##############################
Test: Incremental Build with patches - FAIL
Desc: Incremental build per patch in the series
Output:
tools/mgmt-tester.c: In function ‘main’:
tools/mgmt-tester.c:12364:5: note: variable tracking size limit exceeded with ‘-fvar-tracking-assignments’, retrying without
12364 | int main(int argc, char *argv[])
      |     ^~~~
unit/test-avdtp.c: In function ‘main’:
unit/test-avdtp.c:766:5: note: variable tracking size limit exceeded with ‘-fvar-tracking-assignments’, retrying without
  766 | int main(int argc, char *argv[])
      |     ^~~~
unit/test-avrcp.c: In function ‘main’:
unit/test-avrcp.c:989:5: note: variable tracking size limit exceeded with ‘-fvar-tracking-assignments’, retrying without
  989 | int main(int argc, char *argv[])
      |     ^~~~
In file included from mesh/mesh-io-generic.c:26:
./mesh/mesh-mgmt.h:18:5: error: unknown type name ‘mgmt_request_func_t’
   18 |     mgmt_request_func_t callback,
      |     ^~~~~~~~~~~~~~~~~~~
./mesh/mesh-mgmt.h:19:22: error: unknown type name ‘mgmt_destroy_func_t’; did you mean ‘bt_hci_destroy_func_t’?
   19 |     void *user_data, mgmt_destroy_func_t destroy);
      |                      ^~~~~~~~~~~~~~~~~~~
      |                      bt_hci_destroy_func_t
./mesh/mesh-mgmt.h:21:5: error: unknown type name ‘mgmt_notify_func_t’
   21 |     mgmt_notify_func_t callback,
      |     ^~~~~~~~~~~~~~~~~~
./mesh/mesh-mgmt.h:22:22: error: unknown type name ‘mgmt_destroy_func_t’; did you mean ‘bt_hci_destroy_func_t’?
   22 |     void *user_data, mgmt_destroy_func_t destroy);
      |                      ^~~~~~~~~~~~~~~~~~~
      |                      bt_hci_destroy_func_t
mesh/mesh-io-generic.c: In function ‘dev_init’:
mesh/mesh-io-generic.c:448:25: error: passing argument 1 of ‘mesh_mgmt_list’ from incompatible pointer type [-Werror=incompatible-pointer-types]
  448 |   return mesh_mgmt_list(read_info, io);
      |                         ^~~~~~~~~
      |                         |
      |                         void (*)(int,  void *)
In file included from mesh/mesh-io-generic.c:26:
./mesh/mesh-mgmt.h:15:48: note: expected ‘mesh_mgmt_read_info_func_t’ {aka ‘void (*)(int,  _Bool,  _Bool,  _Bool,  void *)’} but argument is of type ‘void (*)(int,  void *)’
   15 | bool mesh_mgmt_list(mesh_mgmt_read_info_func_t cb, void *user_data);
      |                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7271: mesh/mesh-io-generic.o] Error 1
make: *** [Makefile:4307: all] Error 2




---
Regards,
Linux Bluetooth