All of lore.kernel.org
 help / color / mirror / Atom feed
diff for duplicates of <001901d95263$eac4b460$c04e1d20$@samsung.com>

diff --git a/a/1.txt b/N1/1.txt
index bb47b22..8499a5c 100644
--- a/a/1.txt
+++ b/N1/1.txt
@@ -61,65 +61,57 @@
 > >>>                 };
 > >>> There is no iommu-names property for compatible
 > "samsung,exynos3250-
-> >> mfc, samsung,mfc-v7", that’s why I kept
-> >>>         iommus:
-> >>>           minItems: 1
-> >>>           maxItems: 2
-> >>> I would even go with below if you agree?
-> >>>         iommus:
-> >>>           minItems: 1
-> >>>           maxItems: 2
-> >>>         iommus-names: false
-> >>
-> >> No, I don't agree. Why do you need two IOMMUs for Exynos3250?
-> >>>
-> > Here IOMMUs minItems:1 is meant for Exynos3250 while IOMMUs
-> maxItems:2 meant for samsung,mfc-v5(Exynos 4).
-> > As you suggested
-> > iommus:
-> >   maxItems: 1
-> > iommu-names: false
-> >
-> > won't fit here.
-> > Since IOMMUs maxItems is 2 here for Exynos4. Giving maxItems:2 alone
-> will also break the bindings. Thanks why suggested
-> >          iommus:
-> >            minItems: 1
-> >            maxItems: 2
-> 
-> Are you sure you are putting this in correct place? This is if:then for
-> Exynos3250, isn't it?
-> 
-Its not only for Exynos3250, its along with Exynos4 since their clock properties are same
-allOf:
-  - if:
-      properties:
-        compatible:
-          contains:
-            enum:
-              - samsung,mfc-v5        #Exynos4
-              - samsung,exynos3250-mfc
-    then:
-      properties:
-        clocks:
-          maxItems: 2
-        clock-names:
-          items:
-            - const: mfc
-            - const: sclk_mfc
-        iommus:
-          minItems: 1
-          maxItems: 2
-
-or do I keep different if:then condition for both Exynos3250 and Exynos4 for IOMMUs property ?
-
-> Best regards,
+> >> mfc, samsung,mfc-v7", that’s why I kept\r
+> >>>         iommus:\r
+> >>>           minItems: 1\r
+> >>>           maxItems: 2\r
+> >>> I would even go with below if you agree?\r
+> >>>         iommus:\r
+> >>>           minItems: 1\r
+> >>>           maxItems: 2\r
+> >>>         iommus-names: false\r
+> >>\r
+> >> No, I don't agree. Why do you need two IOMMUs for Exynos3250?\r
+> >>>\r
+> > Here IOMMUs minItems:1 is meant for Exynos3250 while IOMMUs\r
+> maxItems:2 meant for samsung,mfc-v5(Exynos 4).\r
+> > As you suggested\r
+> > iommus:\r
+> >   maxItems: 1\r
+> > iommu-names: false\r
+> >\r
+> > won't fit here.\r
+> > Since IOMMUs maxItems is 2 here for Exynos4. Giving maxItems:2 alone\r
+> will also break the bindings. Thanks why suggested\r
+> >          iommus:\r
+> >            minItems: 1\r
+> >            maxItems: 2\r
+> \r
+> Are you sure you are putting this in correct place? This is if:then for\r
+> Exynos3250, isn't it?\r
+> \r
+Its not only for Exynos3250, its along with Exynos4 since their clock properties are same\r
+allOf:\r
+  - if:\r
+      properties:\r
+        compatible:\r
+          contains:\r
+            enum:\r
+              - samsung,mfc-v5        #Exynos4\r
+              - samsung,exynos3250-mfc\r
+    then:\r
+      properties:\r
+        clocks:\r
+          maxItems: 2\r
+        clock-names:\r
+          items:\r
+            - const: mfc\r
+            - const: sclk_mfc\r
+        iommus:\r
+          minItems: 1\r
+          maxItems: 2\r
+\r
+or do I keep different if:then condition for both Exynos3250 and Exynos4 for IOMMUs property ?\r
+\r
+> Best regards,\r
 > Krzysztof
-
-
-
-
-_______________________________________________
-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 98f7f57..8bd9ade 100644
--- a/a/content_digest
+++ b/N1/content_digest
@@ -96,67 +96,59 @@
  "> >>>                 };\n"
  "> >>> There is no iommu-names property for compatible\n"
  "> \"samsung,exynos3250-\n"
