diff for duplicates of <20160614002550.GA4882@voom.fritz.box> diff --git a/a/1.txt b/N1/1.txt index a961873..c917981 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -9,7 +9,7 @@ On Fri, Jun 10, 2016 at 05:28:11PM +0300, Pantelis Antoniou wrote: > > > > Add a new function to merge overlays with a base device tree. > > -> > Signed-off-by: Maxime Ripard <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> +> > Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com> > > --- > > include/libfdt.h | 30 ++++ > > lib/libfdt/Makefile | 2 +- @@ -41,7 +41,7 @@ On Fri, Jun 10, 2016 at 05:28:11PM +0300, Pantelis Antoniou wrote: > for booting. A valid strategy would be to scribble over the DT magic > number so that that blob is invalidated. > -> What are the other people’s opinion on this? +> What are the other people?s opinion on this? > > > + * returns: > > + * 0, on success @@ -446,7 +446,7 @@ On Fri, Jun 10, 2016 at 05:28:11PM +0300, Pantelis Antoniou wrote: > > + continue; > > + > -> This is incorrect. The use of “fragment” is a convention only. +> This is incorrect. The use of ?fragment? is a convention only. > The real test whether the node is an overlay fragment is that > it contains a target property. @@ -462,7 +462,7 @@ a fragment also happens to have a property called "target". > > + return target; > > + > -> So you could do ‘if (target < 0) continue;’ or handle a more complex error code. +> So you could do ?if (target < 0) continue;? or handle a more complex error code. > > > + overlay = fdt_subnode_offset(dto, fragment, "__overlay__"); > > + if (overlay < 0) @@ -517,14 +517,14 @@ a fragment also happens to have a property called "target". > > + return ret; > > +} > -> I would caution against the liberal use of malloc in libfdt. We’re possibly running in a constrained +> I would caution against the liberal use of malloc in libfdt. We?re possibly running in a constrained > environment; a custom extents based (non freeing) allocator should be better. > > We need some figures about memory consumption when this is enabled, and a CONFIG option to disable it. > > Regards > -> — Pantelis +> ? Pantelis > > @@ -533,3 +533,10 @@ David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson +-------------- next part -------------- +A non-text attachment was scrubbed... +Name: signature.asc +Type: application/pgp-signature +Size: 819 bytes +Desc: not available +URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160614/c3d022f2/attachment.sig> diff --git a/a/2.bin b/a/2.bin deleted file mode 100644 index 3f42bb6..0000000 --- a/a/2.bin +++ /dev/null @@ -1,17 +0,0 @@ ------BEGIN PGP SIGNATURE----- -Version: GnuPG v1 - -iQIcBAEBAgAGBQJXX08OAAoJEGw4ysog2bOSJckQAJ8LFgv+m/LKJhhz3B3Cwfto -PI8/or3EQ2awtdGziVXVcbII3D23e+BI01ANKT4nfg3NJofzPjGoK7dOUkEru5u6 -Us+mXQQwQW0py3RBFrUk8WSHtI1EwtyrL8ZQ7PS22K/RcYrUsSBpZuswg2NgvKtT -1p6rLSbYOnnRxaH+60vopLetyO2RoMFBf82WODjtyr2xAc78wrhO8sOLGXUQHAW3 -yPmR9qyYI51UIXUPGA0m+IYtcN6s86pSMNu3/6lvEJG7Hhx6stsf0Qjcv8pHsM+V -ljACYRQG1prJPtE1+kwPr5joWycAQDPG5vRwX+FvQk8N6rMJDWu00dYRIRE8o09y -syo6W2XR0yhQ3k4wSl3+Hi7xzKhtTMEhEPWGD5fxe6LkI8iJ+sDfMGR82/oefhcN -4jmoNwuE3DFCfGx3hZu09kt+ahbUiutdsOt3hQoQSsw1CwdhU06qUthweA2HIp6o -zy0ZNy9mly+q9RWBMxy8hEQllkp81/2GzyaI5kUra/x9ZOXLlPYCEcjOqBqwa8TM -X9rkuUgllPN5CK+Tk39oBhUb3HS/fXwhd/Fpqh05jeHwk71S6wMfBFhHeYrvmVM6 -XcGsSHkJ8hSBF/UKrgDHIeWj/NUhA78rqa0oy8Ym4iO2HowR300em1QXxd+9eHQB -b+9k4+u9g23cGYKQWNVN -=Oi/m ------END PGP SIGNATURE----- diff --git a/a/2.hdr b/a/2.hdr deleted file mode 100644 index 5e5352c..0000000 --- a/a/2.hdr +++ /dev/null @@ -1 +0,0 @@ -Content-Type: application/pgp-signature; name="signature.asc" diff --git a/a/content_digest b/N1/content_digest index eaf3938..15eae32 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,23 +1,11 @@ "ref\01464340402-2249-1-git-send-email-maxime.ripard@free-electrons.com\0" "ref\01464340402-2249-8-git-send-email-maxime.ripard@free-electrons.com\0" "ref\034997AD3-B621-4823-920E-22E4A6F0E0D1@konsulko.com\0" - "ref\034997AD3-B621-4823-920E-22E4A6F0E0D1-OWPKS81ov/FWk0Htik3J/w@public.gmane.org\0" - "From\0David Gibson <david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+@public.gmane.org>\0" - "Subject\0Re: [PATCH v2 7/9] libfdt: Add overlay application function\0" + "From\0David Gibson <david@gibson.dropbear.id.au>\0" + "Subject\0[U-Boot] [PATCH v2 7/9] libfdt: Add overlay application function\0" "Date\0Tue, 14 Jun 2016 10:25:50 +1000\0" - "To\0Pantelis Antoniou <pantelis.antoniou-OWPKS81ov/FWk0Htik3J/w@public.gmane.org>\0" - "Cc\0Maxime Ripard <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>" - Simon Glass <sjg-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org> - Boris Brezillon <boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> - Alexander Kaplan <alex-MflLfwwFzuz+yO7R74ARew@public.gmane.org> - Thomas Petazzoni <thomas.petazzoni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> - devicetree-compiler-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - " Antoine T\303\251nart <antoine.tenart-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>" - Hans de Goede <hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> - Tom Rini <trini-OWPKS81ov/FWk0Htik3J/w@public.gmane.org> - u-boot-0aAXYlwwYIKGBzrmiIFOJg@public.gmane.org - " Stefan Agner <stefan-XLVq0VzYD2Y@public.gmane.org>\0" - "\01:1\0" + "To\0u-boot@lists.denx.de\0" + "\00:1\0" "b\0" "On Fri, Jun 10, 2016 at 05:28:11PM +0300, Pantelis Antoniou wrote:\n" "> Hi Maxime,\n" @@ -30,7 +18,7 @@ "> > \n" "> > Add a new function to merge overlays with a base device tree.\n" "> > \n" - "> > Signed-off-by: Maxime Ripard <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>\n" + "> > Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>\n" "> > ---\n" "> > include/libfdt.h | 30 ++++\n" "> > lib/libfdt/Makefile | 2 +-\n" @@ -62,7 +50,7 @@ "> for booting. A valid strategy would be to scribble over the DT magic\n" "> number so that that blob is invalidated.\n" "> \n" - "> What are the other people\342\200\231s opinion on this?\n" + "> What are the other people?s opinion on this?\n" "> \n" "> > + * returns:\n" "> > + *\t0, on success\n" @@ -467,7 +455,7 @@ "> > +\t\t\tcontinue;\n" "> > +\n" "> \n" - "> This is incorrect. The use of \342\200\234fragment\342\200\235 is a convention only.\n" + "> This is incorrect. The use of ?fragment? is a convention only.\n" "> The real test whether the node is an overlay fragment is that\n" "> it contains a target property.\n" "\n" @@ -483,7 +471,7 @@ "> > +\t\t\treturn target;\n" "> > +\n" "> \n" - "> So you could do \342\200\230if (target < 0) continue;\342\200\231 or handle a more complex error code.\n" + "> So you could do ?if (target < 0) continue;? or handle a more complex error code.\n" "> \n" "> > +\t\toverlay = fdt_subnode_offset(dto, fragment, \"__overlay__\");\n" "> > +\t\tif (overlay < 0)\n" @@ -538,14 +526,14 @@ "> > +\treturn ret;\n" "> > +}\n" "> \n" - "> I would caution against the liberal use of malloc in libfdt. We\342\200\231re possibly running in a constrained\n" + "> I would caution against the liberal use of malloc in libfdt. We?re possibly running in a constrained\n" "> environment; a custom extents based (non freeing) allocator should be better.\n" "> \n" "> We need some figures about memory consumption when this is enabled, and a CONFIG option to disable it.\n" "> \n" "> Regards\n" "> \n" - "> \342\200\224 Pantelis\n" + "> ? Pantelis\n" "> \n" "> \n" "\n" @@ -553,26 +541,13 @@ "David Gibson\t\t\t| I'll have my music baroque, and my code\n" "david AT gibson.dropbear.id.au\t| minimalist, thank you. NOT _the_ _other_\n" "\t\t\t\t| _way_ _around_!\n" - http://www.ozlabs.org/~dgibson - "\01:2\0" - "fn\0signature.asc\0" - "b\0" - "-----BEGIN PGP SIGNATURE-----\n" - "Version: GnuPG v1\n" - "\n" - "iQIcBAEBAgAGBQJXX08OAAoJEGw4ysog2bOSJckQAJ8LFgv+m/LKJhhz3B3Cwfto\n" - "PI8/or3EQ2awtdGziVXVcbII3D23e+BI01ANKT4nfg3NJofzPjGoK7dOUkEru5u6\n" - "Us+mXQQwQW0py3RBFrUk8WSHtI1EwtyrL8ZQ7PS22K/RcYrUsSBpZuswg2NgvKtT\n" - "1p6rLSbYOnnRxaH+60vopLetyO2RoMFBf82WODjtyr2xAc78wrhO8sOLGXUQHAW3\n" - "yPmR9qyYI51UIXUPGA0m+IYtcN6s86pSMNu3/6lvEJG7Hhx6stsf0Qjcv8pHsM+V\n" - "ljACYRQG1prJPtE1+kwPr5joWycAQDPG5vRwX+FvQk8N6rMJDWu00dYRIRE8o09y\n" - "syo6W2XR0yhQ3k4wSl3+Hi7xzKhtTMEhEPWGD5fxe6LkI8iJ+sDfMGR82/oefhcN\n" - "4jmoNwuE3DFCfGx3hZu09kt+ahbUiutdsOt3hQoQSsw1CwdhU06qUthweA2HIp6o\n" - "zy0ZNy9mly+q9RWBMxy8hEQllkp81/2GzyaI5kUra/x9ZOXLlPYCEcjOqBqwa8TM\n" - "X9rkuUgllPN5CK+Tk39oBhUb3HS/fXwhd/Fpqh05jeHwk71S6wMfBFhHeYrvmVM6\n" - "XcGsSHkJ8hSBF/UKrgDHIeWj/NUhA78rqa0oy8Ym4iO2HowR300em1QXxd+9eHQB\n" - "b+9k4+u9g23cGYKQWNVN\n" - "=Oi/m\n" - "-----END PGP SIGNATURE-----\n" + "http://www.ozlabs.org/~dgibson\n" + "-------------- next part --------------\n" + "A non-text attachment was scrubbed...\n" + "Name: signature.asc\n" + "Type: application/pgp-signature\n" + "Size: 819 bytes\n" + "Desc: not available\n" + URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160614/c3d022f2/attachment.sig> -51700d2478018408d32f0e7108d74fff15563af39b0b90ffc0abf98b6bbe0bf9 +4cea2315d72e53d75b964428bb04ec030ea23a330d8c98078110033e9a1de9d7
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.