diff for duplicates of <74dca3fa90d5cd78286281e2ced45842bee21f91.camel@mediatek.com> diff --git a/a/1.txt b/N1/1.txt index 56554d0..fb6669e 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -200,3 +200,7 @@ Nancy > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel +_______________________________________________ +linux-arm-kernel mailing list +linux-arm-kernel@lists.infradead.org +http://lists.infradead.org/mailman/listinfo/linux-arm-kernel diff --git a/a/content_digest b/N1/content_digest index 8ba9684..a63a317 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -236,6 +236,10 @@ "> _______________________________________________\n" "> linux-arm-kernel mailing list\n" "> linux-arm-kernel@lists.infradead.org\n" - > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel + "> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel\n" + "_______________________________________________\n" + "linux-arm-kernel mailing list\n" + "linux-arm-kernel@lists.infradead.org\n" + http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -8eb2afc03dea21f855bbe4624557649dfaa178f00653b1c1c1b7859500640164 +6566d5bfe2b6a235a8202693c2dc5873499e5c7ac4ff790dd5f001db50f46279
diff --git a/N2/1.bin b/N2/1.bin new file mode 100644 index 0000000..3c3e8ca --- /dev/null +++ b/N2/1.bin @@ -0,0 +1,217 @@ +<pre> +Dear Angelo, + +Sorry for late reply. + +On Fri, 2023-03-17 at 10:58 +0100, AngeloGioacchino Del Regno wrote: +> Il 17/03/23 10:52, Nancy Lin (林欣螢) ha scritto: +> > On Fri, 2023-03-17 at 10:37 +0100, AngeloGioacchino Del Regno +> > wrote: +> > > Il 17/03/23 10:03, Krzysztof Kozlowski ha scritto: +> > > > On 17/03/2023 08:55, Nancy Lin (林欣螢) wrote: +> > > > > On Thu, 2023-03-16 at 12:36 +0100, Krzysztof Kozlowski wrote: +> > > > > > On 16/03/2023 10:53, AngeloGioacchino Del Regno wrote: +> > > > > >  +> > > > > > > Hello Krzysztof, Nancy, +> > > > > > >  +> > > > > > > Since this series has reached v29, can we please reach an +> > > > > > > agreement +> > > > > > > on the bindings +> > > > > > > to use here, so that we can get this finally upstreamed? +> > > > > > >  +> > > > > > > I will put some examples to try to get this issue +> > > > > > > resolved. +> > > > > > >  +> > > > > > > ### Example 1: Constrain the number of GCE entries to +> > > > > > > *seven* +> > > > > > > array +> > > > > > > elements (7x4!) +> > > > > > >  +> > > > > > >      mediatek,gce-client-reg: +> > > > > > >        $ref: /schemas/types.yaml#/definitions/phandle- +> > > > > > > array +> > > > > > >        maxItems: 1 +> > > > > > >        description: The register of display function +> > > > > > > block to +> > > > > > > be set +> > > > > > > by gce. +> > > > > > >          There are 4 arguments in this property, gce +> > > > > > > node, +> > > > > > > subsys id, +> > > > > > > offset and +> > > > > > >          register size. The subsys id is defined in the +> > > > > > > gce +> > > > > > > header of +> > > > > > > each chips +> > > > > > >          include/dt-bindings/gce/<chip>-gce.h, mapping to +> > > > > > > the +> > > > > > > register of display +> > > > > > >          function block. +> > > > > > >        items: +> > > > > > >          minItems: 28 +> > > > > > >          maxItems: 28 +> > > > > > >          items:                     <----- this block +> > > > > > > doesn't +> > > > > > > seem to +> > > > > > > get checked :\ +> > > > > > >            - description: phandle of GCE +> > > > > > >            - description: GCE subsys id +> > > > > > >            - description: register offset +> > > > > > >            - description: register size +> > > > > >  +> > > > > > This is what we would like to have but it requires +> > > > > > exception in +> > > > > > dtschema. Thus: +> > > > > >  +> > > > > > >  +> > > > > > >  +> > > > > > > ### Example 2: Don't care about constraining the number +> > > > > > > of +> > > > > > > arguments +> > > > > > >  +> > > > > > >      mediatek,gce-client-reg: +> > > > > > >        $ref: /schemas/types.yaml#/definitions/phandle- +> > > > > > > array +> > > > > > >        maxItems: 1 +> > > > > > >        description: The register of display function +> > > > > > > block to +> > > > > > > be set +> > > > > > > by gce. +> > > > > > >          There are 4 arguments in this property, gce +> > > > > > > node, +> > > > > > > subsys id, +> > > > > > > offset and +> > > > > > >          register size. The subsys id is defined in the +> > > > > > > gce +> > > > > > > header of +> > > > > > > each chips +> > > > > > >          include/dt-bindings/gce/<chip>-gce.h, mapping to +> > > > > > > the +> > > > > > > register of display +> > > > > > >          function block. +> > > > > >  +> > > > > > use this. +> > > > > >  +> > > > > > Best regards, +> > > > > > Krzysztof +> > > > >  +> > > > >  +> > > > > Hi Krzysztof, Angelo, +> > > > >  +> > > > > Thanks for the comment. +> > > > > The Example 2 can pass dt_binding_check. +> > > > >  +> > > > > But the example in the binding has 7 items [1] and dts [2]. +> > > > > Does +> > > > > the +> > > > > "maxItems: 1" affect any other schema or dts check? +> > > >  +> > > > Ah, then it should be maxItems: 7, not 1. +> > > >  +> > >  +> > > Keep in mind for your v30: +> > >  +> > > maxItems: 7 will pass - but only if minItems is *not* 7 :-) +> > >  +> > > -> (so, do not declare minItems, as default is 1) <- +> > >  +> > > Regards, +> > > Angelo +> > >  +> >  +> > Hi Angelo, +> >  +> > I still have one message [1] when runing dt_binding_check for +> > "example +> > 2 + maxItems: 7" [2]. +> >  +> > [1] +> > /proj/mtk19347/cros/src/third_party/kernel/v5.10/Documentation/devi +> > cetr +> > ee/bindings/display/mediatek/mediatek,ethdr.example.dtb: +> > hdr-engine@1c114000: mediatek,gce-client-reg: [[4294967295, 7, +> > 16384, +> > 4096, 4294967295, 7, 20480, 4096, 4294967295, 7, 28672, 4096, +> > 4294967295, 7, 36864, 4096, 4294967295, 7, 40960, 4096, 4294967295, +> > 7, +> > 45056, 4096, 4294967295, 7, 49152, 4096]] is too short +> >  +> >  +> > [2] +> >     mediatek,gce-client-reg: +> >       $ref: /schemas/types.yaml#/definitions/phandle-array +> >       maxItems: 7 +> >       description: The register of display function block to be set +> > by +> > gce. +> >         There are 4 arguments in this property, gce node, subsys +> > id, +> > offset and +> >         register size. The subsys id is defined in the gce header +> > of +> > each chips +> >         include/dt-bindings/gce/<chip>-gce.h, mapping to the +> > register of +> > display +> >         function block. +> >  +>  +> Maybe I'm wrong about the "do not declare minItems"... try with +>  +> minItems: 1 +> maxItems: 7 +>  +>  +> ...does it work now? +>  + +Yes, It works well with "example2 + minItems:1 + maxItems: 7" [1] + +[1] +  mediatek,gce-client-reg: +    $ref: /schemas/types.yaml#/definitions/phandle-array +    minItems: 1 +    maxItems: 7 +    description: The register of display function block to be set by +gce. +      There are 4 arguments in this property, gce node, subsys id, +offset and +      register size. The subsys id is defined in the gce header of each +chips +      include/dt-bindings/gce/<chip>-gce.h, mapping to the register of +display +      function block. + +Regards, +Nancy + + +> > Regards, +> > Nancy +> >  +> >  +> > > > Best regards, +> > > > Krzysztof +> > > >  +>  +>  +>  +>  +> _______________________________________________ +> linux-arm-kernel mailing list +> linux-arm-kernel@lists.infradead.org +> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel + +</pre><!--type:text--><!--{--><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><!--}--> diff --git a/N2/2.hdr b/N2/2.hdr new file mode 100644 index 0000000..45d76fc --- /dev/null +++ b/N2/2.hdr @@ -0,0 +1,3 @@ +Content-Type: text/plain; + charset="utf-8" +Content-Transfer-Encoding: base64 diff --git a/a/1.txt b/N2/2.txt similarity index 100% rename from a/1.txt rename to N2/2.txt diff --git a/a/content_digest b/N2/content_digest index 8ba9684..4718503 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -21,19 +21,236 @@ chunkuang.hu@kernel.org <chunkuang.hu@kernel.org> robh+dt@kernel.org <robh+dt@kernel.org> " krzysztof.kozlowski+dt@linaro.org <krzysztof.kozlowski+dt@linaro.org>\0" - "Cc\0linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org>" - linux-mediatek@lists.infradead.org <linux-mediatek@lists.infradead.org> + "Cc\0devicetree@vger.kernel.org <devicetree@vger.kernel.org>" + Project_Global_Chrome_Upstream_Group <Project_Global_Chrome_Upstream_Group@mediatek.com> " Singo Chang (\345\274\265\350\210\210\345\234\213) <Singo.Chang@mediatek.com>" - nathan@kernel.org <nathan@kernel.org> - devicetree@vger.kernel.org <devicetree@vger.kernel.org> - daniel@ffwll.ch <daniel@ffwll.ch> - " CK Hu (\350\203\241\344\277\212\345\205\211) <ck.hu@mediatek.com>" + ndesaulniers@google.com <ndesaulniers@google.com> + linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org> dri-devel@lists.freedesktop.org <dri-devel@lists.freedesktop.org> - Project_Global_Chrome_Upstream_Group <Project_Global_Chrome_Upstream_Group@mediatek.com> - linux-arm-kernel@lists.infradead.org <linux-arm-kernel@lists.infradead.org> + nathan@kernel.org <nathan@kernel.org> clang-built-linux@googlegroups.com <clang-built-linux@googlegroups.com> - " ndesaulniers@google.com <ndesaulniers@google.com>\0" - "\00:1\0" + linux-mediatek@lists.infradead.org <linux-mediatek@lists.infradead.org> + " linux-arm-kernel@lists.infradead.org <linux-arm-kernel@lists.infradead.org>\0" + "\01:1\0" + "b\0" + "<pre>\r\n" + "Dear Angelo,\r\n" + "\r\n" + "Sorry for late reply.\r\n" + "\r\n" + "On Fri, 2023-03-17 at 10:58 +0100, AngeloGioacchino Del Regno wrote:\r\n" + "> Il 17/03/23 10:52, Nancy Lin (林欣螢) ha scritto:\r\n" + "> > On Fri, 2023-03-17 at 10:37 +0100, AngeloGioacchino Del Regno\r\n" + "> > wrote:\r\n" + "> > > Il 17/03/23 10:03, Krzysztof Kozlowski ha scritto:\r\n" + "> > > > On 17/03/2023 08:55, Nancy Lin (林欣螢) wrote:\r\n" + "> > > > > On Thu, 2023-03-16 at 12:36 +0100, Krzysztof Kozlowski wrote:\r\n" + "> > > > > > On 16/03/2023 10:53, AngeloGioacchino Del Regno wrote:\r\n" + "> > > > > > \r\n" + "> > > > > > > Hello Krzysztof, Nancy,\r\n" + "> > > > > > > \r\n" + "> > > > > > > Since this series has reached v29, can we please reach an\r\n" + "> > > > > > > agreement\r\n" + "> > > > > > > on the bindings\r\n" + "> > > > > > > to use here, so that we can get this finally upstreamed?\r\n" + "> > > > > > > \r\n" + "> > > > > > > I will put some examples to try to get this issue\r\n" + "> > > > > > > resolved.\r\n" + "> > > > > > > \r\n" + "> > > > > > > ### Example 1: Constrain the number of GCE entries to\r\n" + "> > > > > > > *seven*\r\n" + "> > > > > > > array\r\n" + "> > > > > > > elements (7x4!)\r\n" + "> > > > > > > \r\n" + "> > > > > > >      mediatek,gce-client-reg:\r\n" + "> > > > > > >        $ref: /schemas/types.yaml#/definitions/phandle-\r\n" + "> > > > > > > array\r\n" + "> > > > > > >        maxItems: 1\r\n" + "> > > > > > >        description: The register of display function\r\n" + "> > > > > > > block to\r\n" + "> > > > > > > be set\r\n" + "> > > > > > > by gce.\r\n" + "> > > > > > >          There are 4 arguments in this property, gce\r\n" + "> > > > > > > node,\r\n" + "> > > > > > > subsys id,\r\n" + "> > > > > > > offset and\r\n" + "> > > > > > >          register size. The subsys id is defined in the\r\n" + "> > > > > > > gce\r\n" + "> > > > > > > header of\r\n" + "> > > > > > > each chips\r\n" + "> > > > > > >          include/dt-bindings/gce/<chip>-gce.h, mapping to\r\n" + "> > > > > > > the\r\n" + "> > > > > > > register of display\r\n" + "> > > > > > >          function block.\r\n" + "> > > > > > >        items:\r\n" + "> > > > > > >          minItems: 28\r\n" + "> > > > > > >          maxItems: 28\r\n" + "> > > > > > >          items:                     <----- this block\r\n" + "> > > > > > > doesn't\r\n" + "> > > > > > > seem to\r\n" + "> > > > > > > get checked :\\r\n" + "> > > > > > >            - description: phandle of GCE\r\n" + "> > > > > > >            - description: GCE subsys id\r\n" + "> > > > > > >            - description: register offset\r\n" + "> > > > > > >            - description: register size\r\n" + "> > > > > > \r\n" + "> > > > > > This is what we would like to have but it requires\r\n" + "> > > > > > exception in\r\n" + "> > > > > > dtschema. Thus:\r\n" + "> > > > > > \r\n" + "> > > > > > > \r\n" + "> > > > > > > \r\n" + "> > > > > > > ### Example 2: Don't care about constraining the number\r\n" + "> > > > > > > of\r\n" + "> > > > > > > arguments\r\n" + "> > > > > > > \r\n" + "> > > > > > >      mediatek,gce-client-reg:\r\n" + "> > > > > > >        $ref: /schemas/types.yaml#/definitions/phandle-\r\n" + "> > > > > > > array\r\n" + "> > > > > > >        maxItems: 1\r\n" + "> > > > > > >        description: The register of display function\r\n" + "> > > > > > > block to\r\n" + "> > > > > > > be set\r\n" + "> > > > > > > by gce.\r\n" + "> > > > > > >          There are 4 arguments in this property, gce\r\n" + "> > > > > > > node,\r\n" + "> > > > > > > subsys id,\r\n" + "> > > > > > > offset and\r\n" + "> > > > > > >          register size. The subsys id is defined in the\r\n" + "> > > > > > > gce\r\n" + "> > > > > > > header of\r\n" + "> > > > > > > each chips\r\n" + "> > > > > > >          include/dt-bindings/gce/<chip>-gce.h, mapping to\r\n" + "> > > > > > > the\r\n" + "> > > > > > > register of display\r\n" + "> > > > > > >          function block.\r\n" + "> > > > > > \r\n" + "> > > > > > use this.\r\n" + "> > > > > > \r\n" + "> > > > > > Best regards,\r\n" + "> > > > > > Krzysztof\r\n" + "> > > > > \r\n" + "> > > > > \r\n" + "> > > > > Hi Krzysztof, Angelo,\r\n" + "> > > > > \r\n" + "> > > > > Thanks for the comment.\r\n" + "> > > > > The Example 2 can pass dt_binding_check.\r\n" + "> > > > > \r\n" + "> > > > > But the example in the binding has 7 items [1] and dts [2].\r\n" + "> > > > > Does\r\n" + "> > > > > the\r\n" + "> > > > > "maxItems: 1" affect any other schema or dts check?\r\n" + "> > > > \r\n" + "> > > > Ah, then it should be maxItems: 7, not 1.\r\n" + "> > > > \r\n" + "> > > \r\n" + "> > > Keep in mind for your v30:\r\n" + "> > > \r\n" + "> > > maxItems: 7 will pass - but only if minItems is *not* 7 :-)\r\n" + "> > > \r\n" + "> > > -> (so, do not declare minItems, as default is 1) <-\r\n" + "> > > \r\n" + "> > > Regards,\r\n" + "> > > Angelo\r\n" + "> > > \r\n" + "> > \r\n" + "> > Hi Angelo,\r\n" + "> > \r\n" + "> > I still have one message [1] when runing dt_binding_check for\r\n" + "> > "example\r\n" + "> > 2 + maxItems: 7" [2].\r\n" + "> > \r\n" + "> > [1]\r\n" + "> > /proj/mtk19347/cros/src/third_party/kernel/v5.10/Documentation/devi\r\n" + "> > cetr\r\n" + "> > ee/bindings/display/mediatek/mediatek,ethdr.example.dtb:\r\n" + "> > hdr-engine@1c114000: mediatek,gce-client-reg: [[4294967295, 7,\r\n" + "> > 16384,\r\n" + "> > 4096, 4294967295, 7, 20480, 4096, 4294967295, 7, 28672, 4096,\r\n" + "> > 4294967295, 7, 36864, 4096, 4294967295, 7, 40960, 4096, 4294967295,\r\n" + "> > 7,\r\n" + "> > 45056, 4096, 4294967295, 7, 49152, 4096]] is too short\r\n" + "> > \r\n" + "> > \r\n" + "> > [2]\r\n" + "> >     mediatek,gce-client-reg:\r\n" + "> >       $ref: /schemas/types.yaml#/definitions/phandle-array\r\n" + "> >       maxItems: 7\r\n" + "> >       description: The register of display function block to be set\r\n" + "> > by\r\n" + "> > gce.\r\n" + "> >         There are 4 arguments in this property, gce node, subsys\r\n" + "> > id,\r\n" + "> > offset and\r\n" + "> >         register size. The subsys id is defined in the gce header\r\n" + "> > of\r\n" + "> > each chips\r\n" + "> >         include/dt-bindings/gce/<chip>-gce.h, mapping to the\r\n" + "> > register of\r\n" + "> > display\r\n" + "> >         function block.\r\n" + "> > \r\n" + "> \r\n" + "> Maybe I'm wrong about the "do not declare minItems"... try with\r\n" + "> \r\n" + "> minItems: 1\r\n" + "> maxItems: 7\r\n" + "> \r\n" + "> \r\n" + "> ...does it work now?\r\n" + "> \r\n" + "\r\n" + "Yes, It works well with "example2 + minItems:1 + maxItems: 7" [1]\r\n" + "\r\n" + "[1]\r\n" + "  mediatek,gce-client-reg:\r\n" + "    $ref: /schemas/types.yaml#/definitions/phandle-array\r\n" + "    minItems: 1\r\n" + "    maxItems: 7\r\n" + "    description: The register of display function block to be set by\r\n" + "gce.\r\n" + "      There are 4 arguments in this property, gce node, subsys id,\r\n" + "offset and\r\n" + "      register size. The subsys id is defined in the gce header of each\r\n" + "chips\r\n" + "      include/dt-bindings/gce/<chip>-gce.h, mapping to the register of\r\n" + "display\r\n" + "      function block.\r\n" + "\r\n" + "Regards,\r\n" + "Nancy\r\n" + "\r\n" + "\r\n" + "> > Regards,\r\n" + "> > Nancy\r\n" + "> > \r\n" + "> > \r\n" + "> > > > Best regards,\r\n" + "> > > > Krzysztof\r\n" + "> > > > \r\n" + "> \r\n" + "> \r\n" + "> \r\n" + "> \r\n" + "> _______________________________________________\r\n" + "> linux-arm-kernel mailing list\r\n" + "> linux-arm-kernel@lists.infradead.org\r\n" + "> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel\r\n" + "\r\n" + "</pre><!--type:text--><!--{--><pre>************* MEDIATEK Confidentiality Notice ********************\r\n" + "The information contained in this e-mail message (including any \r\n" + "attachments) may be confidential, proprietary, privileged, or otherwise\r\n" + "exempt from disclosure under applicable laws. It is intended to be \r\n" + "conveyed only to the designated recipient(s). Any use, dissemination, \r\n" + "distribution, printing, retaining or copying of this e-mail (including its \r\n" + "attachments) by unintended recipient(s) is strictly prohibited and may \r\n" + "be unlawful. If you are not an intended recipient of this e-mail, or believe \r\n" + "that you have received this e-mail in error, please notify the sender \r\n" + "immediately (by replying to this e-mail), delete any and all copies of \r\n" + "this e-mail (including any attachments) from your system, and do not\r\n" + "disclose the content of this e-mail to any other person. Thank you!\r\n" + </pre><!--}--> + "\01:2\0" "b\0" "Dear Angelo,\n" "\n" @@ -238,4 +455,4 @@ "> linux-arm-kernel@lists.infradead.org\n" > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -8eb2afc03dea21f855bbe4624557649dfaa178f00653b1c1c1b7859500640164 +a1c0d5e467a793928338dcec1478fa332c67b54ca80b71ea1dac7ea851ddeab4
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.