diff for duplicates of <1466095574.4442.53.camel@synopsys.com> diff --git a/a/1.txt b/N1/1.txt index 5dfd536..7903765 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,6 +1,6 @@ Hi Marek, -We used to use?of_reserved_mem_device_init() in such a context in GPU drivers: +We used to use of_reserved_mem_device_init() in such a context in GPU drivers: -------------------->8------------------- /* Get the optional framebuffer memory resource */ ret = of_reserved_mem_device_init(drm->dev); @@ -12,8 +12,8 @@ The point is we may have a dedicated reserved memory area or may not have (depen Our expectation is if reserved memory area is missing then of_reserved_mem_device_init() just returns -ENODEV and it used to work like this. -Now with your commit?59ce4039727e "of: reserved_mem: add support for using more than one region for given device" -behavior is different.?of_reserved_mem_device_init_by_idx() has this: +Now with your commit 59ce4039727e "of: reserved_mem: add support for using more than one region for given device" +behavior is different. of_reserved_mem_device_init_by_idx() has this: -------------------->8------------------- target = of_parse_phandle(np, "memory-region", idx); if (!target) @@ -21,10 +21,14 @@ behavior is different.?of_reserved_mem_device_init_by_idx() has this: -------------------->8------------------- So I'm wondering which part should be fixed: -?1)?of_reserved_mem itself or -?2) users of of_reserved_mem_device_init() + 1) of_reserved_mem itself or + 2) users of of_reserved_mem_device_init() Any thoughts? Regards, Alexey +_______________________________________________ +linux-snps-arc mailing list +linux-snps-arc@lists.infradead.org +http://lists.infradead.org/mailman/listinfo/linux-snps-arc diff --git a/a/content_digest b/N1/content_digest index c9c7ec9..69df264 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,12 +1,19 @@ - "From\0Alexey.Brodkin@synopsys.com (Alexey Brodkin)\0" + "From\0Alexey Brodkin <Alexey.Brodkin@synopsys.com>\0" "Subject\0of_reserved_mem_device_init_by_idx() returns -EINVAL if \"memory-region\" is missing\0" "Date\0Thu, 16 Jun 2016 16:47:08 +0000\0" - "To\0linux-snps-arc@lists.infradead.org\0" + "To\0m.szyprowski@samsung.com <m.szyprowski@samsung.com>\0" + "Cc\0robh@kernel.org <robh@kernel.org>" + s.nawrocki@samsung.com <s.nawrocki@samsung.com> + airlied@linux.ie <airlied@linux.ie> + Liviu.Dudau@arm.com <Liviu.Dudau@arm.com> + dri-devel@lists.freedesktop.org <dri-devel@lists.freedesktop.org> + daniel@ffwll.ch <daniel@ffwll.ch> + " linux-snps-arc@lists.infradead.org <linux-snps-arc@lists.infradead.org>\0" "\00:1\0" "b\0" "Hi Marek,\n" "\n" - "We used to use?of_reserved_mem_device_init() in such a context in GPU drivers:\n" + "We used to use\302\240of_reserved_mem_device_init() in such a context in GPU drivers:\n" "-------------------->8-------------------\n" "\t/* Get the optional framebuffer memory resource */\n" "\tret = of_reserved_mem_device_init(drm->dev);\n" @@ -18,8 +25,8 @@ "Our expectation is if reserved memory area is missing then of_reserved_mem_device_init()\n" "just returns -ENODEV and it used to work like this.\n" "\n" - "Now with your commit?59ce4039727e \"of: reserved_mem: add support for using more than one region for given device\"\n" - "behavior is different.?of_reserved_mem_device_init_by_idx() has this:\n" + "Now with your commit\302\24059ce4039727e \"of: reserved_mem: add support for using more than one region for given device\"\n" + "behavior is different.\302\240of_reserved_mem_device_init_by_idx() has this:\n" "-------------------->8-------------------\n" "\ttarget = of_parse_phandle(np, \"memory-region\", idx);\n" "\tif (!target)\n" @@ -27,12 +34,16 @@ "-------------------->8-------------------\n" "\n" "So I'm wondering which part should be fixed:\n" - "?1)?of_reserved_mem itself or\n" - "?2) users of of_reserved_mem_device_init()\n" + "\302\2401)\302\240of_reserved_mem itself or\n" + "\302\2402) users of of_reserved_mem_device_init()\n" "\n" "Any thoughts?\n" "\n" "Regards,\n" - Alexey + "Alexey\n" + "_______________________________________________\n" + "linux-snps-arc mailing list\n" + "linux-snps-arc@lists.infradead.org\n" + http://lists.infradead.org/mailman/listinfo/linux-snps-arc -6513f56f01f71e3c5e063377a662d4dd78a7e9dcac98b48edf711595175b5c7e +20f87c836de229c07aee4079fd3268cf12869aeba52f0b4a8f00dfed18035797
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.