<html><body><p> <pre> Hi Krzysztof, Thanks for the reviews. On Wed, 2024-12-11 at 10:39 +0100, Krzysztof Kozlowski wrote: > External email : Please do not click links or open attachments until > you have verified the sender or the content. > > > On Wed, Dec 11, 2024 at 11:22:50AM +0800, Jason-JH.Lin wrote: > > Add compatible name and iommus property for MT8196. > > > > Signed-off-by: Jason-JH.Lin <jason-jh.lin@mediatek.com> > > --- > > .../devicetree/bindings/mailbox/mediatek,gce-mailbox.yaml | 4 > > ++++ > > 1 file changed, 4 insertions(+) > > Headers are never separate from the bindings patch... Do you mean I need to merge [PATCH 1/8] and [PATCH 2/8] into 1 patch? > > And you reveived exactly the same comments before which you just > ignored. > I checked the dt_binding_check history and checkpatch history, but I didn't see the WARNING like `header is separated from bindings`. But I did see the WARNING of maintainers updating that appeared every time when I added the new file. Here is my history of dt_binding_check: make dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/mailbox/mediatek,gce- mailbox.yaml O=/tmp/src_kernel/kernel/out -C kernel/linux-next -k make[1]: Entering directory '/tmp/src_kernel/kernel/kernel/linux-next' make[2]: Entering directory '/tmp/src_kernel/kernel/out' HOSTCC scripts/basic/fixdep HOSTCC scripts/dtc/dtc.o HOSTCC scripts/dtc/data.o HOSTCC scripts/dtc/srcpos.o HOSTCC scripts/dtc/flattree.o HOSTCC scripts/dtc/fstree.o HOSTCC scripts/dtc/livetree.o HOSTCC scripts/dtc/checks.o HOSTCC scripts/dtc/util.o HOSTCC scripts/dtc/treesource.o LEX scripts/dtc/dtc-lexer.lex.c YACC scripts/dtc/dtc-parser.tab.[ch] HOSTCC scripts/dtc/dtc-lexer.lex.o HOSTCC scripts/dtc/dtc-parser.tab.o SCHEMA Documentation/devicetree/bindings/processed-schema.json HOSTLD scripts/dtc/dtc CHKDT /tmp/src_kernel/kernel/kernel/linux- next/Documentation/devicetree/bindings LINT /tmp/src_kernel/kernel/kernel/linux- next/Documentation/devicetree/bindings DTEX Documentation/devicetree/bindings/mailbox/mediatek,gce- mailbox.example.dts DTC [C] Documentation/devicetree/bindings/mailbox/mediatek,gce- mailbox.example.dtb make[2]: Leaving directory '/tmp/src_kernel/kernel/out' make[1]: Leaving directory '/tmp/src_kernel/kernel/kernel/linux-next' [09:53:30] - - - - - - - - - -> [ OK ] KERN_DIR=kernel/linux-next make -k -j32 dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/mailbox/mediatek,gce- mailbox.yaml Here is my history of checkpatch: ~/linux$ ./scripts/checkpatch.pl --strict --ignore=GERRIT_CHANGE_ID -- show-types 000* ----------------------- 0000-cover-letter.patch ----------------------- total: 0 errors, 0 warnings, 0 checks, 0 lines checked 0000-cover-letter.patch has no obvious style problems and is ready for submission. --------------------------------------------------------------- 0001-dt-bindings-mailbox-mediatek-Add-GCE-header-file-for.patch --------------------------------------------------------------- Traceback (most recent call last): File "scripts/spdxcheck.py", line 6, in <module> from ply import lex, yacc ModuleNotFoundError: No module named 'ply' WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #17: new file mode 100644 total: 0 errors, 1 warnings, 0 checks, 1439 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. 0001-dt-bindings-mailbox-mediatek-Add-GCE-header-file-for.patch has style problems, please review. --------------------------------------------------------------- 0002-dt-bindings-mailbox-mediatek-Add-MT8196-support-for-.patch --------------------------------------------------------------- total: 0 errors, 0 warnings, 0 checks, 16 lines checked 0002-dt-bindings-mailbox-mediatek-Add-MT8196-support-for-.patch has no obvious style problems and is ready for submission. --------------------------------------------------------------- Did I miss some configuration? Regards, Jason-JH.Lin > NAK. > > Best regards, > Krzysztof > </pre> </p></body></html><pre>************* MEDIATEK Confidentiality Notice ******************** The information contained in this e-mail message (including any attachments) may be confidential, proprietary, privileged, or otherwise exempt from disclosure under applicable laws. It is intended to be conveyed only to the designated recipient(s). Any use, dissemination, distribution, printing, retaining or copying of this e-mail (including its attachments) by unintended recipient(s) is strictly prohibited and may be unlawful. If you are not an intended recipient of this e-mail, or believe that you have received this e-mail in error, please notify the sender immediately (by replying to this e-mail), delete any and all copies of this e-mail (including any attachments) from your system, and do not disclose the content of this e-mail to any other person. Thank you! </pre>