diff for duplicates of <52209278.8040106@samsung.com> diff --git a/a/1.txt b/N1/1.txt index be40ac7..6f34b6e 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -196,9 +196,9 @@ Does it really makes sense to have such configuration with different > > + * global autoconfigured region for contiguous allocations > > + * (used only with Contiguous Memory Allocator) > > + */ -> > + contig_region at 0 { +> > + contig_region@0 { > -> I would expect this to be "region at 0", not "contig_region". Also, the '_' +> I would expect this to be "region@0", not "contig_region". Also, the '_' > character is strongly discouraged in node names. > > > + compatible = "linux,contiguous-memory-region"; @@ -238,7 +238,7 @@ but Rob already pointed that such code is quite hard to understand. > > + /* > > + * special region for framebuffer > > + */ -> > + display_region: region at 78000000 { +> > + display_region: region@78000000 { > > + compatible = "linux,contiguous-memory-region", "reserved-memory-region"; > > + reg = <0x78000000 0x800000>; > > + }; @@ -246,7 +246,7 @@ but Rob already pointed that such code is quite hard to understand. > > + /* > > + * special region for multimedia processing devices > > + */ -> > + multimedia_region: region at 77000000 { +> > + multimedia_region: region@77000000 { > > + compatible = "linux,contiguous-memory-region"; > > + reg = <0x77000000 0x4000000>; > > + }; @@ -255,17 +255,17 @@ but Rob already pointed that such code is quite hard to understand. > > + > > + /* ... */ > > + -> > + fb0: fb at 12300000 { +> > + fb0: fb@12300000 { > > + status = "okay"; > > + memory-region = <&display_region>; > > + }; > > + -> > + scaler: scaler at 12500000 { +> > + scaler: scaler@12500000 { > > + status = "okay"; > > + memory-region = <&multimedia_region>; > > + }; > > + -> > + codec: codec at 12600000 { +> > + codec: codec@12600000 { > > + status = "okay"; > > + memory-region = <&multimedia_region>; > > + }; diff --git a/a/content_digest b/N1/content_digest index 9f16324..092fd5c 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,10 +1,29 @@ "ref\01377527959-5080-1-git-send-email-m.szyprowski@samsung.com\0" "ref\01377527959-5080-4-git-send-email-m.szyprowski@samsung.com\0" "ref\020130829224637.D09183E1222@localhost\0" - "From\0m.szyprowski@samsung.com (Marek Szyprowski)\0" - "Subject\0[PATCH v7 3/4] drivers: of: add initialization code for dma reserved memory\0" + "From\0Marek Szyprowski <m.szyprowski@samsung.com>\0" + "Subject\0Re: [PATCH v7 3/4] drivers: of: add initialization code for dma reserved memory\0" "Date\0Fri, 30 Aug 2013 14:39:20 +0200\0" - "To\0linux-arm-kernel@lists.infradead.org\0" + "To\0Grant Likely <grant.likely@secretlab.ca>\0" + "Cc\0Mark Rutland <mark.rutland@arm.com>" + devicetree@vger.kernel.org + Laura Abbott <lauraa@codeaurora.org> + Pawel Moll <pawel.moll@arm.com> + Arnd Bergmann <arnd@arndb.de> + Stephen Warren <swarren@wwwdotorg.org> + Tomasz Figa <t.figa@samsung.com> + Tomasz Figa <tomasz.figa@gmail.com> + Michal Nazarewicz <mina86@mina86.com> + linaro-mm-sig@lists.linaro.org + Marc <marc.ceeeee@gmail.com> + Kyungmin Park <kyungmin.park@samsung.com> + Sylwester Nawrocki <s.nawrocki@samsung.com> + Kumar Gala <galak@codeaurora.org> + Olof Johansson <olof@lixom.net> + Nishanth Peethambaran <nishanth.p@gmail.com> + Sascha Hauer <s.hauer@pengutronix.de> + linux-arm-kernel@lists.infradead.org + " Ian Campbell <ian.campbell@citrix.com>\0" "\00:1\0" "b\0" "Hello,\n" @@ -205,9 +224,9 @@ "> > +\t\t\t * global autoconfigured region for contiguous allocations\n" "> > +\t\t\t * (used only with Contiguous Memory Allocator)\n" "> > +\t\t\t */\n" - "> > +\t\t\tcontig_region at 0 {\n" + "> > +\t\t\tcontig_region@0 {\n" ">\n" - "> I would expect this to be \"region at 0\", not \"contig_region\". Also, the '_'\n" + "> I would expect this to be \"region@0\", not \"contig_region\". Also, the '_'\n" "> character is strongly discouraged in node names.\n" ">\n" "> > +\t\t\t\tcompatible = \"linux,contiguous-memory-region\";\n" @@ -247,7 +266,7 @@ "> > +\t\t\t/*\n" "> > +\t\t\t * special region for framebuffer\n" "> > +\t\t\t */\n" - "> > +\t\t\tdisplay_region: region at 78000000 {\n" + "> > +\t\t\tdisplay_region: region@78000000 {\n" "> > +\t\t\t\tcompatible = \"linux,contiguous-memory-region\", \"reserved-memory-region\";\n" "> > +\t\t\t\treg = <0x78000000 0x800000>;\n" "> > +\t\t\t};\n" @@ -255,7 +274,7 @@ "> > +\t\t\t/*\n" "> > +\t\t\t * special region for multimedia processing devices\n" "> > +\t\t\t */\n" - "> > +\t\t\tmultimedia_region: region at 77000000 {\n" + "> > +\t\t\tmultimedia_region: region@77000000 {\n" "> > +\t\t\t\tcompatible = \"linux,contiguous-memory-region\";\n" "> > +\t\t\t\treg = <0x77000000 0x4000000>;\n" "> > +\t\t\t};\n" @@ -264,17 +283,17 @@ "> > +\n" "> > +\t/* ... */\n" "> > +\n" - "> > +\tfb0: fb at 12300000 {\n" + "> > +\tfb0: fb@12300000 {\n" "> > +\t\tstatus = \"okay\";\n" "> > +\t\tmemory-region = <&display_region>;\n" "> > +\t};\n" "> > +\n" - "> > +\tscaler: scaler at 12500000 {\n" + "> > +\tscaler: scaler@12500000 {\n" "> > +\t\tstatus = \"okay\";\n" "> > +\t\tmemory-region = <&multimedia_region>;\n" "> > +\t};\n" "> > +\n" - "> > +\tcodec: codec at 12600000 {\n" + "> > +\tcodec: codec@12600000 {\n" "> > +\t\tstatus = \"okay\";\n" "> > +\t\tmemory-region = <&multimedia_region>;\n" "> > +\t};\n" @@ -559,4 +578,4 @@ "Marek Szyprowski\n" Samsung R&D Institute Poland -969418dc1ff7ef75f1b696bb4e84d14b96b4ce47ab0df32fdd846ccb3fb9bf4f +0cfea33251a7cad5d90751a49bd3058563025debd20e5055c83d83035bee3ca0
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.