- "> >> mfc, samsung,mfc-v7\", that\342\200\231s why I kept\n"
- "> >>>         iommus:\n"
- "> >>>           minItems: 1\n"
- "> >>>           maxItems: 2\n"
- "> >>> I would even go with below if you agree?\n"
- "> >>>         iommus:\n"
- "> >>>           minItems: 1\n"
- "> >>>           maxItems: 2\n"
- "> >>>         iommus-names: false\n"
- "> >>\n"
- "> >> No, I don't agree. Why do you need two IOMMUs for Exynos3250?\n"
- "> >>>\n"
- "> > Here IOMMUs minItems:1 is meant for Exynos3250 while IOMMUs\n"
- "> maxItems:2 meant for samsung,mfc-v5(Exynos 4).\n"
- "> > As you suggested\n"
- "> > iommus:\n"
- "> >   maxItems: 1\n"
- "> > iommu-names: false\n"
- "> >\n"
- "> > won't fit here.\n"
- "> > Since IOMMUs maxItems is 2 here for Exynos4. Giving maxItems:2 alone\n"
- "> will also break the bindings. Thanks why suggested\n"
- "> >          iommus:\n"
- "> >            minItems: 1\n"
- "> >            maxItems: 2\n"
- "> \n"
- "> Are you sure you are putting this in correct place? This is if:then for\n"
- "> Exynos3250, isn't it?\n"
- "> \n"
- "Its not only for Exynos3250, its along with Exynos4 since their clock properties are same\n"
- "allOf:\n"
- "  - if:\n"
- "      properties:\n"
- "        compatible:\n"
- "          contains:\n"
- "            enum:\n"
- "              - samsung,mfc-v5        #Exynos4\n"
- "              - samsung,exynos3250-mfc\n"
- "    then:\n"
- "      properties:\n"
- "        clocks:\n"
- "          maxItems: 2\n"
- "        clock-names:\n"
- "          items:\n"
- "            - const: mfc\n"
- "            - const: sclk_mfc\n"
- "        iommus:\n"
- "          minItems: 1\n"
- "          maxItems: 2\n"
- "\n"
- "or do I keep different if:then condition for both Exynos3250 and Exynos4 for IOMMUs property ?\n"
- "\n"
- "> Best regards,\n"
- "> Krzysztof\n"
- "\n"
- "\n"
- "\n"
- "\n"
- "_______________________________________________\n"
- "linux-arm-kernel mailing list\n"
- "linux-arm-kernel@lists.infradead.org\n"
- http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
+ "> >> mfc, samsung,mfc-v7\", that\342\200\231s why I kept\r\n"
+ "> >>>         iommus:\r\n"
+ "> >>>           minItems: 1\r\n"
+ "> >>>           maxItems: 2\r\n"
+ "> >>> I would even go with below if you agree?\r\n"
+ "> >>>         iommus:\r\n"
+ "> >>>           minItems: 1\r\n"
+ "> >>>           maxItems: 2\r\n"
+ "> >>>         iommus-names: false\r\n"
+ "> >>\r\n"
+ "> >> No, I don't agree. Why do you need two IOMMUs for Exynos3250?\r\n"
+ "> >>>\r\n"
+ "> > Here IOMMUs minItems:1 is meant for Exynos3250 while IOMMUs\r\n"
+ "> maxItems:2 meant for samsung,mfc-v5(Exynos 4).\r\n"
+ "> > As you suggested\r\n"
+ "> > iommus:\r\n"
+ "> >   maxItems: 1\r\n"
+ "> > iommu-names: false\r\n"
+ "> >\r\n"
+ "> > won't fit here.\r\n"
+ "> > Since IOMMUs maxItems is 2 here for Exynos4. Giving maxItems:2 alone\r\n"
+ "> will also break the bindings. Thanks why suggested\r\n"
+ "> >          iommus:\r\n"
+ "> >            minItems: 1\r\n"
+ "> >            maxItems: 2\r\n"
+ "> \r\n"
+ "> Are you sure you are putting this in correct place? This is if:then for\r\n"
+ "> Exynos3250, isn't it?\r\n"
+ "> \r\n"
+ "Its not only for Exynos3250, its along with Exynos4 since their clock properties are same\r\n"
+ "allOf:\r\n"
+ "  - if:\r\n"
+ "      properties:\r\n"
+ "        compatible:\r\n"
+ "          contains:\r\n"
+ "            enum:\r\n"
+ "              - samsung,mfc-v5        #Exynos4\r\n"
+ "              - samsung,exynos3250-mfc\r\n"
+ "    then:\r\n"
+ "      properties:\r\n"
+ "        clocks:\r\n"
+ "          maxItems: 2\r\n"
+ "        clock-names:\r\n"
+ "          items:\r\n"
+ "            - const: mfc\r\n"
+ "            - const: sclk_mfc\r\n"
+ "        iommus:\r\n"
+ "          minItems: 1\r\n"
+ "          maxItems: 2\r\n"
+ "\r\n"
+ "or do I keep different if:then condition for both Exynos3250 and Exynos4 for IOMMUs property ?\r\n"
+ "\r\n"
+ "> Best regards,\r\n"
+ > Krzysztof
 
-1ca4bd2a8de40ed0e6acec4cb9583b161d90620191c422fce622f7a44e632483
+20d88bb4c60f559bdc7df202f0fa3d83eaff5e586e85c017992f5a2abb866207

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.